HomePhabricator

[avalanche] Fix threading issue in processor_tests.cpp

Description

[avalanche] Fix threading issue in processor_tests.cpp

Summary:
The test called the PeerManager directly instead of using the processor
interface that holds the appropriated locks.

Test Plan:
With Clang and Debug:

ninja check-avalanche

Run the TSAN build.

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Maniphest Tasks: T1611

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

Details

Provenance
FabienAuthored on Jun 4 2021, 19:23
FabienPushed on Jun 5 2021, 06:59
Reviewer
Restricted Project
Differential Revision
D9634: [avalanche] Fix threading issue in processor_tests.cpp
Parents
rABC359045064944: [avalanche] Download the proof from our peers that send avahello
Branches
Unknown
Tags
Unknown
Tasks
Restricted Maniphest Task