qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-09-29 08:14:29 -06:00
Code Releases Activity

os/exec: ignore context.Canceled errors in TestConcurrentExec

Browse Source 
We cancel the Context to unblock the test as soon as all of the "exit"
processes have completed. If that happens to occur before all of the
"hang" processes have started, the Start calls may fail with
context.Canceled.

Since those errors are possible in normal operation of the test,
ignore them.

Fixes #61277.
Updates #61080.

Change-Id: I20db083ec89ca88eb085ceb2892b9f87f83705ac
Reviewed-on: https://go-review.googlesource.com/c/go/+/508755
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Bryan Mills <bcmills@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
This commit is contained in:
Bryan C. Mills 2023-07-11 10:08:48 -04:00 committed by Gopher Robot
parent 651869716a
commit cb7a091d72
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/os/exec/exec_test.go
View File

@ -1754,7 +1754,9 @@ func TestConcurrentExec(t *testing.T) {
ready.Wait()
if err := cmd.Start(); err != nil {
t.Error(err)
if !errors.Is(err, context.Canceled) {
t.Error(err)
}
return
}