fix: return transaction objects on repeated bridge swaps

src/__tests__/Transaction.test.ts

@@ -0,0 +1,28 @@
+import { Transaction } from '../bridge/Transaction';
+
+jest.mock('@naftalimurgor/jsbgl', () => ({
+  asyncInit: jest.fn().mockResolvedValue(undefined),
+}));
+
+describe('Transaction factory', () => {
+  const originalTransaction = globalThis.Transaction;
+
+  afterEach(() => {
+    globalThis.Transaction = originalTransaction;
+  });
+
+  it('returns a transaction object every time it is called', () => {
+    let transactionId = 0;
+
+    globalThis.Transaction = class {
+      public readonly id = ++transactionId;
+    } as typeof globalThis.Transaction;
+
+    const firstTxObject = Transaction.makeTxObject();
+    const secondTxObject = Transaction.makeTxObject();
+
+    expect(firstTxObject).toBeDefined();
+    expect(secondTxObject).toBeDefined();
+    expect(secondTxObject).not.toBe(firstTxObject);
+  });
+});

src/bridge/Transaction.ts

@@ -1,23 +1,24 @@
-import jsbgl from '@naftalimurgor/jsbgl'
+import jsbgl from '@naftalimurgor/jsbgl';
 
 export class Transaction {
-  private static txInstance: Transaction
+  private static txInstance: Transaction;
   private constructor() {
-    this._initJsbglModule()
+    this._initJsbglModule();
   }
 
   public static makeTxObject() {
     if (!this.txInstance) {
-      this.txInstance = new Transaction()
-      return new globalThis.Transaction()
+      this.txInstance = new Transaction();
     }
+
+    return new globalThis.Transaction();
   }
 
   /**
-   * 
+   *
    * Initializes the jsbgl module, by injecting the module to the global scope
-  */
+   */
   private _initJsbglModule() {
-    return Promise.resolve(jsbgl.asyncInit())
+    return Promise.resolve(jsbgl.asyncInit());
   }
-}
+}