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
Branch
activateschnorr
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6957
Build 11961: Bitcoin ABC Buildbot (legacy)
Build 11960: arc lint + arc unit

Event Timeline

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 edited the test plan for this revision. (Show Details)

update feature_nulldummy test to only run pre-activation

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

test/functional/abc-schnorrmultisig-activation.py
88 ↗(On Diff #10517)

this can be removed (leftover from copy)

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