From: Katie Hockman Date: Fri, 15 Jan 2021 16:26:34 +0000 (-0500) Subject: [dev.fuzz] cmd/go/testdata: fix flaky test X-Git-Tag: go1.18beta1~1282^2~105 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=d45df5de32e555a0386b7e473d30516d744df70a;p=gostls13.git [dev.fuzz] cmd/go/testdata: fix flaky test Change-Id: I7702aa12a1ed9bb0645af774dd584e661d7c8fa5 Reviewed-on: https://go-review.googlesource.com/c/go/+/284193 Trust: Katie Hockman Run-TryBot: Katie Hockman TryBot-Result: Go Bot Reviewed-by: Jay Conrod --- diff --git a/src/cmd/go/testdata/script/test_fuzz_mutator.txt b/src/cmd/go/testdata/script/test_fuzz_mutator.txt index f858dcf354..bc7edf3cc5 100644 --- a/src/cmd/go/testdata/script/test_fuzz_mutator.txt +++ b/src/cmd/go/testdata/script/test_fuzz_mutator.txt @@ -14,7 +14,7 @@ go run check_logs.go fuzz fuzz.worker ! go test -v -fuzz=Fuzz -parallel=1 -fuzztime=30s mutator_test.go ! stdout ok stdout FAIL -stdout 'mutator found enough edge cases' +stdout 'mutator found enough unique mutations' -- go.mod -- module m @@ -162,7 +162,6 @@ func checkWorkerLog(r io.Reader) error { package fuzz_test import ( - "strings" "testing" ) @@ -175,22 +174,9 @@ func Fuzz(f *testing.F) { crashes := make(map[string]bool) // No seed corpus initiated f.Fuzz(func(t *testing.T, b []byte) { - if len(crashes) >= 150 { - panic("mutator found enough edge cases") - } - - if len(b) < 5 { - return // continue - } - - for i := 0; i < 256; i++ { - s := string(byte(i)) - if strings.HasPrefix(string(b), s) { - crashes["pre-" + s] = true - } - if strings.HasSuffix(string(b), s) { - crashes["suffix-" + s] = true - } + crashes[string(b)] = true + if len(crashes) >= 1000 { + panic("mutator found enough unique mutations") } }) } \ No newline at end of file