added a batch tx caller service

aalavandhan · aalavandhan · commit c37397531827 · 2021-09-07T13:03:47.000-07:00
diff --git a/contracts/_utilities/BatchTxCaller.sol b/contracts/_utilities/BatchTxCaller.sol
@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: GPL-3.0-or-later
+pragma solidity 0.7.3;
+pragma experimental ABIEncoderV2;
+
+/**
+ * @title BatchTxCaller
+ * @notice Utility that executes an array of provided transactions.
+ */
+contract BatchTxCaller {
+    event TransactionFailed(address indexed destination, uint256 index, bytes data, bytes reason);
+
+    /**
+     * @notice Executes all transactions marked enabled.
+     *         If any transaction in the transaction list reverts, it returns false.
+     */
+    function executeAll(
+        address[] memory destinations,
+        bytes[] memory data,
+        uint256[] memory values
+    ) external payable returns (bool) {
+        bool exeuctionSuccess = true;
+
+        for (uint256 i = 0; i < destinations.length; i++) {
+            (bool result, bytes memory reason) = destinations[i].call{value: values[i]}(data[i]);
+            if (!result) {
+                emit TransactionFailed(destinations[i], i, data[i], reason);
+                exeuctionSuccess = false;
+            }
+        }
+
+        return exeuctionSuccess;
+    }
+}
diff --git a/contracts/_utilities/BatchTxExecutor.sol b/contracts/_utilities/BatchTxExecutor.sol
@@ -23,6 +23,20 @@ contract BatchTxExecutor is Ownable {
 
     Transaction[] public transactions;
 
+    /**
+     * @notice Computes and returns the total value for all enabled transactions.
+     */
+    function totalValue() external view returns (uint256) {
+        uint256 fee;
+        for (uint256 i = 0; i < transactions.length; i++) {
+            Transaction storage t = transactions[i];
+            if (t.enabled) {
+                fee += t.value;
+            }
+        }
+        return fee;
+    }
+
     /**
      * @notice Executes all transactions marked enabled.
      *         If any transaction in the transaction list reverts, it returns false.
diff --git a/helpers/contracts.js b/helpers/contracts.js
@@ -26,6 +26,7 @@ const ContractABIPaths = {
   XCAmple: 'contracts/satellite-chain/xc-ampleforth',
   XCAmpleController: 'contracts/satellite-chain/xc-ampleforth',
   BatchTxExecutor: 'contracts/_utilities',
+  BatchTxCaller: 'contracts/_utilities',
 
   TokenVault: 'contracts/base-chain',
 
diff --git a/sdk/deployments/dev1RopstenBaseChain.json b/sdk/deployments/dev1RopstenBaseChain.json
@@ -336,22 +336,12 @@
     "blockNumber": 10359882
   },
   "batchRebaseReporter": {
-    "address": "0x5F4241DC1e35D88ABE8C588B3EDD165AdF1F27fB",
+    "address": "0x0b9116D4d08C4D3842C2818F4A03419d230B1370",
     "abi": [
-      "event OwnershipTransferred(address indexed previousOwner, address indexed newOwner)",
       "event TransactionFailed(address indexed destination, uint256 index, bytes data, bytes reason)",
-      "function addTransaction(address destination, uint256 value, bytes data)",
-      "function checkExecution(uint256 index) payable returns (bool)",
-      "function executeAll() payable returns (bool)",
-      "function owner() view returns (address)",
-      "function removeTransaction(uint256 index)",
-      "function renounceOwnership()",
-      "function setTransactionEnabled(uint256 index, bool enabled)",
-      "function transactions(uint256) view returns (bool enabled, address destination, uint256 value, bytes data)",
-      "function transactionsSize() view returns (uint256)",
-      "function transferOwnership(address newOwner)"
+      "function executeAll(address[] destinations, bytes[] data, uint256[] values) payable returns (bool)"
     ],
-    "hash": "0x255c39c2c7084c934bd764bc82ad5d316b57068a387845960d775c01c0809209",
-    "blockNumber": 10542785
+    "hash": "0x52d27c364e745da890655c957294a415dba03435fe6255e42fcb2ce7a279674a",
+    "blockNumber": 10909164
   }
-}
+}
diff --git a/sdk/deployments/dev2GoerliBaseChain.json b/sdk/deployments/dev2GoerliBaseChain.json
@@ -245,22 +245,12 @@
     "blockNumber": 5031511
   },
   "batchRebaseReporter": {
-    "address": "0x36ebc6ff9aDb001Ac216bA9F55dcd384B8cFeC6A",
+    "address": "0xFECBF6202508943fd4d169c4aBC5D7C9f9EfE221",
     "abi": [
-      "event OwnershipTransferred(address indexed previousOwner, address indexed newOwner)",
       "event TransactionFailed(address indexed destination, uint256 index, bytes data, bytes reason)",
-      "function addTransaction(address destination, uint256 value, bytes data)",
-      "function checkExecution(uint256 index) payable returns (bool)",
-      "function executeAll() payable returns (bool)",
-      "function owner() view returns (address)",
-      "function removeTransaction(uint256 index)",
-      "function renounceOwnership()",
-      "function setTransactionEnabled(uint256 index, bool enabled)",
-      "function transactions(uint256) view returns (bool enabled, address destination, uint256 value, bytes data)",
-      "function transactionsSize() view returns (uint256)",
-      "function transferOwnership(address newOwner)"
+      "function executeAll(address[] destinations, bytes[] data, uint256[] values) payable returns (bool)"
     ],
-    "hash": "0x0deba6485ddef65b18e70b03f4eca3a66c5b29df28dea3da825d6eac5ccb914d",
-    "blockNumber": 5060206
+    "hash": "0xff9863647fcb87f8c54e70bfc955172effc86198451a362e3b8ad9faed9139b7",
+    "blockNumber": 5387916
   }
 }
diff --git a/sdk/deployments/prodBscSatChain.json b/sdk/deployments/prodBscSatChain.json
@@ -111,7 +111,7 @@
   },
   "isBaseChain": false,
   "chainBridge/bridge": {
-    "address": "0x223fafbc2cA53A75CcfF5B2369128d3d1a828F36",
+    "address": "0xFd55eBc7bBde603A048648C6eAb8775c997C1001",
     "abi": [
       "constructor(uint8 chainID, address[] initialRelayers, uint256 initialRelayerThreshold, uint256 fee, uint256 expiry)",
       "event Deposit(uint8 indexed destinationChainID, bytes32 indexed resourceID, uint64 indexed depositNonce)",
@@ -136,21 +136,29 @@
       "function _proposals(uint72, bytes32) view returns (bytes32 _resourceID, bytes32 _dataHash, uint8 _status, uint256 _proposedBlock)",
       "function _relayerThreshold() view returns (uint256)",
       "function _resourceIDToHandlerAddress(bytes32) view returns (address)",
+      "function _specialFee() view returns (uint256)",
+      "function _specialFeeChainID() view returns (uint8)",
       "function _totalProposals() view returns (uint256)",
       "function _totalRelayers() view returns (uint256)",
+      "function _wtokenAddress() view returns (address)",
       "function adminAddRelayer(address relayerAddress)",
       "function adminChangeFee(uint256 newFee)",
       "function adminChangeRelayerThreshold(uint256 newThreshold)",
+      "function adminChangeSpecialFee(uint256 newFee, uint8 chainID)",
       "function adminPauseTransfers()",
       "function adminRemoveRelayer(address relayerAddress)",
       "function adminSetBurnable(address handlerAddress, address tokenAddress)",
       "function adminSetGenericResource(address handlerAddress, bytes32 resourceID, address contractAddress, bytes4 depositFunctionSig, uint256 depositFunctionDepositerOffset, bytes4 executeFunctionSig)",
       "function adminSetResource(address handlerAddress, bytes32 resourceID, address tokenAddress)",
       "function adminUnpauseTransfers()",
+      "function adminUpdateBridgeAddress(address handlerAddress, address newBridgeAddress)",
       "function adminWithdraw(address handlerAddress, address tokenAddress, address recipient, uint256 amountOrTokenID)",
       "function cancelProposal(uint8 chainID, uint64 depositNonce, bytes32 dataHash)",
       "function deposit(uint8 destinationChainID, bytes32 resourceID, bytes data) payable",
+      "function depositETH(uint8 destinationChainID, bytes32 resourceID, bytes data) payable",
       "function executeProposal(uint8 chainID, uint64 depositNonce, bytes data, bytes32 resourceID)",
+      "function getFee(uint8 destinationChainID) view returns (uint256)",
+      "function getFees() view returns (uint256, uint256, uint8)",
       "function getProposal(uint8 originChainID, uint64 depositNonce, bytes32 dataHash) view returns (tuple(bytes32 _resourceID, bytes32 _dataHash, address[] _yesVotes, address[] _noVotes, uint8 _status, uint256 _proposedBlock))",
       "function getRoleAdmin(bytes32 role) view returns (bytes32)",
       "function getRoleMember(bytes32 role, uint256 index) view returns (address)",
diff --git a/sdk/deployments/prodEthereumBaseChain.json b/sdk/deployments/prodEthereumBaseChain.json
@@ -154,7 +154,7 @@
     ]
   },
   "chainBridge/bridge": {
-    "address": "0xbD515E41DF155112Cc883f8981CB763a286261be",
+    "address": "0xa2A22B46B8df38cd7C55E6bf32Ea5a32637Cf2b1",
     "abi": [
       "constructor(uint8 chainID, address[] initialRelayers, uint256 initialRelayerThreshold, uint256 fee, uint256 expiry)",
       "event Deposit(uint8 indexed destinationChainID, bytes32 indexed resourceID, uint64 indexed depositNonce)",
@@ -179,21 +179,29 @@
       "function _proposals(uint72, bytes32) view returns (bytes32 _resourceID, bytes32 _dataHash, uint8 _status, uint256 _proposedBlock)",
       "function _relayerThreshold() view returns (uint256)",
       "function _resourceIDToHandlerAddress(bytes32) view returns (address)",
+      "function _specialFee() view returns (uint256)",
+      "function _specialFeeChainID() view returns (uint8)",
       "function _totalProposals() view returns (uint256)",
       "function _totalRelayers() view returns (uint256)",
+      "function _wtokenAddress() view returns (address)",
       "function adminAddRelayer(address relayerAddress)",
       "function adminChangeFee(uint256 newFee)",
       "function adminChangeRelayerThreshold(uint256 newThreshold)",
+      "function adminChangeSpecialFee(uint256 newFee, uint8 chainID)",
       "function adminPauseTransfers()",
       "function adminRemoveRelayer(address relayerAddress)",
       "function adminSetBurnable(address handlerAddress, address tokenAddress)",
       "function adminSetGenericResource(address handlerAddress, bytes32 resourceID, address contractAddress, bytes4 depositFunctionSig, uint256 depositFunctionDepositerOffset, bytes4 executeFunctionSig)",
       "function adminSetResource(address handlerAddress, bytes32 resourceID, address tokenAddress)",
       "function adminUnpauseTransfers()",
+      "function adminUpdateBridgeAddress(address handlerAddress, address newBridgeAddress)",
       "function adminWithdraw(address handlerAddress, address tokenAddress, address recipient, uint256 amountOrTokenID)",
       "function cancelProposal(uint8 chainID, uint64 depositNonce, bytes32 dataHash)",
       "function deposit(uint8 destinationChainID, bytes32 resourceID, bytes data) payable",
+      "function depositETH(uint8 destinationChainID, bytes32 resourceID, bytes data) payable",
       "function executeProposal(uint8 chainID, uint64 depositNonce, bytes data, bytes32 resourceID)",
+      "function getFee(uint8 destinationChainID) view returns (uint256)",
+      "function getFees() view returns (uint256, uint256, uint8)",
       "function getProposal(uint8 originChainID, uint64 depositNonce, bytes32 dataHash) view returns (tuple(bytes32 _resourceID, bytes32 _dataHash, address[] _yesVotes, address[] _noVotes, uint8 _status, uint256 _proposedBlock))",
       "function getRoleAdmin(bytes32 role) view returns (bytes32)",
       "function getRoleMember(bytes32 role, uint256 index) view returns (address)",
@@ -361,22 +369,12 @@
     "blockNumber": 12795542
   },
   "batchRebaseReporter": {
-    "address": "0x25fbC7D475B5013f631E5BB7f9Da57A8d1522179",
+    "address": "0x36aAf46CAB4B6E5390Fb44311b5664a71A398400",
     "abi": [
-      "event OwnershipTransferred(address indexed previousOwner, address indexed newOwner)",
       "event TransactionFailed(address indexed destination, uint256 index, bytes data, bytes reason)",
-      "function addTransaction(address destination, uint256 value, bytes data)",
-      "function checkExecution(uint256 index) payable returns (bool)",
-      "function executeAll() payable returns (bool)",
-      "function owner() view returns (address)",
-      "function removeTransaction(uint256 index)",
-      "function renounceOwnership()",
-      "function setTransactionEnabled(uint256 index, bool enabled)",
-      "function transactions(uint256) view returns (bool enabled, address destination, uint256 value, bytes data)",
-      "function transactionsSize() view returns (uint256)",
-      "function transferOwnership(address newOwner)"
+      "function executeAll(address[] destinations, bytes[] data, uint256[] values) payable returns (bool)"
     ],
-    "hash": "0x3974b71ec5143a95f56f8ec20707f3d6bc46a0c9c31c407a96bf2d07d6c1e029",
-    "blockNumber": 12801448
+    "hash": "0x062d0fb489800960d83da2b93a8ca129e5f170d6dac5e44ea55669c3e8c88dd2",
+    "blockNumber": 13096932
   }
 }
diff --git a/tasks/deploy/chain_bridge.js b/tasks/deploy/chain_bridge.js
@@ -355,7 +355,7 @@ txTask(
     );
     const transferGateway = await getDeployedContractInstance(
       network,
-      'chainBridge/rebaseGateway',
+      'chainBridge/transferGateway',
       provider,
     );
 
@@ -434,7 +434,7 @@ txTask(
       );
       const transferGateway = await getDeployedContractInstance(
         network,
-        'chainBridge/rebaseGateway',
+        'chainBridge/transferGateway',
         provider,
       );
       const adminRole = await bridge.DEFAULT_ADMIN_ROLE();
diff --git a/tasks/deploy/rebase_reporter.js b/tasks/deploy/rebase_reporter.js
diff --git a/test/unit/_utilities/batch_tx_executor.js b/test/unit/_utilities/batch_tx_executor.js
