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
  1. theStack commented at 3:55 pm on November 26, 2025: contributor
    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.
  2. test: check for output to stdout in `TestShell` test 52230a7f69
  3. DrahtBot added the label Tests on Nov 26, 2025
  4. DrahtBot commented at 3:55 pm on November 26, 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/33951.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK maflcko, rkrux

    If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

  5. maflcko commented at 8:44 am on November 27, 2025: member
    lgtm ACK 52230a7f697fd99abdc4550d6a60737be024e246
  6. rkrux approved
  7. rkrux commented at 9:00 am on November 27, 2025: contributor

    crACK 52230a7f697fd99abdc4550d6a60737be024e246

    I didn’t know about TestShell, looks quite handy!

  8. fanquake merged this on Nov 27, 2025
  9. fanquake closed this on Nov 27, 2025

  10. theStack deleted the branch on Nov 27, 2025

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-12-23 03:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me