HomePhabricator

Merge #13517: qa: Remove need to handle the network thread in tests

Description

Merge #13517: qa: Remove need to handle the network thread in tests

Summary:
Merge #13517: qa: Remove need to handle the network thread in tests
fa87da2f172ae2e6dc15e9ed156a3564a8ecfbdd qa: Avoid start/stop of the network thread mid-test (MarcoFalke)

Pull request description:

This simplifies test writing by removing the need to handle the network thread in tests. E.g. start thread, join thread, restart thread mid-test, adding p2p connections at the "right" time, ...

Tree-SHA512: 533642f12fef5496f1933855edcdab1a7ed901d088d34911749cd0f9e044c8a6cb1f89985ac3a7f41a512943663e4e270a61978f6f072143ae050cd102d4eab8

Backport of Core PR13517
https://github.com/bitcoin/bitcoin/pull/13517/files
Completes T676
Depends on D4002, D4011


Merge #13715: tests: fixes mininode's P2PConnection sending messages on closing transport
ea5340c9d2 tests: fixes mininode's P2PConnection sending messages on closing transport (marcoagner)

Pull request description:

Fixes #13579.
I think one possible solution is to check for [`_transport.is_closing()`](https://docs.python.org/3.4/library/asyncio-protocol.html#asyncio.BaseTransport.is_closing) in the lambda before sending a message (compatible with Python 3.4 too). Let me know if I missed any side effects this introduces.

Tree-SHA512: cab46f81dccfec7b4460fda478a617845564520694449a9e85bf8a5f1e75f35f52cafd7c64966712c3d6c29956344d5a9dbad8851424f061eb3748bc621b900b

Backport of Core PR13715
https://github.com/bitcoin/bitcoin/pull/13715/files
Completes T678


Both backports included because feature_assumevalid.py breaks without PR13715

Test Plan: test_runner.py

Reviewers: #bitcoin_abc, deadalnix, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

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

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Jun 29 2018, 16:04
jasonbcoxCommitted on Sep 6 2019, 15:52
jasonbcoxPushed on Sep 6 2019, 16:45
Reviewer
Restricted Project
Differential Revision
D4012: Merge #13517: qa: Remove need to handle the network thread in tests
Parents
rSTAGING7fe97790bd89: Merge #11818: I accidentally [deliberately] killed it [theā€¦
Branches
Unknown
Tags
Unknown