Differential D10084

[avalanche] Add some test coverage for the buildavalancheproof RPC
ClosedPublic
Actions

Authored by Fabien on Sep 9 2021, 12:46.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC25e6fd9b6a17: [avalanche] Add some test coverage for the buildavalancheproof RPC

Summary

The RPC is used everywhere in the tests but is not tested itself.
This diff does not intend to extend the safety checks but only check for the existing errors to prevent regressions.

Test Plan

./test/functional/test_runner.py abc_rpc_buildavalancheproof

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_test_buildavalancheproof

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16605
Build 33065: Build Diff	build-without-wallet · build-debug · build-clang-tidy · build-diff · build-clang
Build 33064: arc lint + arc unit

Event Timeline

Fabien created this revision.Sep 9 2021, 12:46

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 9 2021, 12:46

Fabien requested review of this revision.Sep 9 2021, 12:46

Fabien added a child revision: D10086: [avalanche] Add an error message for duplicated stakes in buildavalancheproof.Sep 9 2021, 12:54

Harbormaster completed remote builds in B16605: Diff 29876.Sep 9 2021, 12:59

PiRK accepted this revision.Sep 9 2021, 15:32

This revision is now accepted and ready to land.Sep 9 2021, 15:32

Closed by commit rABC25e6fd9b6a17: [avalanche] Add some test coverage for the buildavalancheproof RPC (authored by Fabien). · Explain WhySep 9 2021, 18:00

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC25e6fd9b6a17: [avalanche] Add some test coverage for the buildavalancheproof RPC.

Revision Contents
Changeset List

Path

Size

test/

functional/

abc_rpc_buildavalancheproof.py

90 lines

Diff 29876

test/functional/abc_rpc_buildavalancheproof.py

Loading...