Skip to content

Commit

Permalink
tests: Use updated test names test_{function}_{case}
Browse files Browse the repository at this point in the history 
Resolves Issue: Uniswap#390
  • Loading branch information
@prosperring
prosperring committed Jul 14, 2024
1 parent ac47532 commit a768543
Show file tree
Hide file tree
Showing 16 changed files with 47 additions and 47 deletions.
4 changes: 2 additions & 2 deletions test/CustomAccounting.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -152,11 +152,11 @@ contract CustomAccountingTest is Test, Deployers, GasSnapshot {
assertEq(currency1.balanceOf(address(this)), balanceBefore1 + amountToSwap, "amount 1");
}

// maximum available liquidity in each direction for the pool in test_fuzz_swap_beforeSwap_returnsDeltaSpecified
// maximum available liquidity in each direction for the pool in test_swap_beforeSwap_returnsDeltaSpecified_fuzz
int128 maxPossibleIn_fuzz_test = -6018336102428409;
int128 maxPossibleOut_fuzz_test = 5981737760509662;

function test_fuzz_swap_beforeSwap_returnsDeltaSpecified(
function test_swap_beforeSwap_returnsDeltaSpecified_fuzz(
int128 hookDeltaSpecified,
int256 amountSpecified,
bool zeroForOne
Expand Down
2 changes: 1 addition & 1 deletion test/DynamicFees.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ contract TestDynamicFees is Test, Deployers, GasSnapshot {
assertEq(_fetchPoolLPFee(key), 123);
}

function test_fuzz_ProtocolAndLPFee(uint24 lpFee, uint16 protocolFee0, uint16 protocolFee1, int256 amountSpecified)
function test_ProtocolAndLPFee_fuzz(uint24 lpFee, uint16 protocolFee0, uint16 protocolFee1, int256 amountSpecified)
public
{
assertEq(_fetchPoolLPFee(key), 0);
Expand Down
2 changes: 1 addition & 1 deletion test/DynamicReturnFees.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ contract TestDynamicReturnFees is Test, Deployers, GasSnapshot {
);
}

function test_fuzz_dynamicReturnSwapFee(uint24 fee) public {
function test_dynamicReturnSwapFee_fuzz(uint24 fee) public {
// hook will handle adding the override flag
dynamicReturnFeesHook.setFee(fee);

Expand Down
2 changes: 1 addition & 1 deletion test/Extsload.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ contract ExtsloadTest is Test, GasSnapshot {
}
}

function test_fuzz_extsload(uint256 length, uint256 seed, bytes memory dirtyBits) public {
function test_extsload_fuzz(uint256 length, uint256 seed, bytes memory dirtyBits) public {
length = bound(length, 0, 1000);
bytes32[] memory slots = new bytes32[](length);
bytes32[] memory expected = new bytes32[](length);
Expand Down
2 changes: 1 addition & 1 deletion test/ModifyLiquidity.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ contract ModifyLiquidityTest is Test, Logger, Deployers, JavascriptFfi, Fuzzers,
/// forge-config: default.fuzz.runs = 10
/// forge-config: pr.fuzz.runs = 10
/// forge-config: ci.fuzz.runs = 500
function test_ffi_fuzz_addLiquidity_defaultPool_ReturnsCorrectLiquidityDelta(
function test_ffi_addLiquidity_defaultPool_ReturnsCorrectLiquidityDelta_fuzz(
IPoolManager.ModifyLiquidityParams memory paramSeed
) public {
// Sanitize the fuzzed params to get valid tickLower, tickUpper, and liquidityDelta.
Expand Down
2 changes: 1 addition & 1 deletion test/PoolManagerInitialize.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ contract PoolManagerInitializeTest is Test, Deployers, GasSnapshot {
});
}

function test_fuzz_initialize(PoolKey memory key0, uint160 sqrtPriceX96) public {
function test_initialize_fuzz(PoolKey memory key0, uint160 sqrtPriceX96) public {
// Assumptions tested in Pool.t.sol
sqrtPriceX96 = uint160(bound(sqrtPriceX96, TickMath.MIN_SQRT_PRICE, TickMath.MAX_SQRT_PRICE - 1));

Expand Down
6 changes: 3 additions & 3 deletions test/ProtocolFeesImplementation.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ contract ProtocolFeesTest is Test, GasSnapshot, Deployers {
protocolFees.setProtocolFee(key, MAX_PROTOCOL_FEE_BOTH_TOKENS + (1 << 12));
}

function test_fuzz_setProtocolFee(PoolKey memory key, uint24 protocolFee) public {
function test_setProtocolFee_fuzz(PoolKey memory key, uint24 protocolFee) public {
protocolFees.setProtocolFeeController(feeController);
// Set price to pretend that the pool is initialized
protocolFees.setPrice(key, Constants.SQRT_PRICE_1_1);
Expand Down Expand Up @@ -120,7 +120,7 @@ contract ProtocolFeesTest is Test, GasSnapshot, Deployers {
assertEq(currency0.balanceOf(address(this)), 100);
}

function test_fuzz_collectProtocolFees(address recipient, uint256 amount, uint256 feesAccrued) public {
function test_collectProtocolFees_fuzz(address recipient, uint256 amount, uint256 feesAccrued) public {
vm.assume(feesAccrued <= currency0.balanceOf(address(protocolFees)));

uint256 recipientBalanceBefore = currency0.balanceOf(recipient);
Expand Down Expand Up @@ -161,7 +161,7 @@ contract ProtocolFeesTest is Test, GasSnapshot, Deployers {
assertEq(protocolFees.protocolFeesAccrued(currency0), 300);
}

function test_fuzz_updateProtocolFees(uint256 amount, uint256 startingAmount) public {
function test_updateProtocolFees_fuzz(uint256 amount, uint256 startingAmount) public {
// set a starting balance of protocol fees
protocolFees.updateProtocolFees(currency0, startingAmount);
assertEq(protocolFees.protocolFeesAccrued(currency0), startingAmount);
Expand Down
2 changes: 1 addition & 1 deletion test/Reserves.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ contract ReservesTest is Test {
assertEq(bytes32(uint256(keccak256("ReservesOf")) - 1), Reserves.RESERVES_OF_SLOT);
}

function test_fuzz_get_set(Currency currency, uint256 value) public {
function test_get_set_fuzz(Currency currency, uint256 value) public {
vm.assume(value != type(uint256).max);
currency.setReserves(value);
assertEq(currency.getReserves(), value);
Expand Down
4 changes: 2 additions & 2 deletions test/libraries/BitMath.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ contract TestBitMath is Test, GasSnapshot {
assertEq(BitMath.mostSignificantBit(type(uint256).max), 255);
}

function test_fuzz_mostSignificantBit(uint256 x) public pure {
function test_mostSignificantBit_fuzz(uint256 x) public pure {
vm.assume(x != 0);
assertEq(BitMath.mostSignificantBit(x), mostSignificantBitReference(x));
}
Expand Down Expand Up @@ -80,7 +80,7 @@ contract TestBitMath is Test, GasSnapshot {
assertEq(BitMath.leastSignificantBit(type(uint256).max), 0);
}

function test_fuzz_leastSignificantBit(uint256 x) public pure {
function test_leastSignificantBit_fuzz(uint256 x) public pure {
vm.assume(x != 0);
assertEq(BitMath.leastSignificantBit(x), leastSignificantBitReference(x));
}
Expand Down
8 changes: 4 additions & 4 deletions test/libraries/FullMath.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ contract FullMathTest is Test {
uint256 constant Q128 = 2 ** 128;
uint256 constant MAX_UINT256 = type(uint256).max;

function test_fuzz_mulDiv_revertsWith0Denominator(uint256 x, uint256 y) public {
function test_mulDiv_revertsWith0Denominator_fuzz(uint256 x, uint256 y) public {
vm.expectRevert();
x.mulDiv(y, 0);
}
Expand Down Expand Up @@ -49,14 +49,14 @@ contract FullMathTest is Test {
assertEq(Q128.mulDiv(1000 * Q128, 3000 * Q128), result);
}

function test_fuzz_mulDiv(uint256 x, uint256 y, uint256 d) public pure {
function test_mulDiv_fuzz(uint256 x, uint256 y, uint256 d) public pure {
vm.assume(d != 0);
vm.assume(y != 0);
x = bound(x, 0, type(uint256).max / y);
assertEq(FullMath.mulDiv(x, y, d), x * y / d);
}

function test_fuzz_mulDivRoundingUp_revertsWith0Denominator(uint256 x, uint256 y) public {
function test_mulDivRoundingUp_revertsWith0Denominator_fuzz(uint256 x, uint256 y) public {
vm.expectRevert();
x.mulDivRoundingUp(y, 0);
}
Expand Down Expand Up @@ -94,7 +94,7 @@ contract FullMathTest is Test {
);
}

function test_fuzz_mulDivRoundingUp(uint256 x, uint256 y, uint256 d) public pure {
function test_mulDivRoundingUp_fuzz(uint256 x, uint256 y, uint256 d) public pure {
vm.assume(d != 0);
vm.assume(y != 0);
x = bound(x, 0, type(uint256).max / y);
Expand Down
42 changes: 21 additions & 21 deletions test/libraries/Hooks.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
}

// hook validation
function test_fuzz_validateHookPermissions_noHooks(uint160 addr) public view {
function test_validateHookPermissions_noHooks_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;

IHooks hookAddr = IHooks(address(preAddr));
Expand Down Expand Up @@ -233,7 +233,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeInitialize(uint160 addr) public view {
function test_validateHookPermissions_beforeInitialize_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;

IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_INITIALIZE_FLAG)));
Expand Down Expand Up @@ -272,7 +272,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_afterInitialize(uint160 addr) public view {
function test_validateHookPermissions_afterInitialize_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;

IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.AFTER_INITIALIZE_FLAG)));
Expand Down Expand Up @@ -311,7 +311,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAndAfterInitialize(uint160 addr) public view {
function test_validateHookPermissions_beforeAndAfterInitialize_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_INITIALIZE_FLAG | Hooks.AFTER_INITIALIZE_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -349,7 +349,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAddLiquidity(uint160 addr) public view {
function test_validateHookPermissions_beforeAddLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_ADD_LIQUIDITY_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -387,7 +387,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_afterAddLiquidity(uint160 addr) public view {
function test_validateHookPermissions_afterAddLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.AFTER_ADD_LIQUIDITY_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -425,7 +425,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAndAfterAddLiquidity(uint160 addr) public view {
function test_validateHookPermissions_beforeAndAfterAddLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr =
IHooks(address(uint160(preAddr | Hooks.BEFORE_ADD_LIQUIDITY_FLAG | Hooks.AFTER_ADD_LIQUIDITY_FLAG)));
Expand Down Expand Up @@ -464,7 +464,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeRemoveLiquidity(uint160 addr) public view {
function test_validateHookPermissions_beforeRemoveLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_REMOVE_LIQUIDITY_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -502,7 +502,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_afterRemoveLiquidity(uint160 addr) public view {
function test_validateHookPermissions_afterRemoveLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.AFTER_REMOVE_LIQUIDITY_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -540,7 +540,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAfterRemoveLiquidity(uint160 addr) public view {
function test_validateHookPermissions_beforeAfterRemoveLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr =
IHooks(address(uint160(preAddr | Hooks.BEFORE_REMOVE_LIQUIDITY_FLAG | Hooks.AFTER_REMOVE_LIQUIDITY_FLAG)));
Expand Down Expand Up @@ -579,7 +579,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeInitializeAfterAddLiquidity(uint160 addr) public view {
function test_validateHookPermissions_beforeInitializeAfterAddLiquidity_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr =
IHooks(address(uint160(preAddr | Hooks.BEFORE_INITIALIZE_FLAG | Hooks.AFTER_ADD_LIQUIDITY_FLAG)));
Expand Down Expand Up @@ -618,7 +618,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeSwap(uint160 addr) public view {
function test_validateHookPermissions_beforeSwap_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_SWAP_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -656,7 +656,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_afterSwap(uint160 addr) public view {
function test_validateHookPermissions_afterSwap_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.AFTER_SWAP_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -694,7 +694,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAndAfterSwap(uint160 addr) public view {
function test_validateHookPermissions_beforeAndAfterSwap_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_SWAP_FLAG | Hooks.AFTER_SWAP_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -732,7 +732,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeDonate(uint160 addr) public view {
function test_validateHookPermissions_beforeDonate_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_DONATE_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -770,7 +770,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_afterDonate(uint160 addr) public view {
function test_validateHookPermissions_afterDonate_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.AFTER_DONATE_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -808,7 +808,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_beforeAndAfterDonate(uint160 addr) public view {
function test_validateHookPermissions_beforeAndAfterDonate_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_DONATE_FLAG | Hooks.AFTER_DONATE_FLAG)));
Hooks.validateHookPermissions(
Expand Down Expand Up @@ -846,7 +846,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookPermissions_allHooks(uint160 addr) public view {
function test_validateHookPermissions_allHooks_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
uint160 allHookBitsFlipped = uint160((1 << hookPermissionCount) - 1);
IHooks hookAddr = IHooks(address(uint160(preAddr) | allHookBitsFlipped));
Expand Down Expand Up @@ -885,7 +885,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertTrue(hookAddr.hasPermission(Hooks.AFTER_REMOVE_LIQUIDITY_RETURNS_DELTA_FLAG));
}

function test_fuzz_validateHookAddress_failsAllHooks(uint160 addr, uint16 mask) public {
function test_validateHookAddress_failsAllHooks_fuzz(uint160 addr, uint16 mask) public {
uint160 preAddr = addr & clearAllHookPermissionsMask;
// Set the upper `hooksPermissionCount` number of bits to get the full mask in uint16.
uint16 allHooksMask = uint16(~uint16(0));
Expand Down Expand Up @@ -914,7 +914,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
);
}

function test_fuzz_validateHookAddress_failsNoHooks(uint160 addr, uint16 mask) public {
function test_validateHookAddress_failsNoHooks_fuzz(uint160 addr, uint16 mask) public {
// we only want hookPermissionCount of mask
mask = mask >> (16 - hookPermissionCount);
uint160 preAddr = addr & clearAllHookPermissionsMask;
Expand Down Expand Up @@ -965,7 +965,7 @@ contract HooksTest is Test, Deployers, GasSnapshot {
assertFalse(Hooks.isValidHookAddress(IHooks(address(0)), LPFeeLibrary.DYNAMIC_FEE_FLAG));
}

function test_fuzz_isValidHookAddress_invalid_returnsDeltaWithoutHookFlag(uint160 addr) public view {
function test_isValidHookAddress_invalid_returnsDeltaWithoutHookFlag_fuzz(uint160 addr) public view {
uint160 preAddr = addr & clearAllHookPermissionsMask;
IHooks hookAddr = IHooks(address(uint160(preAddr | Hooks.BEFORE_SWAP_RETURNS_DELTA_FLAG)));
assertFalse(Hooks.isValidHookAddress(hookAddr, 3000));
Expand Down
6 changes: 3 additions & 3 deletions test/libraries/LPFeeLibrary.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ contract LPFeeLibraryTest is Test {
assertFalse(LPFeeLibrary.isDynamicFee(dynamicFee));
}

function test_fuzz_isDynamicFee(uint24 fee) public pure {
function test_isDynamicFee_fuzz(uint24 fee) public pure {
assertEq(fee == LPFeeLibrary.DYNAMIC_FEE_FLAG, LPFeeLibrary.isDynamicFee(fee));
}

Expand All @@ -46,7 +46,7 @@ contract LPFeeLibraryTest is Test {
LPFeeLibrary.validate(fee);
}

function test_fuzz_validate(uint24 fee) public {
function test_validate_fuzz(uint24 fee) public {
if (fee > 1000000) {
vm.expectRevert(LPFeeLibrary.FeeTooLarge.selector);
}
Expand Down Expand Up @@ -75,7 +75,7 @@ contract LPFeeLibraryTest is Test {
LPFeeLibrary.getInitialLPFee(dynamicFee);
}

function test_fuzz_getInitialLPFee(uint24 fee) public {
function test_getInitialLPFee_fuzz(uint24 fee) public {
if (fee == LPFeeLibrary.DYNAMIC_FEE_FLAG) {
assertEq(LPFeeLibrary.getInitialLPFee(fee), 0);
} else if (fee > 1000000) {
Expand Down
2 changes: 1 addition & 1 deletion test/libraries/NonZeroDeltaCount.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ contract NonZeroDeltaCountTest is Test {

// Reading from right to left. Bit of 0: call increase. Bit of 1: call decrease.
// The library allows over over/underflow so we dont check for that here
function test_fuzz_nonZeroDeltaCount(uint256 instructions) public {
function test_nonZeroDeltaCount_fuzz(uint256 instructions) public {
assertEq(NonZeroDeltaCount.read(), 0);
uint256 expectedCount;
for (uint256 i = 0; i < 256; i++) {
Expand Down
Loading

0 comments on commit a768543

Please sign in to comment.