feat(aave steward): rm max variable borrow rate check

Author: DhairyaSethi

src/contracts/misc/GhoAaveSteward.sol

@@ -21,9 +21,6 @@ import {RiskCouncilControlled} from './RiskCouncilControlled.sol';
 contract GhoAaveSteward is Ownable, RiskCouncilControlled, IGhoAaveSteward {
   using ReserveConfiguration for DataTypes.ReserveConfigurationMap;
 
-  /// @inheritdoc IGhoAaveSteward
-  uint32 public constant GHO_BORROW_RATE_MAX = 0.25e4; // 25.00%
-
   uint256 internal constant BPS_MAX = 100_00;
 
   /// @inheritdoc IGhoAaveSteward
@@ -227,14 +224,6 @@ contract GhoAaveSteward is Ownable, RiskCouncilControlled, IGhoAaveSteward {
       ),
       'INVALID_VARIABLE_RATE_SLOPE2'
     );
-
-    require(
-      uint256(newRates.baseVariableBorrowRate) +
-        uint256(newRates.variableRateSlope1) +
-        uint256(newRates.variableRateSlope2) <=
-        GHO_BORROW_RATE_MAX,
-      'BORROW_RATE_HIGHER_THAN_MAX'
-    );
   }
 
   /**

src/contracts/misc/interfaces/IGhoAaveSteward.sol

@@ -90,12 +90,6 @@ interface IGhoAaveSteward {
    */
   function getGhoTimelocks() external view returns (GhoDebounce memory);
 
-  /**
-   * @notice Returns maximum value that can be assigned to GHO borrow rate.
-   * @return The maximum value that can be assigned to GHO borrow rate in ray (e.g. 0.01e27 results in 1.0%)
-   */
-  function GHO_BORROW_RATE_MAX() external view returns (uint32);
-
   /**
    * @notice The address of pool data provider of the POOL the steward controls
    */

src/test/TestGhoAaveSteward.t.sol

@@ -327,6 +327,21 @@ contract TestGhoAaveSteward is TestGhoBase {
     assertEq(currentBorrowRate, newBorrowRate);
   }
 
+  function testUpdateGhoBorrowRateUpwardsFromHigh() public {
+    // set a very high borrow rate of 80%
+    uint32 highBaseBorrowRate = 0.80e4;
+    _setGhoBorrowRateViaConfigurator(highBaseBorrowRate);
+    highBaseBorrowRate += 0.04e4;
+    vm.prank(RISK_COUNCIL);
+    GHO_AAVE_STEWARD.updateGhoBorrowRate(
+      defaultRateParams.optimalUsageRatio,
+      highBaseBorrowRate,
+      defaultRateParams.variableRateSlope1,
+      defaultRateParams.variableRateSlope2
+    );
+    assertEq(highBaseBorrowRate, _getGhoBorrowRate());
+  }
+
   function testUpdateGhoBorrowRateDownwards() public {
     uint32 oldBorrowRate = _getGhoBorrowRate();
     uint32 newBorrowRate = oldBorrowRate - 1;
@@ -341,18 +356,19 @@ contract TestGhoAaveSteward is TestGhoBase {
     assertEq(currentBorrowRate, newBorrowRate);
   }
 
-  function testUpdateGhoBorrowRateMaxValue() public {
-    uint32 ghoBorrowRateMax = GHO_AAVE_STEWARD.GHO_BORROW_RATE_MAX();
-    _setGhoBorrowRateViaConfigurator(ghoBorrowRateMax - 1);
+  function testUpdateGhoBorrowRateDownwardsFromHigh() public {
+    // set a very high borrow rate of 80%
+    uint32 highBaseBorrowRate = 0.80e4;
+    _setGhoBorrowRateViaConfigurator(highBaseBorrowRate);
+    highBaseBorrowRate -= 0.04e4;
     vm.prank(RISK_COUNCIL);
     GHO_AAVE_STEWARD.updateGhoBorrowRate(
       defaultRateParams.optimalUsageRatio,
-      ghoBorrowRateMax,
+      highBaseBorrowRate,
       defaultRateParams.variableRateSlope1,
       defaultRateParams.variableRateSlope2
     );
-    uint32 currentBorrowRate = _getGhoBorrowRate();
-    assertEq(currentBorrowRate, ghoBorrowRateMax);
+    assertEq(highBaseBorrowRate, _getGhoBorrowRate());
   }
 
   function testUpdateGhoBorrowRateMaxIncrement() public {
@@ -660,19 +676,6 @@ contract TestGhoAaveSteward is TestGhoBase {
     );
   }
 
-  function testRevertUpdateGhoBorrowRateIfValueMoreThanMax() public {
-    uint32 maxGhoBorrowRate = GHO_BORROW_RATE_MAX;
-    _setGhoBorrowRateViaConfigurator(maxGhoBorrowRate);
-    vm.prank(RISK_COUNCIL);
-    vm.expectRevert('BORROW_RATE_HIGHER_THAN_MAX');
-    GHO_AAVE_STEWARD.updateGhoBorrowRate(
-      defaultRateParams.optimalUsageRatio,
-      maxGhoBorrowRate + 1,
-      defaultRateParams.variableRateSlope1,
-      defaultRateParams.variableRateSlope2
-    );
-  }
-
   function testRevertUpdateGhoBorrowRateIfMaxExceededUpwards() public {
     uint32 oldBorrowRate = _getGhoBorrowRate();
     uint32 newBorrowRate = oldBorrowRate + GHO_BORROW_RATE_CHANGE_MAX + 1;

src/test/helpers/Constants.sol

@@ -52,7 +52,6 @@ contract Constants {
   // Gho Stewards
   uint32 constant GHO_BORROW_RATE_CHANGE_MAX = 0.05e4;
   uint256 constant GSM_FEE_RATE_CHANGE_MAX = 0.0050e4;
-  uint32 constant GHO_BORROW_RATE_MAX = 0.25e4;
   uint256 constant MINIMUM_DELAY_V2 = 1 days;
   uint256 constant FIXED_RATE_STRATEGY_FACTORY_REVISION = 1;