Skip to content

Commit

Permalink
Merge #18447: test: Add coverage for script parse error in ParseScript
Browse files Browse the repository at this point in the history
dcda81c test: add coverage for script parse error in ParseScript (pierrenn)

Pull request description:

  Follow up on this suggestion :  bitcoin/bitcoin#18416 (comment)

  This adds a test case to raise the `script parse error` in `ParseScript`.

ACKs for top commit:
  instagibbs:
    utACK bitcoin/bitcoin@dcda81c

Tree-SHA512: ae0ef2c00f34cee818c83582f190d5f4043159e922862f2b442b7b895b8ff3ca421533699247c12c367be77813b5205830a771cd47a18e8932807ccace2d6a1c
  • Loading branch information
MarcoFalke committed Mar 27, 2020
2 parents bdc2644 + dcda81c commit 210b533
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions test/util/data/bitcoin-util-test.json
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,12 @@
"output_cmp": "txcreate2.json",
"description": "Parses a transaction with no inputs and a single output script (output in json)"
},
{ "exec": "./bitcoin-tx",
"args": ["-create", "outscript=0:123badscript"],
"return_code": 1,
"error_txt": "error: script parse error",
"description": "Create a new transaction with an invalid output script"
},
{ "exec": "./bitcoin-tx",
"args": ["-create", "outscript=0:OP_DROP", "nversion=1"],
"output_cmp": "txcreatescript1.hex",
Expand Down

0 comments on commit 210b533

Please sign in to comment.