Page MenuHomePhabricator

New Schnorr multisig activation

Authored by markblundeberg on Jul 25 2019, 05:29.



Based on D2483, but the functional test uses the new features recently
introduced (see current

Depends on D3474

Test Plan

make check --with-gravitonactivation

Diff Detail

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

Event Timeline

markblundeberg created this revision.Jul 25 2019, 05:29
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 25 2019, 05:29

Hmm, I tried also running ./test/functional/ --with-gravitonactivation which actually causes a failure in, which is not surprising because the reject reason will have changed.

markblundeberg planned changes to this revision.Jul 25 2019, 05:42
markblundeberg edited the test plan for this revision. (Show Details)

update feature_nulldummy test to only run pre-activation

markblundeberg added inline comments.Jul 26 2019, 21:13
29 ↗(On Diff #10505)

Alternatively, we can just delete this test now since the test cases seem to be covered by anyway.

rebase onto the updated parent

markblundeberg added inline comments.Jul 29 2019, 18:54
88 ↗(On Diff #10517)

this can be removed (leftover from copy)

deadalnix accepted this revision.Aug 2 2019, 00:36
deadalnix added inline comments.
224 ↗(On Diff #10517)

You probably want to have an else that throw an error.

This revision is now accepted and ready to land.Aug 2 2019, 00:36