Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to using native fuzz tests #346

Closed
joshuagl opened this issue Jul 20, 2022 · 3 comments
Closed

Switch to using native fuzz tests #346

joshuagl opened this issue Jul 20, 2022 · 3 comments

Comments

@joshuagl
Copy link
Member

#275 introduced some JSON unmarshal hardening and uses a fuzzer to generate test data for the hardening.

We should port the test to use the native fuzzer available in Go 1.18 (and newer) to reduce our third-party dependencies.

Originally posted by @Zenithar in #275 (comment)

@abs007
Copy link
Contributor

abs007 commented Sep 10, 2022

Does this mean importing the native fuzzer in place of the 3rd party fuzzer in the source code and then running go mod tidy?
(New to golang. Hence a bit verbose)

@joshuagl
Copy link
Member Author

I'm not familiar with the state of fuzzing in Go, but it might also be necessary to port the fuzzing logic to the API of the native fuzzer.

@rdimitrov
Copy link
Contributor

Closing since the code base changed and we no longer have the same tests.

Thanks for raising this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants