diff options
author | MarcoFalke <falke.marco@gmail.com> | 2020-04-16 12:56:29 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2020-04-16 12:56:36 -0400 |
commit | d8dfcea5d956126fe0e2b1f36db74c8ef805e7ab (patch) | |
tree | 9074acfe2c29d835191562fb6a19ed7610524ec5 /src/test/script_tests.cpp | |
parent | f4c0ad4aefe0a59d629c415d0ceac63bf2c0bb58 (diff) | |
parent | bee88b8c5887e6beb75f26f0db97888a48fa7e7c (diff) | |
download | bitcoin-d8dfcea5d956126fe0e2b1f36db74c8ef805e7ab.tar.xz |
Merge #17669: tests: have coins simulation test also use CCoinsViewDB
bee88b8c5887e6beb75f26f0db97888a48fa7e7c tests: have coins simulation test also use CCoinsViewDB (James O'Beirne)
Pull request description:
Before this change, the coins simulation test uses a base view of type
CCoinsViewTest, which has no relevance outside of the unittest suite. Might as
well reuse this testcase with a more realistic configuration that has
CCoinsViewDB (i.e. in-memory leveldb) at the bottom of the view structure.
This adds explicit use of CCoinsViewDB in the unittest suite.
#### Before change
```
./src/test/test_bitcoin --run_test=coins_tests --catch_system_errors=no 21.99s user 0.04s system 99% cpu 22.057 total
```
#### After change
```
./src/test/test_bitcoin --run_test=coins_tests --catch_system_errors=no 78.80s user 0.04s system 100% cpu 1:18.82 total
```
ACKs for top commit:
ryanofsky:
Code review ACK bee88b8c5887e6beb75f26f0db97888a48fa7e7c
Tree-SHA512: 75296b2bcbae2f46e780489aafb032592544a15c384d569d016005692fe79fe60d7f05857cf25cc7b0f9ab1c53b47886a6c71cca074a03fb9afec30e1f376858
Diffstat (limited to 'src/test/script_tests.cpp')
0 files changed, 0 insertions, 0 deletions