Skip to content

Commit a538cf6

Browse files
authored
Merge pull request #450 from nblumhardt/extend-process-exit-timeout
Give test processes more time to exit cleanly
2 parents ec27789 + 1f25ced commit a538cf6

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

test/SeqCli.EndToEnd/Support/CaptiveProcess.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,12 @@ public int WaitForExit(TimeSpan? timeout = null)
111111

112112
if (_captureOutput)
113113
{
114-
if (!_outputComplete.WaitOne(TimeSpan.FromSeconds(5)))
115-
throw new IOException("STDOUT did not complete in the fixed 5-second window.");
114+
const int secondsWait = 15;
115+
if (!_outputComplete.WaitOne(TimeSpan.FromSeconds(secondsWait)))
116+
throw new IOException($"STDOUT did not complete in the fixed {secondsWait}-second window.");
116117

117-
if (!_errorComplete.WaitOne(TimeSpan.FromSeconds(5)))
118-
throw new IOException("STDERR did not complete in the fixed 5-second window.");
118+
if (!_errorComplete.WaitOne(TimeSpan.FromSeconds(secondsWait)))
119+
throw new IOException($"STDERR did not complete in the fixed {secondsWait}-second window.");
119120
}
120121

121122
return _process.ExitCode;

0 commit comments

Comments
 (0)