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

Improve contracts coverage

Open a17 opened this issue 1 year ago • 0 comments

Here we collect all the uncovered areas in the code base with a description of how to cover them.

Core

  • ...

Strategies

  • DefiEdgeQuickSwapMerklFarmStrategy
    • [ ] _normalise, _getChainlinkPrice: test with asset that have decimals > 18
    • [ ] _calculateShares: test with totalShares == 0 DefiEdge underlying ALM or make such underlyings unsupportable
    • [ ] _getPriceInUSD: test with not _isBase asset that need chainlink price
  • GammaQuickSwapMerklFarmStrategy
    • [ ] _previewDepositAssets: try to write fuzzing test or something like it
    • [ ] _assetsAmounts: read on empty vault in test
  • GammaRetroMerklFarmStrategy
    • [ ] initVariants: test with not active farm
    • [ ] _previewDepositAssets: try to write fuzzing test or something like it
    • [ ] _assetsAmounts: read on empty vault in test
  • IchiQuickSwapMerklFarmStrategy
    • [ ] getAssetsProportions: test with other underlyings
  • ...

a17 avatar May 15 '24 16:05 a17