refactor: Cleared some unnecessary comments and changed variable assi…

…gment

Signed-off-by: ivaylogarnev-limechain <[email protected]>

src/transaction/Transaction.js

@@ -876,12 +876,9 @@ export default class Transaction extends Executable {
             if (transaction.sigMap && transaction.sigMap.sigPair) {
                 transaction.sigMap.sigPair = transaction.sigMap.sigPair.filter(
                     (sigPair) => {
-                        let sigPairPublicKeyHex;
-                        if (sigPair.pubKeyPrefix) {
-                            sigPairPublicKeyHex = hex.encode(
-                                sigPair.pubKeyPrefix,
-                            );
-                        }
+                        let sigPairPublicKeyHex = hex.encode(
+                            sigPair?.pubKeyPrefix || new Uint8Array(),
+                        );
 
                         const matchesPublicKey =
                             sigPairPublicKeyHex === publicKeyHex;
@@ -944,9 +941,9 @@ export default class Transaction extends Executable {
         }
 
         // Clear the internal tracking of signer public keys and other relevant arrays
-        this._signerPublicKeys.clear(); // Clear the map of signer public keys
-        this._publicKeys = []; // Clear the array of public keys
-        this._transactionSigners = []; // Clear the array of transaction signers
+        this._signerPublicKeys.clear();
+        this._publicKeys = [];
+        this._transactionSigners = [];
 
         return this;
     }

test/unit/Transaction.js

@@ -314,7 +314,7 @@ describe("Transaction", function () {
         });
     });
 
-    describe.only("Transaction removeSignature/clearAllSignatures methods", function () {
+    describe("Transaction removeSignature/clearAllSignatures methods", function () {
         let key1, key2, key3;
         let transaction;
 
@@ -354,6 +354,7 @@ describe("Transaction", function () {
                 key3,
             );
 
+            //Check if the transaction internal tracking of signer public keys is correct
             expect(transaction._signerPublicKeys.size).to.equal(3);
             expect(transaction._publicKeys.length).to.equal(3);
             expect(transaction._transactionSigners.length).to.equal(3);
@@ -368,6 +369,7 @@ describe("Transaction", function () {
             //Check if the transaction is frozen
             expect(transaction.isFrozen()).to.be.true;
 
+            //Check if the transaction internal tracking of signer public keys is correct
             expect(transaction._signerPublicKeys.size).to.equal(2);
             expect(transaction._publicKeys.length).to.equal(2);
             expect(transaction._transactionSigners.length).to.equal(2);
@@ -391,7 +393,7 @@ describe("Transaction", function () {
             //Check if the transaction is frozen
             expect(transaction.isFrozen()).to.be.true;
 
-            //Check if the transaction keys are empty
+            //Check if the transaction internal tracking of signer public keys is cleared
             expect(transaction._signerPublicKeys.size).to.equal(0);
             expect(transaction._publicKeys.length).to.equal(0);
             expect(transaction._transactionSigners.length).to.equal(0);