fix: compilers warnings

AngleProtocol · Jul 23, 2024 · be3ca53 · be3ca53
1 parent 4730bec
commit be3ca53
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 11 deletions.
diff --git a/test/fuzz/Redeem.t.sol b/test/fuzz/Redeem.t.sol
@@ -19,10 +19,6 @@ contract RedeemFuzzTest is ERC4626StrategyTest {
 
         for (uint256 i = 0; i < 5; i++) {
             uint256 previousBalance = strategy.balanceOf(alice);
-
-            uint256 totalAssets = strategy.totalAssets();
-            uint256 lastTotalAssets = strategy.lastTotalAssets();
-
             uint256 previousAssetBalance = IERC20(asset).balanceOf(alice);
 
             vm.startPrank(alice);

diff --git a/test/fuzz/Swap.t.sol b/test/fuzz/Swap.t.sol
@@ -67,7 +67,7 @@ contract SwapFuzzTest is ERC4626StrategyTest {
                     .zeroFloorSub(strategy.lockedProfit())
             );
 
-            uint256 lastLockedProfit = strategy.lockedProfit();
+            lastLockedProfit = strategy.lockedProfit();
 
             vm.warp(block.timestamp + timeOffsets[i]);
         }

diff --git a/test/unit/Accumulate.t.sol b/test/unit/Accumulate.t.sol
@@ -35,8 +35,6 @@ contract AccumulateTest is ERC4626StrategyTest {
         IERC20(asset).approve(address(strategy), 100e18);
         strategy.deposit(100e18, alice);
         vm.stopPrank();
-        uint256 balance = ERC4626(strategyAsset).balanceOf(address(strategy));
-        uint256 lastTotalAssets = strategy.lastTotalAssets();
         vm.warp(block.timestamp + 1 weeks);
 
         vm.mockCall(strategyAsset, abi.encodeWithSelector(IERC20.balanceOf.selector), abi.encode(9e18));

diff --git a/test/unit/Constructor.t.sol b/test/unit/Constructor.t.sol
@@ -4,7 +4,7 @@ pragma solidity 0.8.26;
 import "../ERC4626StrategyTest.t.sol";
 
 contract ConstructorTest is ERC4626StrategyTest {
-    function test_constructor_Normal() public {
+    function test_constructor_Normal() public view {
         assertEq(strategy.vestingPeriod(), 1 weeks);
         assertEq(strategy.performanceFee(), 10_000);
         assertEq(strategy.developerFee(), 20_000);
@@ -19,7 +19,7 @@ contract ConstructorTest is ERC4626StrategyTest {
         assertEq(strategy.lockedProfit(), 0);
     }
 
-    function test_constructor_CorrectRoles() public {
+    function test_constructor_CorrectRoles() public view {
         assertTrue(strategy.hasRole(strategy.DEVELOPER_ROLE(), developer));
         assertTrue(strategy.hasRole(strategy.INTEGRATOR_ROLE(), integrator));
         assertTrue(strategy.hasRole(strategy.KEEPER_ROLE(), keeper));

diff --git a/test/unit/Deposit.t.sol b/test/unit/Deposit.t.sol
@@ -67,7 +67,6 @@ contract DepositTest is ERC4626StrategyTest {
 
         vm.mockCall(strategyAsset, abi.encodeWithSelector(IERC20.balanceOf.selector), abi.encode(9e18));
         uint256 previewedDeposit = strategy.previewDeposit(100e18);
-        uint256 previousBalance = ERC4626(strategyAsset).balanceOf(address(strategy));
 
         vm.prank(alice);
         vm.mockCall(strategyAsset, abi.encodeWithSelector(IERC20.balanceOf.selector), abi.encode(9e18));

diff --git a/test/unit/MaxMint.t.sol b/test/unit/MaxMint.t.sol
@@ -4,7 +4,7 @@ pragma solidity 0.8.26;
 import "../ERC4626StrategyTest.t.sol";
 
 contract MaxMintTest is ERC4626StrategyTest {
-    function test_MaxMint_Normal() public {
+    function test_MaxMint_Normal() public view {
         assertEq(strategy.maxMint(alice), ERC4626(strategyAsset).maxMint(address(strategy)));
     }