[avalanche] Fix a potential race in the quorum functional test
ClosedPublic
Actions

Authored by Fabien on Fri, Apr 19, 13:21.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCda36689fd3ab: [avalanche] Fix a potential race in the quorum functional test

Summary

Wait for the ready_to_poll status to change after disconnecting the nodes to avoid getting an unexpected vote from the node.

Test Plan

./test/functional/test_runner.py abc_p2p_avalanche_quorum

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix_avalanche_quorum_test

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28668
Build 56877: Build Diff	build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 56876: arc lint + arc unit

Event Timeline

Fabien created this revision.Fri, Apr 19, 13:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Apr 19, 13:21

Fabien requested review of this revision.Fri, Apr 19, 13:21

PiRK accepted this revision.Fri, Apr 19, 13:52

This revision is now accepted and ready to land.Fri, Apr 19, 13:52

Harbormaster completed remote builds in B28668: Diff 47309.Fri, Apr 19, 14:05

Closed by commit rABCda36689fd3ab: [avalanche] Fix a potential race in the quorum functional test (authored by Fabien). · Explain WhyFri, Apr 19, 14:12

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCda36689fd3ab: [avalanche] Fix a potential race in the quorum functional test.

Revision Contents
Changeset List

Path

Size

test/

functional/

abc_p2p_avalanche_quorum.py

1 line

Diff 47309

View Options

test/functional/abc_p2p_avalanche_quorum.py

[avalanche] Fix a potential race in the quorum functional testClosedPublicActions