Skip to content

test: worker children in two more test files should exit with the test process #712

Closed

Description

@tps-flint

Follow-up to #693 / #705.

Two more test files park worker child processes whose teardown runs only in a finally, so a hard-killed or timed-out test run can leave those workers running:

  • packages/cli/test/bridge-ack-path.test.ts (workers waiting on release-… file gates, ~:294 and ~:309)
  • packages/cli/test/mail-existing-updates.test.ts (~:150)

The fix: start these workers so they exit when the test process goes away (the parent-pid and stdin watch from the live-process helper in #705, or an equivalent), and prove it with a test that hard-kills the owning process and asserts the workers are gone within a bounded deadline.

Do this after #705 merges, since it reuses that helper.

Activity

  1. moved this to Triage in Office queueon Nov 3, 2026
  2. self-assigned this
    on Nov 10, 2026
  3. moved this from Triage to In review in Office queueon Nov 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions