Backports #33001 to 29.x.
[29.x] test: Do not pass tests on unhandled exceptions #33046
pull fanquake wants to merge 3 commits into bitcoin:29.x from fanquake:29_x_unhandled_exceptions changing 2 files +4 −15-
fanquake commented at 4:03 PM on July 23, 2025: member
-
79e1a3c9c6
test: Do not pass tests on unhandled exceptions
This adds a missing catch for BaseException (e.g. SystemExit), which would otherwise be silently ignored. Also, remove the redundant other catches, which are just calling log.exception with a redundant log message. Github-Pull: #33001 Rebased-From: fa30b34026f76a5b8af997152fced2d281782e0d
-
5e327e6703
test: Log KeyboardInterrupt as exception
log.exception is more verbose and useful to debug timeouts. Also, log stderr for CalledProcessError to make debugging easier. Github-Pull: #33001 Rebased-From: faa3e684118bffa7a98cf76eeeb59243219df900
-
doc: update release notes for 29.x 411e15194b
- fanquake added this to the milestone 29.1 on Jul 23, 2025
- DrahtBot added the label Backport on Jul 23, 2025
-
DrahtBot commented at 4:03 PM on July 23, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33046.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK pablomartin4btc, marcofleon If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- pablomartin4btc approved
-
pablomartin4btc commented at 4:09 PM on July 23, 2025: member
ACK 411e15194b3a770ff455d413a0fe2495f0362297
- fanquake requested review from marcofleon on Jul 24, 2025
-
marcofleon commented at 10:51 AM on July 24, 2025: contributor
lgtm ACK 411e15194b3a770ff455d413a0fe2495f0362297
- fanquake merged this on Jul 24, 2025
- fanquake closed this on Jul 24, 2025
- fanquake deleted the branch on Jul 24, 2025
Contributors
Labels
Milestone
29.1