HomePhabricator

test: Fail if connect_nodes fails

Description

test: Fail if connect_nodes fails

Summary:
Pull request description:

Currently, `connect_nodes` will return silently when the connection is disconnected while connecting. This is confusing, so fix it.

Can be tested by reverting the signet test change and observing the failure when running the test.

Partial backport of core#25443

Depends on D17529

Test Plan:

ninja check-functional

Reviewers: #bitcoin_abc, Fabien, PiRK

Reviewed By: #bitcoin_abc, Fabien, PiRK

Subscribers: PiRK

Differential Revision: https://reviews.bitcoinabc.org/D17530

Details

Provenance
laanwj <126646+laanwj@users.noreply.github.com>Authored on Jun 22 2022, 10:19
roqqitCommitted on Wed, Jan 15, 22:25
roqqitPushed on Wed, Jan 15, 22:25
Reviewer
Restricted Project
Differential Revision
D17530: test: Fail if connect_nodes fails
Parents
rABCc801d8fc621a: [test] Cleanup unnecessary sync_all in rpc_net.py
Branches
Unknown
Tags
Unknown

Event Timeline