Add timeout to testDoesNotReapUnrelatedChildProcess() (#267)

broken-circle · web-flow · commit 370f301ad0d7 · 2026-06-03T11:37:18.000-07:00
Add a timeout to prevent non-deterministically hangs which consume the
full CI job timeout with no actionable signal.
diff --git a/Tests/SubprocessTests/ProcessMonitoringTests.swift b/Tests/SubprocessTests/ProcessMonitoringTests.swift
@@ -229,7 +229,8 @@ extension SubprocessProcessMonitoringTests {
         }
     }
 
-    @Test func testDoesNotReapUnrelatedChildProcess() async throws {
+    @Test(.timeLimit(.minutes(1)))
+    func testDoesNotReapUnrelatedChildProcess() async throws {
         // Make sure we don't reap child exit status that we didn't spawn
         let child1 = self.immediateExitProcess(withExitCode: 0)
         let child2 = self.immediateExitProcess(withExitCode: 0)

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,8 @@ extension SubprocessProcessMonitoringTests {`
`229`	`229`	`}`
`230`	`230`	`}`
`231`	`231`
`232`		`- @Test func testDoesNotReapUnrelatedChildProcess() async throws {`
	`232`	`+ @Test(.timeLimit(.minutes(1)))`
	`233`	`+ func testDoesNotReapUnrelatedChildProcess() async throws {`
`233`	`234`	`// Make sure we don't reap child exit status that we didn't spawn`
`234`	`235`	`let child1 = self.immediateExitProcess(withExitCode: 0)`
`235`	`236`	`let child2 = self.immediateExitProcess(withExitCode: 0)`