MarcoFalke 38389dd3a0 Merge #19252: test: wait for disconnect in disconnect_p2ps + bloomfilter test followups ... 9a40cfc558 [refactor] use waiting inside disconnect_p2ps (gzhao408) aeb9fb414e [test] wait for disconnect_p2ps to be reflected in getpeerinfo (gzhao408) e81942d2e1 [test] logging and style followups for bloomfilter tests (gzhao408) Pull request description: Followup to #19083 which adds bloomfilter-related tests. 1. Make test_node `disconnect_p2ps` wait until disconnection is complete to avoid race conditions (and not place the burden on tests) from MarcoFalke's [comment](https://github.com/bitcoin/bitcoin/pull/19083#discussion_r437383989). And clean up any redundant `wait_until`s in the functional tests. 2. Clean up style + logging in p2p_filter.py and p2p_nobloomfilter_messages.py and jonatack's other [comments](https://github.com/bitcoin/bitcoin/pull/19083#pullrequestreview-428955784) ACKs for top commit: jonatack: Code review ACK 9a40cfc from re-reviewing the diff and `git range-diff 5cafb46 8386ad5 9a40cfc` MarcoFalke: ACK 9a40cfc558 🐂 Tree-SHA512: 2e14b1c12fc08a355bd5ccad7a2a734a4ccda4bc7dc7bac171cb57359819fc1599d764290729af74832fac3e2be258c5d406c701e78ab6d7262835859b9a7d87		2020-06-17 06:20:10 -04:00
..
__init__.py	…
address.py	…
authproxy.py	…
blocktools.py	…
coverage.py	…
descriptors.py	…
key.py	[tests] move generate_wif_key to wallet_util.py	2020-06-10 12:10:02 -04:00
messages.py	…
mininode.py	…
netutil.py	…
script.py	…
script_util.py	…
segwit_addr.py	…
siphash.py	…
socks5.py	…
test_framework.py	test: refactor: Inline adjust_bitcoin_conf_for_pre_17	2020-06-16 08:27:57 -04:00
test_node.py	[test] wait for disconnect_p2ps to be reflected in getpeerinfo	2020-06-16 08:26:13 -07:00
test_shell.py	…
util.py	pep-8 test/functional/test_framework/util.py	2020-06-16 08:28:06 -04:00
wallet_util.py	[tests] move generate_wif_key to wallet_util.py	2020-06-10 12:10:02 -04:00