chore(sdk-core): sui pass gas limit param in prebuild

linyah9912 · linyah9912 · commit 94cf97c997a6 · 2022-12-02T13:46:20.000-05:00
Ticket: BG-63034
diff --git a/modules/bitgo/test/v2/unit/wallet.ts b/modules/bitgo/test/v2/unit/wallet.ts
@@ -1300,6 +1300,22 @@ describe('V2 Wallet:', function () {
 
       response.isDone().should.be.true();
     });
+
+    it('should pass gas limit parameter through when building transaction for sui', async function () {
+      const params = { gasLimit: 100 };
+      const path = `/api/v2/${wallet.coin()}/wallet/${wallet.id()}/tx/build`;
+      const response = nock(bgUrl)
+        .post(path, _.matches(params)) // use _.matches to do a partial match on request body object instead of strict matching
+        .reply(200);
+      try {
+        await wallet.prebuildTransaction(params);
+      } catch (e) {
+        // the prebuildTransaction method will probably throw an exception for not having all of the correct nocks
+        // we only care about /tx/build and whether reservation is an allowed parameter
+      }
+
+      response.isDone().should.be.true();
+    });
   });
 
   describe('Maximum Spendable', function maximumSpendable() {
diff --git a/modules/sdk-core/src/bitgo/utils/mpcUtils.ts b/modules/sdk-core/src/bitgo/utils/mpcUtils.ts
@@ -161,6 +161,16 @@ export abstract class MpcUtils {
       }
     }
 
+    if (params.feeOptions !== undefined) {
+      return {
+        ...baseIntent,
+        memo: params.memo?.value,
+        token: params.tokenName,
+        enableTokens: params.enableTokens,
+        feeOptions: params.feeOptions,
+      };
+    }
+
     return {
       ...baseIntent,
       memo: params.memo?.value,
diff --git a/modules/sdk-core/src/bitgo/wallet/wallet.ts b/modules/sdk-core/src/bitgo/wallet/wallet.ts
@@ -2554,6 +2554,8 @@ export class Wallet implements IWallet {
               maxPriorityFeePerGas: Number(params.eip1559?.maxPriorityFeePerGas),
               gasLimit: params.gasLimit,
             };
+    } else if (params.gasLimit !== undefined) {
+      feeOptions = { gasLimit: params.gasLimit };
     } else {
       feeOptions = undefined;
     }

Original file line number	Diff line number	Diff line change
`@@ -2554,6 +2554,8 @@ export class Wallet implements IWallet {`
`2554`	`2554`	`maxPriorityFeePerGas: Number(params.eip1559?.maxPriorityFeePerGas),`
`2555`	`2555`	`gasLimit: params.gasLimit,`
`2556`	`2556`	`};`
	`2557`	`+ } else if (params.gasLimit !== undefined) {`
	`2558`	`+ feeOptions = { gasLimit: params.gasLimit };`
`2557`	`2559`	`} else {`
`2558`	`2560`	`feeOptions = undefined;`
`2559`	`2561`	`}`