Add additional script edge condition tests. #6075

pull davecgh wants to merge 1 commits into bitcoin:master from davecgh:more_negative_script_tests changing 1 files +18 −0

davecgh commented at 7:46 AM on April 28, 2015: contributor

This pull request adds several tests to the script_invalid.json data which exercise some edge conditions that are not currently being tested.

These are mainly being added to cover several cases a branch coverage analysis of btcd showed are not already being covered, but given more tests of edge conditions are always a good thing, I'm contributing them upstream.

Add additional script edge condition tests.

This commit adds several tests to the script_invalid.json data which
exercise some edge conditions that are not currently being tested.

These are mainly being added to cover several cases a branch coverage
analysis of btcd showed are not already being covered, but given more
tests of edge conditions are always a good thing, I'm contributing
them upstream.

9fadf1c874

laanwj commented at 8:01 AM on April 28, 2015: member

ACK, thanks
laanwj added the label Tests on Apr 28, 2015
sipa commented at 11:47 AM on April 29, 2015: member

ACK
laanwj merged this on Apr 29, 2015
laanwj closed this on Apr 29, 2015
laanwj referenced this in commit df1609f314 on Apr 29, 2015
davecgh deleted the branch on May 11, 2015
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels