diff options
| author | Michael Pratt <mpratt@google.com> | 2025-01-22 17:18:19 -0500 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-05-19 11:00:01 -0700 |
| commit | 2c929d6f4c8fcd1021dc3cd57b2eedff5ae9a592 (patch) | |
| tree | bd5dc12ab00823fa1f4450eb838cf0eb4d2abfb8 /src/os/exec/exec_test.go | |
| parent | 5afada035ced1f89267d3177a9fb75fab4df81ff (diff) | |
| download | go-2c929d6f4c8fcd1021dc3cd57b2eedff5ae9a592.tar.xz | |
runtime: pass through -asan/-msan/-race to testprog tests
The tests using testprog / testprogcgo are currently not covered on the
asan/msan/race builders because they don't build testprog with the
sanitizer flag.
Explicitly pass the flag if the test itself is built with the sanitizer.
There were a few tests that explicitly passed -race (even on non-race
builders). These tests will now only run on race builders.
For #71395.
Cq-Include-Trybots: luci.golang.try:gotip-linux-amd64-asan-clang15,gotip-linux-amd64-msan-clang15,gotip-linux-amd64-race
Change-Id: I6a6a636ce8271246316a80d426c0e4e2f6ab99c5
Reviewed-on: https://go-review.googlesource.com/c/go/+/643897
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>
Diffstat (limited to 'src/os/exec/exec_test.go')
| -rw-r--r-- | src/os/exec/exec_test.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/os/exec/exec_test.go b/src/os/exec/exec_test.go index 8c62387193..3bded3dea6 100644 --- a/src/os/exec/exec_test.go +++ b/src/os/exec/exec_test.go @@ -683,7 +683,10 @@ func TestExtraFiles(t *testing.T) { // This test runs with cgo disabled. External linking needs cgo, so // it doesn't work if external linking is required. - testenv.MustInternalLink(t, false) + // + // N.B. go build below explictly doesn't pass through + // -asan/-msan/-race, so we don't care about those. + testenv.MustInternalLink(t, testenv.NoSpecialBuildTypes) if runtime.GOOS == "windows" { t.Skipf("skipping test on %q", runtime.GOOS) |