Skip to content

Commit

Permalink
fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
wjmelements committed Oct 8, 2024
1 parent 0c762cb commit 13afb31
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .forge-snapshots/erc20 collect protocol fees.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
57607
57491
2 changes: 1 addition & 1 deletion .forge-snapshots/native collect protocol fees.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
59762
59646
2 changes: 1 addition & 1 deletion .forge-snapshots/poolManager bytecode size.snap
Original file line number Diff line number Diff line change
@@ -1 +1 @@
24056
23978
10 changes: 0 additions & 10 deletions test/ProtocolFeesImplementation.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -110,16 +110,6 @@ contract ProtocolFeesTest is Test, GasSnapshot, Deployers {
protocolFees.collectProtocolFees(address(1), currency0, 0);
}

function test_collectProtocolFees_revertsWithContractUnlocked() public {
protocolFees.setIsUnlocked(true);

protocolFees.setProtocolFeeController(feeController);
vm.prank(address(feeController));

vm.expectRevert(IProtocolFees.ContractUnlocked.selector);
protocolFees.collectProtocolFees(address(1), currency0, 0);
}

function test_collectProtocolFees_succeeds() public {
// set a balance of protocol fees that can be collected
protocolFees.updateProtocolFees(currency0, 100);
Expand Down

0 comments on commit 13afb31

Please sign in to comment.