test: check for importprunedfunds RPC errors #24510

pull theStack wants to merge 1 commits into bitcoin:master from theStack:202203-test-check_for_importprunedfunds_RPC_errors changing 1 files +19 −1

theStack commented at 3:21 PM on March 9, 2022: member

This PR adds missing test coverage for the following errors of the importprunedfunds RPC: https://github.com/bitcoin/bitcoin/blob/7003b6ab24f6adfffd71d7b7d4182afde52ff859/src/wallet/rpc/backup.cpp#L320-L322 https://github.com/bitcoin/bitcoin/blob/7003b6ab24f6adfffd71d7b7d4182afde52ff859/src/wallet/rpc/backup.cpp#L332-L334 https://github.com/bitcoin/bitcoin/blob/7003b6ab24f6adfffd71d7b7d4182afde52ff859/src/wallet/rpc/backup.cpp#L338-L340 https://github.com/bitcoin/bitcoin/blob/7003b6ab24f6adfffd71d7b7d4182afde52ff859/src/wallet/rpc/backup.cpp#L343-L345
test: check for importprunedfunds RPC errors 7573789925
fanquake added the label Tests on Mar 9, 2022
DrahtBot commented at 6:54 AM on March 11, 2022: member


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #20892 (tests: Run both descriptor and legacy tests within a single test invocation by achow101)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
MarcoFalke commented at 3:03 PM on March 15, 2022: member

review ACK 7573789925f6b5c3d73f43d2ebf8c732945c1824
MarcoFalke merged this on Mar 15, 2022
MarcoFalke closed this on Mar 15, 2022
theStack deleted the branch on Mar 15, 2022
sidhujag referenced this in commit 24cf00ba5b on Mar 16, 2022
DrahtBot locked this on Mar 15, 2023

Contributors

Labels

test: check for importprunedfunds RPC errors #24510

Conflicts