feat: adds test coverage for HAPI transactions in conformity tests (#4675)

Signed-off-by: Simeon Nakov <[email protected]>

Author: simzzz

docs/openrpc.json

@@ -750,10 +750,7 @@
         "name": "Txpool content",
         "schema": {
           "type": "object",
-          "required": [
-            "pending",
-            "queued"
-          ],
+          "required": ["pending", "queued"],
           "properties": {
             "pending": {
               "type": "object",
@@ -799,10 +796,7 @@
         "name": "Txpool contentFrom",
         "schema": {
           "type": "object",
-          "required": [
-            "pending",
-            "queued"
-          ],
+          "required": ["pending", "queued"],
           "properties": {
             "pending": {
               "type": "object",
@@ -1754,22 +1748,16 @@
         ],
         "properties": {
           "blockHash": {
-            "type": [
-              "string"
-            ],
+            "type": ["string"],
             "pattern": "^0x([A-Fa-f0-9]{64})$",
             "description": "Zero hex 32 bytes."
           },
           "blockNumber": {
-            "type": [
-              "null"
-            ],
+            "type": ["null"],
             "description": "Null."
           },
           "transactionIndex": {
-            "type": [
-              "null"
-            ],
+            "type": ["null"],
             "description": "Null."
           },
           "from": {
@@ -1798,10 +1786,7 @@
             "description": "Nonce of the transaction."
           },
           "to": {
-            "type": [
-              "string",
-              "null"
-            ],
+            "type": ["string", "null"],
             "pattern": "^0x[a-fA-F0-9]{40}$",
             "description": "Recipient address (null for contract creation)."
           },
@@ -1811,9 +1796,7 @@
             "description": "Value transferred."
           },
           "type": {
-            "type": [
-              "string"
-            ],
+            "type": ["string"],
             "pattern": "^0x[0-9a-fA-F]+$",
             "description": "Transaction type (EIP-2718 typed transactions)."
           },
@@ -1833,34 +1816,22 @@
             "description": "Signature s component."
           },
           "chainId": {
-            "type": [
-              "string",
-              "null"
-            ],
+            "type": ["string", "null"],
             "pattern": "^0x[0-9a-fA-F]+$",
             "description": "Chain ID for EIP-155 transactions."
           },
           "gasPrice": {
-            "type": [
-              "string",
-              "null"
-            ],
+            "type": ["string", "null"],
             "pattern": "^0x[0-9a-fA-F]+$",
             "description": "Gas price."
           },
           "maxFeePerGas": {
-            "type": [
-              "string",
-              "null"
-            ],
+            "type": ["string", "null"],
             "pattern": "^0x[0-9a-fA-F]+$",
             "description": "For EIP-1559 transactions: maximum fee per gas."
           },
           "maxPriorityFeePerGas": {
-            "type": [
-              "string",
-              "null"
-            ],
+            "type": ["string", "null"],
             "pattern": "^0x[0-9a-fA-F]+$",
             "description": "For EIP-1559 transactions: max priority fee per gas."
           }
@@ -1960,19 +1931,14 @@
       "BlockTracerType": {
         "title": "Block tracer type",
         "type": "string",
-        "enum": [
-          "callTracer",
-          "prestateTracer"
-        ]
+        "enum": ["callTracer", "prestateTracer"]
       },
       "TransactionWithSender": {
         "title": "Transaction object with sender",
         "type": "object",
         "allOf": [
           {
-            "required": [
-              "from"
-            ],
+            "required": ["from"],
             "properties": {
               "from": {
                 "title": "from",
@@ -2057,14 +2023,7 @@
       "TransactionLegacyUnsigned": {
         "type": "object",
         "title": "Legacy transaction.",
-        "required": [
-          "type",
-          "nonce",
-          "gas",
-          "value",
-          "input",
-          "gasPrice"
-        ],
+        "required": ["type", "nonce", "gas", "value", "input", "gasPrice"],
         "properties": {
           "type": {
             "title": "type",
@@ -2120,12 +2079,7 @@
           },
           {
             "title": "EIP-1559 transaction signature properties.",
-            "required": [
-              "yParity",
-              "r",
-              "s",
-              "v"
-            ],
+            "required": ["yParity", "r", "s", "v"],
             "properties": {
               "yParity": {
                 "title": "yParity",
@@ -2182,11 +2136,7 @@
           },
           {
             "title": "Legacy transaction signature properties.",
-            "required": [
-              "v",
-              "r",
-              "s"
-            ],
+            "required": ["v", "r", "s"],
             "properties": {
               "v": {
                 "title": "v",
@@ -2227,13 +2177,7 @@
         "allOf": [
           {
             "title": "Contextual information",
-            "required": [
-              "blockHash",
-              "blockNumber",
-              "from",
-              "hash",
-              "transactionIndex"
-            ],
+            "required": ["blockHash", "blockNumber", "from", "hash", "transactionIndex"],
             "properties": {
               "blockHash": {
                 "title": "block hash",
@@ -2265,9 +2209,7 @@
       "Log": {
         "title": "log",
         "type": "object",
-        "required": [
-          "transactionHash"
-        ],
+        "required": ["transactionHash"],
         "properties": {
           "removed": {
             "title": "removed",
@@ -2329,7 +2271,17 @@
         "properties": {
           "type": {
             "title": "type",
-            "$ref": "#/components/schemas/byte"
+            "description": "Transaction type. Null for HAPI transactions.",
+            "oneOf": [
+              {
+                "title": "Transaction Type",
+                "$ref": "#/components/schemas/byte"
+              },
+              {
+                "title": "Null (HAPI transaction)",
+                "type": "null"
+              }
+            ]
           },
           "transactionHash": {
             "title": "transaction hash",
@@ -2531,10 +2483,7 @@
         "properties": {
           "type": {
             "type": "string",
-            "enum": [
-              "CALL",
-              "CREATE"
-            ],
+            "enum": ["CALL", "CREATE"],
             "description": "The type of action (CALL for function calls or CREATE for contract creation)."
           },
           "from": {
@@ -2581,11 +2530,7 @@
             "description": "Sub-calls made during this call frame."
           }
         },
-        "required": [
-          "from",
-          "gas",
-          "input"
-        ]
+        "required": ["from", "gas", "input"]
       }
     }
   }

packages/server/tests/acceptance/conformityTests.spec.ts

@@ -7,11 +7,14 @@
 // } from '@open-rpc/meta-schema';
 // import { parseOpenRPCDocument } from '@open-rpc/schema-utils-js';
 // import { expect } from 'chai';
+import { TransferTransaction } from '@hashgraph/sdk';
 import fs from 'fs';
 import path from 'path';
 
 // import WebSocket from 'ws';
 import openRpcData from '../../../../docs/openrpc.json';
+import { Utils } from '../helpers/utils';
+import genesisData from './data/conformity/genesis.json';
 // import CallerContract from '../contracts/Caller.json';
 // import LogsContract from '../contracts/Logs.json';
 import {
@@ -22,6 +25,7 @@ import {
   sendAccountAddress,
   setCreateContractLegacyTransactionAndBlockHash,
   setCurrentBlockHash,
+  setHapiTransactionHash,
   setLegacyTransactionAndBlockHash,
   setTransaction1559_2930AndBlockHash,
   setTransaction1559AndBlockHash,
@@ -98,7 +102,7 @@ const overwritesDirectoryPath = path.resolve(__dirname, 'data/conformity/overwri
  * their behavior will remain consistent with the expectations.
  */
 const initGenesisData = async function () {
-  for (const data of require('./data/conformity/genesis.json')) {
+  for (const data of genesisData) {
     const options = { maxPriorityFeePerGas: gasPrice, maxFeePerGas: gasPrice, gasLimit: gasLimit };
     options['to'] = data.account ? data.account : null;
     if (data.balance) options['value'] = `0x${data.balance.toString(16)}`;
@@ -119,6 +123,37 @@ describe('@api-conformity', async function () {
         await signAndSendRawTransaction(RELAY_URL, createContractLegacyTransaction),
       );
       await initGenesisData();
+
+      // Execute a native HAPI transaction (token transfer via SDK) to test synthetic receipt handling
+      const servicesNode = global.servicesNode;
+      const hapiTestAccount = await Utils.createAliasAccount(
+        global.mirrorNode,
+        global.accounts[0],
+        '1000000000', // 10 HBAR
+      );
+      const tokenId = await servicesNode.createToken(1000);
+      try {
+        await hapiTestAccount.client.associateToken(tokenId);
+      } catch (e: any) {
+        // Ignore if already associated
+        if (!e.message?.includes('TOKEN_ALREADY_ASSOCIATED_TO_ACCOUNT')) {
+          throw e;
+        }
+      }
+      const hapiTransaction = new TransferTransaction()
+        .addTokenTransfer(tokenId, servicesNode._thisAccountId(), -10)
+        .addTokenTransfer(tokenId, hapiTestAccount.accountId, 10)
+        .setTransactionMemo('Conformity test HAPI token transfer');
+      const hapiResp = await hapiTransaction.execute(servicesNode.client);
+      await hapiResp.getRecord(servicesNode.client);
+      // Wait for mirror node to index the transaction
+      await new Promise((r) => setTimeout(r, 3000));
+      // Get the transaction hash from mirror node logs
+      const logsRes = await global.mirrorNode.get(`/contracts/results/logs?limit=1`);
+      if (logsRes.logs && logsRes.logs.length > 0 && logsRes.logs[0].contract_id === tokenId.toString()) {
+        setHapiTransactionHash(logsRes.logs[0].transaction_hash);
+      }
+
       setCurrentBlockHash(await getLatestBlockHash(RELAY_URL));
     });
     //Reading the directories within the ethereum execution api repo

packages/server/tests/acceptance/data/conformity/overwrites/eth_getTransactionReceipt/get-hapi-receipt.io

@@ -0,0 +1,14 @@
+// gets the receipt for a HAPI (native Hedera) transaction
+// HAPI transactions are submitted via the Hedera SDK (e.g., TransferTransaction)
+// and appear as synthetic transactions in the relay.
+//
+// Key differences from Ethereum transactions:
+// - type is null (not 0x0, 0x1, or 0x2)
+// - from is the zero address
+// - gasUsed and cumulativeGasUsed are 0x0
+
+## wildcard: result.blockHash, result.blockNumber, result.transactionIndex, result.transactionHash, result.to, result.contractAddress, result.logs, result.logsBloom, result.effectiveGasPrice, result.root
+
+>> {"jsonrpc":"2.0","id":1,"method":"eth_getTransactionReceipt","params":["0x0000000000000000000000000000000000000000000000000000000000000000"]}
+<< {"jsonrpc":"2.0","id":1,"result":{"blockHash":"0x0000000000000000000000000000000000000000000000000000000000000000","blockNumber":"0x1","from":"0x0000000000000000000000000000000000000000","to":"0x0000000000000000000000000000000000000000","cumulativeGasUsed":"0x0","gasUsed":"0x0","contractAddress":"0x0000000000000000000000000000000000000000","logs":[],"logsBloom":"0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000","transactionHash":"0x0000000000000000000000000000000000000000000000000000000000000000","transactionIndex":"0x0","effectiveGasPrice":"0x0","root":"0x0000000000000000000000000000000000000000000000000000000000000000","status":"0x1","type":null}}
+

packages/server/tests/acceptance/data/conformity/utils/constants.ts

@@ -23,6 +23,7 @@ export let transaction2930AndBlockHash: TransactionResponse;
 export let transaction1559AndBlockHash: TransactionResponse;
 export let createContractLegacyTransactionAndBlockHash: TransactionResponse;
 export let transaction1559_2930AndBlockHash: TransactionResponse;
+export let hapiTransactionHash: string;
 
 export function setCurrentBlockHash(value: string) {
   currentBlockHash = value;
@@ -47,3 +48,7 @@ export function setTransaction1559_2930AndBlockHash(value: TransactionResponse)
 export function setCreateContractLegacyTransactionAndBlockHash(value: TransactionResponse) {
   createContractLegacyTransactionAndBlockHash = value;
 }
+
+export function setHapiTransactionHash(value: string) {
+  hapiTransactionHash = value;
+}

packages/server/tests/acceptance/data/conformity/utils/overwrites.ts

@@ -6,6 +6,7 @@ import {
   currentBlockHash,
   EMPTY_TX_HASH,
   ETHEREUM_NETWORK_ACCOUNT_HASH,
+  hapiTransactionHash,
   legacyTransactionAndBlockHash,
   localNodeAccountPrivateKey,
   NONEXISTENT_TX_HASH,
@@ -117,6 +118,9 @@ function buildTransactionOverrides() {
     ['overwrites/eth_getTransactionReceipt/get-legacy-input.io']: {
       '0': createContractLegacyTransactionAndBlockHash.transactionHash,
     },
+    ['overwrites/eth_getTransactionReceipt/get-hapi-receipt.io']: {
+      '0': hapiTransactionHash,
+    },
     ['overwrites/eth_getBalance/get-balance.io']: {
       '0': ETHEREUM_NETWORK_ACCOUNT_HASH,
       '1': 'latest',