morpho-org / morpho-v1-optimizers-vaults

ERC4626 vaults to ease interaction with Morpho Optimizers.

Home Page:https://vaults.morpho.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[R&D review] Make sure that every case is tested

MathisGD opened this issue · comments

As seen with #152, we missed some tests. We must make sure that every case is tested, particularly with senders, receivers and owners (every case should be tested). Maybe with fuzzing

For example, removing this line in deposit mint withdraw and redeem does not break any test.
https://github.com/morpho-dao/morpho-tokenized-vaults/blob/6f182b04cf3326cf4244292f8533524236f0b773/src/compound/SupplyVaultBase.sol#L112