docs: apply suggestions

Signed-off-by: Merlin Egalite <[email protected]>
morpho-org · Aug 15, 2023 · 4aa66f5 · 4aa66f5
1 parent 85fc322
commit 4aa66f5
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/test/forge/integration/TestIntegrationBorrow.t.sol b/test/forge/integration/TestIntegrationBorrow.t.sol
@@ -10,23 +10,22 @@ contract IntegrationBorrowTest is BaseTest {
     function testBorrowMarketNotCreated(
         Market memory marketFuzz,
         address borrowerFuzz,
-        address RECEIVER,
         uint256 amount
     ) public {
-        vm.assume(neq(marketFuzz, market) && RECEIVER != address(0));
+        vm.assume(neq(marketFuzz, market));
 
         vm.prank(borrowerFuzz);
         vm.expectRevert(bytes(ErrorsLib.MARKET_NOT_CREATED));
         morpho.borrow(marketFuzz, amount, 0, borrowerFuzz, RECEIVER);
     }
 
-    function testBorrowZeroAmount(address borrowerFuzz, address RECEIVER) public {
+    function testBorrowZeroAmount(address borrowerFuzz) public {
         vm.prank(borrowerFuzz);
         vm.expectRevert(bytes(ErrorsLib.INCONSISTENT_INPUT));
         morpho.borrow(market, 0, 0, borrowerFuzz, RECEIVER);
     }
 
-    function testBorrowInconsistentInput(address borrowerFuzz, uint256 amount, uint256 shares, address RECEIVER)
+    function testBorrowInconsistentInput(address borrowerFuzz, uint256 amount, uint256 shares)
         public
     {
         amount = bound(amount, 1, MAX_TEST_AMOUNT);
@@ -47,8 +46,8 @@ contract IntegrationBorrowTest is BaseTest {
         morpho.borrow(market, amount, 0, borrowerFuzz, address(0));
     }
 
-    function testBorrowUnauthorized(address supplier, address attacker, address RECEIVER, uint256 amount) public {
-        vm.assume(supplier != attacker && supplier != address(0) && RECEIVER != address(0));
+    function testBorrowUnauthorized(address supplier, address attacker, uint256 amount) public {
+        vm.assume(supplier != attacker && supplier != address(0));
         amount = bound(amount, 1, MAX_TEST_AMOUNT);
 
         _supply(amount);

diff --git a/test/forge/integration/TestIntegrationSupply.t.sol b/test/forge/integration/TestIntegrationSupply.t.sol
@@ -8,7 +8,7 @@ contract IntegrationSupplyTest is BaseTest {
     using SharesMathLib for uint256;
 
     function testSupplyMarketNotCreated(Market memory marketFuzz, uint256 amount) public {
-        vm.assume(neq(marketFuzz, market) && SUPPLIER != address(0));
+        vm.assume(neq(marketFuzz, market));
 
         vm.prank(SUPPLIER);
         vm.expectRevert(bytes(ErrorsLib.MARKET_NOT_CREATED));

diff --git a/test/forge/integration/TestIntegrationWithdrawCollateral.t.sol b/test/forge/integration/TestIntegrationWithdrawCollateral.t.sol
@@ -6,16 +6,15 @@ import "../BaseTest.sol";
 contract IntegrationWithdrawCollateralTest is BaseTest {
     using MathLib for uint256;
 
-    function testWithdrawCollateralMarketNotCreated(Market memory marketFuzz, address RECEIVER) public {
-        vm.assume(neq(marketFuzz, market) && RECEIVER != address(0));
+    function testWithdrawCollateralMarketNotCreated(Market memory marketFuzz) public {
+        vm.assume(neq(marketFuzz, market));
 
         vm.prank(SUPPLIER);
         vm.expectRevert(bytes(ErrorsLib.MARKET_NOT_CREATED));
         morpho.withdrawCollateral(marketFuzz, 1, SUPPLIER, RECEIVER);
     }
 
     function testWithdrawCollateralZeroAmount(uint256 amount) public {
-        vm.assume(SUPPLIER != address(0));
         amount = bound(amount, 1, MAX_TEST_AMOUNT);
 
         collateralToken.setBalance(SUPPLIER, amount);