fix: strategy unit tests

Author: ypatil12

src/test/mocks/ERC20Mock.sol

@@ -46,6 +46,13 @@ contract ERC20Mock is Context, IERC20 {
     constructor() {
         _mint(msg.sender, initSupply);
     }
+
+    /**
+     * @dev Returns the decimals places of the token.
+     */
+    function decimals() external pure returns (uint8) {
+        return 18;
+    }
 
     /**
      * @dev See {IERC20-totalSupply}.

src/test/mocks/MockDecimals.sol

@@ -0,0 +1,8 @@
+// SPDX-License-Identifier: BUSL-1.1
+pragma solidity ^0.8.12;
+
+contract MockDecimals {
+    function decimals() public pure returns (uint8) {
+        return 18;
+    }
+}

src/test/mocks/Reverter.sol

@@ -2,8 +2,13 @@
 pragma solidity ^0.8.9;
 
 contract Reverter {
-
     fallback() external {
         revert("Reverter: I am a contract that always reverts");
     }
+}
+
+contract ReverterWithDecimals is Reverter {
+    function decimals() external pure returns (uint8) {
+        return 18;
+    }
 }

src/test/unit/StrategyManagerUnit.t.sol

@@ -9,6 +9,7 @@ import "src/test/mocks/ERC20Mock.sol";
 import "src/test/mocks/ERC20_SetTransferReverting_Mock.sol";
 import "src/test/mocks/Reverter.sol";
 import "src/test/mocks/Reenterer.sol";
+import "src/test/mocks/MockDecimals.sol";
 import "src/test/events/IStrategyManagerEvents.sol";
 import "src/test/utils/EigenLayerUnitTestSetup.sol";
 
@@ -367,7 +368,7 @@ contract StrategyManagerUnitTests_depositIntoStrategy is StrategyManagerUnitTest
 
     function test_Revert_WhenTokenSafeTransferFromReverts() external {
         // replace 'dummyStrat' with one that uses a reverting token
-        dummyToken = IERC20(address(new Reverter()));
+        dummyToken = IERC20(address(new ReverterWithDecimals()));
         dummyStrat = _deployNewStrategy(dummyToken, strategyManager, pauserRegistry, dummyAdmin);
 
         // whitelist the strategy for deposit
@@ -390,8 +391,8 @@ contract StrategyManagerUnitTests_depositIntoStrategy is StrategyManagerUnitTest
     }
 
     function test_Revert_WhenTokenDoesNotExist() external {
-        // replace 'dummyStrat' with one that uses a non-existent token
-        dummyToken = IERC20(address(5678));
+        // replace 'dummyStrat' with one that uses a non-existent token, but will pass the initializer decimals check
+        dummyToken = IERC20(address(new MockDecimals()));
         dummyStrat = _deployNewStrategy(dummyToken, strategyManager, pauserRegistry, dummyAdmin);
 
         // whitelist the strategy for deposit
@@ -408,7 +409,7 @@ contract StrategyManagerUnitTests_depositIntoStrategy is StrategyManagerUnitTest
         IStrategy strategy = dummyStrat;
 
         cheats.prank(staker);
-        cheats.expectRevert("Address: call to non-contract");
+        cheats.expectRevert("SafeERC20: low-level call failed");
         strategyManager.depositIntoStrategy(strategy, token, amount);
     }