[avalanche] split the functional test suite into 3 suites
AbandonedPublic
Actions

Authored by PiRK on Mar 25 2021, 11:55.

Details

Reviewers

None

Group Reviewers

Restricted Project

Summary

The avalanche functional test was getting complicated. This splits the
current test suite into 3 separate tests:

fork resolution and voting
proofs and node response to invalid proofs
peer-discovery

This will hopefully reduce the conflicts and the need for rebasing while working on distinct aspects of the avalanche codebase.

Depends on D9341

Test Plan

ninja check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

arcpatch-D9350

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15409
Build 30740: Build Diff	build-diff · build-clang · build-without-wallet · build-clang-tidy · build-debug
Build 30739: arc lint + arc unit

Event Timeline

PiRK created this revision.Mar 25 2021, 11:55

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 25 2021, 11:55

PiRK requested review of this revision.Mar 25 2021, 11:55

Harbormaster completed remote builds in B15407: Diff 28018.Mar 25 2021, 12:04

PiRK retitled this revision from [avalanche] split the functional test into 3 tests to [avalanche] split the functional test suite into 3 suites.Mar 25 2021, 14:23

PiRK edited the summary of this revision. (Show Details)

use new names for the new test classes

Harbormaster completed remote builds in B15409: Diff 28021.Mar 25 2021, 16:18

D9327 will probably be replaced, so this needs a lot of work to become independant of D9327

PiRK mentioned this in D9351: [avalanche] split the test suite into 2 suites.Mar 26 2021, 09:51

abandoned in favor of D9351

The new diff does not depend on the peer-discovery stuff (D9327) that requires more work.

PiRK mentioned this in rABCebc6189c51a3: [avalanche] split the test suite into 2 suites.Mar 26 2021, 10:55