diff --git a/src/test/PoolModifyPositionTest.sol b/src/test/PoolModifyPositionTest.sol
deleted file mode 100644
index f6c3026d3..000000000
--- a/src/test/PoolModifyPositionTest.sol
+++ /dev/null
@@ -1,73 +0,0 @@
-// SPDX-License-Identifier: UNLICENSED
-pragma solidity ^0.8.20;
-
-import {CurrencyLibrary, Currency} from "../types/Currency.sol";
-import {IPoolManager} from "../interfaces/IPoolManager.sol";
-import {BalanceDelta} from "../types/BalanceDelta.sol";
-import {PoolKey} from "../types/PoolKey.sol";
-import {PoolTestBase} from "./PoolTestBase.sol";
-import {IHooks} from "../interfaces/IHooks.sol";
-import {Hooks} from "../libraries/Hooks.sol";
-import {Test} from "forge-std/Test.sol";
-import {FeeLibrary} from "../libraries/FeeLibrary.sol";
-
-contract PoolModifyPositionTest is Test, PoolTestBase {
-    using CurrencyLibrary for Currency;
-    using Hooks for IHooks;
-    using FeeLibrary for uint24;
-
-    constructor(IPoolManager _manager) PoolTestBase(_manager) {}
-
-    struct CallbackData {
-        address sender;
-        PoolKey key;
-        IPoolManager.ModifyLiquidityParams params;
-        bytes hookData;
-    }
-
-    function modifyPosition(PoolKey memory key, IPoolManager.ModifyLiquidityParams memory params, bytes memory hookData)
-        external
-        payable
-        returns (BalanceDelta delta)
-    {
-        delta = abi.decode(
-            manager.lock(address(this), abi.encode(CallbackData(msg.sender, key, params, hookData))), (BalanceDelta)
-        );
-
-        uint256 ethBalance = address(this).balance;
-        if (ethBalance > 0) {
-            CurrencyLibrary.NATIVE.transfer(msg.sender, ethBalance);
-        }
-    }
-
-    function lockAcquired(address, bytes calldata rawData) external returns (bytes memory) {
-        require(msg.sender == address(manager));
-
-        CallbackData memory data = abi.decode(rawData, (CallbackData));
-
-        BalanceDelta delta = manager.modifyLiquidity(data.key, data.params, data.hookData);
-        // Checks that the current hook is cleared if there is an access lock. Note that if this router is ever used in a nested lock this will fail.
-        assertEq(address(manager.getCurrentHook()), address(0));
-
-        (,,, int256 delta0) = _fetchBalances(data.key.currency0, data.sender);
-        (,,, int256 delta1) = _fetchBalances(data.key.currency1, data.sender);
-
-        // These assertions only apply in non lock-accessing pools.
-        if (!data.key.hooks.hasPermission(Hooks.ACCESS_LOCK_FLAG)) {
-            if (data.params.liquidityDelta > 0) {
-                assert(delta0 > 0 || delta1 > 0 || data.key.hooks.hasPermission(Hooks.NO_OP_FLAG));
-                assert(!(delta0 < 0 || delta1 < 0));
-            } else {
-                assert(delta0 < 0 || delta1 < 0 || data.key.hooks.hasPermission(Hooks.NO_OP_FLAG));
-                assert(!(delta0 > 0 || delta1 > 0));
-            }
-        }
-
-        if (delta0 > 0) _settle(data.key.currency0, data.sender, int128(delta0), true);
-        if (delta1 > 0) _settle(data.key.currency1, data.sender, int128(delta1), true);
-        if (delta0 < 0) _take(data.key.currency0, data.sender, int128(delta0), true);
-        if (delta1 < 0) _take(data.key.currency1, data.sender, int128(delta1), true);
-
-        return abi.encode(delta);
-    }
-}