openzeppelin-contracts icon indicating copy to clipboard operation
openzeppelin-contracts copied to clipboard

Remove pseudo-fuzzing tests in hardhat in favor of foundry fuzzing.

Open Amxx opened this issue 1 year ago • 3 comments

Some unit tests are using loops generating random values. In some cases, this could be replaced with proper fuzzing. There are even cases where the fuzzing tests have already been written.

I propose we remove the random and/or looped unit tests when an equivalent fuzzing tests exists.

Amxx avatar Feb 13 '24 17:02 Amxx

@Amxx If you are looking for someone to work on this. I would like to contribute.

allwin199 avatar Feb 13 '24 17:02 allwin199

@allwin199 Thank you for the proposal.

We don't want to take any PR yet. You could help however by identifying tests that may be candidate for removal.

Amxx avatar Feb 13 '24 19:02 Amxx

Sure!

allwin199 avatar Feb 14 '24 03:02 allwin199