remove fuzz tests for now

Author: anna-carroll

.gas-snapshot

@@ -1,14 +1,18 @@
-OrdersFuzzTest:testFuzz_initiate((address,uint256)[],(address,uint256,address,uint32)[]) (runs: 256, μ: 494675, ~: 498424)
-OrdersTest:test_initiate_ERC20() (gas: 81300)
-OrdersTest:test_initiate_ETH() (gas: 44846)
-OrdersTest:test_initiate_both() (gas: 118596)
-OrdersTest:test_initiate_multiERC20() (gas: 688359)
-OrdersTest:test_initiate_multiETH() (gas: 75333)
+OrdersTest:test_fill_ERC20() (gas: 70364)
+OrdersTest:test_fill_ETH() (gas: 68414)
+OrdersTest:test_fill_both() (gas: 166580)
+OrdersTest:test_fill_multiETH() (gas: 131926)
+OrdersTest:test_fill_underflowETH() (gas: 115281)
+OrdersTest:test_initiate_ERC20() (gas: 81435)
+OrdersTest:test_initiate_ETH() (gas: 44949)
+OrdersTest:test_initiate_both() (gas: 118677)
+OrdersTest:test_initiate_multiERC20() (gas: 688417)
+OrdersTest:test_initiate_multiETH() (gas: 75304)
 OrdersTest:test_onlyBuilder() (gas: 12815)
-OrdersTest:test_orderExpired() (gas: 28015)
-OrdersTest:test_sweepERC20() (gas: 60370)
-OrdersTest:test_sweepETH() (gas: 81923)
-OrdersTest:test_underflowETH() (gas: 63477)
+OrdersTest:test_orderExpired() (gas: 27956)
+OrdersTest:test_sweepERC20() (gas: 60402)
+OrdersTest:test_sweepETH() (gas: 81940)
+OrdersTest:test_underflowETH() (gas: 63528)
 PassageTest:test_configureEnter() (gas: 82311)
 PassageTest:test_disallowedEnter() (gas: 17916)
 PassageTest:test_enter() (gas: 25563)

test/Orders.t.sol

@@ -236,80 +236,3 @@ contract OrdersTest is Test {
         target.fill{value: amount}(outputs);
     }
 }
-
-contract OrdersFuzzTest is Test {
-    RollupOrders public target;
-    mapping(address => bool) isToken;
-
-    uint256 deadline = block.timestamp + 100;
-
-    event Filled(Output[] outputs);
-
-    event Order(uint256 deadline, Input[] inputs, Output[] outputs);
-
-    event Sweep(address indexed recipient, address indexed token, uint256 amount);
-
-    function setUp() public {
-        target = new RollupOrders();
-
-        // deploy ERC20 tokens
-        for (uint256 i; i < 15; i++) {
-            address token = address(new TestERC20("hi", "HI"));
-            isToken[token] = true;
-        }
-    }
-
-    function testFuzz_initiate(Input[] memory fuzzInputs, Output[] memory fuzzOutputs) public {
-        // can only use vm.assume 256 * 256 times, and we do 256 fuzz runs
-        vm.assume(fuzzInputs.length < 250);
-        // setup the fuzz test by ensuring ETH and ERC20 amounts are available
-        uint256 totalETH;
-        for (uint256 i; i < fuzzInputs.length; i++) {
-            if (fuzzInputs[i].token == address(0)) {
-                // tally up total ETH amount in the inputs
-                totalETH += fuzzInputs[i].amount;
-                continue;
-            }
-
-            _setupToken(fuzzInputs[i].token, fuzzInputs[i].amount, address(target));
-        }
-        // deal the total ETH amount to ensure the address has it
-        vm.deal(address(this), address(this).balance + totalETH);
-
-        vm.expectEmit();
-        emit Order(deadline, fuzzInputs, fuzzOutputs);
-        target.initiate{value: totalETH}(deadline, fuzzInputs, fuzzOutputs);
-    }
-
-    function testFuzz_fill(Output[] memory fuzzOutputs) public {
-        // can only use vm.assume 256 * 256 times, and we do 256 fuzz runs
-        vm.assume(fuzzOutputs.length < 250);
-        // setup the fuzz test by ensuring ETH and ERC20 amounts are available
-        uint256 totalETH;
-        for (uint256 i; i < fuzzOutputs.length; i++) {
-            if (fuzzOutputs[i].token == address(0)) {
-                totalETH += fuzzOutputs[i].amount;
-                continue;
-            }
-
-            _setupToken(fuzzOutputs[i].token, fuzzOutputs[i].amount, fuzzOutputs[i].recipient);
-        }
-        // deal the total ETH amount to ensure the address has it
-        vm.deal(address(this), address(this).balance + totalETH);
-
-        vm.expectEmit();
-        emit Filled(fuzzOutputs);
-        target.fill{value: totalETH}(fuzzOutputs);
-    }
-
-    function _setupToken(address token, uint256 amount, address recipient) internal {
-        // ERC20 inputs must be one of the tokens we've deployed
-        vm.assume(isToken[token]);
-        // mint the token amount to ensure the address has it
-        TestERC20(token).mint(address(this), amount);
-        // approve the target to spend the total token amount
-        TestERC20(token).approve(address(target), type(uint256).max);
-        // expect the token amount to be transferred
-        vm.expectCall(token, abi.encodeWithSelector(ERC20.transferFrom.selector, address(this), recipient, amount));
-    }
-}