Merge pull request #6845 from BitGo/WP-000000/fix-opeth-staking-call

fix(sdk-core): send staking tx signatures based on wallet config

Author: noel-bitgo

modules/bitgo/test/v2/unit/staking/stakingWalletNonTSS.ts

@@ -364,6 +364,14 @@ describe('non-TSS Staking Wallet', function () {
       signStub.restore();
     });
 
+    it('should return true for isSendCallRequired for topethWctStakingWallet', function () {
+      // Access the private method using type assertion
+      const isSendCallRequired = (topethWctStakingWallet as any).isSendCallRequired();
+
+      // Check that isSendCallRequired returns true because this is not ETH TSS
+      isSendCallRequired.should.equal(true);
+    });
+
     it('should build and validate transaction', async function () {
       const unsignedTransaction: PrebuildTransactionResult = {
         walletId: topethWctStakingWallet.walletId,

modules/sdk-core/src/bitgo/staking/stakingWallet.ts

@@ -1,6 +1,8 @@
 /**
  * @prettier
  */
+import { CoinFamily } from '@bitgo/statics';
+
 import {
   DelegationOptions,
   DelegationResults,
@@ -333,9 +335,9 @@ export class StakingWallet implements IStakingWallet {
    * @returns true if send API call to staking service is required else false
    */
   private isSendCallRequired(): boolean {
-    if (this.wallet.baseCoin.getFamily() === 'eth') {
+    if (this.wallet.baseCoin.getFamily() === CoinFamily.ETH) {
       return !this.isEthTss;
-    } else if (this.wallet.baseCoin.supportsTss()) {
+    } else if (this.wallet.multisigType() === 'tss') {
       return this.wallet.baseCoin.getMPCAlgorithm() !== 'ecdsa';
     } else {
       return true;