diff --git a/src/libraries/PositionInfoLibrary.sol b/src/libraries/PositionInfoLibrary.sol index ac9885f1..85650585 100644 --- a/src/libraries/PositionInfoLibrary.sol +++ b/src/libraries/PositionInfoLibrary.sol @@ -33,7 +33,6 @@ type PositionInfo is uint256; using PositionInfoLibrary for PositionInfo global; library PositionInfoLibrary { - PositionInfo internal constant EMPTY_POSITION_INFO = PositionInfo.wrap(0); uint256 internal constant MASK_UPPER_200_BITS = 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF00000000000000; diff --git a/test/BaseActionsRouter.t.sol b/test/BaseActionsRouter.t.sol index 3e1481d0..a411dbc1 100644 --- a/test/BaseActionsRouter.t.sol +++ b/test/BaseActionsRouter.t.sol @@ -10,7 +10,6 @@ import {Test} from "forge-std/Test.sol"; import {Deployers} from "@uniswap/v4-core/test/utils/Deployers.sol"; contract BaseActionsRouterTest is Test, Deployers { - MockBaseActionsRouter router; function setUp() public { diff --git a/test/DeltaResolver.t.sol b/test/DeltaResolver.t.sol index 8788630c..8dc620c5 100644 --- a/test/DeltaResolver.t.sol +++ b/test/DeltaResolver.t.sol @@ -10,7 +10,6 @@ import {ERC20} from "solmate/src/tokens/ERC20.sol"; import {MockDeltaResolver} from "./mocks/MockDeltaResolver.sol"; contract DeltaResolverTest is Test, Deployers { - MockDeltaResolver resolver; function setUp() public { diff --git a/test/libraries/PositionInfoLibrary.t.sol b/test/libraries/PositionInfoLibrary.t.sol index bf80a341..877f8ba7 100644 --- a/test/libraries/PositionInfoLibrary.t.sol +++ b/test/libraries/PositionInfoLibrary.t.sol @@ -6,7 +6,6 @@ import {PoolKey} from "@uniswap/v4-core/src/types/PoolKey.sol"; import {PositionInfo, PositionInfoLibrary, PoolId} from "../../src/libraries/PositionInfoLibrary.sol"; contract PositionInfoLibraryTest is Test { - function setUp() public {} function test_fuzz_initialize(PoolKey memory poolKey, int24 tickLower, int24 tickUpper) public pure { diff --git a/test/mocks/MockDeltaResolver.sol b/test/mocks/MockDeltaResolver.sol index 1982a122..514379ce 100644 --- a/test/mocks/MockDeltaResolver.sol +++ b/test/mocks/MockDeltaResolver.sol @@ -10,7 +10,6 @@ import {ERC20} from "solmate/src/tokens/ERC20.sol"; import {Test} from "forge-std/Test.sol"; contract MockDeltaResolver is Test, DeltaResolver, IUnlockCallback { - uint256 public payCallCount; constructor(IPoolManager _poolManager) ImmutableState(_poolManager) {} diff --git a/test/router/Payments.gas.t.sol b/test/router/Payments.gas.t.sol index 51bc6c42..f27e3617 100644 --- a/test/router/Payments.gas.t.sol +++ b/test/router/Payments.gas.t.sol @@ -10,7 +10,6 @@ import {Actions} from "../../src/libraries/Actions.sol"; import {ActionConstants} from "../../src/libraries/ActionConstants.sol"; contract PaymentsTests is RoutingTestHelpers { - function setUp() public { setupRouterCurrenciesAndPoolsWithLiquidity(); plan = Planner.init(); diff --git a/test/router/Payments.t.sol b/test/router/Payments.t.sol index 091c510e..d572bd1d 100644 --- a/test/router/Payments.t.sol +++ b/test/router/Payments.t.sol @@ -11,7 +11,6 @@ import {ActionConstants} from "../../src/libraries/ActionConstants.sol"; import {BipsLibrary} from "../../src/libraries/BipsLibrary.sol"; contract PaymentsTests is RoutingTestHelpers { - address bob = makeAddr("BOB"); function setUp() public { diff --git a/test/router/V4Router.gas.t.sol b/test/router/V4Router.gas.t.sol index 270bf996..7c155a01 100644 --- a/test/router/V4Router.gas.t.sol +++ b/test/router/V4Router.gas.t.sol @@ -10,7 +10,6 @@ import {Actions} from "../../src/libraries/Actions.sol"; import {ActionConstants} from "../../src/libraries/ActionConstants.sol"; contract V4RouterTest is RoutingTestHelpers { - function setUp() public { setupRouterCurrenciesAndPoolsWithLiquidity(); plan = Planner.init(); diff --git a/test/router/V4Router.t.sol b/test/router/V4Router.t.sol index f6360947..c4799715 100644 --- a/test/router/V4Router.t.sol +++ b/test/router/V4Router.t.sol @@ -9,7 +9,6 @@ import {Actions} from "../../src/libraries/Actions.sol"; import {ActionConstants} from "../../src/libraries/ActionConstants.sol"; contract V4RouterTest is RoutingTestHelpers { - address alice = makeAddr("ALICE"); function setUp() public { diff --git a/test/shared/Planner.sol b/test/shared/Planner.sol index f86fd527..d3c3b8be 100644 --- a/test/shared/Planner.sol +++ b/test/shared/Planner.sol @@ -15,7 +15,6 @@ struct Plan { using Planner for Plan global; library Planner { - function init() internal pure returns (Plan memory plan) { return Plan({actions: bytes(""), params: new bytes[](0)}); } diff --git a/test/shared/RoutingTestHelpers.sol b/test/shared/RoutingTestHelpers.sol index d0464b86..8c62cbea 100644 --- a/test/shared/RoutingTestHelpers.sol +++ b/test/shared/RoutingTestHelpers.sol @@ -21,7 +21,6 @@ import {ActionConstants} from "../../src/libraries/ActionConstants.sol"; /// @notice A shared test contract that wraps the v4-core deployers contract and exposes basic helpers for swapping with the router. contract RoutingTestHelpers is Test, Deployers { - PoolModifyLiquidityTest positionManager; MockV4Router router; diff --git a/test/shared/fuzz/LiquidityFuzzers.sol b/test/shared/fuzz/LiquidityFuzzers.sol index 86fa9812..da14514c 100644 --- a/test/shared/fuzz/LiquidityFuzzers.sol +++ b/test/shared/fuzz/LiquidityFuzzers.sol @@ -12,7 +12,6 @@ import {Actions} from "../../../src/libraries/Actions.sol"; import {Planner, Plan} from "../../shared/Planner.sol"; contract LiquidityFuzzers is Fuzzers { - uint128 constant _MAX_SLIPPAGE_INCREASE = type(uint128).max; function addFuzzyLiquidity(