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

Add Fuzz test for #90 #9

Closed
jeromedoucet opened this issue Sep 30, 2022 · 3 comments
Closed

Add Fuzz test for #90 #9

jeromedoucet opened this issue Sep 30, 2022 · 3 comments
Labels
community mistake Mistakes proposed by the community

Comments

@jeromedoucet
Copy link

Not considering Fuzz test

Since go 1.18, this kind of tests may help us to find out missing edge case.

Solution

Use it ;) .

@jeromedoucet jeromedoucet added the community mistake Mistakes proposed by the community label Sep 30, 2022
@teivah
Copy link
Owner

teivah commented Sep 30, 2022

@jeromedoucet Do you have a specific context in which fuzzing should be applied?

@jeromedoucet
Copy link
Author

jeromedoucet commented Sep 30, 2022

@teivah Yes. At least two :

1 fuzzing is a great help to achieve building a golden master testing strategy, when dealing with an untested piece of spaghetti code for instance.

2 Where the code must deals with "external" (and then uncontrolled) input with "complex" checks. It may be a HTTP handler or a database driver or something like that. In such piece of code, the controls over inputs may be critical, and fuzzing may help us to find bugs.

That's the uses case I have for the moment ;)

@teivah
Copy link
Owner

teivah commented Jun 6, 2023

Added, thanks @jeromedoucet 🙏

@teivah teivah closed this as completed Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community mistake Mistakes proposed by the community
Projects
None yet
Development

No branches or pull requests

2 participants