diff options
| author | W. J. van der Laan <laanwj@protonmail.com> | 2021-11-29 13:48:17 +0100 |
|---|---|---|
| committer | W. J. van der Laan <laanwj@protonmail.com> | 2021-11-29 13:51:27 +0100 |
| commit | a574f4ad390908ace794eb10fdb917d66c57094b (patch) | |
| tree | a2b9b1b1d5ba2bb5c34f6c9224dd31eea198d249 /test/functional/feature_anchors.py | |
| parent | 014cae2ee8386bfd074d1882148dc684b094cf57 (diff) | |
| parent | 988024fe37fe00c2deff28e44593e330aa6191f4 (diff) | |
Merge bitcoin/bitcoin#23392: test: move check_node_connections to util
988024fe37fe00c2deff28e44593e330aa6191f4 test: add check_node_connections in util (brunoerg)
Pull request description:
This function (`check_node_connections`) exists in `feature_anchors.py` and `p2p_add_connections.py` and does the same thing in both, this PR moves this function to util to avoid repetition and facilitate maintenance.
ACKs for top commit:
laanwj:
Code review ACK 988024fe37fe00c2deff28e44593e330aa6191f4
Tree-SHA512: bf86c5659933539c72cb91ad587552b45c918be74d36fb429e78f3b954f01ed0855a85dd49aea35b432fbd18227c05eb3fec8b99c139c3509c39b19bccf6b7fd
Diffstat (limited to 'test/functional/feature_anchors.py')
| -rwxr-xr-x | test/functional/feature_anchors.py | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/test/functional/feature_anchors.py b/test/functional/feature_anchors.py index 01b6c5f9eb..713c0826d3 100755 --- a/test/functional/feature_anchors.py +++ b/test/functional/feature_anchors.py @@ -8,18 +8,12 @@ import os from test_framework.p2p import P2PInterface from test_framework.test_framework import BitcoinTestFramework -from test_framework.util import assert_equal +from test_framework.util import check_node_connections INBOUND_CONNECTIONS = 5 BLOCK_RELAY_CONNECTIONS = 2 -def check_node_connections(*, node, num_in, num_out): - info = node.getnetworkinfo() - assert_equal(info["connections_in"], num_in) - assert_equal(info["connections_out"], num_out) - - class AnchorsTest(BitcoinTestFramework): def set_test_params(self): self.num_nodes = 1 |