Skip to content

Commit e7f3450

Browse files
author
MarcoFalke
committed
Merge #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 #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
2 parents fd925e9 + fa81525 commit e7f3450

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
@@ -203,6 +203,8 @@ def test_balances(*, fee_node_1=0):
203203
self.log.info('Put txs back into mempool of node 1 (not node 0)')
204204
self.nodes[0].invalidateblock(block_reorg)
205205
self.nodes[1].invalidateblock(block_reorg)
206+
self.sync_blocks()
207+
self.nodes[0].syncwithvalidationinterfacequeue()
206208
assert_equal(self.nodes[0].getbalance(minconf=0), 0) # wallet txs not in the mempool are untrusted
207209
self.nodes[0].generatetoaddress(1, ADDRESS_WATCHONLY)
208210
assert_equal(self.nodes[0].getbalance(minconf=0), 0) # wallet txs not in the mempool are untrusted

0 commit comments

Comments
 (0)