grandizzy

Results 349 comments of grandizzy

I run the test and can see 2 different values for fuzzed `UserOperation` input, e.g. in 2 different runs showing ``` [PASS] test_snapshot((bytes,bytes,uint256,uint256,uint256)) (runs: 256, μ: 417326, ~: 417135) [PASS]...

@cruzdanilo this happens due to fuzzing from mutated state values which are not deterministic, if adding locally ``` /// forge-config: default.fuzz.dictionary-weight = 0 function test_snapshot(UserOperation calldata userOp) external { ```...

@klkvr makes perfect sense, thanks! If you can push your local fix I can use it to test some others projects that were having similar issues

> @grandizzy pushed my local approach, Ig you'd have to drop your previous commit for it to actually take effect for snapshot @klkvr I tested https://github.com/foundry-rs/foundry/issues/5689#issuecomment-1705094827 and it looks good...

#6579 was done for custom errors within setup, will check the scenario you hit

yep, I managed to recreate with the project above, the error is `ErrorNotEnoughBalance` thrown by the `ERC20Bridged` contract https://basescan.org/address/0x69ce2505CE515C0203160450157366F927243309 which is not an artifact created during test so the abi...

I took the code for failing contract and run `forge inspect errors` on it, see https://book.getfoundry.sh/reference/cli/forge/inspect?#forge-inspect You can also use `forge selectors list` https://book.getfoundry.sh/reference/cli/forge/selectors/list (to get the code you can...

lgtm, pls address the nit above

that's correct, it is because bounds are logged in forge std https://github.com/foundry-rs/forge-std/blob/d44c4fbbb9ff054fb334babbdd34f9b6e899b3d6/src/StdUtils.sol#L62 https://github.com/foundry-rs/forge-std/blob/d44c4fbbb9ff054fb334babbdd34f9b6e899b3d6/src/StdUtils.sol#L87 @mattsse I am going to revert this commit for now as it looks it needs some more...

> So this PR now print console logs for every iteration of a fuzz set? I think previously we only printed them for the last successful run? Printing for every...