diff options
| author | Jay Conrod <jayconrod@google.com> | 2021-02-09 17:32:08 -0500 |
|---|---|---|
| committer | Jay Conrod <jayconrod@google.com> | 2021-02-10 18:33:14 +0000 |
| commit | 25bd2e962e33d15922111464311c4a94ec910773 (patch) | |
| tree | 9585dd8ea2f58078275239839129fccc6f0d697a /src/internal/fuzz/encoding_test.go | |
| parent | 1b5cf71ccf0dc95f121830cfdad8280c4f6c1f28 (diff) | |
| download | go-25bd2e962e33d15922111464311c4a94ec910773.tar.xz | |
[dev.fuzz] testing: move inFuzzFn checks from common to F
inFuzzFn is set when the fuzz function is called. While it's set,
F methods that have side effects like Skip and Fail may not be called.
Previously, (CL 259657) inFuzzFn was in common, and we checked it in
the common implementation of those methods. This causes problems in
CL 290693 for recursive methods like common.Fail. If T.Fail is
called by the fuzz function, it calls common.Fail on the parent F's
common. That should not panic.
Change-Id: I841b12f77d9c77f5021370d03313e71b4ef50102
Reviewed-on: https://go-review.googlesource.com/c/go/+/290811
Trust: Jay Conrod <jayconrod@google.com>
Trust: Katie Hockman <katie@golang.org>
Reviewed-by: Katie Hockman <katie@golang.org>
Diffstat (limited to 'src/internal/fuzz/encoding_test.go')
0 files changed, 0 insertions, 0 deletions
