Merge pull request #6897 from BitGo/harit/fix-jito-tx-not-confirmed

fix(jito): remove output for staking activate

Author: haritkapadia

modules/sdk-coin-sol/src/lib/transaction.ts

@@ -40,6 +40,7 @@ import {
   requiresAllSignatures,
   validateRawMsgInstruction,
 } from './utils';
+import { SolStakingTypeEnum } from '@bitgo/public-types';
 
 export class Transaction extends BaseTransaction {
   protected _solTransaction: SolTransaction;
@@ -359,11 +360,13 @@ export class Transaction extends BaseTransaction {
             value: instruction.params.amount,
             coin: this._coinConfig.name,
           });
-          outputs.push({
-            address: instruction.params.stakingAddress,
-            value: instruction.params.amount,
-            coin: this._coinConfig.name,
-          });
+          if (instruction.params.stakingType !== SolStakingTypeEnum.JITO) {
+            outputs.push({
+              address: instruction.params.stakingAddress,
+              value: instruction.params.amount,
+              coin: this._coinConfig.name,
+            });
+          }
           break;
         case InstructionBuilderTypes.StakingDeactivate:
           if (instruction.params.amount && instruction.params.unstakingAddress) {

modules/sdk-coin-sol/test/unit/transactionBuilder/stakingActivateBuilder.ts

@@ -202,12 +202,7 @@ describe('Sol Staking Activate Builder', () => {
       value: amount,
       coin: 'tsol',
     });
-    tx.outputs.length.should.equal(1);
-    tx.outputs[0].should.deepEqual({
-      address: JITO_STAKE_POOL_ADDRESS,
-      value: amount,
-      coin: 'tsol',
-    });
+    tx.outputs.length.should.equal(0);
   };
 
   describe('Succeed', () => {