test: fix StopIteration exception in p2p_node_network_limited.py #29753
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #29731
The
next()
call throws an exception if the default parameter is omitted and the iterator is exhausted.Fix it by providing a default value.
The failure can be tested by commenting out lines 90 and 91 in the test (the
self.connect_nodes(2, 0)
). Since there is no connection, the node in question retrieves a single element in the 'getchaintips()' call. This scenario without the fix, aborts the test right away, throwing anStopIteration
exception, and with the fix, the test properly waits until the timeout (wait_until()
call).