mirror of
https://github.com/kaspanet/kaspad.git
synced 2025-09-13 13:00:10 +00:00

* [NOD-1249] Add pruning related constants (#869) * [NOD-1249] Add pruning related constants * [NOD-1249] Change status suspect to UTXONotVerified * [NOD-1249] Add TestPruningDepth * [NOD-1249] Add comment to pruningDepth * [NOD-1249] Add pruning helper functions (#875) * [NOD-1249] Added node.blockAtDepth * [NOD-1249] Added node.finalityPoint() * [NOD-1249] Add hasFinalityPointInOthersSelectedChain * [NOD-1249] Add nonFinalityViolatingBlues * [NOD-1249] Added isInPastOfAny * [NOD-1249] Updated all calls to blockNode functions that require dag * [NOD-1249] Add blockNode.reds field and persist it * [NOD-1249] Add checkObjectiveFinality * [NOD-1249] Add isViolatingSubjectiveFinality * [NOD-1249] Added to TestGHOSTDAG check that reds are as expected * [NOD-1249] Add checkMergeLimit and checkDAGRelations * [NOD-1249] Invert condition in blockInDepth * [NOD-1249] Make isInPastOfAny resemble isInPast * [NOD-1249] Added comments to isInPast and isInPastOfAny * [NOD-1252] Remove any references to legacy finality (#876) * [NOD-1032] validateParents: check number of parents and that no parents were manually rejected (#877) * [NOD-1254] Block verification changes (#882) * [NOD-1254] Call checkDAGRelations and move it to correct place * [NOD-1254] Use blockStatuses properly * [NOD-1254] Add support for setting node's verification flag and set it to UTXONotVerified once block passes basic verification * [NOD-1254] Check for subjctiveFinality, and for node not being in the selectedParentChain * [NOD-1254] Make blockStatus an ordinary value - not bit flags * [NOD-1254] Isolate all utxo-requiring validation into a single separate if branches * [NOD-1254] Re-arrange connectBloc so that things that happen in UTXO-validated blocks only are all grouped together * [NOD-1254] Resolve and check selectedParent's status before validatingUTXO * [NOD-1254] Separate virtualUTXODiff from utxoVerificationOutput * [NOD-1254] Stylistic fixes * [NOD-1254] Use dag.index.(Set)BlockNodeStatus instead of accessing node.status * [NOD-1288] Sub-routinize checkConnectToPastUTXO * [NOD-1288] Re-write checkConnectToPastUTXO in a way that allows to filter-out invalid transactions * [NOD-1288] Make checkTxSequenceLock use already calculated utxo outputs * [NOD-1288] Make checkTxMass use already calculated utxo outputs * [NOD-1288] Use dag.sigCache for ValidateTransactionScripts * [NOD-1288] Use checkConnectTransactionToPastUTXO in applyBlueBlocks * [NOD-1288] Clean-up old code-path from no longer used functions * [NOD-1288] Skip any irrelevant parts of txo verification if block is genesis * [NOD-1288] Set where it should have been * [NOD-1288] Fix reachability checks to never use the new node + make isInSelectedParentChainOf return true if node == other * [NOD-1288] invert the condition for isNewSelectedTip * [NOD-1288] Separate checkIsAccepted to own function, and properly handle coinbase * [NOD-1288] Don't update utxo-diff for UTXONotVerified parents/tips + Make PrepareBlockForTest resolve the selectedParent's UTXOSet if needed * [NOD-1288] Include mass off coinbase transactions * [NOD-1288] Move comment to correct position * [NOD-1288] If blockAtDepth should return genesis - do it immidiately * [NOD-1288] Comment-out instead of removeing scriptval_test.go * [NOD-1288] Rename: entry -> utxoEntry * [NOD-1288] Remove special function for calcCoinbaseTxMass * [NOD-1288] Remove redundant check from checkEntryAmounts * [NOD-1288] Rename: MaxMassPerBlock -> MaxMassAcceptedByBlock * [NOD-1255] Implement boundedMergeBreakingParents * [NOD-1255] Implement selectAllowedTips * [NOD-1255] Integrate virtual parent selection into block verification process * [NOD-1255] Add node to tips all the time, remove it from candidates and add it's parents if it's disqualified * [NOD-1255] remove tips from virtaulBlock * [NOD-1255] Rename: didVirtualParentsChanged -> didVirtualParentsChange * [NOD-1255] Remove redundant sanity check * [NOD-1255] Handle a forgotten error * [NOD-1255] Prettify selectVirtualParents * [NOD-1255] UpdateTipsUTXO should run over all UTXO-Verified tips, even if they are not parents of virtual * [NOD-1311] Make isInPast inclusive * [NOD-1032] Handle finality conflicts (#904) * [NOD-1312] AddTip should not include finalityViolating and manuallyRejected blocks * [NOD-1312] Implement resolveFinalityConflict * [NOD-1312] Implement dag notifications for finalityChanges + updateing DAG state * [NOD-1312] Added finality conflict rpc boilerplate * [NOD-1312] Implement handling of getFinalityConflicts + resolveFinalityConflict RPCs * [NOD-1312] Implement finality conflict related notifications * [NOD-1312] Move all time to millisecond time * [NOD-1312] Add comments + unexport some methods * [NOD-1312] Add clarification in comments * [NOD-1312] Move updateFinalityConflictResolution to finality_conflicts.go * [NOD-1312] Rename: currentSelectedTip -> selectedTip * [NOD-1312] Add long comment to ResolveFinalityConflict * [NOD-1312] Convert areAllInSelectedParentChainOf into isInSelectedParentChainOfAll * [NOD-1312] Rename chainUpdates (type) -> selectedParentChainUpdates, to distinguish from the variable chainUpdates * [NOD-1032] Make all blockdag tests compile * [NOD-1278] Fix finality-related tests (#910) * [NOD-1032] Don't return node.dag.genesis from blockAtDepth because it might still not exist * [NOD-1032] Actually add a tip in dag.addTip * [NOD-1278] Add transaction to utxo set if it's coinbase * [NOD-1278] Use VirtualParentHashes instead of TipHashes where appropriate * [NOD-1278] If no valid virtual parent candidates - return error, don't wait for panic * [NOD-1278] Transition TestCalcSequenceLock from newTestDAG to DAGSetup * [NOD-1278] Fix .bluest() tie-breaker * [NOD-1278] Remove feeData structure, as it no longer works, store feeData in acceptanceData * [NOD-1278] Remove dag parameter from blockNode methods * [NOD-1278] Fix TestBlueBlockWindow * [NOD-1278] Don't subject selectedParent to MaxMergeSet * [NOD-1278] se PrepareAndProcessBlockForTest instead of .addTip in TestSelectedPath * [NOD-1278] Fixed TestDAGStateSerialization * [NOD-1278] Fix TestAcceptanceIndexRecover * [NOD-1278] Fix TestCheckConnectBlockTemplate * [NOD-1278] Fix TestChainUpdates * [NOD-1278] Fix and rename TestVirtualBlock -> TestTips * [NOD-1278] Rename checkIsAccepted -> maybeAcceptTx * [NOD-1278] Re-activate TestDoubleSpends * Revert "[NOD-1278] Fixed TestDAGStateSerialization" This reverts commit 845095d6de7207b07cf819d05f3f38ad94da9cf6. * [NOD-1278] Remove dag parameter from expectedCoinbaseTransaction * [NOD-1348] Implemented simplified Finality Conflict Resolution scheme (#911) * [NOD-1348] Rename functions according to Research spec * [NOD-1348] Added blockSet.areAllIn * [NOD-1348] Implemented simplified finality conflict resolution scheme * [NOD-1348] Refactorings and fixes in selectVirtualParents * [NOD-1278] Fix bugs discovered by unit-tests + Fix unit-tests (#916) * Updated to version v0.3.1 * [NOD-858] Don't switch sync peer if the syncing process hasn't yet started with the current sync peer (#700) * [NOD-858] Don't switch sync peer if the syncing process hasn't yet started with the current sync peer * [NOD-858] SetShouldSendBlockLocator(false) on OnBlockLocator * [NOD-858] Rename shouldSendBlockLocator->wasBlockLocatorRequested * [NOD-858] Move panic to shouldReplaceSyncPeer * [NOD-869] Add a print after os.Exit(1) to see if it is ever called (#701) * [NOD-1238] Fix acceptance index never being initialized. (#859) * [NOD-1278] Genesis never violates finality * [NOD-1348] Refactorings and fixes in selectVirtualParents * [NOD-1278] Don't call dag.selectVirtualParents for genesis * [NOD-1278] Properly organize errors in maybeAcceptBlock * [NOD-1278] updateTipsUTXO should only run on tips whose status is * [NOD-1278] updateTipsUTXO should only run on tips whose status is `valid` * [NOD-1278] Fix TestDoubleSpends * [NOD-1278] Fix TestDAGIndexFailedStatus * [NOD-1278] IsFinalizedTransaction should use uint64 everywhere * [NOD-1278] If tx is coinbase and not selectedParent - don't update pastUTXO * [NOD-1278] Store tips and validTips separately * [NOD-1278] Store ValidTips and VirtualParents in dagState * [NOD-1278] Fix TestProcessOrphans * [NOD-1278] Fix TestProcessOrphans * [NOD-1278] Fix TestOrderInDiffFromAcceptanceData * [NOD-1278] Fix TestHelp * [NOD-1278] Remove mining.PrepareBlockForTest; use blockdag.PrepareBlockForTest instead * [NOD-1278] Explicitly disallow chained transactions * [NOD-1278] * [NOD-1278] Fix some comments Co-authored-by: Ori Newman <orinewman1@gmail.com> Co-authored-by: stasatdaglabs <39559713+stasatdaglabs@users.noreply.github.com> Co-authored-by: Yuval Shaul <yuval.shaul@gmail.com> * [NOD-1355] Add unit-test for finality + When resolving finalityConflict - make sure the block that will come out selectedTip is statusValid (#919) * [NOD-1355] Added test for finality * [NOD-1355] When resolving finalityConflict - make sure the block that will come out selectedTip is statusValid * [NOD-1032] Renames: anything about inputsWithReferencedUTXOEntries -> remove 'Referenced' * [NOD-1032] Don't ignore non-rule errors * [NOD-1032] Fix comment * [NOD-1032] Enhanced comments on TestChainUpdates * [NOD-1032] Remove scriptval_test.go * [NOD-1032] Extracted isNewSelectedTip to a method * [NOD-1032] Use dag.Now() instead of mstime.Now() * [NOD-1032] Print block status when accepting block * [NOD-1032] Add comment explaining boundedMergeBreakingParents * [NOD-1032] Enhanced test and imporved comments in TestFinality * [NOD-1032] Rename: Objective finality -> bounded merge depth * [NOD-1032] No need to check that validTips are valid * [NOD-1032] Remove dag from arguments of updateDiffAndDiffChild * [NOD-1032] Correct variable names in LookupNodes [NOD-1032] Correct variable names in LookupNodes * [NOD-1032] Fix some comments * [NOD-1032] Some style changes * [NOD-1032] Refactor isAnyInPastOf * [NOD-1032] Enhance comment in updateVirtualParents * [NOD-1032] Flip condition in updateVirtualParents * [NOD-1032] Stylistic and grammatic fixes in dag.go and dag_test.go * [NOD-1032] Explain why updateSelectedParentSet receives geneses on init * [NOD-1032] Remove ErrParentManuallyRejected * [NOD-1032] Added wrapper for resolveNodeStatus that creates a special transaction for it * [NOD-1032] Rename: statusUTXONotVerified -> statusUTXOPendingVerification * [NOD-1032] Use virtual parents in CurrentBits() * [NOD-1032] rename: isViolatingSubjectiveFinality -> isViolatingFinality * [NOD-1032] Move updateVirtualAndTips to applyDAGChanges * [NOD-1032] Invert condition for isFinalityPointInPast * [NOD-1032] Fix antiPastBetween isInPast became inclusive * [NOD-1032] Remove redundant call for addTip * [NOD-1032] Use calcCoinbaseTxMass where appropriate * [NOD-1032] Remove time fields from conflict notifications * [NOD-1032] Assign the correct thing to i * [NOD-1032] unify checkOutputsAmounts and checkTxOutputAmounts * [NOD-1032] Cleanup in CheckTransactionInputsAndCalulateFee * [NOD-1032] Fixed some style and comments * [NOD-1032] If selectedParent is disqualifiedFromChain - validateAndApplyUTXOSet should return this as a ruleError * [NOD-1032] Set the status in resolveNodeStatus * [NOD-1032] Correct comment on boundedMergeBreakingParents * [NOD-1032] Fix a typo. * [NOD-1032] Update a variable name. * [NOD-1032] Fix a comment. * [NOD-1032] Fix merge errors. * [NOD-1032] Add VirtualParentHashes to getBlockDagInfo. * [NOD-1032] Update handleGetBlockTemplate. * [NOD-1032] Comment out all the old RPC stuff. * [NOD-1032] Remove irrelevant type. * [NOD-1032] Implement ResolveFinalityConflict. * [NOD-1032] Remove irrelevant comments. * [NOD-1032] Implement NotifyFinalityConflicts. * [NOD-1032] Add FinalityConflictNotification and FinalityConflictResolvedNotification. * [NOD-1032] Finish implementing finality conflict notifications. * [NOD-1032] Remove old RPC stuff. * [NOD-1032] Fix grammar in a comment. Co-authored-by: Ori Newman <orinewman1@gmail.com> Co-authored-by: stasatdaglabs <39559713+stasatdaglabs@users.noreply.github.com> Co-authored-by: Yuval Shaul <yuval.shaul@gmail.com> Co-authored-by: stasatdaglabs <stas@daglabs.com>
311 lines
8.2 KiB
Go
311 lines
8.2 KiB
Go
package blockdag
|
|
|
|
import (
|
|
"reflect"
|
|
"testing"
|
|
|
|
"github.com/kaspanet/kaspad/util/daghash"
|
|
)
|
|
|
|
func TestHashes(t *testing.T) {
|
|
bs := blockSetFromSlice(
|
|
&blockNode{
|
|
hash: &daghash.Hash{3},
|
|
},
|
|
&blockNode{
|
|
hash: &daghash.Hash{1},
|
|
},
|
|
&blockNode{
|
|
hash: &daghash.Hash{0},
|
|
},
|
|
&blockNode{
|
|
hash: &daghash.Hash{2},
|
|
},
|
|
)
|
|
|
|
expected := []*daghash.Hash{
|
|
{0},
|
|
{1},
|
|
{2},
|
|
{3},
|
|
}
|
|
|
|
hashes := bs.hashes()
|
|
if !daghash.AreEqual(hashes, expected) {
|
|
t.Errorf("TestHashes: hashes order is %s but expected %s", hashes, expected)
|
|
}
|
|
}
|
|
|
|
func TestBlockSetSubtract(t *testing.T) {
|
|
node1 := &blockNode{hash: &daghash.Hash{10}}
|
|
node2 := &blockNode{hash: &daghash.Hash{20}}
|
|
node3 := &blockNode{hash: &daghash.Hash{30}}
|
|
|
|
tests := []struct {
|
|
name string
|
|
setA blockSet
|
|
setB blockSet
|
|
expectedResult blockSet
|
|
}{
|
|
{
|
|
name: "both sets empty",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(),
|
|
},
|
|
{
|
|
name: "subtract an empty set",
|
|
setA: blockSetFromSlice(node1),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "subtract from empty set",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(node1),
|
|
expectedResult: blockSetFromSlice(),
|
|
},
|
|
{
|
|
name: "subtract unrelated set",
|
|
setA: blockSetFromSlice(node1),
|
|
setB: blockSetFromSlice(node2),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "typical case",
|
|
setA: blockSetFromSlice(node1, node2),
|
|
setB: blockSetFromSlice(node2, node3),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
result := test.setA.subtract(test.setB)
|
|
if !reflect.DeepEqual(result, test.expectedResult) {
|
|
t.Errorf("blockSet.subtract: unexpected result in test '%s'. "+
|
|
"Expected: %v, got: %v", test.name, test.expectedResult, result)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestBlockSetAddSet(t *testing.T) {
|
|
node1 := &blockNode{hash: &daghash.Hash{10}}
|
|
node2 := &blockNode{hash: &daghash.Hash{20}}
|
|
node3 := &blockNode{hash: &daghash.Hash{30}}
|
|
|
|
tests := []struct {
|
|
name string
|
|
setA blockSet
|
|
setB blockSet
|
|
expectedResult blockSet
|
|
}{
|
|
{
|
|
name: "both sets empty",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(),
|
|
},
|
|
{
|
|
name: "add an empty set",
|
|
setA: blockSetFromSlice(node1),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "add to empty set",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(node1),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "add already added member",
|
|
setA: blockSetFromSlice(node1, node2),
|
|
setB: blockSetFromSlice(node1),
|
|
expectedResult: blockSetFromSlice(node1, node2),
|
|
},
|
|
{
|
|
name: "typical case",
|
|
setA: blockSetFromSlice(node1, node2),
|
|
setB: blockSetFromSlice(node2, node3),
|
|
expectedResult: blockSetFromSlice(node1, node2, node3),
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
test.setA.addSet(test.setB)
|
|
if !reflect.DeepEqual(test.setA, test.expectedResult) {
|
|
t.Errorf("blockSet.addSet: unexpected result in test '%s'. "+
|
|
"Expected: %v, got: %v", test.name, test.expectedResult, test.setA)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestBlockSetAddSlice(t *testing.T) {
|
|
node1 := &blockNode{hash: &daghash.Hash{10}}
|
|
node2 := &blockNode{hash: &daghash.Hash{20}}
|
|
node3 := &blockNode{hash: &daghash.Hash{30}}
|
|
|
|
tests := []struct {
|
|
name string
|
|
set blockSet
|
|
slice []*blockNode
|
|
expectedResult blockSet
|
|
}{
|
|
{
|
|
name: "add empty slice to empty set",
|
|
set: blockSetFromSlice(),
|
|
slice: []*blockNode{},
|
|
expectedResult: blockSetFromSlice(),
|
|
},
|
|
{
|
|
name: "add an empty slice",
|
|
set: blockSetFromSlice(node1),
|
|
slice: []*blockNode{},
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "add to empty set",
|
|
set: blockSetFromSlice(),
|
|
slice: []*blockNode{node1},
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "add already added member",
|
|
set: blockSetFromSlice(node1, node2),
|
|
slice: []*blockNode{node1},
|
|
expectedResult: blockSetFromSlice(node1, node2),
|
|
},
|
|
{
|
|
name: "typical case",
|
|
set: blockSetFromSlice(node1, node2),
|
|
slice: []*blockNode{node2, node3},
|
|
expectedResult: blockSetFromSlice(node1, node2, node3),
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
test.set.addSlice(test.slice)
|
|
if !reflect.DeepEqual(test.set, test.expectedResult) {
|
|
t.Errorf("blockSet.addSlice: unexpected result in test '%s'. "+
|
|
"Expected: %v, got: %v", test.name, test.expectedResult, test.set)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestBlockSetUnion(t *testing.T) {
|
|
node1 := &blockNode{hash: &daghash.Hash{10}}
|
|
node2 := &blockNode{hash: &daghash.Hash{20}}
|
|
node3 := &blockNode{hash: &daghash.Hash{30}}
|
|
|
|
tests := []struct {
|
|
name string
|
|
setA blockSet
|
|
setB blockSet
|
|
expectedResult blockSet
|
|
}{
|
|
{
|
|
name: "both sets empty",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(),
|
|
},
|
|
{
|
|
name: "union against an empty set",
|
|
setA: blockSetFromSlice(node1),
|
|
setB: blockSetFromSlice(),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "union from an empty set",
|
|
setA: blockSetFromSlice(),
|
|
setB: blockSetFromSlice(node1),
|
|
expectedResult: blockSetFromSlice(node1),
|
|
},
|
|
{
|
|
name: "union with subset",
|
|
setA: blockSetFromSlice(node1, node2),
|
|
setB: blockSetFromSlice(node1),
|
|
expectedResult: blockSetFromSlice(node1, node2),
|
|
},
|
|
{
|
|
name: "typical case",
|
|
setA: blockSetFromSlice(node1, node2),
|
|
setB: blockSetFromSlice(node2, node3),
|
|
expectedResult: blockSetFromSlice(node1, node2, node3),
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
result := test.setA.union(test.setB)
|
|
if !reflect.DeepEqual(result, test.expectedResult) {
|
|
t.Errorf("blockSet.union: unexpected result in test '%s'. "+
|
|
"Expected: %v, got: %v", test.name, test.expectedResult, result)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestBlockSetAreAllIn(t *testing.T) {
|
|
node1 := &blockNode{hash: &daghash.Hash{10}}
|
|
node2 := &blockNode{hash: &daghash.Hash{20}}
|
|
node3 := &blockNode{hash: &daghash.Hash{30}}
|
|
|
|
tests := []struct {
|
|
name string
|
|
set blockSet
|
|
other blockSet
|
|
expectedResult bool
|
|
}{
|
|
{
|
|
name: "two empty sets",
|
|
set: blockSetFromSlice(),
|
|
other: blockSetFromSlice(),
|
|
expectedResult: true,
|
|
},
|
|
{
|
|
name: "set empty, other full",
|
|
set: blockSetFromSlice(),
|
|
other: blockSetFromSlice(node1, node2, node3),
|
|
expectedResult: true,
|
|
},
|
|
{
|
|
name: "set full, other empty",
|
|
set: blockSetFromSlice(node1, node2, node3),
|
|
other: blockSetFromSlice(),
|
|
expectedResult: false,
|
|
},
|
|
{
|
|
name: "same node in both",
|
|
set: blockSetFromSlice(node1),
|
|
other: blockSetFromSlice(node1),
|
|
expectedResult: true,
|
|
},
|
|
{
|
|
name: "different node in both",
|
|
set: blockSetFromSlice(node1),
|
|
other: blockSetFromSlice(node2),
|
|
expectedResult: false,
|
|
},
|
|
{
|
|
name: "set is subset of other",
|
|
set: blockSetFromSlice(node1, node2),
|
|
other: blockSetFromSlice(node2, node1, node3),
|
|
expectedResult: true,
|
|
},
|
|
{
|
|
name: "other is subset of set",
|
|
set: blockSetFromSlice(node2, node1, node3),
|
|
other: blockSetFromSlice(node1, node2),
|
|
expectedResult: false,
|
|
},
|
|
}
|
|
|
|
for _, test := range tests {
|
|
result := test.set.areAllIn(test.other)
|
|
|
|
if result != test.expectedResult {
|
|
t.Errorf("blockSet.areAllIn: unexpected result in test '%s'. "+
|
|
"Expected: '%t', got: '%t'", test.name, test.expectedResult, result)
|
|
}
|
|
}
|
|
}
|