Merge pull request #49 from wildcat-finance/remove-repayOutstanding-repayDelinquent

Remove secondary repay functions [issue 47]

Author: laurenceday

src/market/WildcatMarket.sol

@@ -176,20 +176,6 @@ contract WildcatMarket is
     hooks.onRepay(amount, state, baseCalldataSize);
   }
 
-  function repayOutstandingDebt() external nonReentrant sphereXGuardExternal {
-    MarketState memory state = _getUpdatedState();
-    uint256 outstandingDebt = state.totalDebts().satSub(totalAssets());
-    _repay(state, outstandingDebt, 0x04);
-    _writeState(state);
-  }
-
-  function repayDelinquentDebt() external nonReentrant sphereXGuardExternal {
-    MarketState memory state = _getUpdatedState();
-    uint256 delinquentDebt = state.liquidityRequired().satSub(totalAssets());
-    _repay(state, delinquentDebt, 0x04);
-    _writeState(state);
-  }
-
   /**
    * @dev Transfers funds from the caller to the market.
    *

test/helpers/fuzz/AccessControlHooksFuzzContext.sol

@@ -281,7 +281,7 @@ library LibAccessControlHooksFuzzContext {
         context.functionKind == FunctionKind.DepositFunction
       ) {
         vm.expectEmit(address(context.hooks));
-        emit AccessControlHooks.AccountMadeFirstDeposit(context.account);
+        emit AccessControlHooks.AccountMadeFirstDeposit(context.market, context.account);
       }
     }
     if (context.expectations.expectedError != 0) {

test/market/WildcatMarket.t.sol

@@ -351,70 +351,6 @@ contract WildcatMarketTest is BaseMarketTest {
     market.repay(1e18);
   }
 
-  /* ========================================================================== */
-  /*                           repayOutstandingDebt()                           */
-  /* ========================================================================== */
-
-  function test_repayOutstandingDebt() external {
-    _depositBorrowWithdraw(alice, 1e18, 8e17, 1e18);
-    asset.mint(address(this), 8e17);
-    asset.approve(address(market), 8e17);
-    vm.expectEmit(address(market));
-    emit DebtRepaid(address(this), 8e17);
-    market.repayOutstandingDebt();
-  }
-
-  function test_repayOutstandingDebt_NullRepayAmount() external {
-    vm.expectRevert(IMarketEventsAndErrors.NullRepayAmount.selector);
-    market.repayOutstandingDebt();
-  }
-
-  /* ========================================================================== */
-  /*                            repayDelinquentDebt()                           */
-  /* ========================================================================== */
-
-  function test_repayDelinquentDebt() external {
-    _depositBorrowWithdraw(alice, 1e18, 8e17, 2e17); // 20% of 8e17
-    asset.mint(address(this), 1.6e17);
-    asset.approve(address(market), 1.6e17);
-    vm.expectEmit(address(market));
-    emit DebtRepaid(address(this), 1.6e17);
-    market.repayDelinquentDebt();
-  }
-
-  function test_repayDelinquentDebt2() public asAccount(borrower) {
-    assertEq(market.delinquencyGracePeriod(), 2000);
-    parameters.delinquencyGracePeriod = 86_400;
-    parameters.withdrawalBatchDuration = 0;
-    setUp();
-
-    assertEq(market.delinquencyGracePeriod(), 86_400, 'delinquencyGracePeriod');
-    assertEq(market.delinquencyFeeBips(), 1_000, 'delinquencyFeeBips');
-    _depositBorrowWithdraw(alice, 1e18, 8e17, 1e18);
-
-    assertTrue(market.currentState().isDelinquent, 'should be delinquent');
-    fastForward(365 days);
-
-    MarketState memory state = pendingState();
-    updateState(state);
-    market.updateState();
-    _checkState();
-    uint delinquentDebt = state.liquidityRequired().satSub(lastTotalAssets);
-    asset.mint(borrower, delinquentDebt);
-    asset.approve(address(market), delinquentDebt);
-
-    state = pendingState();
-    _trackRepay(state, borrower, delinquentDebt);
-    updateState(state);
-    market.repayDelinquentDebt();
-    _checkState();
-  }
-
-  function test_repayDelinquentDebt_NullRepayAmount() external {
-    vm.expectRevert(IMarketEventsAndErrors.NullRepayAmount.selector);
-    market.repayDelinquentDebt();
-  }
-
   // ========================================================================== //
   //                            Credentials Provider                            //
   // ========================================================================== //