google / gofuzz

Fuzz testing for go.
Apache License 2.0
1.5k stars 119 forks source link

Fix testing methodology; passes accumulate now. #53

Closed lavalamp closed 4 years ago

lavalamp commented 4 years ago

go test -count=10000 . passes now.

I spot checked by introducing errors and the new tests continue to catch bugs. Running 10k tests as above caught the one place I messed up the fail/pass logic.

lavalamp commented 4 years ago

@dnephin @punkeel Would you like to review?

lavalamp commented 4 years ago

Added a commit to make all the tests work the same way, which has been bugging me since I wrote this library.

lavalamp commented 4 years ago

Thanks for looking, I'm gonna call that good enough, I can send a follow up if anyone finds a bug. :)