diff options
author | MarcoFalke <falke.marco@gmail.com> | 2020-06-17 14:45:16 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2020-06-17 14:45:20 -0400 |
commit | 35ed88f187c9bc7eedc3a1cf12193e0fbf222057 (patch) | |
tree | 609bbb0c4cf00d7972e3fa083be3870c6a9fa5b0 /test/functional | |
parent | 39bd9ddb8783807b9cde6288233e86ad7c85d61f (diff) | |
parent | fa195d4eba6397262e3e76c8525eb48dcb5e7f2d (diff) | |
download | bitcoin-35ed88f187c9bc7eedc3a1cf12193e0fbf222057.tar.xz |
Merge #19298: test: Add missing sync_blocks
fa195d4eba6397262e3e76c8525eb48dcb5e7f2d test: Add missing sync_blocks (MarcoFalke)
Pull request description:
Bitcoin Core does not sort block and tx announcements for other peers, so generating 100 blocks and then sending out a transaction might reject it if it arrives too early. (non-final)
Fix that by syncing the blocks first.
Fix #19265
Fix #19311
ACKs for top commit:
Sjors:
utACK fa195d4eba6397262e3e76c8525eb48dcb5e7f2d: sounds plausible
Tree-SHA512: fdc46aed59595e4189509e71bd4a3607a93893933cc01d806cec2ee7701d54d7422c5f22dd83b81ddb021f9113b3119a688fdd8cf8a6474fc12fea422aedd064
Diffstat (limited to 'test/functional')
-rwxr-xr-x | test/functional/feature_notifications.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/test/functional/feature_notifications.py b/test/functional/feature_notifications.py index 33a308ad1b..dd4c318cee 100755 --- a/test/functional/feature_notifications.py +++ b/test/functional/feature_notifications.py @@ -93,6 +93,7 @@ class NotificationsTest(BitcoinTestFramework): self.nodes[0].sethdseed(seed=self.nodes[1].dumpprivkey(keyhash_to_p2pkh(hex_str_to_bytes(self.nodes[1].getwalletinfo()['hdseedid'])[::-1]))) self.nodes[0].rescanblockchain() self.nodes[0].generatetoaddress(100, ADDRESS_BCRT1_UNSPENDABLE) + self.sync_blocks() # Generate transaction on node 0, sync mempools, and check for # notification on node 1. |