HomePhabricator

Move NodeContext from TestingSetup to BasicTestingSetup

Description

Move NodeContext from TestingSetup to BasicTestingSetup

Summary:
This will allow for the scheduler to be destructed even for tests that
only use the BasicTestingSetup. This fixes a TSAN lock order inversion
issue when running the reverselock_tests and the scheduler_tests
together.

Extracted from core PR18571.

Test Plan:

./src/test/test_bitcoin -t reverselock_tests,scheduler_tests
ninja check

Reviewers: #bitcoin_abc, deadalnix, jasonbcox

Reviewed By: #bitcoin_abc, deadalnix, jasonbcox

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

Details

Provenance
FabienAuthored on Jun 29 2020, 14:43
FabienPushed on Jun 29 2020, 15:52
Reviewer
Restricted Project
Differential Revision
D6766: Move NodeContext from TestingSetup to BasicTestingSetup
Parents
rABCb23802b86950: test: Check that wait_until returns if time point is in the past
Branches
Unknown
Tags
Unknown