@ensuro/core 2.6.2 → 2.8.0-beta1

package/contracts/PolicyPool.sol

@@ -1,20 +1,23 @@
 // SPDX-License-Identifier: Apache-2.0
 pragma solidity ^0.8.0;
 
-import {IERC20Metadata} from "@openzeppelin/contracts/token/ERC20/extensions/IERC20Metadata.sol";
 import {AddressUpgradeable} from "@openzeppelin/contracts-upgradeable/utils/AddressUpgradeable.sol";
-import {UUPSUpgradeable} from "@openzeppelin/contracts-upgradeable/proxy/utils/UUPSUpgradeable.sol";
+import {ERC165Checker} from "@openzeppelin/contracts/utils/introspection/ERC165Checker.sol";
 import {ERC721Upgradeable} from "@openzeppelin/contracts-upgradeable/token/ERC721/ERC721Upgradeable.sol";
+import {IERC20Metadata} from "@openzeppelin/contracts/token/ERC20/extensions/IERC20Metadata.sol";
+import {PausableUpgradeable} from "@openzeppelin/contracts-upgradeable/security/PausableUpgradeable.sol";
 import {SafeERC20} from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
+import {UUPSUpgradeable} from "@openzeppelin/contracts-upgradeable/proxy/utils/UUPSUpgradeable.sol";
+
 import {IAccessManager} from "./interfaces/IAccessManager.sol";
-import {PausableUpgradeable} from "@openzeppelin/contracts-upgradeable/security/PausableUpgradeable.sol";
-import {IPremiumsAccount} from "./interfaces/IPremiumsAccount.sol";
-import {IPolicyPool} from "./interfaces/IPolicyPool.sol";
-import {IRiskModule} from "./interfaces/IRiskModule.sol";
-import {IPolicyPoolComponent} from "./interfaces/IPolicyPoolComponent.sol";
 import {IEToken} from "./interfaces/IEToken.sol";
 import {IPolicyHolder} from "./interfaces/IPolicyHolder.sol";
+import {IPolicyPool} from "./interfaces/IPolicyPool.sol";
+import {IPolicyPoolComponent} from "./interfaces/IPolicyPoolComponent.sol";
+import {IPremiumsAccount} from "./interfaces/IPremiumsAccount.sol";
+import {IRiskModule} from "./interfaces/IRiskModule.sol";
 import {Policy} from "./Policy.sol";
+
 import {WadRayMath} from "./dependencies/WadRayMath.sol";
 
 /**
@@ -156,11 +159,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * @param kind Value indicating the kind of component. See {ComponentKind}
    * @param newStatus The status of the component after the operation. See {ComponentStatus}
    */
-  event ComponentStatusChanged(
-    IPolicyPoolComponent indexed component,
-    ComponentKind kind,
-    ComponentStatus newStatus
-  );
+  event ComponentStatusChanged(IPolicyPoolComponent indexed component, ComponentKind kind, ComponentStatus newStatus);
 
   /**
    * @dev Modifier that checks the caller has a given role
@@ -201,11 +200,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * @param symbol_ The symbol of the ERC721 token.
    * @param treasury_ The address of the treasury that will receive the protocol fees.
    */
