fix: Add visibility accessors to state variables of mocks

aave · Jan 7, 2022 · b76dcb7 · b76dcb7
1 parent 9faa22f
commit b76dcb7
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 10 deletions.
diff --git a/.solhint.json b/.solhint.json
@@ -0,0 +1,5 @@
+{
+    "rules": {
+      "max-line-length": "warn"
+    }
+  }
diff --git a/contracts/mocks/flashloan/MockFlashLoanReceiver.sol b/contracts/mocks/flashloan/MockFlashLoanReceiver.sol
@@ -13,9 +13,9 @@ contract MockFlashLoanReceiver is FlashLoanReceiverBase {
   event ExecutedWithFail(address[] _assets, uint256[] _amounts, uint256[] _premiums);
   event ExecutedWithSuccess(address[] _assets, uint256[] _amounts, uint256[] _premiums);
 
-  bool _failExecution;
-  uint256 _amountToApprove;
-  bool _simulateEOA;
+  bool internal _failExecution;
+  uint256 internal _amountToApprove;
+  bool internal _simulateEOA;
 
   constructor(IPoolAddressesProvider provider) FlashLoanReceiverBase(provider) {}
 

diff --git a/contracts/mocks/flashloan/MockSimpleFlashLoanReceiver.sol b/contracts/mocks/flashloan/MockSimpleFlashLoanReceiver.sol
@@ -16,9 +16,9 @@ contract MockFlashLoanSimpleReceiver is FlashLoanSimpleReceiverBase {
   event ExecutedWithFail(address asset, uint256 amount, uint256 premium);
   event ExecutedWithSuccess(address asset, uint256 amount, uint256 premium);
 
-  bool _failExecution;
-  uint256 _amountToApprove;
-  bool _simulateEOA;
+  bool internal _failExecution;
+  uint256 internal _amountToApprove;
+  bool internal _simulateEOA;
 
   constructor(IPoolAddressesProvider provider) FlashLoanSimpleReceiverBase(provider) {}
 

diff --git a/contracts/mocks/helpers/MockPool.sol b/contracts/mocks/helpers/MockPool.sol
@@ -7,8 +7,8 @@ contract MockPool {
   // Reserved storage space to avoid layout collisions.
   uint256[100] private ______gap;
 
-  address _addressesProvider;
-  address[] _reserveList;
+  address internal _addressesProvider;
+  address[] internal _reserveList;
 
   function initialize(address provider) external {
     _addressesProvider = provider;

diff --git a/contracts/mocks/oracle/PriceOracle.sol b/contracts/mocks/oracle/PriceOracle.sol
@@ -4,8 +4,9 @@ pragma solidity 0.8.10;
 import {IPriceOracle} from '../../interfaces/IPriceOracle.sol';
 
 contract PriceOracle is IPriceOracle {
-  mapping(address => uint256) prices;
-  uint256 ethPriceUsd;
+  mapping(address => uint256) internal prices;
+
+  uint256 internal ethPriceUsd;
 
   event AssetPriceUpdated(address asset, uint256 price, uint256 timestamp);
   event EthPriceUpdated(uint256 price, uint256 timestamp);