Merge #12425: Add some script tests
ClosedPublic
Actions

Authored by markblundeberg on May 24 2019, 21:30.

Details

Reviewers

jasonbcox
Fabien
Mengerian
deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGINGd15798062587: Merge #12425: Add some script tests
rABCd15798062587: Merge #12425: Add some script tests

Summary

PR12425 backport https://github.com/bitcoin/bitcoin/pull/12425/files
be45a67 Add some script tests related to BOOL ops and odd values like negative 0. (Richard Kiss)

Pull request description:

Add some script tests related to BOOL ops and odd values like negative 0.

Backport note: 0x02 0x00 0x00 --> 0x02 0x0000 for our ParseScript variant.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markblundeberg created this revision.May 24 2019, 21:30

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 24 2019, 21:30

Herald added a reviewer: deadalnix. · View Herald Transcript

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

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

Harbormaster completed remote builds in B5978: Diff 8862.May 24 2019, 21:45

jasonbcox accepted this revision.May 24 2019, 23:05

This revision is now accepted and ready to land.May 24 2019, 23:05

Closed by commit rABCd15798062587: Merge #12425: Add some script tests (authored by Wladimir J. van der Laan <laanwj@gmail.com>, committed by Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>). · Explain WhyMay 24 2019, 23:10

This revision was automatically updated to reflect the committed changes.

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> added a commit: rABCd15798062587: Merge #12425: Add some script tests.

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> added a commit: rSTAGINGd15798062587: Merge #12425: Add some script tests.Sep 18 2019, 23:44

Revision Contents
Changeset List

Path

Size

src/

test/

data/

script_tests.json

12 lines

Diff 8866

View Options

src/test/data/script_tests.json

Merge #12425: Add some script testsClosedPublicActions