Merge pull request #4135 from BitGo/WP-1123-whitelist-apiVersion-for-…

…buildAccountConsolidations

feat(root): whitelist apiVersion for buildAccountConsolidations

modules/express/src/clientRoutes.ts

@@ -826,7 +826,7 @@ function createSendParams(req: express.Request) {
 }
 
 function createTSSSendParams(req: express.Request) {
-  if (req.config.externalSignerUrl !== undefined) {
+  if (req.config?.externalSignerUrl !== undefined) {
     const coin = req.bitgo.coin(req.params.coin);
     if (coin.getMPCAlgorithm() === MPCType.EDDSA) {
       return {

modules/express/test/unit/clientRoutes/consolidateAccount.ts

@@ -104,6 +104,25 @@ describe('Consolidate account', () => {
     consolidationStub.should.be.calledOnceWith(body);
   });
 
+  it('should pass the apiVersion param to bitgo api consolidate/build', async () => {
+    const result = { success: [], failure: [] };
+    const body = { apiVersion: 'full' };
+    const { bitgoStub, consolidationStub } = createConsolidateMocks(result, true, true);
+    const mockRequest = {
+      bitgo: bitgoStub,
+      params: {
+        coin: 'tsol',
+        id: '23423423423423',
+      },
+      body,
+    };
+
+    await handleV2ConsolidateAccount(mockRequest as express.Request & typeof mockRequest).should.be.resolvedWith(
+      result
+    );
+    consolidationStub.should.be.calledOnceWith(body);
+  });
+
   it('should return 202 when some transactions fail', async () => {
     const result = { success: [0], failure: [0] };
     const body = 'testbody';

modules/sdk-core/src/bitgo/wallet/wallet.ts

@@ -200,6 +200,7 @@ export class Wallet implements IWallet {
       'validToBlock',
       'preview',
       'keepAlive',
+      'apiVersion',
     ];
   }