refactor: deprecate fuzzed integration valid order test in favour of …

…unit test

test/integration/BCoWHelper.t.sol

@@ -91,31 +91,6 @@ contract ConstantProductHelperForkedTest is Test {
     assertEq(postSpotPrice, 1_052_631_578_947_368);
   }
 
-  // NOTE: this test checks that the order generated by the helper is valid
-  function testValidOrder(uint256 priceSkewness) public {
-    IBCoWPool pool = IBCoWPool(address(basicPool));
-
-    // skew the price by max 50% (more could result in reverts bc of max swap ratio)
-    priceSkewness = bound(priceSkewness, 5000, 15_000);
-    vm.assume(priceSkewness != 10_000); // avoids no-skewness revert
-
-    uint256[] memory prices = new uint256[](2);
-    prices[0] = INITIAL_WETH_BALANCE;
-    prices[1] = INITIAL_DAI_BALANCE * priceSkewness / 10_000;
-
-    // the helper generates the AMM order
-    (GPv2Order.Data memory ammOrder,,,) = helper.order(address(pool), prices);
-
-    if (priceSkewness > 10_000) {
-      assertEq(address(ammOrder.buyToken), address(DAI));
-    } else {
-      assertEq(address(ammOrder.buyToken), address(WETH));
-    }
-
-    // it should not revert
-    pool.verify(ammOrder);
-  }
-
   // NOTE: reverting test, weighted pools are not supported
   function testWeightedOrder() public {
     IBCoWPool pool = IBCoWPool(address(weightedPool));

test/unit/BCoWHelper.t.sol

@@ -172,6 +172,8 @@ contract BCoWHelperTest is Test {
     // it should return a valid pool order
     (GPv2Order.Data memory ammOrder,,,) = helper.order(address(pool), prices);
 
+    assertEq(address(ammOrder.buyToken), priceSkewness > 10_000 ? tokens[0] : tokens[1]);
+
     // this call should not revert
     pool.verify(ammOrder);
   }