Remove unused maxSlippage param

Author: davidlee1435

src/ForeignController.sol

@@ -837,7 +837,6 @@ contract ForeignController is AccessControl {
                 tokenIn      : tokenIn,
                 amountIn     : amountIn,
                 minAmountOut : minAmountOut,
-                maxSlippage  : maxSlippages[pool],
                 tickDelta    : swapMaxTickDelta,
                 poolParams   : uniswapV3PoolParams[pool]
             })

src/MainnetController.sol

@@ -653,7 +653,6 @@ contract MainnetController is AccessControl {
                 tokenIn      : tokenIn,
                 amountIn     : amountIn,
                 minAmountOut : minAmountOut,
-                maxSlippage  : maxSlippages[pool],
                 tickDelta    : swapMaxTickDelta,
                 poolParams   : uniswapV3PoolParams[pool]
             })

src/libraries/UniswapV3Lib.sol

@@ -51,7 +51,6 @@ library UniswapV3Lib {
         uint256             amountIn;
         uint256             minAmountOut;
         uint24              tickDelta; // The maximum that the tick can move by after completing the swap; cannot exceed MAX_TICK_DELTA
-        uint256             maxSlippage;
     }
 
     struct SwapCache {
@@ -86,7 +85,6 @@ library UniswapV3Lib {
 
     // Rate limit decreased by value of tokenIn (the amount actually spent)
     function swap(UniV3Context calldata context, SwapParams calldata params) external returns (uint256 amountOut) {
-        require(params.maxSlippage > 0,                                 "UniswapV3Lib/max-slippage-not-set");
         require(params.tickDelta <= params.poolParams.swapMaxTickDelta, "UniswapV3Lib/invalid-max-tick-delta");
         require(params.poolParams.twapSecondsAgo != 0,                  "UniswapV3Lib/zero-twap-seconds");
 

test/grove-base-fork/UniswapV3.t.sol

@@ -289,25 +289,6 @@ contract ForeignControllerSwapUniswapV3FailureTests is UniswapV3TestBase {
         );
     }
 
-    function test_swapUniswapV3_maxSlippageNotSet() public {
-        uint256 amountIn = 100_000e6;
-        _fundProxy(amountIn, 0);
-
-        vm.prank(GROVE_EXECUTOR);
-        foreignController.setMaxSlippage(_getPool(), 0);
-
-        vm.startPrank(ALM_RELAYER);
-        vm.expectRevert("UniswapV3Lib/max-slippage-not-set");
-        foreignController.swapUniswapV3(
-            _getPool(),
-            address(token0),
-            amountIn,
-            0,
-            200
-        );
-        vm.stopPrank();
-    }
-
     function test_swapUniswapV3_invalidTokenIn() public {
         vm.startPrank(ALM_RELAYER);
         vm.expectRevert("UniswapV3Lib/invalid-token-pair");

test/grove-mainnet-fork/UniswapV3.t.sol

@@ -250,25 +250,6 @@ contract MainnetControllerSwapUniswapV3FailureTests is UniswapV3TestBase {
         );
     }
 
-    function test_swapUniswapV3_maxSlippageNotSet() public {
-        uint256 amountIn = 100_000e6;
-        _fundProxy(amountIn, 0);
-
-        vm.prank(GROVE_PROXY);
-        mainnetController.setMaxSlippage(_getPool(), 0);
-
-        vm.startPrank(relayer);
-        vm.expectRevert("UniswapV3Lib/max-slippage-not-set");
-        mainnetController.swapUniswapV3(
-            _getPool(),
-            address(token0),
-            amountIn,
-            0,
-            200
-        );
-        vm.stopPrank();
-    }
-
     function test_swapUniswapV3_invalidTokenIn() public {
         uint256 amountIn = 100_000e6;
         deal(address(dai), address(almProxy), amountIn);