From cb7a091d729eab75ccfdaeba5a0605f05addf422 Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Tue, 11 Jul 2023 10:08:48 -0400 Subject: [PATCH] os/exec: ignore context.Canceled errors in TestConcurrentExec 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 Run-TryBot: Bryan Mills Auto-Submit: Bryan Mills Reviewed-by: Ian Lance Taylor --- src/os/exec/exec_test.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/os/exec/exec_test.go b/src/os/exec/exec_test.go index d37fffd39d..473f92ba8e 100644 --- a/src/os/exec/exec_test.go +++ b/src/os/exec/exec_test.go @@ -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 }