[script_tests] consolidate multisig order tests
AbandonedPublic
Actions

Authored by markblundeberg on Jul 9 2019, 16:17.

Details

Reviewers

deadalnix
jasonbcox
Fabien
Mengerian

Group Reviewers

Restricted Project

Summary

The valid and invalid tests used to be broken up into two files. Since
the files are now merged it makes sense to put the related tests
together.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 6761
Build 11569: Bitcoin ABC Buildbot (legacy)
Build 11568: arc lint + arc unit

Event Timeline

markblundeberg created this revision.Jul 9 2019, 16:17

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 9 2019, 16:17

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

markblundeberg added a child revision: D3613: [script_tests] add tests for pubkey encoding with strictenc + nullfail.Jul 9 2019, 16:19

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6761: Diff 10134.Jul 9 2019, 16:25

These are tests we inherited from core. I don't see a good reason to reorder it all.

This revision now requires changes to proceed.Jul 9 2019, 20:20

markblundeberg abandoned this revision.Jul 10 2019, 00:40

Revision Contents
Changeset List

Path

Size

src/

test/

data/

script_tests.json

49 lines

Diff 10134

View Options

src/test/data/script_tests.json

[script_tests] consolidate multisig order testsAbandonedPublicActions