Skip to content

Commit 43d5d31

Browse files
committed
test: update branch in collectFees
1 parent b5e32cb commit 43d5d31

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

Diff for: tests/integration/concrete/lockup-base/collect-fees/collectFees.t.sol

+3-3
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@ import { Errors } from "src/libraries/Errors.sol";
77
import { Integration_Test } from "../../../Integration.t.sol";
88

99
contract CollectFees_Integration_Concrete_Test is Integration_Test {
10-
function test_WhenAdminIsNotContract() external {
10+
function test_GivenAdminIsNotContract() external {
1111
_test_CollectFees(users.admin);
1212
}
1313

14-
function test_RevertWhen_AdminDoesNotImplementReceiveFunction() external whenAdminIsContract {
14+
function test_RevertWhen_AdminDoesNotImplementReceiveFunction() external givenAdminIsContract {
1515
// Transfer the admin to a contract that does not implement the receive function.
1616
resetPrank({ msgSender: users.admin });
1717
lockup.transferAdmin(address(contractWithoutReceive));
@@ -32,7 +32,7 @@ contract CollectFees_Integration_Concrete_Test is Integration_Test {
3232
lockup.collectFees();
3333
}
3434

35-
function test_WhenAdminImplementsReceiveFunction() external whenAdminIsContract {
35+
function test_GivenAdminImplementsReceiveFunction() external givenAdminIsContract {
3636
// Transfer the admin to a contract that implements the receive function.
3737
resetPrank({ msgSender: users.admin });
3838
lockup.transferAdmin(address(contractWithReceive));
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
CollectFees_Integration_Concrete_Test
2-
├── when admin is not contract
2+
├── given admin is not contract
33
│ ├── it should transfer fee
44
│ ├── it should decrease contract balance to zero
55
│ └── it should emit a {CollectFees} event
6-
└── when admin is contract
7-
├── when admin does not implement receive function
6+
└── given admin is contract
7+
├── given admin does not implement receive function
88
│ └── it should revert
9-
└── when admin implements receive function
9+
└── given admin implements receive function
1010
├── it should transfer fee
1111
├── it should decrease contract balance to zero
1212
└── it should emit a {CollectFees} event

Diff for: tests/utils/Modifiers.sol

+2-2
Original file line numberDiff line numberDiff line change
@@ -437,10 +437,10 @@ abstract contract Modifiers is Fuzzers {
437437
}
438438

439439
/*//////////////////////////////////////////////////////////////////////////
440-
WITHDRAW-FEES
440+
COLLECT-FEES
441441
//////////////////////////////////////////////////////////////////////////*/
442442

443-
modifier whenAdminIsContract() {
443+
modifier givenAdminIsContract() {
444444
_;
445445
}
446446

0 commit comments

Comments
 (0)