Skip to content

Commit b6cf0f8

Browse files
author
MacroFake
committed
Merge bitcoin/bitcoin#25511: test: non-positive integer value to -peertimeout should throw an error
d22bd54 test: passing a non-positive integer value to `-peertimeout` should throw an error (brunoerg) Pull request description: This PR adds test coverage for #25506, since #25505 gets closed. ACKs for top commit: kristapsk: ACK d22bd54 w0xlt: ACK bitcoin/bitcoin@d22bd54 1440000bytes: ACK bitcoin/bitcoin@d22bd54 Tree-SHA512: 89c8a097606cb52569d816cc2227baac832df70e381d07c4a12aeb024c500d334c8102218fc6519eebb3819159d8308119d7253d9192a6bebe13b8e738b286b9
2 parents bae8a66 + d22bd54 commit b6cf0f8

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/functional/p2p_timeouts.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,11 @@ def run_test(self):
9494
no_version_node.wait_for_disconnect(timeout=1)
9595
no_send_node.wait_for_disconnect(timeout=1)
9696

97+
self.stop_nodes(0)
98+
self.nodes[0].assert_start_raises_init_error(
99+
expected_msg='Error: peertimeout must be a positive integer.',
100+
extra_args=['-peertimeout=0'],
101+
)
97102

98103
if __name__ == '__main__':
99104
TimeoutsTest().main()

0 commit comments

Comments
 (0)