Page MenuHomePhabricator

Merge #16302: test: Add missing syncwithvalidationinterfacequeue to wallet_balance test
ClosedPublic

Authored by jasonbcox on Jul 16 2020, 20:08.

Details

Summary

fa815255c70d32809aac640db4a8762c7d71e8db 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

Backport of Core PR16302

Test Plan

test_runner.py wallet_balance

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.Jul 16 2020, 20:47