diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-12-09 09:24:19 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-12-09 09:24:22 +0100 |
| commit | 529ed3336205e30d94e741a4eb93dd945ad518e7 (patch) | |
| tree | fd456921be0ca66e4043d58dce8085b29ff0ac2e /src/rpc/blockchain.cpp | |
| parent | 68ca8677e039a60b9f51938987d784cb58a3c995 (diff) | |
| parent | 84bc35d7a5b91c50cb58ff844e7fc7d9f026cc76 (diff) | |
Merge bitcoin/bitcoin#23715: test: feature_rbf.py: check specified wallet type availability
84bc35d7a5b91c50cb58ff844e7fc7d9f026cc76 test: feature_rbf.py: check specified wallet type availability (Sebastian Falbesoner)
Pull request description:
The test currently leads to a failure if in general wallet support is compiled, but the library for the specified type (BDB/SQLite) is not, i.e. if started with the `--legacy-wallet` parameter, but bitcoind is compiled without BDB support, see e.g. https://github.com/bitcoin/bitcoin/pull/23682#issuecomment-989044207
Fix this by checking if the specified wallet type (BDB for legacy wallet, SQLite for descriptor wallet) is available.
Also move the helper `is_specified_wallet_compiled()` to the
test framework's class BitcoinTestFramework first, so it can be reused.
Should further pave the way for #23682. On my local instance without BDB compiled, all targets in test_runner pass now.
ACKs for top commit:
mzumsande:
ACK 84bc35d7a5b91c50cb58ff844e7fc7d9f026cc76, changes loook good for me and I confirmed that this fixes the error encountered on master when running the test `--without-bdb`.
Tree-SHA512: 1575c03c793c8e0ac195d0914eff75d02604301c8fb77d0fdb7c0b245561569c0c7db387ef4de499044b68ab6e14b4b78b955f6e74c84197bcaed95f439c9824
Diffstat (limited to 'src/rpc/blockchain.cpp')
0 files changed, 0 insertions, 0 deletions