HomePhabricator

[tests] Make p2p-leaktests.py more robust

Description

[tests] Make p2p-leaktests.py more robust

Summary:
Also throw in various nits.

Backport Core PR #11078: [tests] Make p2p-leaktests.py more robust

0063d2c3d [tests] Make p2p-leaktests.py more robust (John Newbery)

Pull request description:

There has been an example of p2p-leaktests.py failing on travis in the new service bits test (introduced in #11001 . It appeared to me that the previous p2p connections had not been fully disconnected before attempting to add new p2p connections.

I've added a sleep and restarted the NetworkThread, but I don't know whether this will fix the problem, since I'm unable to reproduce the failure locally.

Test Plan:

./test/functional/test_runner.py p2p-leaktests

Reviewers: #bitcoin_abc, schancel

Reviewed By: #bitcoin_abc, schancel

Subscribers: teamcity

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

Details

Provenance
John Newbery <john@johnnewbery.com>Authored on Aug 17 2017, 15:35
deadalnixCommitted on Mar 7 2018, 21:19
deadalnixPushed on Mar 7 2018, 21:35
Reviewer
Restricted Project
Differential Revision
D1183: [tests] Make p2p-leaktests.py more robust
Parents
rSTAGINGef4ee06f6d0f: Remove unused `plTxnReplaced` parameter to AcceptToMemoryPool
Branches
Unknown
Tags
Unknown