Page MenuHomePhabricator
Differential D6548

Update avalanche integration test to use a quorum
ClosedPublic
Actions

Authored by deadalnix on Jun 12 2020, 15:48.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCefef30853a6d: Update avalanche integration test to use a quorum
Summary

Instead of one node, the test will now provide a quorum of 8 fake nodes to bitcoind so we can simulate more realitic conditions.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Jun 12 2020, 15:48
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 12 2020, 15:48
deadalnix requested review of this revision.Jun 12 2020, 15:48
Harbormaster completed remote builds in B11443: Diff 21321.Jun 12 2020, 16:02
deadalnix added a child revision: D6559: [avalanche] Process AvalancheResponse and act based on the result..Jun 12 2020, 21:02
Fabien accepted this revision.Jun 15 2020, 07:54
Fabien added a subscriber: Fabien.
Fabien added inline comments.
test/functional/abc_p2p_avalanche.py
98 ↗(On Diff #21321)

nit: on node fromt he => one node from the

This revision is now accepted and ready to land.Jun 15 2020, 07:54
Closed by commit rABCefef30853a6d: Update avalanche integration test to use a quorum (authored by deadalnix). · Explain WhyJun 15 2020, 12:55
This revision was automatically updated to reflect the committed changes.
deadalnix added a commit: rABCefef30853a6d: Update avalanche integration test to use a quorum.

Revision Contents
Changeset List

PathSize
test/
functional/
63 lines

Diff 21374

View Options

test/functional/abc_p2p_avalanche.py

Loading...