refactor(wp): send many uniform error messages changes

 Ticket: COIN-1957

Author: krupanand-bitgo

modules/abstract-eth/src/abstractEthLikeNewCoins.ts

@@ -2281,7 +2281,7 @@ export abstract class AbstractEthLikeNewCoins extends AbstractEthLikeCoin {
       throw new Error(`tx cannot be both a batch and hop transaction`);
     }
     if (txPrebuild.recipients.length !== 1) {
-      throw new Error(`txPrebuild should only have 1 recipient but ${txPrebuild.recipients.length} found`);
+      throw new Error(`Cannot send to more than 1 destination address`);
     }
     if (txParams.hop && txPrebuild.hopTransaction) {
       // Check recipient amount for hop transaction

modules/abstract-eth/test/unit/coin.ts

@@ -348,7 +348,7 @@ export function runTransactionVerificationTests(coinName: string, bitgo: TestBit
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async () => {

modules/sdk-coin-arbeth/test/unit/arbeth.ts

@@ -402,7 +402,7 @@ describe('Arbitrum', function () {
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {

modules/sdk-coin-avaxc/src/avaxc.ts

@@ -171,7 +171,7 @@ export class AvaxC extends AbstractEthLikeNewCoins {
       throw new Error(`tx cannot be both a batch and hop transaction`);
     }
     if (txPrebuild.recipients.length !== 1) {
-      throw new Error(`txPrebuild should only have 1 recipient but ${txPrebuild.recipients.length} found`);
+      throw new Error(`Cannot send to more than 1 destination address`);
     }
     if (txParams.hop && txPrebuild.hopTransaction) {
       // Check recipient amount for hop transaction

modules/sdk-coin-avaxc/test/unit/avaxc.ts

@@ -610,7 +610,7 @@ describe('Avalanche C-Chain', function () {
 
       await tavaxCoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {

modules/sdk-coin-bera/test/unit/bera.ts

@@ -302,7 +302,7 @@ describe('Bera', function () {
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a normal txPrebuild from the bitgo server with the wrong amount', async function () {

modules/sdk-coin-eth/test/unit/eth.ts

@@ -411,7 +411,7 @@ describe('ETH:', function () {
 
       await coin
         .verifyTransaction({ txParams, txPrebuild: txPrebuild as any, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {

modules/sdk-coin-opeth/test/unit/opeth.ts

@@ -398,7 +398,7 @@ describe('Optimism', function () {
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {

modules/sdk-coin-polygon/test/unit/polygon.ts

@@ -387,7 +387,7 @@ describe('Polygon', function () {
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {

modules/sdk-coin-zketh/test/unit/zketh.ts

@@ -400,7 +400,7 @@ describe('zkSync', function () {
 
       await basecoin
         .verifyTransaction({ txParams, txPrebuild, wallet, verification })
-        .should.be.rejectedWith('txPrebuild should only have 1 recipient but 2 found');
+        .should.be.rejectedWith('Cannot send to more than 1 destination address');
     });
 
     it('should reject a hop txPrebuild that does not send to its hop address', async function () {