Add tests for removed Ownable and ERC4626 methods, update test covera…

…ge report
brritoxyz · Jan 12, 2024 · 7c8a9c6 · 7c8a9c6
1 parent b735b1c
commit 7c8a9c6
Show file tree

Hide file tree

Showing 2 changed files with 222 additions and 112 deletions.
diff --git a/lcov.info b/lcov.info
@@ -1,128 +1,182 @@
 TN:
 SF:script/BrrETH.s.sol
-FN:7,BrrETHScript.setUp
-FNDA:0,BrrETHScript.setUp
-FN:9,BrrETHScript.run
+FN:12,BrrETHScript.run
 FNDA:0,BrrETHScript.run
-DA:10,0
-FNF:2
+DA:13,0
+DA:15,0
+DA:17,0
+DA:19,0
+FNF:1
 FNH:0
-LF:1
+LF:4
+LH:0
+BRF:0
+BRH:0
+end_of_record
+TN:
+SF:script/BrrETHRedeemHelper.s.sol
+FN:11,BrrETHRedeemHelperScript.run
+FNDA:0,BrrETHRedeemHelperScript.run
+DA:12,0
+DA:14,0
+FNF:1
+FNH:0
+LF:2
 LH:0
 BRF:0
 BRH:0
 end_of_record
 TN:
 SF:src/BrrETH.sol
-FN:62,BrrETH.name
+FN:78,BrrETH.name
 FNDA:1,BrrETH.name
-DA:63,1
-FN:66,BrrETH.symbol
+DA:79,1
+FN:86,BrrETH.symbol
 FNDA:1,BrrETH.symbol
-DA:67,1
-FN:70,BrrETH.asset
+DA:87,1
+FN:94,BrrETH.asset
 FNDA:1,BrrETH.asset
-DA:71,2159
-FN:75,BrrETH.approveTokens
+DA:95,2033
+FN:99,BrrETH.approveTokens
 FNDA:1,BrrETH.approveTokens
-DA:76,1
-DA:80,1
-DA:86,1
-FN:96,BrrETH.maxDeposit
-FNDA:0,BrrETH.maxDeposit
-DA:97,533
-FN:108,BrrETH.convertToShares
-FNDA:534,BrrETH.convertToShares
-DA:116,1077
-FN:127,BrrETH._deposit
-FNDA:543,BrrETH._deposit
-DA:133,543
-DA:135,543
-FN:143,BrrETH.deposit
-FNDA:267,BrrETH.deposit
-DA:144,267
-DA:146,267
-DA:148,267
-DA:150,267
-DA:152,267
-DA:154,267
-FN:166,BrrETH.deposit
-FNDA:533,BrrETH.deposit
-DA:170,533
-BRDA:170,0,0,257
-BRDA:170,0,1,276
-DA:172,276
-DA:174,276
-DA:176,276
-DA:183,276
-FN:187,BrrETH.harvest
-FNDA:1,BrrETH.harvest
-DA:188,1
-DA:190,1
-DA:192,1
-DA:194,1
-BRDA:194,1,0,-
-BRDA:194,1,1,1
-DA:198,1
-DA:204,1
-DA:215,1
-DA:218,1
-BRDA:218,2,0,-
-BRDA:218,2,1,1
-DA:221,1
-DA:223,1
-DA:225,1
-DA:226,1
-DA:230,1
-DA:232,1
-FN:243,BrrETH.setCometRewards
-FNDA:259,BrrETH.setCometRewards
-DA:244,258
-BRDA:244,3,0,1
-BRDA:244,3,1,257
-DA:246,257
-DA:248,257
-FN:255,BrrETH.setRouter
-FNDA:259,BrrETH.setRouter
-DA:256,258
-BRDA:256,4,0,1
-BRDA:256,4,1,257
-DA:258,257
-DA:260,257
-FN:267,BrrETH.setRewardFee
-FNDA:260,BrrETH.setRewardFee
-DA:268,258
-BRDA:268,5,0,1
-BRDA:268,5,1,257
-DA:270,257
-DA:272,257
-FN:279,BrrETH.setFeeDistributor
-FNDA:2,BrrETH.setFeeDistributor
-DA:280,2
-BRDA:280,6,0,1
-BRDA:280,6,1,1
-DA:282,1
-DA:284,1
-FN:291,BrrETH.transferOwnership
-FNDA:0,BrrETH.transferOwnership
-FN:293,BrrETH.renounceOwnership
-FNDA:0,BrrETH.renounceOwnership
-FN:299,BrrETH.maxMint
-FNDA:0,BrrETH.maxMint
-FN:301,BrrETH.maxWithdraw
-FNDA:0,BrrETH.maxWithdraw
-FN:303,BrrETH.previewMint
-FNDA:0,BrrETH.previewMint
-FN:305,BrrETH.previewWithdraw
-FNDA:0,BrrETH.previewWithdraw
-FN:307,BrrETH.mint
-FNDA:0,BrrETH.mint
-FN:309,BrrETH.withdraw
-FNDA:0,BrrETH.withdraw
+DA:100,102
+DA:104,102
+DA:110,102
+FN:121,BrrETH.convertToShares
+FNDA:334,BrrETH.convertToShares
+DA:129,818
+FN:140,BrrETH._deposit
+FNDA:483,BrrETH._deposit
+DA:146,483
+DA:148,483
+FN:157,BrrETH.deposit
+FNDA:272,BrrETH.deposit
+DA:161,272
+DA:163,272
+DA:165,272
+DA:167,272
+DA:168,272
+DA:170,272
+BRDA:170,0,0,1
+BRDA:170,0,1,271
+DA:172,271
+FN:184,BrrETH.deposit
+FNDA:313,BrrETH.deposit
+DA:189,313
+BRDA:189,1,0,101
+BRDA:189,1,1,212
+DA:190,101
+DA:192,212
+DA:194,212
+DA:196,212
+DA:203,212
+FN:207,BrrETH.harvest
+FNDA:100,BrrETH.harvest
+DA:208,158
+DA:210,158
+DA:212,158
+DA:214,158
+BRDA:214,2,0,-
+BRDA:214,2,1,158
+DA:218,158
+DA:224,158
+DA:235,158
+DA:238,158
+BRDA:238,3,0,-
+BRDA:238,3,1,158
+DA:241,158
+DA:243,158
+DA:245,158
+DA:249,158
+DA:256,158
+DA:258,158
+FN:270,BrrETH.setCometRewards
+FNDA:104,BrrETH.setCometRewards
+DA:274,103
+BRDA:274,4,0,1
+BRDA:274,4,1,102
+DA:275,102
+BRDA:275,5,0,58
+BRDA:275,5,1,102
+DA:277,102
+DA:279,102
+FN:286,BrrETH.setRouter
+FNDA:103,BrrETH.setRouter
+DA:287,102
+BRDA:287,6,0,1
+BRDA:287,6,1,101
+DA:289,101
+DA:292,101
+DA:294,101
+FN:301,BrrETH.setRewardFee
+FNDA:203,BrrETH.setRewardFee
+DA:302,202
+BRDA:302,7,0,101
+BRDA:302,7,1,101
+DA:304,101
+DA:306,101
+FN:313,BrrETH.setProtocolFeeReceiver
+FNDA:3,BrrETH.setProtocolFeeReceiver
+DA:316,2
+BRDA:316,8,0,1
+BRDA:316,8,1,1
+DA:317,1
+DA:319,1
+DA:321,1
+FN:328,BrrETH.setFeeDistributor
+FNDA:56,BrrETH.setFeeDistributor
+DA:329,55
+BRDA:329,9,0,1
+BRDA:329,9,1,54
+DA:331,54
+DA:333,54
+FN:340,BrrETH.transferOwnership
+FNDA:1,BrrETH.transferOwnership
+DA:341,1
+FN:344,BrrETH.renounceOwnership
+FNDA:1,BrrETH.renounceOwnership
+DA:345,1
+FN:352,BrrETH.maxMint
+FNDA:1,BrrETH.maxMint
+DA:353,1
+FN:356,BrrETH.maxWithdraw
+FNDA:1,BrrETH.maxWithdraw
+DA:357,1
+FN:360,BrrETH.previewMint
+FNDA:1,BrrETH.previewMint
+DA:361,1
+FN:364,BrrETH.previewWithdraw
+FNDA:1,BrrETH.previewWithdraw
+DA:365,1
+FN:368,BrrETH.mint
+FNDA:1,BrrETH.mint
+DA:369,1
+FN:372,BrrETH.withdraw
+FNDA:1,BrrETH.withdraw
+DA:377,1
 FNF:22
