From: Bryan C. Mills <bcmills@google.com>
Date: Tue, 11 Jul 2023 14:08:48 +0000 (-0400)
Subject: os/exec: ignore context.Canceled errors in TestConcurrentExec
X-Git-Tag: go1.21rc3~2^2~8
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=cb7a091d729eab75ccfdaeba5a0605f05addf422;p=gostls13.git

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 <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>
---

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
 			}