-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
[dev.fuzz] cmd/go: add fuzz timeout flag #46220
Comments
Will https://golang.org/pkg/testing/#T.Deadline be supported? |
EDIT: |
Change https://golang.org/cl/330509 mentions this issue: |
It would be nice to support this for 1.18, but it's not a release blocker. We'll see if we get to it. |
#48157 is a proposal to add a flag and a T method for per-test timeouts. We'll wait for that to be resolved. If approved, we'll use the same flag and method. |
We may want to consider adding a
-fuzztimeout flag
, which acts similar to a -timeout flag, but for a run of a single input to thef.Fuzz
function.If -fuzztimeout=100ms, then we give each execution of the fuzz function 100 milliseconds to run, and consider it a crasher if that execution takes longer than 100ms to complete.
Note that libFuzzer does this with a -timeout flag.
We could also consider retrofitting the existing -timeout flag, but that might cause confusion. The description for timeout is "If a test binary runs longer than duration d, panic." If someone sets timeout while fuzzing, we don't want to panic when we hit that timeout (which we will if there is no crash).
Related to #44483.
The text was updated successfully, but these errors were encountered: