test: fix RPC coverage check #33064

1. 
   fanquake commented at 2:02 pm on July 25, 2025: member

   This is #27593 cleaned up / rebased, now that the legacy wallet has been dropped.

   Closes #27593.

2. DrahtBot added the label Tests on Jul 25, 2025
3. 
   DrahtBot commented at 2:02 pm on July 25, 2025: contributor

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

   Code Coverage & Benchmarks

   For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33064.

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	cedwies, maflcko

   If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #31668 (Added rescan option for import descriptors by saikiran57)

   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.

4. DrahtBot added the label CI failed on Jul 25, 2025
5. test: add abortrescan RPC test 2630b64f81
6. test: fix RPC coverage check 8aed477c33
7. fanquake force-pushed on Jul 26, 2025
8. fanquake marked this as ready for review on Jul 26, 2025
9. DrahtBot removed the label CI failed on Jul 26, 2025
10. 
    cedwies commented at 3:52 pm on July 27, 2025: none

    Tested on MacOS 15.5 (Debug build).

    • unit tests pass (0/143 failures)
    • ./test/functional/wallet_transactiontime_rescan.py passes (11 s)

    Question: would it make sense to also test “abortrescan” during an active rescan to hit the True path and ensure the scan halts as expected?

    ACK 8aed477

11. 
    maflcko commented at 8:41 am on July 28, 2025: member

    lgtm ACK 8aed477c3322212a636ab69d4923f89e2d9a63a2
12. 
    fanquake commented at 9:40 am on July 28, 2025: member

    Question: would it make sense to also test “abortrescan” during an active rescan to hit the True path and ensure the scan halts as expected?

    Feel free to open a new PR, adding additional test changes.

13. fanquake merged this on Jul 28, 2025
14. fanquake closed this on Jul 28, 2025

    ---

15. fanquake deleted the branch on Jul 28, 2025

Contributors
fanquake DrahtBot cedwies maflcko

Labels
Tests