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

Disable Fuzz testing #336

Closed
niconiconi opened this issue Jul 13, 2022 · 1 comment
Closed

Disable Fuzz testing #336

niconiconi opened this issue Jul 13, 2022 · 1 comment

Comments

@niconiconi
Copy link
Contributor

Fuzz testing tend to feed invalid input to the proof system,

For example, when implementing integer division with Hint functions, the fuzz testing tends to feed 0 to the divisor and causing divided by zero error. (Panic in the hint function.)

A hint function failure should be a failed proof instead of directly terminate the Go program.

Is there any way to disable fuzz testing?

@ivokub
Copy link
Collaborator

ivokub commented Jul 13, 2022

There is a draft PR #296 which implements the testing option to disable fuzzing. The long-term idea is to remove fuzzing from the assert.ProverSucceeded method and implement a different test suite.

I rebased the PR on top of develop, so if you just run your tests on top of the feat/nofuzzing branch with test.NoFuzzing() test option, then it should skip the fuzzing tests for now.

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

No branches or pull requests

2 participants