Skip to content

Conversation

@pierreN
Copy link
Contributor

@pierreN pierreN commented Mar 27, 2020

Follow up on this suggestion : #18416 (comment)

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

@pierreN pierreN changed the title test: add coverage for script parse error in ParseScript test: Add coverage for script parse error in ParseScript Mar 27, 2020
@fanquake fanquake added the Tests label Mar 27, 2020
@instagibbs
Copy link
Member

utACK dcda81c

@DrahtBot
Copy link
Contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Coverage

Coverage Change (pull 18447, 5444a82031a651e899e17e0fc6a6a6b5054e1348) Reference (master, 7eed413)
Lines +0.0201 % 90.1791 %
Functions +0.0000 % 85.9654 %
Branches +0.0085 % 51.7481 %

Updated at: 2020-03-27T13:52:20.335074.

@maflcko maflcko merged commit 210b533 into bitcoin:master Mar 27, 2020
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Jan 13, 2021
Summary: This is a backport of Core [[bitcoin/bitcoin#18447 | PR18447]]

Test Plan:  ./test/util/bitcoin-util-test.py -v

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D8889
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants