Differential D10216

[avalanche] Make regular proof format deserialization tests in par with the legacy
ClosedPublic
Actions

Authored by Fabien on Sep 29 2021, 19:16.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABCfd3cc02e5ebc: [avalanche] Make regular proof format deserialization tests in par with the…

Summary

This clones the legacy test vectors and use them for the regular format check. This makes sure that no regression from the legacy format is introduced. Of course these tests needs to be updated to conform with the new format, so a valid payout script is added to each case and the stakes signatures updated accordingly.

Ref T1854.

Test Plan

ninja check-avalanche-proof_tests

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Sep 29 2021, 19:16

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 29 2021, 19:16

Fabien requested review of this revision.Sep 29 2021, 19:16

Fabien mentioned this in D10140: [avalanche] Sign the avalanche proof.Sep 29 2021, 19:21

Fabien added a child revision: D10140: [avalanche] Sign the avalanche proof.Sep 29 2021, 19:26

Harbormaster completed remote builds in B16815: Diff 30217.Sep 29 2021, 19:30

deadalnix accepted this revision.Sep 30 2021, 15:41

This revision is now accepted and ready to land.Sep 30 2021, 15:41

Closed by commit rABCfd3cc02e5ebc: [avalanche] Make regular proof format deserialization tests in par with the… (authored by Fabien). · Explain WhySep 30 2021, 19:24

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCfd3cc02e5ebc: [avalanche] Make regular proof format deserialization tests in par with the….

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

proof_tests.cpp

261 lines

Diff 30253

src/avalanche/test/proof_tests.cpp

Loading...