Skip to content

Commit a5e2035

Browse files
MarcoFalkeMunkybooty
authored andcommitted
Merge bitcoin#16302: test: Add missing syncwithvalidationinterfacequeue to wallet_balance test
fa81525 test: Add missing sync_all to wallet_balance test (MarcoFalke) Pull request description: A `syncwithvalidationinterfacequeue` should be sufficient. Fixes bitcoin#16020 ACKs for top commit: promag: ACK fa81525. This can be tested by adding sleep in `CWallet::BlockConnected` just before `LOCK(cs_wallet)` - master will always fail while this PR will succeed. Tree-SHA512: 07e067c698627f90f0b9848f921b7067adc70c27105db3258e056384197e50dbee055c87839d238cc11bde11179d3f5879b39e1c8e15465f8f07558c694b677d
1 parent 5b4de7f commit a5e2035

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

test/functional/wallet_balance.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,8 @@ def test_balances(*, fee_node_1=0):
189189
self.log.info('Put txs back into mempool of node 1 (not node 0)')
190190
self.nodes[0].invalidateblock(block_reorg)
191191
self.nodes[1].invalidateblock(block_reorg)
192+
self.sync_blocks()
193+
self.nodes[0].syncwithvalidationinterfacequeue()
192194
assert_equal(self.nodes[0].getbalance(minconf=0), 0) # wallet txs not in the mempool are untrusted
193195
self.nodes[0].generatetoaddress(1, ADDRESS_WATCHONLY)
194196
assert_equal(self.nodes[0].getbalance(minconf=0), 0) # wallet txs not in the mempool are untrusted

0 commit comments

Comments
 (0)