diff options
| author | Kir Kolyshkin <kolyshkin@gmail.com> | 2024-08-29 18:23:25 -0700 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2024-09-03 19:33:06 +0000 |
| commit | 76a42d74356e3c5bee0851c99665b21bf29f0c27 (patch) | |
| tree | 5703997880dff2c711e9c0217a702c59986ba44f /src/internal/testenv/exec.go | |
| parent | 995c816a7a9190db1ac0870cf2c424385b03ac4b (diff) | |
| download | go-76a42d74356e3c5bee0851c99665b21bf29f0c27.tar.xz | |
internal/testenv: add missing t.Helper calls
...and move a few so they won't be called when not needed.
Change-Id: I024b9552ed5ed839cde4fbae4815ec6ba8b67265
Reviewed-on: https://go-review.googlesource.com/c/go/+/609300
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/internal/testenv/exec.go')
| -rw-r--r-- | src/internal/testenv/exec.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/internal/testenv/exec.go b/src/internal/testenv/exec.go index 0e6a5f9a1a..9f21b323ab 100644 --- a/src/internal/testenv/exec.go +++ b/src/internal/testenv/exec.go @@ -32,6 +32,7 @@ import ( // for the resulting error. func MustHaveExec(t testing.TB) { if err := tryExec(); err != nil { + t.Helper() t.Skipf("skipping test: cannot exec subprocess on %s/%s: %v", runtime.GOOS, runtime.GOARCH, err) } } @@ -103,6 +104,7 @@ func MustHaveExecPath(t testing.TB, path string) { err, _ = execPaths.LoadOrStore(path, err) } if err != nil { + t.Helper() t.Skipf("skipping test: %s: %s", path, err) } } |