HomePhabricator

add a couple more Schnorr checkmultisig tests

Description

add a couple more Schnorr checkmultisig tests

Summary:

  • a test with a one-byte checkbits not pushed using OP_1-OP_1NEGATE.
  • a test with a two-byte checkbits.

Also re-disable auto script gen (was inadvertently activated in D3474).
The auto script gen should generally be kept off since it allows the tester
to emit "Missing auto script_valid test" in case that the generator has
been changed somehow and impacted existing tests.

Test Plan: make check

Reviewers: deadalnix, #bitcoin_abc

Reviewed By: deadalnix, #bitcoin_abc

Differential Revision: https://reviews.bitcoinabc.org/D4455

Details

Provenance
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Authored on Nov 17 2019, 02:05
markblundebergPushed on Nov 20 2019, 23:13
Reviewer
Restricted Project
Differential Revision
D4455: add a couple more Schnorr checkmultisig tests
Parents
rSTAGINGd32a1d5f5fe6: Make SigningProvider expose key origin information
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/14280

Event Timeline

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> committed rSTAGING7b6969f08418: add a couple more Schnorr checkmultisig tests (authored by Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>).Nov 20 2019, 22:59