-FNH:13
-LF:47
-LH:47
-BRF:14
-BRH:12
+FNH:22
+LF:62
+LH:62
+BRF:20
+BRH:18
+end_of_record
+TN:
+SF:src/BrrETHRedeemHelper.sol
+FN:32,BrrETHRedeemHelper.redeem
+FNDA:102,BrrETHRedeemHelper.redeem
+DA:34,102
+DA:37,102
+DA:39,102
+DA:41,102
+BRDA:41,0,0,1
+BRDA:41,0,1,101
+DA:42,1
+DA:44,101
+FNF:1
+FNH:1
+LF:6
+LH:6
+BRF:2
+BRH:2
 end_of_record
diff --git a/test/BrrETH.t.sol b/test/BrrETH.t.sol
@@ -834,4 +834,60 @@ contract BrrETHTest is Helper {
 
         assertEq(feeDistributor, vault.feeDistributor());
     }
+
+    /*//////////////////////////////////////////////////////////////
+                    Removed Ownable methods
+    //////////////////////////////////////////////////////////////*/
+
+    function testCannotTransferOwnershipRemovedOwnableMethod() external {
+        vm.expectRevert(BrrETH.RemovedOwnableMethod.selector);
+
+        vault.transferOwnership(address(0));
+    }
+
+    function testCannotRenounceOwnershipRemovedOwnableMethod() external {
+        vm.expectRevert(BrrETH.RemovedOwnableMethod.selector);
+
+        vault.renounceOwnership();
+    }
+
+    /*//////////////////////////////////////////////////////////////
+                    Removed ERC4626 methods
+    //////////////////////////////////////////////////////////////*/
+
+    function testCannotMaxMintRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.maxMint(address(0));
+    }
+
+    function testCannotMaxWithdrawRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.maxWithdraw(address(0));
+    }
+
+    function testCannotPreviewMintRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.previewMint(0);
+    }
+
+    function testCannotPreviewWithdrawRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.previewWithdraw(0);
+    }
+
+    function testCannotMintRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.mint(0, address(0));
+    }
+
+    function testCannotWithdrawRemovedERC4626Method() external {
+        vm.expectRevert(BrrETH.RemovedERC4626Method.selector);
+
+        vault.withdraw(0, address(0), address(0));
+    }
 }