highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-02-03 22:03:01 +01:00
Code Issues Packages Projects Releases Wiki Activity

test: Verify peer usage after assumeutxo validation completes

Browse Source 
Add test coverage to ensure peers without the snapshot block in their chain can be used for block downloads after background validation completes. The test fails without the fix in the previous commit.
This commit is contained in:
stringintech
2025-12-18 02:06:57 +03:30
parent 0067abe153
commit 7d9e1a8102
1 changed files with 25 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
test/functional/feature_assumeutxo.py
View File

@@ -347,6 +347,29 @@ class AssumeutxoTest(BitcoinTestFramework):
assert 'NETWORK' in ibd_node.getpeerinfo()[0]['servicesnames']
self.sync_blocks(nodes=(ibd_node, snapshot_node))
def test_sync_to_most_work_chain_after_background_validation(self):
"""
After background validation completes, node should be able
to download and process blocks from peers without the snapshot block in their chain.
"""
self.log.info("Testing sync to the most-work chain without the snapshot block after background validation")
forking_node = self.nodes[0]
snapshot_node = self.nodes[2] # Has already completed background validation
self.log.info("Forking node switches to an alternative chain that forks one block before the snapshot block")
fork_point = SNAPSHOT_BASE_HEIGHT - 1
forking_node_old_height = forking_node.getblockcount()
forking_node_old_chainwork = int(forking_node.getblockchaininfo()['chainwork'], 16)
forking_node.invalidateblock(forking_node.getblockhash(fork_point + 1))
self.log.info("Mine one more block than original chain to make the new chain have most work")
self.generate(forking_node, nblocks=(forking_node_old_height - fork_point) + 1, sync_fun=self.no_op)
assert int(forking_node.getblockchaininfo()['chainwork'], 16) > forking_node_old_chainwork
self.log.info("Snapshot node should reorg to the most-work chain without the snapshot block")
self.sync_blocks(nodes=(snapshot_node, forking_node))
def assert_only_network_limited_service(self, node):
node_services = node.getnetworkinfo()['localservicesnames']
assert 'NETWORK' not in node_services
@@ -775,6 +798,8 @@ class AssumeutxoTest(BitcoinTestFramework):
# The following test cleans node2 and node3 chain directories.
self.test_sync_from_assumeutxo_node(snapshot=dump_output)
self.test_sync_to_most_work_chain_after_background_validation()
@dataclass
class Block:
hash: str