Page MenuHomePhabricator

Add additional unit tests for segwit recovery
ClosedPublic

Authored by florian on Apr 10 2019, 22:39.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCdb32590c400b: Add additional unit tests for segwit recovery
Summary

New unit tests from the revised specification.

Test Plan

make check

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

florian created this revision.Apr 10 2019, 22:39
Owners added a reviewer: Restricted Owners Package.Apr 10 2019, 22:39
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 10 2019, 22:39
deadalnix requested changes to this revision.Apr 10 2019, 23:17

Se comments in the spec PR.

This revision now requires changes to proceed.Apr 10 2019, 23:17
florian updated this revision to Diff 8244.Apr 23 2019, 19:24

Update tests to match the merged spec (test vectors: V6, V7, I10, I4, I5, I2, I1).

deadalnix accepted this revision.May 4 2019, 18:26
This revision is now accepted and ready to land.May 4 2019, 18:26
florian updated this revision to Diff 8385.May 4 2019, 20:58

rebase

This revision was automatically updated to reflect the committed changes.