Merge pull request #7 from icon-project/feat/add-setters-for-evm-intents

feat: add setters for evm intents

Author: ibrizsabin

contracts/evm/contracts/Intents/Intents.sol

@@ -5,6 +5,8 @@ pragma abicoder v2;
 import "openzeppelin-contracts/contracts/token/ERC20/utils/SafeERC20.sol";
 import "openzeppelin-contracts/contracts/utils/Strings.sol";
 import "@iconfoundation/xcall-solidity-library/utils/ParseAddress.sol";
+import "openzeppelin-contracts/contracts/access/Ownable.sol";
+
 
 import "./Types.sol";
 import "./Encoding.sol";
@@ -16,7 +18,7 @@ import {console} from "forge-std/console.sol";
 
 /// @title ICONIntents
 /// @notice Implements the intent-based swapping protocol for cross-chain swaps.
-contract Intents is GeneralizedConnection {
+contract Intents is GeneralizedConnection, Ownable {
     using Encoding for *;
     using Strings for string;
     using SafeERC20 for IERC20;
@@ -71,6 +73,13 @@ contract Intents is GeneralizedConnection {
         permit2 = IPermit2(_premit2);
     }
 
+    function setFeeHandler(address _feeHandler) external onlyOwner {
+        feeHandler = _feeHandler;
+    }
+
+    function setProtocolFee(uint16 _protocolFee) external onlyOwner {
+        protocolFee = _protocolFee;
+    }
 
     function swap(
         Types.SwapOrder memory order

contracts/evm/remappings.txt

@@ -1,4 +1,4 @@
-@xcall/contracts/=./contracts/
+@intents/contracts/=./contracts/
 @iconfoundation/xcall-solidity-library/=./library/xcall/
 @xcall/utils/=./library/utils/
 ds-test/=lib/forge-std/lib/ds-test/src/

contracts/evm/test/Intents/Intents.t.sol

@@ -642,4 +642,28 @@ contract IntentsTest is Test {
         // Assert
         assertTrue(intents.finishedOrders(keccak256(order.encode())));
     }
+
+    function testSetFeeHandler() public {
+        address newFeeHandler = address(0x891);
+        intents.setFeeHandler(newFeeHandler);
+        assertEq(intents.feeHandler(), newFeeHandler);
+
+        address nonOwner = address(0x892);
+        vm.startPrank(nonOwner);
+
+        vm.expectRevert("Ownable: caller is not the owner");
+        intents.setFeeHandler(newFeeHandler);
+    }
+
+    function testSetProtocolFee() public {
+        uint16 newFee = 13;
+        intents.setProtocolFee(newFee);
+        assertEq(intents.protocolFee(), newFee);
+
+        address nonOwner = address(0x892);
+        vm.startPrank(nonOwner);
+
+        vm.expectRevert("Ownable: caller is not the owner");
+        intents.setProtocolFee(newFee);
+    }
 }