mirror of
https://github.com/golang/go
synced 2024-09-15 22:20:06 +00:00
cmd/cgo/internal/testsanitizers: skip some libfuzzer tests in short mode
These tests are newly-running now that we have installed a C++ toolchain on the clang builders, but one of them doesn't actually complete in short mode. For #67698. Change-Id: Id1ff4c05e380426d920277c13495f30e23561a4e Reviewed-on: https://go-review.googlesource.com/c/go/+/589295 Auto-Submit: Michael Knyszek <mknyszek@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
This commit is contained in:
parent
cad3ed207f
commit
1cf862e66c
|
@ -33,8 +33,9 @@ func TestLibFuzzer(t *testing.T) {
|
||||||
goSrc string
|
goSrc string
|
||||||
cSrc string
|
cSrc string
|
||||||
expectedError string
|
expectedError string
|
||||||
|
short bool
|
||||||
}{
|
}{
|
||||||
{goSrc: "libfuzzer1.go", expectedError: "panic: found it"},
|
{goSrc: "libfuzzer1.go", expectedError: "panic: found it", short: true},
|
||||||
{goSrc: "libfuzzer2.go", cSrc: "libfuzzer2.c", expectedError: "panic: found it"},
|
{goSrc: "libfuzzer2.go", cSrc: "libfuzzer2.c", expectedError: "panic: found it"},
|
||||||
}
|
}
|
||||||
for _, tc := range cases {
|
for _, tc := range cases {
|
||||||
|
@ -43,6 +44,11 @@ func TestLibFuzzer(t *testing.T) {
|
||||||
t.Run(name, func(t *testing.T) {
|
t.Run(name, func(t *testing.T) {
|
||||||
t.Parallel()
|
t.Parallel()
|
||||||
|
|
||||||
|
// Skip long-running tests in short mode.
|
||||||
|
if testing.Short() && !tc.short {
|
||||||
|
t.Skipf("%s can take upwards of minutes to run; skipping in short mode", name)
|
||||||
|
}
|
||||||
|
|
||||||
dir := newTempDir(t)
|
dir := newTempDir(t)
|
||||||
defer dir.RemoveAll(t)
|
defer dir.RemoveAll(t)
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue