Fix postman eth_getBalance test & misc edits (#466)

Fix postman eth_getBalance test
- Update to get balance on `0.0.98` account

Additional misc fixes
- Fix `eth_call` log
- Add `callerName` to query log
- Fix k6 README

Signed-off-by: Nana Essilfie-Conduah <[email protected]>

Author: Nana-EC

k6/README.md

@@ -40,6 +40,14 @@ For domain specific parameters the following rule is used:
 When the value of a parameter is set with an environment variable, the value will be used, but if no value is set for a
 particular parameter, then its value will be found by querying either the rest or rosetta APIs.
 
+#### Full Suite Test
+
+To run all tests, just do
+
+```shell
+k6 run src/sceanrios/apis.js
+```
+
 The test suite will run the tests sequentially with a configurable graceful stop time in between, so they don't
 interfere with each other.
 
@@ -398,12 +406,11 @@ With the test suite mode, a simplified markdown format report `report.md` will a
 | web3_clientVersion | 1 | 100.00 | 2328.00/s | 2328.00/s | 0.23ms | |
 | web3_client_version | 1 | 100.00 | 859.18/s | 859.18/s | 0.23ms | |
 
-### Single Test
+#### Single Test
 
 To run a single test, such as the `eth_chainid` test, just do
 
 ```shell
-source src/rosetta/k6.env
 k6 run src/sceanrios/test/eth_chainid.js
 ```
 

packages/relay/src/lib/clients/sdkClient.ts

@@ -244,7 +244,7 @@ export class SDKClient {
             const resp = await query.execute(client);
             this.logger.info(`${requestIdPrefix} Consensus Node query response: ${query.constructor.name} ${Status.Success._code}`);
             // local free queries will have a '0.0.0' accountId on transactionId
-            this.logger.trace(`${requestIdPrefix} ${query.paymentTransactionId} query cost ${query._queryPayment}`);
+            this.logger.trace(`${requestIdPrefix} ${query.paymentTransactionId} ${callerName} query cost: ${query._queryPayment}`);
 
             this.captureMetrics(
                 SDKClient.queryMode,
@@ -303,7 +303,7 @@ export class SDKClient {
             }
 
             const transactionRecord: TransactionRecord = await resp.getRecord(this.clientMain);
-            this.logger.trace(`${requestIdPrefix} ${resp.transactionId.toString()} transaction cost: ${transactionRecord.transactionFee}`);
+            this.logger.trace(`${requestIdPrefix} ${resp.transactionId.toString()} ${callerName} transaction cost: ${transactionRecord.transactionFee}`);
             this.captureMetrics(
                 SDKClient.transactionMode,
                 transactionName,

packages/relay/src/lib/eth.ts

@@ -772,7 +772,7 @@ export class EthImpl implements Eth {
     // FIXME: In the future this will be implemented by making calls to the mirror node. For the
     //        time being we'll eat the cost and ask the main consensus nodes instead.
     const requestIdPrefix = formatRequestIdMessage(requestId);
-    this.logger.trace(`${requestIdPrefix} call(hash=${call}, blockParam=${blockParam})`, call, blockParam);
+    this.logger.trace(`${requestIdPrefix} call(hash=${JSON.stringify(call)}, blockParam=${blockParam})`, call, blockParam);
     // The "to" address must always be 42 chars.
     if (call.to.length != 42) {
       throw new Error(requestIdPrefix+

packages/server/tests/postman.json

@@ -277,7 +277,7 @@
 				"header": [],
 				"body": {
 					"mode": "raw",
-					"raw": "{\n    \"id\": \"test_id\",\n    \"jsonrpc\": \"2.0\",\n    \"method\": \"eth_chainId\",\n    \"params\": []\n}",
+					"raw": "{\n    \"id\": \"test_id\",\n    \"jsonrpc\": \"2.0\",\n    \"method\": \"eth_getBalance\",\n    \"params\": [\"0x0000000000000000000000000000000000000062\"]\n}",
 					"options": {
 						"raw": {
 							"language": "json"