Differential D11237

[avalanche] Add a verifyavalanchedelegation RPC
ClosedPublic
Actions

Authored by Fabien on Mar 22 2022, 10:41.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC38e8957aeef3: [avalanche] Add a verifyavalanchedelegation RPC

Summary

This checks the delegation signature stack is correct.
Note that since it shares the error path with other RPCs which have full coverage already, I only added a happy path test to avoid needless redundancy.

Depends on D11236.

Test Plan

./test/functional/test_runner.py abc_rpc_avalancheproof

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Mar 22 2022, 10:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 22 2022, 10:41

Fabien requested review of this revision.Mar 22 2022, 10:41

Harbormaster completed remote builds in B18589: Diff 32880.Mar 22 2022, 11:04

PiRK accepted this revision.Mar 22 2022, 15:51

This revision is now accepted and ready to land.Mar 22 2022, 15:51

Closed by commit rABC38e8957aeef3: [avalanche] Add a verifyavalanchedelegation RPC (authored by Fabien). · Explain WhyMar 22 2022, 19:51

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC38e8957aeef3: [avalanche] Add a verifyavalanchedelegation RPC.

Revision Contents
Changeset List

Path

Size

src/

rpc/

avalanche.cpp

27 lines

test/

functional/

abc_rpc_avalancheproof.py

5 lines

Diff 32900

View Options

src/rpc/avalanche.cpp