add missing functions in interfaces (#219)

* add missing functions in interfaces

* fix interface treasury

Author: GuillaumeNervoXS

contracts/interfaces/ITreasury.sol

@@ -39,4 +39,7 @@ interface ITreasury {
     /// @dev This function removes the minting right to the old flash loan module and grants
     /// it to the new module
     function setFlashLoanModule(address _flashLoanModule) external;
+
+    /// @notice Gets the vault manager list
+    function vaultManagerList(uint256 i) external returns (address);
 }

contracts/interfaces/IVaultManager.sol

@@ -196,6 +196,9 @@ interface IVaultManagerFunctions {
 
     /// @notice Minimum amount of debt a vault can have, expressed in `BASE_TOKENS` that is to say the base of the agTokens
     function dust() external view returns (uint256);
+
+    /// @notice Pauses external permissionless functions of the contract
+    function togglePause() external;
 }
 
 /// @title IVaultManagerStorage

contracts/mock/MockTreasury.sol

@@ -13,6 +13,7 @@ contract MockTreasury is ITreasury {
     address public vaultManager1;
     address public vaultManager2;
     address public flashLoanModule;
+    address[] public vaultManagerList;
 
     constructor(
         IAgToken _stablecoin,

contracts/vaultManager/VaultManager.sol

@@ -84,7 +84,7 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
         if (_oracle.treasury() != _treasury) revert InvalidTreasury();
         treasury = _treasury;
         collateral = _collateral;
-        _collatBase = 10 ** (IERC20Metadata(address(collateral)).decimals());
+        _collatBase = 10**(IERC20Metadata(address(collateral)).decimals());
         stablecoin = IAgToken(_treasury.stablecoin());
         oracle = _oracle;
         string memory _name = string.concat("Angle Protocol ", _symbol, " Vault");
@@ -343,7 +343,7 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
 
         uint256 stablecoinAmountLessFeePaid = (stablecoinAmount *
             (BASE_PARAMS - repayFee_) *
-            (BASE_PARAMS - _borrowFee)) / (BASE_PARAMS ** 2);
+            (BASE_PARAMS - _borrowFee)) / (BASE_PARAMS**2);
         surplus += stablecoinAmount - stablecoinAmountLessFeePaid;
         _repayDebt(vaultID, stablecoinAmountLessFeePaid, 0);
     }
@@ -365,10 +365,11 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
     /// @param liquidator Address of the liquidator which will be performing the liquidation
     /// @return liqOpp Description of the opportunity of liquidation
     /// @dev This function will revert if it's called on a vault that does not exist
-    function checkLiquidation(
-        uint256 vaultID,
-        address liquidator
-    ) external view returns (LiquidationOpportunity memory liqOpp) {
+    function checkLiquidation(uint256 vaultID, address liquidator)
+        external
+        view
+        returns (LiquidationOpportunity memory liqOpp)
+    {
         liqOpp = _checkLiquidation(
             vaultData[vaultID],
             liquidator,
@@ -391,7 +392,15 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
         Vault memory vault,
         uint256 oracleValue,
         uint256 newInterestAccumulator
-    ) internal view returns (uint256 healthFactor, uint256 currentDebt, uint256 collateralAmountInStable) {
+    )
+        internal
+        view
+        returns (
+            uint256 healthFactor,
+            uint256 currentDebt,
+            uint256 collateralAmountInStable
+        )
+    {
         currentDebt = (vault.normalizedDebt * newInterestAccumulator) / BASE_INTEREST;
         collateralAmountInStable = (vault.collateralAmount * oracleValue) / _collatBase;
         if (currentDebt == 0) healthFactor = type(uint256).max;
@@ -798,7 +807,7 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
         // Checking if we're in a situation where the health factor is an increasing or a decreasing function of the
         // amount repaid. In the first case, the health factor is an increasing function which means that the liquidator
         // can bring the vault to the target health ratio
-        if (healthFactor * liquidationDiscount * surcharge >= collateralFactor * BASE_PARAMS ** 2) {
+        if (healthFactor * liquidationDiscount * surcharge >= collateralFactor * BASE_PARAMS**2) {
             // This is the max amount to repay that will bring the person to the target health factor
             // Denom is always positive when a vault gets liquidated in this case and when the health factor
             // is an increasing function of the amount of stablecoins repaid
@@ -807,7 +816,7 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
                 ((targetHealthFactor * currentDebt - collateralAmountInStable * collateralFactor) *
                     BASE_PARAMS *
                     liquidationDiscount) /
-                (surcharge * targetHealthFactor * liquidationDiscount - (BASE_PARAMS ** 2) * collateralFactor);
+                (surcharge * targetHealthFactor * liquidationDiscount - (BASE_PARAMS**2) * collateralFactor);
             // Need to check for the dust as liquidating should not leave a dusty amount in the vault
             uint256 dustParameter = dustLiquidation;
             if (currentDebt * BASE_PARAMS <= maxAmountToRepay * surcharge + dustParameter * BASE_PARAMS) {
@@ -911,7 +920,7 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
         emit LiquidationBoostParametersUpdated(_veBoostProxy, xBoost, yBoost);
     }
 
-    /// @notice Pauses external permissionless functions of the contract
+    /// @inheritdoc IVaultManagerFunctions
     function togglePause() external virtual onlyGovernorOrGuardian {
         paused = !paused;
     }
@@ -945,7 +954,11 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
     /// @param _dustLiquidation New `dustLiquidation` value
     /// @param dustCollateral_ New minimum collateral allowed in a vault after a liquidation
     /// @dev dustCollateral_ is in stable value
-    function setDusts(uint256 _dust, uint256 _dustLiquidation, uint256 dustCollateral_) external onlyGovernor {
+    function setDusts(
+        uint256 _dust,
+        uint256 _dustLiquidation,
+        uint256 dustCollateral_
+    ) external onlyGovernor {
         if (_dust > _dustLiquidation) revert InvalidParameterValue();
         dust = _dust;
         dustLiquidation = _dustLiquidation;
@@ -973,7 +986,11 @@ contract VaultManager is VaultManagerPermit, IVaultManagerFunctions {
     /// @param amount Collateral amount balance of the owner of vaultID increase/decrease
     /// @param add Whether the balance should be increased/decreased
     /// @param vaultID Vault which sees its collateral amount changed
-    function _checkpointCollateral(uint256 vaultID, uint256 amount, bool add) internal virtual {}
+    function _checkpointCollateral(
+        uint256 vaultID,
+        uint256 amount,
+        bool add
+    ) internal virtual {}
 
     /// @notice Get `paused` in storage only if needed
     function _paused() internal view virtual returns (bool) {