-
Notifications
You must be signed in to change notification settings - Fork 1.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
Put fuzz test snapshots behind a flag #1081
Labels
T-feature
Type: feature
Comments
This was referenced Mar 28, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component
Forge
Describe the feature you would like
forge snapshot
currently exports the median and mean gas usage for fuzz teststhis makes using
forge snapshot
for gas optimizing near impossible, as the numbers for fuzz tests will vary wildly even if no actual change to the contract's gas usage was made (due to the random nature of the inputs)imo we should not show fuzz tests in the snapshots by default, and only show them if
--snapshot-fuzz-tests
is passed as a flag or a similar config option is setAdditional context
alternatively replace
forge snapshot
with #887 (my preferred choice)The text was updated successfully, but these errors were encountered: