Differential D11117

[avalanche] Don't assume the legacy proof format in get_proof_ids
ClosedPublic
Actions

Authored by Fabien on Mar 1 2022, 15:00.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCbb1e4d1ad567: [avalanche] Don't assume the legacy proof format in get_proof_ids

Summary

This is a step toward migrating our tests to the new proof format. This removes one of the assumption that legacy format is used.

Test Plan

ninja check-extended

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Mar 1 2022, 15:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 1 2022, 15:00

Fabien requested review of this revision.Mar 1 2022, 15:00

Harbormaster completed remote builds in B18382: Diff 32560.Mar 1 2022, 15:18

PiRK accepted this revision.Mar 1 2022, 18:34

This revision is now accepted and ready to land.Mar 1 2022, 18:34

Closed by commit rABCbb1e4d1ad567: [avalanche] Don't assume the legacy proof format in get_proof_ids (authored by Fabien). · Explain WhyMar 2 2022, 10:33

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCbb1e4d1ad567: [avalanche] Don't assume the legacy proof format in get_proof_ids.

Fabien mentioned this in D11121: [avalanche] Test framework: don't assume the p2p avaproof messages are using the legacy format.Mar 2 2022, 10:50

Fabien mentioned this in rABCb12a12c17154: [avalanche] Test framework: don't assume the p2p avaproof messages are using….Mar 2 2022, 13:41

Revision Contents
Changeset List

Path

Size

test/

functional/

test_framework/

avatools.py

4 lines

Diff 32566

View Options

test/functional/test_framework/avatools.py

[avalanche] Don't assume the legacy proof format in get_proof_idsClosedPublicActions