qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-26 18:16:48 -07:00
Code Releases Activity

cmd/dist: drop remaining conditions on default cgo tests

Browse Source 
Currently, dist registers cmd/cgo/internal{test,testtls,testnocgo}
specially, so they're opted out of "go test std cmd". It has to
register these test packages to run in various non-default build
configurations, but at this point they can also run with the default
build configuration (and for test and testtls, we intentionally want
to test them in the default configuration; this is pointless but
harmless for testnocgo). Hence, this CL drops the special registration
of their default build configurations from registerCgoTests and lets
them be registered as part of registerStdTests.

Change-Id: Id283f3cdcdb202955a854648c0ed1e3c4aa554d1
Reviewed-on: https://go-review.googlesource.com/c/go/+/496179
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
This commit is contained in:
Austin Clements 2023-05-17 16:02:06 -04:00
parent cbd3305f8c
commit 3ebbaf3a4d
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/cmd/dist/test.go vendored
View File

@ -575,11 +575,6 @@ func (t *tester) registerTests() {
// whose test registration happens in a special way.
registerStdTestSpecially := map[string]bool{
"cmd/internal/testdir": true, // Registered at the bottom with sharding.
// cgo tests are registered specially because they involve unusual build
// conditions and flags.
"cmd/cgo/internal/test": true,
"cmd/cgo/internal/testnocgo": true,
"cmd/cgo/internal/testtls": true,
}
// Fast path to avoid the ~1 second of `go list std cmd` when
@ -1128,9 +1123,9 @@ func (t *tester) registerCgoTests(heading string) {
return gt
}
cgoTest("auto", "test", "auto", "")
cgoTest("auto", "testtls", "auto", "")
cgoTest("auto", "testnocgo", "auto", "")
// test, testtls, and testnocgo are run with linkmode="auto", buildmode=""
// as part of go test cmd. Here we only have to register the non-default
// build modes of these tests.
// Stub out various buildmode=pie tests on alpine until 54354 resolved.
builderName := os.Getenv("GO_BUILDER_NAME")