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.

Sign up for GitHub

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

Jump to bottom

feat(fuzz): do not populate dictionary with bytecode metadata #8115

Open
grandizzy opened this issue Jun 8, 2024 · 0 comments · May be fixed by #8139
Open

feat(fuzz): do not populate dictionary with bytecode metadata #8115

grandizzy opened this issue Jun 8, 2024 · 0 comments · May be fixed by #8139
Labels
A-testing Area: testing C-forge Command: forge T-feature Type: feature
Milestone
v1.0.0

Comments

@grandizzy
Copy link
Collaborator

grandizzy commented Jun 8, 2024 •
edited
Loading

Component

Forge

Describe the feature you would like

We currently collect push bytes from contracts and populate fuzz dictionary with (see #1168). However there's no need to use bytecode metadata as it is not relevant and it only adds noise to dict values.
Additionally, any comments added or removed from test contract affects gas snapshots.

Additional context

No response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-testing Area: testing C-forge Command: forge T-feature Type: feature
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

[WIP] fix(fuzz): strip metadata when collecting push bytes grandizzy/foundry
2 participants
@grandizzy @zerosnacks