Fix Transaction.getClaimableBalanceId() to get correct claimable ba…

…lance ID. (#530)
stellar · May 17, 2022 · b6e84fd · b6e84fd
1 parent 98feac6
commit b6e84fd
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/transaction.js b/src/transaction.js
@@ -359,7 +359,7 @@ export class Transaction extends TransactionBase {
     const operationId = xdr.HashIdPreimage.envelopeTypeOpId(
       new xdr.HashIdPreimageOperationId({
         sourceAccount: xdr.AccountId.publicKeyTypeEd25519(account),
-        seqNum: new xdr.SequenceNumber(this.sequence),
+        seqNum: xdr.SequenceNumber.fromString(this.sequence),
         opNum: opIndex
       })
     );

diff --git a/test/unit/transaction_test.js b/test/unit/transaction_test.js
@@ -607,6 +607,19 @@ describe('Transaction', function() {
       );
     });
 
+    // See https://github.com/stellar/js-stellar-base/issues/529
+    it('calculates from transaction src (big number sequence)', function() {
+      let gSource = new StellarBase.Account(address, '114272277834498050');
+
+      let tx = makeBuilder(gSource)
+        .addOperation(makeClaimableBalance())
+        .build();
+      const balanceId = tx.getClaimableBalanceId(0);
+      expect(balanceId).to.be.equal(
+        '000000001cd1e39f422a864b4efca661e11ffaa1c54e69b23aaf096e0cfd361bb4a275bf'
+      );
+    });
+
     it('calculates from muxed transaction src as if unmuxed', function() {
       let gSource = new StellarBase.Account(address, '1234');
       let mSource = new StellarBase.MuxedAccount(gSource, '5678');