Multiply Erc20TransferGasLimit constant by 4 for moonbase

runtime/moonbase/src/lib.rs

@@ -426,6 +426,7 @@ parameter_types! {
 	/// ceil(
 	///     (max_extrinsic.ref_time() / max_extrinsic.proof_size()) / WEIGHT_PER_GAS
 	/// )
+	/// We should re-check `xcm_config::Erc20XcmBridgeTransferGasLimit` when changing this value
 	pub const GasLimitPovSizeRatio: u64 = 16;
 	/// The amount of gas per storage (in bytes): BLOCK_GAS_LIMIT / BLOCK_STORAGE_LIMIT
 	/// (60_000_000 / 160 kb)

runtime/moonbase/src/xcm_config.rs

@@ -691,7 +691,7 @@ parameter_types! {
 
 	// To be able to support almost all erc20 implementations,
 	// we provide a sufficiently hight gas limit.
-	pub Erc20XcmBridgeTransferGasLimit: u64 = 200_000;
+	pub Erc20XcmBridgeTransferGasLimit: u64 = 800_000;
 }
 
 impl pallet_erc20_xcm_bridge::Config for Runtime {

runtime/moonbeam/src/lib.rs

@@ -410,6 +410,7 @@ parameter_types! {
 	/// ceil(
 	///     (max_extrinsic.ref_time() / max_extrinsic.proof_size()) / WEIGHT_PER_GAS
 	/// )
+	/// We should re-check `xcm_config::Erc20XcmBridgeTransferGasLimit` when changing this value
 	pub const GasLimitPovSizeRatio: u64 = 4;
 	/// The amount of gas per storage (in bytes): BLOCK_GAS_LIMIT / BLOCK_STORAGE_LIMIT
 	/// The current definition of BLOCK_STORAGE_LIMIT is 40 KB, resulting in a value of 366.

runtime/moonriver/src/lib.rs

@@ -411,6 +411,7 @@ parameter_types! {
 	/// ceil(
 	///     (max_extrinsic.ref_time() / max_extrinsic.proof_size()) / WEIGHT_PER_GAS
 	/// )
+	/// We should re-check `xcm_config::Erc20XcmBridgeTransferGasLimit` when changing this value
 	pub const GasLimitPovSizeRatio: u64 = 4;
 	/// The amount of gas per storage (in bytes): BLOCK_GAS_LIMIT / BLOCK_STORAGE_LIMIT
 	/// The current definition of BLOCK_STORAGE_LIMIT is 40 KB, resulting in a value of 366.

test/contracts/src/ERC20ExcessGas.sol

@@ -18,7 +18,7 @@ contract ERC20ExcessGas is ERC20WithInitialSupply {
         uint256 amount
     ) public override returns (bool) {
         // Consume gas to over Erc20XcmBridgeTransferGasLimit
-        for (uint i = 0; i < 500; i++) {
+        for (uint i = 0; i < 2000; i++) {
             _gasHog += i;
         }
 

test/suites/dev/moonbase/test-xcm-v3/test-xcm-erc20-excess-gas.ts

@@ -63,7 +63,7 @@ describeSuite({
             // weight
             4_000_000n,
           ],
-          gas: 300_000n,
+          gas: 900_000n,
           rawTxOnly: true,
         });
 
@@ -235,12 +235,12 @@ describeSuite({
                       },
                     },
                     // Override default gas limit with optional GeneralKey.
-                    // b'gas_limit:' + 300000(little endian) + zeros padding
+                    // b'gas_limit:' + 900000(little endian) + zeros padding
                     {
                       GeneralKey: {
                         data: [
-                          103, 97, 115, 95, 108, 105, 109, 105, 116, 58, 224, 147, 4, 0, 0, 0, 0, 0,
-                          0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
+                          103, 97, 115, 95, 108, 105, 109, 105, 116, 58, 160, 187, 13, 0, 0, 0, 0,
+                          0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
                         ],
                         length: 32,
                       },

test/suites/dev/moonbase/test-xcm-v3/test-xcm-erc20-fees-and-trap.ts

@@ -175,7 +175,7 @@ describeSuite({
           })
         ).equals(amountTransferred);
 
-        const feeAssetAmount = 1_000_000_000_000_000n;
+        const feeAssetAmount = 1_100_000_000_000_000n;
 
         // Create xcm message to send ERC20 tokens to Charleth
         const config: XcmFragmentConfig = {

test/suites/dev/moonbase/test-xcm-v3/test-xcm-erc20-v3-filter.ts

@@ -83,7 +83,7 @@ describeSuite({
                   X1: { PalletInstance: Number(balancesPalletIndex) },
                 },
               },
-              fungible: 1_000_000_000_000_000n,
+              fungible: 1_700_000_000_000_000n,
             },
             {
               multilocation: {

test/suites/dev/moonbase/test-xcm-v3/test-xcm-erc20-v3.ts

@@ -83,7 +83,7 @@ describeSuite({
                   X1: { PalletInstance: Number(balancesPalletIndex) },
                 },
               },
-              fungible: 1_000_000_000_000_000n,
+              fungible: 1_100_000_000_000_000n,
             },
             {
               multilocation: {

test/suites/dev/moonbase/test-xcm-v4/test-xcm-erc20-transfer.ts

@@ -61,9 +61,9 @@ describeSuite({
             // Destination as multilocation
             destination,
             // weight
-            500_000n,
+            900_000n,
           ],
-          gas: 500_000n,
+          gas: 900_000n,
           rawTxOnly: true,
         });