Skip to content

Commit

Permalink
Merge bitcoin#18805: tests: Add missing sync_all to wallet_importdesc…
Browse files Browse the repository at this point in the history
…riptors.py

7b2b06d tests: Add missing sync_all to wallet_importdescriptors.py (Andrew Chow)

Pull request description:

  node1 will sometimes do sendtoaddress before it has received a funding transaction which will cause the test to fail. sync_all to ensure it gets the transaction first.

  Fixes bitcoin#18800

ACKs for top commit:
  instagibbs:
    ACK 7b2b06d The wallet endpoint right after is indeed node 1.

Tree-SHA512: b610a771d062b5f955cd70b34337577a1ab8dacbf4be20aa74e1e8234495b0be9faff138eb1713f29decb5574446e0583e221bc2c9a6eea13611b422ea3a296a
  • Loading branch information
MarcoFalke authored and knst committed Mar 6, 2024
1 parent 76e08f9 commit baa6959
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions test/functional/wallet_importdescriptors.py
Original file line number Diff line number Diff line change
Expand Up @@ -306,6 +306,7 @@ def run_test(self):
assert_equal(wmulti_priv.getwalletinfo()['keypoolsize'], 1000)
txid = w0.sendtoaddress(addr, 10)
self.nodes[0].generate(6)
self.sync_all()
wmulti_priv.sendtoaddress(w0.getnewaddress(), 8)
self.nodes[0].generate(6)
self.sync_all()
Expand Down

0 comments on commit baa6959

Please sign in to comment.