Page MenuHomePhabricator

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

Authored by PiRK on Thu, Mar 25, 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 Diffbuild-diff · build-clang · build-without-wallet · build-clang-tidy · build-debug
Build 30739: arc lint + arc unit

Event Timeline

PiRK requested review of this revision.Thu, Mar 25, 11:55
PiRK retitled this revision from [avalanche] split the functional test into 3 tests to [avalanche] split the functional test suite into 3 suites.Thu, Mar 25, 14:23
PiRK edited the summary of this revision. (Show Details)

use new names for the new test classes

PiRK planned changes to this revision.Fri, Mar 26, 08:22

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

abandoned in favor of D9351

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