👷🏻‍♂️ Rename whitelistedTokens -> isWhitelistedTokens

Kwenta · Jul 13, 2023 · e96b0fc · e96b0fc
1 parent 3a22939
commit e96b0fc
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/src/Account.sol b/src/Account.sol
@@ -985,7 +985,7 @@ contract Account is IAccount, Auth, OpsReady {
  /// @dev verify direction and validity of swap (i.e. sUSD <-> whitelisted token)
  if (
  tokenIn == address(MARGIN_ASSET)
- && SETTINGS.whitelistedTokens(tokenOut)
+ && SETTINGS.isWhitelistedTokens(tokenOut)
  ) {
  // if swapping sUSD for another token, ensure sufficient margin
  /// @dev margin is being transferred out of this contract
@@ -994,7 +994,7 @@ contract Account is IAccount, Auth, OpsReady {
  recipient = msg.sender;
  } else if (
  tokenOut == address(MARGIN_ASSET)
- && SETTINGS.whitelistedTokens(tokenIn)
+ && SETTINGS.isWhitelistedTokens(tokenIn)
  ) {
  // if swapping another token for sUSD, token must be transferred to this contract
  /// @dev msg.sender must have approved Permit2 to spend at least the amountIn

diff --git a/src/Settings.sol b/src/Settings.sol
@@ -34,7 +34,7 @@ contract Settings is ISettings, Owned {
  //////////////////////////////////////////////////////////////*/
 
  /// @inheritdoc ISettings
- function whitelistedTokens(address _token)
+ function isWhitelistedTokens(address _token)
  external
  view
  override

diff --git a/src/interfaces/ISettings.sol b/src/interfaces/ISettings.sol
@@ -27,7 +27,7 @@ interface ISettings {
  /// @notice checks if token is whitelisted
  /// @param _token: address of the token to check
  /// @return true if token is whitelisted, false if not
- function whitelistedTokens(address _token) external view returns (bool);
+ function isWhitelistedTokens(address _token) external view returns (bool);
 
  /*//////////////////////////////////////////////////////////////
  SETTERS

diff --git a/test/unit/Settings.t.sol b/test/unit/Settings.t.sol
@@ -60,11 +60,11 @@ contract SettingsTest is Test, ConsolidatedEvents {
  //////////////////////////////////////////////////////////////*/
 
  function test_whitelistedTokens() public {
- assertEq(settings.whitelistedTokens(MARGIN_ASSET), false);
+ assertEq(settings.isWhitelistedTokens(MARGIN_ASSET), false);
  settings.setTokenWhitelistStatus(MARGIN_ASSET, true);
- assertEq(settings.whitelistedTokens(MARGIN_ASSET), true);
+ assertEq(settings.isWhitelistedTokens(MARGIN_ASSET), true);
  settings.setTokenWhitelistStatus(MARGIN_ASSET, false);
- assertEq(settings.whitelistedTokens(MARGIN_ASSET), false);
+ assertEq(settings.isWhitelistedTokens(MARGIN_ASSET), false);
  }
 
  function test_setTokenWhitelistStatus_OnlyOwner() public {
@@ -74,11 +74,11 @@ contract SettingsTest is Test, ConsolidatedEvents {
  }
 
  function test_setTokenWhitelistStatus(address token) public {
- assertEq(settings.whitelistedTokens(token), false);
+ assertEq(settings.isWhitelistedTokens(token), false);
  settings.setTokenWhitelistStatus(token, true);
- assertEq(settings.whitelistedTokens(token), true);
+ assertEq(settings.isWhitelistedTokens(token), true);
  settings.setTokenWhitelistStatus(token, false);
- assertEq(settings.whitelistedTokens(token), false);
+ assertEq(settings.isWhitelistedTokens(token), false);
  }
 
  function test_setTokenWhitelistStatus_Event() public {