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

Gas Reports on Fuzz Tests #2751

Closed
0xfoobar opened this issue Aug 12, 2022 · 2 comments
Closed

Gas Reports on Fuzz Tests #2751

0xfoobar opened this issue Aug 12, 2022 · 2 comments
Labels
A-testing Area: testing C-forge Command: forge Cmd-forge-test Command: forge test T-feature Type: feature

Comments

@0xfoobar
Copy link

Component

Forge

Describe the feature you would like

Log gas reports for tests with fuzzing parameters. I have a codebase that's completely fuzz tests and it would be great to get the gas reports without having to rewrite everything with static method params.

Additional context

No response

@0xfoobar 0xfoobar added the T-feature Type: feature label Aug 12, 2022
@gakonst gakonst added this to Foundry Aug 12, 2022
@gakonst gakonst moved this to Todo in Foundry Aug 12, 2022
@mds1
Copy link
Collaborator

mds1 commented Aug 12, 2022

We should default to using deterministic seed when gas reports are enabled, similar to what is now done with snapshots as of #2591

@rkrasiuk rkrasiuk added A-testing Area: testing Cmd-forge-test Command: forge test C-forge Command: forge labels Aug 14, 2022
@onbjerg
Copy link
Member

onbjerg commented Aug 15, 2022

Duplicate of #1023

@onbjerg onbjerg marked this as a duplicate of #1023 Aug 15, 2022
@onbjerg onbjerg closed this as completed Aug 15, 2022
Repository owner moved this from Todo to Done in Foundry Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing Area: testing C-forge Command: forge Cmd-forge-test Command: forge test T-feature Type: feature
Projects
Archived in project
Development

No branches or pull requests

4 participants