[avalanche] Move the proof accessor tests to the peer manager tests
ClosedPublic
Actions

Authored by Fabien on Jul 12 2021, 13:04.

Details

Reviewers

PiRK
majcosta

Group Reviewers

Restricted Project

Commits

rABC665d9bc0639a: [avalanche] Move the proof accessor tests to the peer manager tests

Summary

The peer manager is responsible for the proof management, so this makes sense to test the proof accessors in its test suite.

Depends on D9766.

Test Plan

ninja check-avalanche

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_move_proof_accessor_tests

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16154
Build 32181: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang · build-clang-tidy
Build 32180: arc lint + arc unit

Event Timeline

Fabien created this revision.Jul 12 2021, 13:04

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 12 2021, 13:04

Fabien requested review of this revision.Jul 12 2021, 13:04

Harbormaster completed remote builds in B16154: Diff 29138.Jul 12 2021, 13:18

PiRK accepted this revision.Jul 12 2021, 13:47

This revision is now accepted and ready to land.Jul 12 2021, 13:47

majcosta accepted this revision.Jul 12 2021, 13:50

Closed by commit rABC665d9bc0639a: [avalanche] Move the proof accessor tests to the peer manager tests (authored by Fabien). · Explain WhyJul 12 2021, 14:06

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC665d9bc0639a: [avalanche] Move the proof accessor tests to the peer manager tests.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

peermanager_tests.cpp

36 lines

processor_tests.cpp

36 lines

Diff 29138

View Options

src/avalanche/test/peermanager_tests.cpp