From dd87e33d88eadaf5302895fb91b0a0e2e16b136e Mon Sep 17 00:00:00 2001 From: cui fliter Date: Fri, 18 Nov 2022 14:06:56 +0800 Subject: os: use testenv.Command instead of exec.Command in tests testenv.Command sets a default timeout based on the test's deadline and sends SIGQUIT (where supported) in case of a hang. Change-Id: I84d469d12ca1f39175b713ca41e7cd2202d11d96 Reviewed-on: https://go-review.googlesource.com/c/go/+/451895 Auto-Submit: Bryan Mills TryBot-Result: Gopher Robot Run-TryBot: Bryan Mills Reviewed-by: Bryan Mills Reviewed-by: Ian Lance Taylor --- src/os/exec/exec_test.go | 2 +- src/os/exec/exec_windows_test.go | 3 ++- src/os/exec/lp_windows_test.go | 4 ++-- 3 files changed, 5 insertions(+), 4 deletions(-) (limited to 'src/os/exec') diff --git a/src/os/exec/exec_test.go b/src/os/exec/exec_test.go index 67e2d256b4..c2f643a645 100644 --- a/src/os/exec/exec_test.go +++ b/src/os/exec/exec_test.go @@ -752,7 +752,7 @@ func TestExtraFiles(t *testing.T) { tempdir := t.TempDir() exe := filepath.Join(tempdir, "read3.exe") - c := exec.Command(testenv.GoToolPath(t), "build", "-o", exe, "read3.go") + c := testenv.Command(t, testenv.GoToolPath(t), "build", "-o", exe, "read3.go") // Build the test without cgo, so that C library functions don't // open descriptors unexpectedly. See issue 25628. c.Env = append(os.Environ(), "CGO_ENABLED=0") diff --git a/src/os/exec/exec_windows_test.go b/src/os/exec/exec_windows_test.go index b39790d61a..efd37107ed 100644 --- a/src/os/exec/exec_windows_test.go +++ b/src/os/exec/exec_windows_test.go @@ -8,6 +8,7 @@ package exec_test import ( "fmt" + "internal/testenv" "io" "os" "os/exec" @@ -69,7 +70,7 @@ func TestPipePassing(t *testing.T) { func TestNoInheritHandles(t *testing.T) { t.Parallel() - cmd := exec.Command("cmd", "/c exit 88") + cmd := testenv.Command(t, "cmd", "/c exit 88") cmd.SysProcAttr = &syscall.SysProcAttr{NoInheritHandles: true} err := cmd.Run() exitError, ok := err.(*exec.ExitError) diff --git a/src/os/exec/lp_windows_test.go b/src/os/exec/lp_windows_test.go index d797b6c53c..50d522948a 100644 --- a/src/os/exec/lp_windows_test.go +++ b/src/os/exec/lp_windows_test.go @@ -164,7 +164,7 @@ func (test lookPathTest) run(t *testing.T, tmpdir, printpathExe string) { // Run "cmd.exe /c test.searchFor" with new environment and // work directory set. All candidates are copies of printpath.exe. // These will output their program paths when run. - should, errCmd := test.runProg(t, env, exec.Command("cmd", "/c", test.searchFor)) + should, errCmd := test.runProg(t, env, testenv.Command(t, "cmd", "/c", test.searchFor)) // Run the lookpath program with new environment and work directory set. have, errLP := test.runProg(t, env, helperCommand(t, "lookpath", test.searchFor)) // Compare results. @@ -572,7 +572,7 @@ func buildPrintPathExe(t *testing.T, dir string) string { t.Fatalf("failed to execute template: %v", err) } outname := name + ".exe" - cmd := exec.Command(testenv.GoToolPath(t), "build", "-o", outname, srcname) + cmd := testenv.Command(t, testenv.GoToolPath(t), "build", "-o", outname, srcname) cmd.Dir = dir out, err := cmd.CombinedOutput() if err != nil { -- cgit v1.3