Page MenuHomePhabricator

New Schnorr multisig activation
ClosedPublic

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

Details

Summary

Based on D2483, but the functional test uses the new features recently
introduced (see current abc-schnorr.py).

Depends on D3474

Test Plan

make check
test_runner.py
test_runner.py --with-gravitonactivation

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

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/test_runner.py --with-gravitonactivation which actually causes a failure in feature_nulldummy.py, 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
test/functional/feature_nulldummy.py
29 ↗(On Diff #10505)

Alternatively, we can just delete this test now since the test cases seem to be covered by abc-schnorrmultisig-activation.py anyway.

rebase onto the updated parent

markblundeberg added inline comments.Jul 29 2019, 18:54
test/functional/abc-schnorrmultisig-activation.py
88 ↗(On Diff #10517)

this can be removed (leftover from copy)

deadalnix accepted this revision.Aug 2 2019, 00:36
deadalnix added inline comments.
test/functional/abc-schnorrmultisig-activation.py
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