This is a small follow-up PR to the recently added TestShell test (#33546), verifying the stdout message "TestShell is already running!" when trying to instantiate a second instance.
test: check for output to stdout in `TestShell` test #33951
pull theStack wants to merge 1 commits into bitcoin:master from theStack:202511-test-TestShell-check-stdout changing 1 files +7 −2-
theStack commented at 3:55 PM on November 26, 2025: contributor
-
test: check for output to stdout in `TestShell` test 52230a7f69
- DrahtBot added the label Tests on Nov 26, 2025
-
DrahtBot commented at 3:55 PM on November 26, 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/33951.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
maflcko commented at 8:44 AM on November 27, 2025: member
lgtm ACK 52230a7f697fd99abdc4550d6a60737be024e246
- rkrux approved
-
rkrux commented at 9:00 AM on November 27, 2025: contributor
crACK 52230a7f697fd99abdc4550d6a60737be024e246
I didn't know about TestShell, looks quite handy!
- fanquake merged this on Nov 27, 2025
- fanquake closed this on Nov 27, 2025
- theStack deleted the branch on Nov 27, 2025
Labels