diff options
| author | fanquake <[email protected]> | 2020-01-29 13:11:58 +0800 |
|---|---|---|
| committer | fanquake <[email protected]> | 2020-01-30 08:41:24 +0800 |
| commit | 1b96a3cd1ebe725896f59614903184289fe62cf8 (patch) | |
| tree | b4f053b219434d6631f2fd5f108b5467fc7c412a /src/validationinterface.cpp | |
| parent | Merge #18022: test: Fix appveyor test_bitcoin build of *.raw (diff) | |
| download | discoin-1b96a3cd1ebe725896f59614903184289fe62cf8.tar.xz discoin-1b96a3cd1ebe725896f59614903184289fe62cf8.zip | |
tests: reset fIsBareMultisigStd after bare-multisig tests
The bug this fixes is two-part.
1.The fIsBareMultisigStd global is being reused by other tests,
i.e script_p2sh_tests(set), after being set to false.
2. The order our tests run in doesn't always? seem to be random,
which meant that the script_p2sh tests would only fail if they
were run in an order where transaction_tests ran first, mutating
the fIsBareMultisigStd global.
This doesn't seem to happen when running make check, but if you
run src/test/test_bitcoin and pass --random=99999, the failure
in script_p2sh:
test/script_p2sh_tests.cpp:200: error: in "script_p2sh_tests/set": txTo[1].IsStandard
will occur (on most systems).
The new test was introduced in 1bb5d517aa616c1d5b5801d2ea36a2de5fb61eba.
Diffstat (limited to 'src/validationinterface.cpp')
0 files changed, 0 insertions, 0 deletions