diff --git a/src/FeeCalculator.sol b/src/FeeCalculator.sol index 362ab42..8d769d7 100644 --- a/src/FeeCalculator.sol +++ b/src/FeeCalculator.sol @@ -46,10 +46,7 @@ contract FeeCalculator is IDepositFeeCalculator, IRedemptionFeeCalculator, Ownab /// @dev Can only be called by the current owner. /// @param _depositFeeRatioScale The new deposit fee ratio scale. function setDepositFeeRatioScale(SD59x18 _depositFeeRatioScale) external onlyOwner { - require( - _depositFeeRatioScale >= zero && _depositFeeRatioScale <= one, - "Deposit fee ratio scale must be between 0 and 1" - ); + require(_depositFeeRatioScale >= zero, "Deposit fee ratio scale must be above 0"); depositFeeRatioScale = _depositFeeRatioScale; } diff --git a/test/FeeCalculator.t.sol b/test/FeeCalculator.t.sol index 380ae65..e468d39 100644 --- a/test/FeeCalculator.t.sol +++ b/test/FeeCalculator.t.sol @@ -665,12 +665,6 @@ contract FeeCalculatorTest is Test { feeCalculator.setDepositFeeScale(invalid); } - function testSetDepositFeeRatioScaleReverts() public { - SD59x18 invalid = sd(1.2 * 1e18); - vm.expectRevert("Deposit fee ratio scale must be between 0 and 1"); - feeCalculator.setDepositFeeRatioScale(invalid); - } - function testSetSingleAssetDepositRelativeFeeReverts() public { SD59x18 invalid = sd(1.3 * 1e18); vm.expectRevert("Single asset deposit relative fee must be between 0 and 1"); @@ -715,7 +709,7 @@ contract FeeCalculatorTest is Test { function testSetDepositFeeRatioScaleNegativeReverts() public { SD59x18 invalid = sd(-0.2 * 1e18); - vm.expectRevert("Deposit fee ratio scale must be between 0 and 1"); + vm.expectRevert("Deposit fee ratio scale must be above 0"); feeCalculator.setDepositFeeRatioScale(invalid); }