From 7c8a9c6451ee1acfae88c5cc9e4e6c48288b350b Mon Sep 17 00:00:00 2001 From: kphed Date: Fri, 12 Jan 2024 13:11:53 -0500 Subject: [PATCH] Add tests for removed Ownable and ERC4626 methods, update test coverage report --- lcov.info | 278 +++++++++++++++++++++++++++------------------- test/BrrETH.t.sol | 56 ++++++++++ 2 files changed, 222 insertions(+), 112 deletions(-) diff --git a/lcov.info b/lcov.info index 56cd4f4..8a2b682 100644 --- 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 index 7235ad8..707c76f 100644 --- 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)); + } }