diff options
| author | Russell Yanofsky <russ@yanofsky.org> | 2016-11-14 09:58:30 -0500 |
|---|---|---|
| committer | Russell Yanofsky <russ@yanofsky.org> | 2016-11-14 13:59:20 -0500 |
| commit | dfa44d1b07a6d1022005dba63dd6372739eee8a0 (patch) | |
| tree | d0c755dc16bdad343a18134a64d6a904685a239e /qa/rpc-tests/mempool_resurrect_test.py | |
| parent | 87ab49e4fe38f0d8d527090ea348081037a68cef (diff) | |
[qa] Wait for specific block announcement in p2p-compactblocks
Change check_announcement_of_new_block() to wait specifically for the
announcement of the newly created block, instead of waiting for any
announcement at all. A difficult to reproduce failure in
check_announcement_of_new_block() that happened in a travis build
(https://travis-ci.org/bitcoin/bitcoin/jobs/175198367) might have happened
because an older announcement was mistaken for the expected one. The error
looked like:
Assertion failed: Failed
File ".../bitcoin/qa/rpc-tests/test_framework/test_framework.py", line 145, in main
self.run_test()
File ".../bitcoin/build/../qa/rpc-tests/p2p-compactblocks.py", line 787, in run_test
self.test_sendcmpct(self.nodes[1], self.segwit_node, 2, old_node=self.old_node)
File ".../bitcoin/build/../qa/rpc-tests/p2p-compactblocks.py", line 201, in test_sendcmpct
check_announcement_of_new_block(node, test_node, lambda p: p.last_cmpctblock is None and p.last_inv is not None)
File ".../bitcoin/build/../qa/rpc-tests/p2p-compactblocks.py", line 194, in check_announcement_of_new_block
assert(predicate(peer))
This commit also changes the assertion failed message above to include more
detailed information for debug.
Diffstat (limited to 'qa/rpc-tests/mempool_resurrect_test.py')
0 files changed, 0 insertions, 0 deletions