Merge pull request #604 from multiversx/TOOL-563-fix-result-parsing-based-on-provider

Update smart contract result data parsing

Author: danielailie

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@multiversx/sdk-core",
-  "version": "14.0.3",
+  "version": "14.0.4",
   "description": "MultiversX SDK for JavaScript and TypeScript",
   "author": "MultiversX",
   "homepage": "https://multiversx.com",

package.json

@@ -76,11 +76,12 @@ export class TransactionOnNetwork {
         const result = TransactionOnNetwork.fromHttpResponse(txHash, response);
         result.smartContractResults =
             response.smartContractResults?.map(
-                (result: Partial<SmartContractResult>) =>
+                (result: any) =>
                     new SmartContractResult({
                         ...result,
                         receiver: result.receiver ? new Address(result.receiver) : undefined,
                         sender: result.sender ? new Address(result.sender) : undefined,
+                        data: Buffer.from(result.data ?? "", "utf-8"),
                         raw: result,
                     }),
             ) ?? [];
@@ -142,11 +143,12 @@ export class TransactionOnNetwork {
         const result = TransactionOnNetwork.fromHttpResponse(txHash, response);
         result.smartContractResults =
             response.results?.map(
-                (result: Partial<SmartContractResult>) =>
+                (result: any) =>
                     new SmartContractResult({
                         ...result,
                         receiver: result.receiver ? new Address(result.receiver) : undefined,
                         sender: result.sender ? new Address(result.sender) : undefined,
+                        data: Buffer.from(result.data ?? "", "base64"),
                         raw: result,
                     }),
             ) ?? [];
@@ -167,7 +169,6 @@ export class TransactionOnNetwork {
         result.gasPrice = BigInt(response.gasPrice) || 0n;
         result.gasLimit = BigInt(response.gasLimit) || 0n;
         result.function = response.function || "";
-        result.data = Buffer.from(response.data || "", "base64");
         result.version = response.version || 1;
         result.options = response.options || 0;
         result.data = Buffer.from(response.data || "", "base64");

src/core/transactionOnNetwork.ts

@@ -162,7 +162,7 @@ describe("ApiNetworkProvider Tests", function () {
         assert.isTrue(transaction.status.isCompleted());
     });
 
-    it("should fetch smart contract invoking transaction", async () => {
+    it("should fetch smart contract invoking transaction with expected data", async () => {
         const transaction = await apiProvider.getTransaction(
             "6fe05e4ca01d42c96ae5182978a77fe49f26bcc14aac95ad4f19618173f86ddb",
         );
@@ -175,6 +175,10 @@ describe("ApiNetworkProvider Tests", function () {
                 "issue@54455354546f6b656e@54455354@016345785d8a0000@06@63616e4368616e67654f776e6572@74727565@63616e55706772616465@74727565@63616e4164645370656369616c526f6c6573@74727565",
             ),
         );
+        assert.deepEqual(
+            transaction.smartContractResults[0].data,
+            Buffer.from("ESDTSetBurnRoleForAll@544553542d666631353565"),
+        );
         assert.equal(Buffer.from(transaction.logs.events[0].topics[0]).toString("hex"), "544553542d666631353565");
         assert.equal(Buffer.from(transaction.logs.events[0].topics[1]).toString("hex"), "");
         assert.equal(Buffer.from(transaction.logs.events[0].topics[2]).toString("hex"), "63616e4368616e67654f776e6572");

src/networkProviders/apiNetworkProvider.dev.net.spec.ts

@@ -168,7 +168,7 @@ describe("ProxyNetworkProvider Tests", function () {
         assert.isTrue(transaction.status.isCompleted());
     });
 
-    it("should fetch smart contract invoking transaction", async () => {
+    it("should fetch smart contract invoking transaction with expected data", async () => {
         const transaction = await proxy.getTransaction(
             "6fe05e4ca01d42c96ae5182978a77fe49f26bcc14aac95ad4f19618173f86ddb",
         );
@@ -181,6 +181,10 @@ describe("ProxyNetworkProvider Tests", function () {
                 "issue@54455354546f6b656e@54455354@016345785d8a0000@06@63616e4368616e67654f776e6572@74727565@63616e55706772616465@74727565@63616e4164645370656369616c526f6c6573@74727565",
             ),
         );
+        assert.deepEqual(
+            transaction.smartContractResults[0].data,
+            Buffer.from("ESDTSetBurnRoleForAll@544553542d666631353565"),
+        );
         assert.equal(Buffer.from(transaction.logs.events[0].topics[0]).toString("hex"), "544553542d666631353565");
         assert.equal(Buffer.from(transaction.logs.events[0].topics[1]).toString("hex"), "");
         assert.equal(Buffer.from(transaction.logs.events[0].topics[2]).toString("hex"), "63616e4368616e67654f776e6572");