fix(sdk-core): add keychain info to transaction verification process, fixes unit tests

Ticket: WP-5945

Author: danielpeng1

modules/bitgo/test/v2/unit/wallet.ts

@@ -1589,6 +1589,7 @@ describe('V2 Wallet:', function () {
           'unsigned'
         )
       );
+      psbts.forEach((psbt) => utxoLib.bitgo.addXpubsToPsbt(psbt, rootWalletKey));
       const txHexes = psbts.map((psbt) => ({ txHex: psbt.toHex() }));
 
       const nocks: nock.Scope[] = [];
@@ -1627,6 +1628,7 @@ describe('V2 Wallet:', function () {
         rootWalletKey,
         'unsigned'
       );
+      utxoLib.bitgo.addXpubsToPsbt(psbt, rootWalletKey);
 
       const nocks: nock.Scope[] = [];
       nocks.push(

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

@@ -735,24 +735,33 @@ export class Wallet implements IWallet {
       return buildResponse;
     }
 
+    const keychains = (await this.baseCoin
+      .keychains()
+      .getKeysForSigning({ wallet: this, reqId })) as unknown as Keychain[];
+
     // Validate that the platform-built transaction matches user parameters
     const txPrebuilds = Array.isArray(buildResponse) ? buildResponse : [buildResponse];
     await Promise.all(
-      txPrebuilds.map((txPrebuild) =>
-        this.baseCoin.verifyTransaction({
-          txParams: params,
-          txPrebuild,
-          wallet: this,
-          verification: params.verification ?? {},
-          reqId,
-        })
-      )
+      txPrebuilds
+        .filter((txPrebuild) => txPrebuild.txHex)
+        .map((txPrebuild) =>
+          this.baseCoin.verifyTransaction({
+            txParams: params,
+            txPrebuild,
+            wallet: this,
+            verification: {
+              ...(params.verification ?? {}),
+              keychains: {
+                user: keychains[0],
+                backup: keychains[1],
+                bitgo: keychains[2],
+              },
+            },
+            reqId,
+          })
+        )
     );
 
-    const keychains = (await this.baseCoin
-      .keychains()
-      .getKeysForSigning({ wallet: this, reqId })) as unknown as Keychain[];
-
     const transactionParams = {
       ...params,
       keychain: keychains[0],