Remove Repaid event from USDC pool

Author: viatrix

contracts/LiquidityPool.sol

@@ -72,7 +72,6 @@ contract LiquidityPool is ILiquidityPool, AccessControl, EIP712 {
     event MPCAddressSet(address oldMPCAddress, address newMPCAddress);
     event Paused(address account);
     event Unpaused(address account);
-    event Repaid(address borrowToken, uint256 repaidAmount);
 
     modifier whenNotPaused() {
         require(!paused, EnforcedPause());
@@ -181,7 +180,7 @@ contract LiquidityPool is ILiquidityPool, AccessControl, EIP712 {
     }
 
     function repay(address[] calldata) external virtual override {
-        emit Repaid(address(0), 0);
+        revert NotImplemented();
     }
 
     // Admin functions

contracts/LiquidityPoolAave.sol

@@ -49,6 +49,7 @@ contract LiquidityPoolAave is LiquidityPool {
     event HealthFactorSet(uint256 oldHealthFactor, uint256 newHealthFactor);
     event DefaultLTVSet(uint256 oldDefaultLTV, uint256 newDefaultLTV);
     event WithdrawnFromAave(address to, uint256 amount);
+    event Repaid(address borrowToken, uint256 repaidAmount);
 
     constructor(
         address liquidityToken,

contracts/Repayer.sol

@@ -146,9 +146,6 @@ contract Repayer is IRepayer, AccessControlUpgradeable, CCTPAdapter {
         if (provider == Provider.CCTP) {
             amount = processTransferCCTP(CCTP_MESSAGE_TRANSMITTER, ASSETS, destinationPool, extraData);
             require(amount > 0, ZeroAmount());
-            address[] memory tokens = new address[](1);
-            tokens[0] = address(ASSETS);
-            ILiquidityPool(destinationPool).repay(tokens);
         } else {
             // Unreachable atm, but could become so when more providers are added to enum.
             revert UnsupportedProvider();
@@ -163,9 +160,6 @@ contract Repayer is IRepayer, AccessControlUpgradeable, CCTPAdapter {
         address destinationPool
     ) internal {
         token.safeTransfer(destinationPool, amount);
-        address[] memory tokens = new address[](1);
-        tokens[0] = address(token);
-        ILiquidityPool(destinationPool).repay(tokens);
         emit ProcessRepay(token, amount, destinationPool, Provider.LOCAL);
     }
 

test/Repayer.ts

@@ -296,8 +296,6 @@ describe("Repayer", function () {
     await expect(tx)
       .to.emit(uni, "Transfer")
       .withArgs(repayer.target, liquidityPool.target, 4n * UNI_DEC);
-    await expect(tx)
-      .to.emit(liquidityPool, "Repaid");
 
     expect(await uni.balanceOf(repayer.target)).to.equal(6n * UNI_DEC);
     expect(await uni.balanceOf(liquidityPool.target)).to.equal(4n * UNI_DEC);
@@ -322,8 +320,6 @@ describe("Repayer", function () {
     await expect(tx)
       .to.emit(usdc, "Transfer")
       .withArgs(repayer.target, liquidityPool2.target, 4n * USDC_DEC);
-    await expect(tx)
-      .to.emit(liquidityPool2, "Repaid");
 
     expect(await usdc.balanceOf(repayer.target)).to.equal(6n * USDC_DEC);
     expect(await usdc.balanceOf(liquidityPool2.target)).to.equal(4n * USDC_DEC);
@@ -438,8 +434,6 @@ describe("Repayer", function () {
     await expect(tx)
       .to.emit(usdc, "Transfer")
       .withArgs(ZERO_ADDRESS, liquidityPool.target, 4n * USDC_DEC);
-    await expect(tx)
-      .to.emit(liquidityPool, "Repaid");
 
     expect(await usdc.balanceOf(liquidityPool.target)).to.equal(4n * USDC_DEC);
     expect(await usdc.balanceOf(repayer.target)).to.equal(0n);