Skip to content

Commit

Permalink
Cleanup test names to best practices
Browse files Browse the repository at this point in the history
  • Loading branch information
jferas committed Dec 12, 2023
1 parent e7a9a3e commit d2f8877
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions test/RadworksGovernor.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -573,7 +573,7 @@ abstract contract _Execute is ProposalTest {
);
}

function testFuzz_grantPauserOnDrips(address _newPauser) public {
function testFuzz_CanGrantPauserOnDrips(address _newPauser) public {
assummeNotTimelock(_newPauser);
address[] memory _originalPausers = drips.allPausers();

Expand All @@ -586,7 +586,7 @@ abstract contract _Execute is ProposalTest {
assertEq(_originalPausers.length + 1, drips.allPausers().length);
}

function testFuzz_grantedPauserCanPauseAndUnPause(address _newPauser) public {
function testFuzz_GrantedPauserCanPauseAndUnPause(address _newPauser) public {
assummeNotTimelock(_newPauser);

_grantNewPauserViaGovernance(_newPauser);
Expand All @@ -602,7 +602,7 @@ abstract contract _Execute is ProposalTest {
assertFalse(drips.isPaused());
}

function testFuzz_revokePauserOnDrips(address _newPauser) public {
function testFuzz_CanRevokePauserOnDrips(address _newPauser) public {
assummeNotTimelock(_newPauser);
_grantNewPauserViaGovernance(_newPauser);

Expand All @@ -615,7 +615,7 @@ abstract contract _Execute is ProposalTest {
assertEq(drips.isPauser(_newPauser), false);
}

function testFuzz_revertWhenRevokedPauserAttemptsPause(address _newPauser) public {
function testFuzz_RevertWhen_PauseAfterRevoke(address _newPauser) public {
assummeNotTimelock(_newPauser);
_grantNewPauserViaGovernance(_newPauser);

Expand All @@ -635,7 +635,7 @@ abstract contract _Execute is ProposalTest {
assertEq(drips.isPauser(_newPauser), false);
}

function test_renounceAdminViaGovernance() public {
function test_CanRenounceAdminViaGovernance() public {
(
address[] memory _targets,
uint256[] memory _values,
Expand All @@ -652,7 +652,7 @@ abstract contract _Execute is ProposalTest {
assertEq(drips.admin(), address(0));
}

function testFuzz_proposeNewAdminViaGovernance(address _newAdmin) public {
function testFuzz_CanProposeNewAdminViaGovernance(address _newAdmin) public {
assummeNotTimelock(_newAdmin);
_proposeNewAdminViaGovernance(_newAdmin);

Expand Down

0 comments on commit d2f8877

Please sign in to comment.