diff --git a/tests/integration/concrete/lockup-base/collect-fees/collectFees.t.sol b/tests/integration/concrete/lockup-base/collect-fees/collectFees.t.sol index 941ce669d..2b454088a 100644 --- a/tests/integration/concrete/lockup-base/collect-fees/collectFees.t.sol +++ b/tests/integration/concrete/lockup-base/collect-fees/collectFees.t.sol @@ -7,11 +7,11 @@ import { Errors } from "src/libraries/Errors.sol"; import { Integration_Test } from "../../../Integration.t.sol"; contract CollectFees_Integration_Concrete_Test is Integration_Test { - function test_WhenAdminIsNotContract() external { + function test_GivenAdminIsNotContract() external { _test_CollectFees(users.admin); } - function test_RevertWhen_AdminDoesNotImplementReceiveFunction() external whenAdminIsContract { + function test_RevertGiven_AdminDoesNotImplementReceiveFunction() external givenAdminIsContract { // Transfer the admin to a contract that does not implement the receive function. resetPrank({ msgSender: users.admin }); lockup.transferAdmin(address(contractWithoutReceive)); @@ -32,7 +32,7 @@ contract CollectFees_Integration_Concrete_Test is Integration_Test { lockup.collectFees(); } - function test_WhenAdminImplementsReceiveFunction() external whenAdminIsContract { + function test_GivenAdminImplementsReceiveFunction() external givenAdminIsContract { // Transfer the admin to a contract that implements the receive function. resetPrank({ msgSender: users.admin }); lockup.transferAdmin(address(contractWithReceive)); diff --git a/tests/integration/concrete/lockup-base/collect-fees/collectFees.tree b/tests/integration/concrete/lockup-base/collect-fees/collectFees.tree index a52fd26f5..6023588ef 100644 --- a/tests/integration/concrete/lockup-base/collect-fees/collectFees.tree +++ b/tests/integration/concrete/lockup-base/collect-fees/collectFees.tree @@ -1,12 +1,12 @@ CollectFees_Integration_Concrete_Test -├── when admin is not contract +├── given admin is not contract │ ├── it should transfer fee │ ├── it should decrease contract balance to zero │ └── it should emit a {CollectFees} event -└── when admin is contract - ├── when admin does not implement receive function +└── given admin is contract + ├── given admin does not implement receive function │ └── it should revert - └── when admin implements receive function + └── given admin implements receive function ├── it should transfer fee ├── it should decrease contract balance to zero └── it should emit a {CollectFees} event diff --git a/tests/utils/Modifiers.sol b/tests/utils/Modifiers.sol index 2988923b5..ac2cb9583 100644 --- a/tests/utils/Modifiers.sol +++ b/tests/utils/Modifiers.sol @@ -437,10 +437,10 @@ abstract contract Modifiers is Fuzzers { } /*////////////////////////////////////////////////////////////////////////// - WITHDRAW-FEES + COLLECT-FEES //////////////////////////////////////////////////////////////////////////*/ - modifier whenAdminIsContract() { + modifier givenAdminIsContract() { _; }