fix: SendTransaction should enqueue call to onNewTx (#854)

Author: r4mmer

src/new/sendTransaction.ts

@@ -447,7 +447,7 @@ export default class SendTransaction extends EventEmitter {
                 // This just returns if the transaction is not a CREATE_TOKEN_TX
                 await addCreatedTokenFromTx(transaction as CreateTokenTransaction, storage);
                 // Add new transaction to the wallet's storage.
-                await wallet.onNewTx({ history: historyTx });
+                wallet.enqueueOnNewTx({ history: historyTx });
               })(this.wallet, this.storage, this.transaction);
             }
             this.emit('send-tx-success', this.transaction);

src/new/wallet.js

@@ -675,7 +675,7 @@ class HathorWallet extends EventEmitter {
         // So we will enqueue this message to be processed later
         this.wsTxQueue.enqueue(wsData);
       } else {
-        this.newTxPromise = this.newTxPromise.then(() => this.onNewTx(wsData));
+        this.enqueueOnNewTx(wsData);
       }
     }
   }
@@ -1330,6 +1330,14 @@ class HathorWallet extends EventEmitter {
     this.emit('state', state);
   }
 
+  /**
+   * Enqueue the call for onNewTx with the given data.
+   * @param {{ history: import('../types').IHistoryTx }} wsData
+   */
+  enqueueOnNewTx(wsData) {
+    this.newTxPromise = this.newTxPromise.then(() => this.onNewTx(wsData));
+  }
+
   /**
    * @param {{ history: import('../types').IHistoryTx }} wsData
    */