Skip to content

Commit 35fb993

Browse files
committed
test: enable v2 transport for p2p_timeouts.py
by skipping the part where we send a non-version message before the version - this message would be interpreted as part of the v2 handshake.
1 parent 2c1669c commit 35fb993

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

test/functional/p2p_timeouts.py

+10-5
Original file line numberDiff line numberDiff line change
@@ -68,23 +68,27 @@ def run_test(self):
6868

6969
with self.nodes[0].assert_debug_log(['Unsupported message "ping" prior to verack from peer=0']):
7070
no_verack_node.send_message(msg_ping())
71-
with self.nodes[0].assert_debug_log(['non-version message before version handshake. Message "ping" from peer=1']):
72-
no_version_node.send_message(msg_ping())
7371

74-
self.mock_forward(1)
72+
# With v2, non-version messages before the handshake would be interpreted as part of the key exchange.
73+
# Therefore, don't execute this part of the test if v2transport is chosen.
74+
if not self.options.v2transport:
75+
with self.nodes[0].assert_debug_log(['non-version message before version handshake. Message "ping" from peer=1']):
76+
no_version_node.send_message(msg_ping())
7577

78+
self.mock_forward(1)
7679
assert "version" in no_verack_node.last_message
7780

7881
assert no_verack_node.is_connected
7982
assert no_version_node.is_connected
8083
assert no_send_node.is_connected
8184

8285
no_verack_node.send_message(msg_ping())
83-
no_version_node.send_message(msg_ping())
86+
if not self.options.v2transport:
87+
no_version_node.send_message(msg_ping())
8488

8589
expected_timeout_logs = [
8690
"version handshake timeout peer=0",
87-
"socket no message in first 3 seconds, 1 0 peer=1",
91+
f"socket no message in first 3 seconds, {'0' if self.options.v2transport else '1'} 0 peer=1",
8892
"socket no message in first 3 seconds, 0 0 peer=2",
8993
]
9094

@@ -100,5 +104,6 @@ def run_test(self):
100104
extra_args=['-peertimeout=0'],
101105
)
102106

107+
103108
if __name__ == '__main__':
104109
TimeoutsTest().main()

test/functional/test_runner.py

+1
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,7 @@
153153
'p2p_invalid_messages.py',
154154
'rpc_createmultisig.py',
155155
'p2p_timeouts.py',
156+
'p2p_timeouts.py --v2transport',
156157
'wallet_dump.py --legacy-wallet',
157158
'rpc_signer.py',
158159
'wallet_signer.py --descriptors',

0 commit comments

Comments
 (0)