coinbase / coinbase/smart-wallet
Improve fuzz testing CI
Open
- Dominant language
- Solidity
- Stars
- 487
- Forks
- 176
- PR merge metrics
- No merged PRs in 30d
Description
Fuzz testing sometimes has noisy gas difference, leading to failures of `forge snapshot --check`, we currently solve this with a 1% tolerance which isn't ideal https://github.com/coinbase/smart-wallet/pull/66/files#r1576297082
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing the CI configuration for `forge snapshot --check` and the discussion linked from the issue. Reproduce the fuzz-testing gas-difference failures and determine a way for expected noise not to cause inappropriate failures; done means the CI check remains meaningful without relying on the current 1% tolerance.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- solidity
- Domain
- ci-cd, testing
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100