diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2018-01-11 08:37:25 -0500 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2018-01-11 08:37:29 -0500 |
| commit | 92a810d04b906722c9efe60e3997243c71ff3d4c (patch) | |
| tree | f90f002452f0e064d52a5e20b58821a5209ad873 /src/rest.cpp | |
| parent | 3c6286873e50248717afd7c56c664cee069c76fa (diff) | |
| parent | f765bb37880443061eb5f0a9cd5897e33732182d (diff) | |
| download | bitcoin-92a810d04b906722c9efe60e3997243c71ff3d4c.tar.xz | |
Merge #12150: Fix ListCoins test failure due to unset g_address_type, g_change_type
f765bb3788 Fix ListCoins test failure due to unset g_address_type, g_change_type (Russell Yanofsky)
Pull request description:
New global variables were introduced in #11403 and not setting them causes:
```
test_bitcoin: wallet/wallet.cpp:4199: CTxDestination GetDestinationForKey(const CPubKey&, OutputType): Assertion `false' failed.
unknown location(0): fatal error in "ListCoins": signal: SIGABRT (application abort requested)
```
It's possible to reproduce the failure reliably by running:
```
src/test/test_bitcoin --log_level=test_suite --run_test=wallet_tests/ListCoins
```
Failures happen nondeterministically because boost test framework doesn't run tests in a specified order, and tests that run previously can set the global variables and mask the bug.
Example travis failure: https://travis-ci.org/bitcoin/bitcoin/builds/327642495
Tree-SHA512: 3e0875716f66bc0304cf92a26457e6b54ecfe15ed962f4343577b05fc56bb577554422b7f53949ad6085ac5798ad7816b8176c5b01e050ddbfbb925d2732767a
Diffstat (limited to 'src/rest.cpp')
0 files changed, 0 insertions, 0 deletions