Skip to content

Commit ca49b8d

Browse files
author
Bryan C. Mills
committed
cmd/go: remove a redundant HasExec check in TestScript/README
The test checks testenv.HasExec right before calling testenv.Command, but testenv.Command already skips the test if HasExec is false. Change-Id: I04716bf823a1b76640394a964973ff34c65afb1f Reviewed-on: https://go-review.googlesource.com/c/go/+/491658 Run-TryBot: Bryan Mills <[email protected]> Auto-Submit: Bryan Mills <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Michael Matloob <[email protected]>
1 parent 3a872a2 commit ca49b8d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

src/cmd/go/scriptreadme_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,6 @@ func checkScriptReadme(t *testing.T, engine *script.Engine, env []string) {
3737
}
3838
args.Conditions = conds.String()
3939

40-
if !testenv.HasExec() {
41-
t.Skipf("updating script README requires os/exec")
42-
}
43-
4440
doc := new(strings.Builder)
4541
cmd := testenv.Command(t, testGo, "doc", "cmd/go/internal/script")
4642
cmd.Env = env

0 commit comments

Comments
 (0)