diff options
| author | glozow <gloriajzhao@gmail.com> | 2024-02-06 10:56:51 +0000 |
|---|---|---|
| committer | glozow <gloriajzhao@gmail.com> | 2024-02-06 11:02:36 +0000 |
| commit | 4de84557d6d1f53255ab19f27c8b6ea0a712934a (patch) | |
| tree | 24e187991e0918b58044ad89036ed6a145984168 /test/functional/rpc_whitelist.py | |
| parent | 4572f48fd5a864311a4f3f09043506d59ad9bda0 (diff) | |
| parent | e7fd70f4b6b163f4ad5b25b4da7fa79899245235 (diff) | |
Merge bitcoin/bitcoin#29356: test: make v2transport arg in addconnection mandatory and few cleanups
e7fd70f4b6b163f4ad5b25b4da7fa79899245235 [test] make v2transport arg in addconnection mandatory and few cleanups (stratospher)
Pull request description:
- make `v2transport` argument in `addconnection` regression-testing only RPC mandatory. https://github.com/bitcoin/bitcoin/pull/24748#discussion_r1470738750
- previously it was an optional arg with default `false` value.
- only place this RPC is used is in the [functional tests](https://github.com/bitcoin/bitcoin/blob/11b436a66af3ceaebb0f907878715f331516a0bc/test/functional/test_framework/test_node.py#L742) where we always pass the appropriate `v2transport` option to the RPC anyways. (and that too just for python dummy peer(`P2PInterface`) and bitcoind(`TestNode`) interactions)
- rename `v2_handshake()` to `_on_data_v2_handshake()` https://github.com/bitcoin/bitcoin/pull/24748#discussion_r1466958424
- more compact return statement in `wait_for_reconnect()` https://github.com/bitcoin/bitcoin/pull/24748#discussion_r1466979708
- assertion to check that empty version packets are received from `TestNode`.
ACKs for top commit:
glozow:
ACK e7fd70f4b6
theStack:
Code-review ACK e7fd70f4b6b163f4ad5b25b4da7fa79899245235
mzumsande:
Code Review ACK e7fd70f4b6b163f4ad5b25b4da7fa79899245235
Tree-SHA512: e66e29baccd91e1e4398b91f7d45c5fc7c2841d77d8a6178734586017bf2be63496721649da91848dec71da605ee31664352407d5bb896e624cc693767c61a1f
Diffstat (limited to 'test/functional/rpc_whitelist.py')
0 files changed, 0 insertions, 0 deletions