chore: reverted "fix: added null check for nullable values in formatContractResult() (#3210)" (#3216)

This reverts commit ca044f1.

Signed-off-by: Logan Nguyen <[email protected]>

Author: quiet-node

packages/relay/src/formatters.ts

@@ -166,15 +166,15 @@ const formatContractResult = (cr: any) => {
   const commonFields = {
     blockHash: toHash32(cr.block_hash),
     blockNumber: nullableNumberTo0x(cr.block_number),
-    from: cr.from?.substring(0, 42) || null,
+    from: cr.from.substring(0, 42),
     gas: nanOrNumberTo0x(cr.gas_used),
     gasPrice,
-    hash: cr.hash?.substring(0, 66) || null,
+    hash: cr.hash.substring(0, 66),
     input: cr.function_parameters,
     nonce: nanOrNumberTo0x(cr.nonce),
     r: cr.r === null ? '0x0' : stripLeadingZeroForSignatures(cr.r.substring(0, 66)),
     s: cr.s === null ? '0x0' : stripLeadingZeroForSignatures(cr.s.substring(0, 66)),
-    to: cr.to?.substring(0, 42) || null,
+    to: cr.to?.substring(0, 42),
     transactionIndex: nullableNumberTo0x(cr.transaction_index),
     type: cr.type === null ? '0x0' : nanOrNumberTo0x(cr.type),
     v: cr.v === null ? '0x0' : nanOrNumberTo0x(cr.v),

packages/relay/tests/lib/eth/eth_getTransactionByBlockHashAndIndex.spec.ts

@@ -120,7 +120,7 @@ describe('@ethGetTransactionByBlockHashAndIndex using MirrorNode', async functio
     verifyAggregatedInfo(result);
   });
 
-  it('eth_getTransactionByBlockHashAndIndex should NOT throw for internal error if `from` field is null', async function () {
+  it('eth_getTransactionByBlockHashAndIndex should throw for internal error', async function () {
     const randomBlock = {
       hash: '0x5f827a801c579c84eca738827b65612b28ed425b7578bfdd10177e24fc3db8d4b1a7f3d56d83c39b950cc5e4d175dd64',
       count: 9,
@@ -131,9 +131,16 @@ describe('@ethGetTransactionByBlockHashAndIndex using MirrorNode', async functio
       .onGet(contractResultsByHashByIndexURL(randomBlock.hash, randomBlock.count))
       .reply(200, defaultContractResultsWithNullableFrom);
 
-    const result = await ethImpl.getTransactionByBlockHashAndIndex(randomBlock.hash, randomBlock.count, requestDetails);
-    expect(result).to.not.be.null;
-    expect(result.from).to.be.null;
+    const args = [randomBlock.hash, numberTo0x(randomBlock.count), requestDetails];
+    const errMessage = "Cannot read properties of null (reading 'substring')";
+
+    await RelayAssertions.assertRejection(
+      predefined.INTERNAL_ERROR(errMessage),
+      ethImpl.getTransactionByBlockHashAndIndex,
+      true,
+      ethImpl,
+      args,
+    );
   });
 
   it('eth_getTransactionByBlockHashAndIndex with no contract result match', async function () {

packages/relay/tests/lib/eth/eth_getTransactionByBlockNumberAndIndex.spec.ts

@@ -159,7 +159,7 @@ describe('@ethGetTransactionByBlockNumberAndIndex using MirrorNode', async funct
     expect(result).to.equal(null);
   });
 
-  it('eth_getTransactionByBlockNumberAndIndex should NOT throw for internal error if `from` field is null', async function () {
+  it('eth_getTransactionByBlockNumberAndIndex should throw for internal error', async function () {
     const defaultContractResultsWithNullableFrom = _.cloneDeep(defaultContractResults);
     defaultContractResultsWithNullableFrom.results[0].from = null;
     const randomBlock = {
@@ -170,14 +170,16 @@ describe('@ethGetTransactionByBlockNumberAndIndex using MirrorNode', async funct
       .onGet(contractResultsByNumberByIndexURL(randomBlock.number, randomBlock.count))
       .reply(200, defaultContractResultsWithNullableFrom);
 
-    const result = await ethImpl.getTransactionByBlockNumberAndIndex(
-      numberTo0x(randomBlock.number),
-      numberTo0x(randomBlock.count),
-      requestDetails,
-    );
+    const args = [numberTo0x(randomBlock.number), numberTo0x(randomBlock.count), requestDetails];
+    const errMessage = "Cannot read properties of null (reading 'substring')";
 
-    expect(result).to.not.be.null;
-    expect(result.from).to.be.null;
+    await RelayAssertions.assertRejection(
+      predefined.INTERNAL_ERROR(errMessage),
+      ethImpl.getTransactionByBlockNumberAndIndex,
+      true,
+      ethImpl,
+      args,
+    );
   });
 
   it('eth_getTransactionByBlockNumberAndIndex with no contract results', async function () {

packages/relay/tests/lib/formatters.spec.ts

@@ -340,15 +340,6 @@ describe('Formatters', () => {
       const formattedResult = formatContractResult({ ...contractResult, type: undefined });
       expect(formattedResult).to.be.null;
     });
-
-    it('Should return formatted result even when `from` and `hash` fields are null', () => {
-      const formattedResult: any = formatContractResult({ ...contractResult, from: null, hash: null, to: null });
-
-      expect(formattedResult).to.not.be.undefined;
-      expect(formattedResult.from).to.be.null;
-      expect(formattedResult.hash).to.be.null;
-      expect(formattedResult.to).to.be.null;
-    });
   });
 
   describe('prepend0x', () => {