-  function initialize(
-    string memory name_,
-    string memory symbol_,
-    address treasury_
-  ) public initializer {
+  function initialize(string memory name_, string memory symbol_, address treasury_) public initializer {
     require(bytes(name_).length > 0, "PolicyPool: name cannot be empty");
     require(bytes(symbol_).length > 0, "PolicyPool: symbol cannot be empty");
     __UUPSUpgradeable_init();
@@ -226,12 +221,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
     _setTreasury(treasury_);
   }
 
-  function _authorizeUpgrade(address newImpl)
-    internal
-    view
-    override
-    onlyRole2(GUARDIAN_ROLE, LEVEL1_ROLE)
-  {
+  function _authorizeUpgrade(address newImpl) internal view override onlyRole2(GUARDIAN_ROLE, LEVEL1_ROLE) {
     IPolicyPool newPool = IPolicyPool(newImpl);
     require(newPool.access() == _access, "Can't upgrade changing the access manager");
     require(newPool.currency() == _currency, "Can't upgrade changing the currency");
@@ -307,20 +297,15 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * to this specific {PolicyPool} and matching the `kind` specified in the next paramter.
    * @param kind The type of component to be added.
    */
-  function addComponent(IPolicyPoolComponent component, ComponentKind kind)
-    external
-    onlyRole(LEVEL1_ROLE)
-  {
+  function addComponent(IPolicyPoolComponent component, ComponentKind kind) external onlyRole(LEVEL1_ROLE) {
     Component storage comp = _components[component];
     require(comp.status == ComponentStatus.inactive, "Component already in the pool");
     require(component.policyPool() == this, "Component not linked to this pool");
 
     require(
       (kind == ComponentKind.eToken && component.supportsInterface(type(IEToken).interfaceId)) ||
-        (kind == ComponentKind.premiumsAccount &&
-          component.supportsInterface(type(IPremiumsAccount).interfaceId)) ||
-        (kind == ComponentKind.riskModule &&
-          component.supportsInterface(type(IRiskModule).interfaceId)),
+        (kind == ComponentKind.premiumsAccount && component.supportsInterface(type(IPremiumsAccount).interfaceId)) ||
+        (kind == ComponentKind.riskModule && component.supportsInterface(type(IRiskModule).interfaceId)),
       "PolicyPool: Not the right kind"
     );
 
@@ -357,15 +342,9 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
     Component storage comp = _components[component];
     require(comp.status == ComponentStatus.deprecated, "Component not deprecated");
     if (comp.kind == ComponentKind.eToken) {
-      require(
-        IEToken(address(component)).totalSupply() == 0,
-        "EToken has liquidity, can't be removed"
-      );
+      require(IEToken(address(component)).totalSupply() == 0, "EToken has liquidity, can't be removed");
     } else if (comp.kind == ComponentKind.riskModule) {
-      require(
-        IRiskModule(address(component)).activeExposure() == 0,
-        "Can't remove a module with active policies"
-      );
+      require(IRiskModule(address(component)).activeExposure() == 0, "Can't remove a module with active policies");
     } else if (comp.kind == ComponentKind.premiumsAccount) {
       IPremiumsAccount pa = IPremiumsAccount(address(component));
       require(pa.purePremiums() == 0, "Can't remove a PremiumsAccount with premiums");
@@ -395,10 +374,10 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * @param component The address of component contract. Must be a component added before.
    * @param newStatus The new status, must be either `active`, `deprecated` or `suspended`.
    */
-  function changeComponentStatus(IPolicyPoolComponent component, ComponentStatus newStatus)
-    external
-    onlyRole2(GUARDIAN_ROLE, LEVEL1_ROLE)
-  {
+  function changeComponentStatus(
+    IPolicyPoolComponent component,
+    ComponentStatus newStatus
+  ) external onlyRole2(GUARDIAN_ROLE, LEVEL1_ROLE) {
     Component storage comp = _components[component];
     require(comp.status != ComponentStatus.inactive, "Component not found");
     require(
@@ -417,11 +396,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * @param component The address of the component
    * @return The status of the component. See {ComponentStatus}
    */
-  function getComponentStatus(IPolicyPoolComponent component)
-    external
-    view
-    returns (ComponentStatus)
-  {
+  function getComponentStatus(IPolicyPoolComponent component) external view returns (ComponentStatus) {
     return _components[component].status;
   }
 
@@ -450,12 +425,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
     eToken.deposit(_msgSender(), _currency.balanceOf(address(eToken)) - balanceBefore);
   }
 
-  function withdraw(IEToken eToken, uint256 amount)
-    external
-    override
-    whenNotPaused
-    returns (uint256)
-  {
+  function withdraw(IEToken eToken, uint256 amount) external override whenNotPaused returns (uint256) {
     ComponentStatus etkStatus = _etkStatus(eToken);
     require(
       etkStatus == ComponentStatus.active || etkStatus == ComponentStatus.deprecated,
@@ -520,19 +490,14 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
     }
   }
 
-  function resolvePolicy(Policy.PolicyData calldata policy, uint256 payout)
-    external
-    override
-    whenNotPaused
-  {
+  function resolvePolicy(Policy.PolicyData calldata policy, uint256 payout) external override whenNotPaused {
     return _resolvePolicy(policy, payout, false);
   }
 
-  function resolvePolicyFullPayout(Policy.PolicyData calldata policy, bool customerWon)
-    external
-    override
-    whenNotPaused
-  {
+  function resolvePolicyFullPayout(
+    Policy.PolicyData calldata policy,
+    bool customerWon
+  ) external override whenNotPaused {
     return _resolvePolicy(policy, customerWon ? policy.payout : 0, false);
   }
 
@@ -555,11 +520,7 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
    * @param payout The amount to paid to the policyholder
    * @param expired True for expiration resolution (`payout` must be 0)
    */
-  function _resolvePolicy(
-    Policy.PolicyData memory policy,
-    uint256 payout,
-    bool expired
-  ) internal {
+  function _resolvePolicy(Policy.PolicyData memory policy, uint256 payout, bool expired) internal {
     // Checks
     _validatePolicy(policy);
     IRiskModule rm = policy.riskModule;
@@ -601,29 +562,16 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
   }
 
   /**
-   * @dev Notifies the payout with a callback if the policyholder is a contract. Only reverts if the policyholder
-   * contract explicitly reverts. Doesn't reverts is the callback is not implemented.
+   * @dev Notifies the payout with a callback if the policyholder is a contract and implementes the IPolicyHolder interface.
+   * Only reverts if the policyholder contract explicitly reverts or it doesn't return the IPolicyHolder.onPayoutReceived selector.
    */
   function _notifyPayout(uint256 policyId, uint256 payout) internal {
     address customer = ownerOf(policyId);
     if (!AddressUpgradeable.isContract(customer)) return;
-    try
-      IPolicyHolder(customer).onPayoutReceived(_msgSender(), address(this), policyId, payout)
-    returns (bytes4 retval) {
-      require(
-        retval == IPolicyHolder.onPayoutReceived.selector,
-        "Invalid return value from Policy Holder"
-      );
-    } catch (bytes memory reason) {
-      if (reason.length == 0) {
-        return; // Not implemented, it's fine
-      } else {
-        // solhint-disable-next-line no-inline-assembly
-        assembly {
-          revert(add(32, reason), mload(reason))
-        }
-      }
-    }
+    if (!ERC165Checker.supportsInterface(customer, type(IPolicyHolder).interfaceId)) return;
+
+    bytes4 retval = IPolicyHolder(customer).onPayoutReceived(_msgSender(), address(this), policyId, payout);
+    require(retval == IPolicyHolder.onPayoutReceived.selector, "PolicyPool: Invalid return value from IPolicyHolder");
   }
 
   /**
@@ -632,9 +580,9 @@ contract PolicyPool is IPolicyPool, PausableUpgradeable, UUPSUpgradeable, ERC721
   function _notifyExpiration(uint256 policyId) internal {
     address customer = ownerOf(policyId);
     if (!AddressUpgradeable.isContract(customer)) return;
-    try IPolicyHolder(customer).onPolicyExpired(_msgSender(), address(this), policyId) returns (
-      bytes4
-    ) {
+    if (!ERC165Checker.supportsInterface(customer, type(IPolicyHolder).interfaceId)) return;
+
+    try IPolicyHolder(customer).onPolicyExpired(_msgSender(), address(this), policyId) returns (bytes4) {
       return;
     } catch {
       return;

package/contracts/PolicyPoolComponent.sol

@@ -24,11 +24,7 @@ import {WadRayMath} from "./dependencies/WadRayMath.sol";
  * @custom:security-contact security@ensuro.co
  * @author Ensuro
  */
-abstract contract PolicyPoolComponent is
-  UUPSUpgradeable,
-  PausableUpgradeable,
-  IPolicyPoolComponent
-{
+abstract contract PolicyPoolComponent is UUPSUpgradeable, PausableUpgradeable, IPolicyPoolComponent {
   using WadRayMath for uint256;
 
   bytes32 public constant GUARDIAN_ROLE = keccak256("GUARDIAN_ROLE");
@@ -80,10 +76,7 @@ abstract contract PolicyPoolComponent is
 
   /// @custom:oz-upgrades-unsafe-allow constructor
   constructor(IPolicyPool policyPool_) {
-    require(
-      address(policyPool_) != address(0),
-      "PolicyPoolComponent: policyPool cannot be zero address"
-    );
+    require(address(policyPool_) != address(0), "PolicyPoolComponent: policyPool cannot be zero address");
     _disableInitializers();
     _policyPool = policyPool_;
   }
@@ -94,29 +87,21 @@ abstract contract PolicyPoolComponent is
     __Pausable_init();
   }
 
-  function _authorizeUpgrade(address newImpl)
-    internal
-    view
-    override
-    onlyGlobalOrComponentRole2(GUARDIAN_ROLE, LEVEL1_ROLE)
-  {
+  function _authorizeUpgrade(
+    address newImpl
+  ) internal view override onlyGlobalOrComponentRole2(GUARDIAN_ROLE, LEVEL1_ROLE) {
     _upgradeValidations(newImpl);
   }
 
   function _upgradeValidations(address newImpl) internal view virtual {
-    require(
-      IPolicyPoolComponent(newImpl).policyPool() == _policyPool,
-      "Can't upgrade changing the PolicyPool!"
-    );
+    require(IPolicyPoolComponent(newImpl).policyPool() == _policyPool, "Can't upgrade changing the PolicyPool!");
   }
 
   /**
    * @dev See {IERC165-supportsInterface}.
    */
   function supportsInterface(bytes4 interfaceId) public view virtual override returns (bool) {
-    return
-      interfaceId == type(IERC165).interfaceId ||
-      interfaceId == type(IPolicyPoolComponent).interfaceId;
+    return interfaceId == type(IERC165).interfaceId || interfaceId == type(IPolicyPoolComponent).interfaceId;
   }
 
   function pause() public onlyGlobalOrComponentRole(GUARDIAN_ROLE) {
@@ -139,11 +124,7 @@ abstract contract PolicyPoolComponent is
     return _policyPool.access().hasComponentRole(address(this), role, _msgSender(), true);
   }
 
-  function _isTweakWad(
-    uint256 oldValue,
-    uint256 newValue,
-    uint256 maxTweak
-  ) internal pure returns (bool) {
+  function _isTweakWad(uint256 oldValue, uint256 newValue, uint256 maxTweak) internal pure returns (bool) {
     if (oldValue == newValue) return true;
     if (oldValue == 0) return maxTweak >= WadRayMath.WAD;
     if (newValue == 0) return false;
@@ -157,11 +138,7 @@ abstract contract PolicyPoolComponent is
   // solhint-disable-next-line no-empty-blocks
   function _validateParameters() internal view virtual {} // Must be reimplemented with specific validations
 
-  function _parameterChanged(
-    IAccessManager.GovernanceActions action,
-    uint256 value,
-    bool tweak
-  ) internal {
+  function _parameterChanged(IAccessManager.GovernanceActions action, uint256 value, bool tweak) internal {
     _validateParameters();
     if (tweak) _registerTweak(action);
     emit GovernanceAction(action, value);

package/contracts/PremiumsAccount.sol

@@ -113,11 +113,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    * @param seniorEtk_ Address of the Senior EToken (2nd loss lender). `address(0)` if not present.
    */
   /// @custom:oz-upgrades-unsafe-allow constructor
-  constructor(
-    IPolicyPool policyPool_,
-    IEToken juniorEtk_,
-    IEToken seniorEtk_
-  ) Reserve(policyPool_) {
+  constructor(IPolicyPool policyPool_, IEToken juniorEtk_, IEToken seniorEtk_) Reserve(policyPool_) {
     _juniorEtk = juniorEtk_;
     _seniorEtk = seniorEtk_;
   }
@@ -169,8 +165,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    * @dev See {IERC165-supportsInterface}.
    */
   function supportsInterface(bytes4 interfaceId) public view virtual override returns (bool) {
-    return
-      super.supportsInterface(interfaceId) || interfaceId == type(IPremiumsAccount).interfaceId;
+    return super.supportsInterface(interfaceId) || interfaceId == type(IPremiumsAccount).interfaceId;
   }
 
   function assetManager() public view override returns (IAssetManager) {
@@ -274,12 +269,12 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
 
   function _toAmount(uint32 value) internal view returns (uint256) {
     // 0 decimals to amount decimals
-    return uint256(value) * 10**currency().decimals();
+    return uint256(value) * 10 ** currency().decimals();
   }
 
   function _toZeroDecimals(uint256 amount) internal view returns (uint32) {
     // Removes the decimals from the amount
-    return (amount / 10**currency().decimals()).toUint32();
+    return (amount / 10 ** currency().decimals()).toUint32();
   }
 
   /**
@@ -316,15 +311,9 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    *                   `_maxDeficit()` and borrows the difference from the eTokens.
    *                   If false and the new ratio leaves `_surplus < -_maxDeficit()`, the operation is reverted.
    */
-  function setDeficitRatio(uint256 newRatio, bool adjustment)
-    external
-    onlyGlobalOrComponentRole(LEVEL2_ROLE)
-  {
+  function setDeficitRatio(uint256 newRatio, bool adjustment) external onlyGlobalOrComponentRole(LEVEL2_ROLE) {
     uint16 truncatedRatio = (newRatio / FOUR_DECIMAL_TO_WAD).toUint16();
-    require(
-      uint256(truncatedRatio) * FOUR_DECIMAL_TO_WAD == newRatio,
-      "Validation: only up to 4 decimals allowed"
-    );
+    require(uint256(truncatedRatio) * FOUR_DECIMAL_TO_WAD == newRatio, "Validation: only up to 4 decimals allowed");
 
     int256 maxDeficit = _maxDeficit(newRatio);
     require(adjustment || _surplus >= maxDeficit, "Validation: surplus must be >= maxDeficit");
@@ -356,10 +345,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    * @param newLimitSr     The new limit to be set for the loans taken from the Senior eToken.
                            If newLimitSr == MAX_UINT, it's ignored. If == 0, means the loans are unbounded.
    */
-  function setLoanLimits(uint256 newLimitJr, uint256 newLimitSr)
-    external
-    onlyGlobalOrComponentRole(LEVEL2_ROLE)
-  {
+  function setLoanLimits(uint256 newLimitJr, uint256 newLimitSr) external onlyGlobalOrComponentRole(LEVEL2_ROLE) {
     if (newLimitJr != type(uint256).max) {
       _params.jrLoanLimit = _toZeroDecimals(newLimitJr);
       require(_toAmount(_params.jrLoanLimit) == newLimitJr, "Validation: no decimals allowed");
@@ -381,11 +367,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    *                 in the context of a policy payout.
    * @param jrEtk If true it indicates that the loan is asked first from the junior eToken.
    */
-  function _borrowFromEtk(
-    uint256 borrow,
-    address receiver,
-    bool jrEtk
-  ) internal {
+  function _borrowFromEtk(uint256 borrow, address receiver, bool jrEtk) internal {
     uint256 left = borrow;
     if (jrEtk) {
       if (_juniorEtk.getLoan(address(this)) + borrow <= jrLoanLimit()) {
@@ -471,11 +453,10 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
    * @param destination The address that will receive the transferred funds.
    * @return Returns the actual amount withdrawn.
    */
-  function withdrawWonPremiums(uint256 amount, address destination)
-    external
-    onlyGlobalOrComponentRole(WITHDRAW_WON_PREMIUMS_ROLE)
-    returns (uint256)
-  {
+  function withdrawWonPremiums(
+    uint256 amount,
+    address destination
+  ) external onlyGlobalOrComponentRole(WITHDRAW_WON_PREMIUMS_ROLE) returns (uint256) {
     require(destination != address(0), "PremiumsAccount: destination cannot be the zero address");
     if (_surplus <= 0) {
       amount = 0;
@@ -489,12 +470,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
     return amount;
   }
 
-  function policyCreated(Policy.PolicyData memory policy)
-    external
-    override
-    onlyPolicyPool
-    whenNotPaused
-  {
+  function policyCreated(Policy.PolicyData memory policy) external override onlyPolicyPool whenNotPaused {
     _activePurePremiums += policy.purePremium;
     if (policy.jrScr > 0) _juniorEtk.lockScr(policy.jrScr, policy.jrInterestRate());
     if (policy.srScr > 0) _seniorEtk.lockScr(policy.srScr, policy.srInterestRate());
@@ -550,10 +526,8 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
     returns (uint256 available)
   {
     available = fundsAvailable();
-    if (available != 0 && address(_seniorEtk) != address(0))
-      available = _repayLoan(available, _seniorEtk);
-    if (available != 0 && address(_juniorEtk) != address(0))
-      available = _repayLoan(available, _juniorEtk);
+    if (available != 0 && address(_seniorEtk) != address(0)) available = _repayLoan(available, _seniorEtk);
+    if (available != 0 && address(_juniorEtk) != address(0)) available = _repayLoan(available, _juniorEtk);
     return available;
   }
 
@@ -589,12 +563,7 @@ contract PremiumsAccount is IPremiumsAccount, Reserve {
     return fundsAvailable_ - repayAmount;
   }
 
-  function policyExpired(Policy.PolicyData memory policy)
-    external
-    override
-    onlyPolicyPool
-    whenNotPaused
-  {
+  function policyExpired(Policy.PolicyData memory policy) external override onlyPolicyPool whenNotPaused {
     _activePurePremiums -= policy.purePremium;
     _storePurePremiumWon(policy.purePremium);
     _unlockScr(policy);

package/contracts/Reserve.sol

@@ -36,7 +36,7 @@ abstract contract Reserve is PolicyPoolComponent {
    */
   /// @custom:oz-upgrades-unsafe-allow constructor
   constructor(IPolicyPool policyPool_) PolicyPoolComponent(policyPool_) {
-    NEGLIGIBLE_AMOUNT = 10**(policyPool_.currency().decimals() / 2);
+    NEGLIGIBLE_AMOUNT = 10 ** (policyPool_.currency().decimals() / 2);
   }
 
   /**
@@ -130,10 +130,10 @@ abstract contract Reserve is PolicyPoolComponent {
    * `force` is true, an error in the deinvestAll() operation is ignored. When `force` is false, if `deinvestAll()`
    * fails, it reverts.
    */
-  function setAssetManager(IAssetManager newAM, bool force)
-    external
-    onlyGlobalOrComponentRole2(GUARDIAN_ROLE, LEVEL1_ROLE)
-  {
+  function setAssetManager(
+    IAssetManager newAM,
+    bool force
+  ) external onlyGlobalOrComponentRole2(GUARDIAN_ROLE, LEVEL1_ROLE) {
     require(
       address(newAM) == address(0) || newAM.supportsInterface(type(IAssetManager).interfaceId),
       "Reserve: asset manager doesn't implements the required interface"
@@ -158,9 +158,7 @@ abstract contract Reserve is PolicyPoolComponent {
          * the storage in the connect() method (as it is the recommended practice in normal changes of AM).
          */
       } else {
-        bytes memory result = am.functionDelegateCall(
-          abi.encodeWithSelector(IAssetManager.deinvestAll.selector)
-        );
+        bytes memory result = am.functionDelegateCall(abi.encodeWithSelector(IAssetManager.deinvestAll.selector));
         _assetEarnings(abi.decode(result, (int256)));
       }
     }
@@ -179,9 +177,7 @@ abstract contract Reserve is PolicyPoolComponent {
    * - Emits {IAssetManager-MoneyInvested} or {IAssetManager-MoneyDeinvested}
    */
   function rebalance() public whenNotPaused {
-    address(assetManager()).functionDelegateCall(
-      abi.encodeWithSelector(IAssetManager.rebalance.selector)
-    );
+    address(assetManager()).functionDelegateCall(abi.encodeWithSelector(IAssetManager.rebalance.selector));
   }
 
   /**
@@ -217,11 +213,9 @@ abstract contract Reserve is PolicyPoolComponent {
    * @param functionCall Abi encoded function call to make.
    * @return Returns the return value of the function called, to be decoded by the receiver.
    */
-  function forwardToAssetManager(bytes memory functionCall)
-    external
-    onlyGlobalOrComponentRole(LEVEL2_ROLE)
-    returns (bytes memory)
-  {
+  function forwardToAssetManager(
+    bytes memory functionCall
+  ) external onlyGlobalOrComponentRole(LEVEL2_ROLE) returns (bytes memory) {
     return address(assetManager()).functionDelegateCall(functionCall);
   }