diff options
| author | Michael Anthony Knyszek <mknyszek@google.com> | 2025-09-11 23:48:04 +0000 |
|---|---|---|
| committer | Michael Knyszek <mknyszek@google.com> | 2025-09-15 14:21:33 -0700 |
| commit | 004858ccddb2f092d3413e96ba8526a8df8e3906 (patch) | |
| tree | 4dd8105c7243cd66c3850ba405b4747995088b65 /src/cmd/cgo/internal/testplugin/plugin_test.go | |
| parent | dbde15800c97e425f4ad1b47c1ee0c302f963fb5 (diff) | |
| download | go-004858ccddb2f092d3413e96ba8526a8df8e3906.tar.xz | |
all: replace os.Getenv("GO_BUILDER_NAME") with testenv.Builder in tests
Some tests still reach for GO_BUILDER_NAME directly. This change makes
it so that they go through testenv.Builder.
There are a couple more, but changing them may also cause tests to start
failing. Done in a follow-up.
Change-Id: Id2453b7b62f5ebf3594e92fa53724a577a97440f
Reviewed-on: https://go-review.googlesource.com/c/go/+/703135
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/cmd/cgo/internal/testplugin/plugin_test.go')
| -rw-r--r-- | src/cmd/cgo/internal/testplugin/plugin_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/cgo/internal/testplugin/plugin_test.go b/src/cmd/cgo/internal/testplugin/plugin_test.go index 5bff81092c..2afb542ec4 100644 --- a/src/cmd/cgo/internal/testplugin/plugin_test.go +++ b/src/cmd/cgo/internal/testplugin/plugin_test.go @@ -46,7 +46,7 @@ func prettyPrintf(format string, args ...interface{}) { } func testMain(m *testing.M) int { - if testing.Short() && os.Getenv("GO_BUILDER_NAME") == "" { + if testing.Short() && testenv.Builder() == "" { globalSkip = func(t *testing.T) { t.Skip("short mode and $GO_BUILDER_NAME not set") } return m.Run() } |