From 7e4191e686c192b9c85065fd3b8775190d3e9c5f Mon Sep 17 00:00:00 2001 From: scott Date: Fri, 11 Nov 2022 03:55:08 +0000 Subject: [PATCH] os/exec: fix TestWaitInterrupt/WaitDelay error message As the comments say. Here we expect err to be nil instead of ctx.Err() Change-Id: I4cd02d62ac0a13c9577a567de36742f13d140d36 GitHub-Last-Rev: 6bedfbc9d2511140d088dc9ee5f40015725f68db GitHub-Pull-Request: golang/go#56698 Reviewed-on: https://go-review.googlesource.com/c/go/+/449737 Reviewed-by: Bryan Mills Run-TryBot: Ian Lance Taylor Reviewed-by: Ian Lance Taylor TryBot-Result: Gopher Robot Auto-Submit: Ian Lance Taylor --- src/os/exec/exec_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/os/exec/exec_test.go b/src/os/exec/exec_test.go index cb1545a651..edff9a201e 100644 --- a/src/os/exec/exec_test.go +++ b/src/os/exec/exec_test.go @@ -1357,7 +1357,7 @@ func TestWaitInterrupt(t *testing.T) { // context expired, a successful exit is valid (even if late) and does // not merit a non-nil error. if err != nil { - t.Errorf("Wait: %v; want %v", err, ctx.Err()) + t.Errorf("Wait: %v; want nil", err) } if ps := cmd.ProcessState; !ps.Exited() { t.Errorf("cmd did not exit: %v", ps) -- 2.50.0