fix: enhanced reliability of eth RPC methods with null checks and retry mechanisms

[quiet-node]

Description:
Numerous requests for eth_getBlockByNumber and eth_getBlockByHash have been reported as failing with errors such as Cannot read properties of null and Received an invalid integer type: null.

Additionally, eth_getTransactionReceipt is also reported to be failing with Error invoking RPC: Invalid parameter: hashOrNumber.

These issues stem from regressions in the getBlock() method, which lacks adequate null checks in certain areas. This pull request addresses the problem by implementing proper null checks and enhancing fallback mechanisms and error handling.

Partially, the results of contracts and log responses from the Mirror Node occasionally contain undefined fields, such as transaction_index, block_hash, block_number, and log_index. This issue may arise because the Mirror Node database does not have sufficient time to save all the information before it is fetched. This PR adds a retry mechanism to the fetching methods that checks for these cases, waits for a specified duration, and then retries the fetch operation.

Related issue(s):

Fixes #3345
Fixes #3351

Notes for reviewer:

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[github-actions]

Test Results

 20 files   -   4  312 suites   - 49   38m 13s ⏱️ - 22m 7s
613 tests  -   6  604 ✅ + 10  4 💤 ±0  5 ❌  - 16 
700 runs   - 201  688 ✅  - 182  4 💤 ±0  8 ❌  - 19 

For more details on these failures, see this check.

Results for commit 051f326. ± Comparison against base commit 90739e1.

This pull request removes 6 tests.

"after all" hook in "RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests "after all" hook in "RPC Server Acceptance Tests"
"after each" hook for "Subscribe to multiple contracts on same subscription" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-3 eth_subscribe "after each" hook for "Subscribe to multiple contracts on same subscription"
"before all" hook for "emits an approval event" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests HTS token should behave like erc20 transfer from when the token owner is not the zero address when the recipient is not the zero address when the spender has enough allowance "before all" hook for "emits an approval event"
"before all" hook for "reverts" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests HTS token should behave like erc20 transfer from when the token owner is not the zero address when the recipient is not the zero address when the spender does not have enough allowance when the token owner has enough balance "before all" hook for "reverts"
"before each" hook for "reverts" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests HTS token should behave like erc20 transfer from when the token owner is not the zero address when the recipient is not the zero address when the spender does not have enough allowance "before each" hook for "reverts"
"before each" hook for "reverts" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests HTS token should behave like erc20 transfer from when the token owner is not the zero address when the recipient is the zero address "before each" hook for "reverts"

♻️ This comment has been updated with latest results.

[Nana-EC]

Good catch.
Some considerations

[natanasow]

LG, one question/thought that might be resolved before merging.

[sonarqubecloud]

Quality Gate failed

Failed conditions
19.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 11 lines in your changes missing coverage. Please review.

Project coverage is 84.83%. Comparing base (13d09e9) to head (051f326).
Report is 8 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/relay/src/lib/clients/mirrorNodeClient.ts	79.16%	2 Missing and 3 partials ⚠️
.../lib/services/ethService/ethCommonService/index.ts	70.58%	3 Missing and 2 partials ⚠️
packages/relay/src/receiptsRootUtils.ts	87.50%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3349      +/-   ##
==========================================
- Coverage   85.30%   84.83%   -0.47%     
==========================================
  Files          69       69              
  Lines        4688     4721      +33     
  Branches     1050     1063      +13     
==========================================
+ Hits         3999     4005       +6     
- Misses        374      399      +25     
- Partials      315      317       +2     

Flag	Coverage Δ
config-service	98.14% <ø> (ø)
relay	79.50% <74.54%> (-0.23%)	⬇️
server	83.28% <ø> (ø)
ws-server	36.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
packages/relay/src/lib/clients/sdkClient.ts	67.81% <100.00%> (-3.31%)	⬇️
packages/relay/src/lib/eth.ts	86.14% <100.00%> (-0.54%)	⬇️
...kages/relay/src/lib/services/debugService/index.ts	85.71% <100.00%> (ø)
packages/relay/src/receiptsRootUtils.ts	96.07% <87.50%> (-1.80%)	⬇️
packages/relay/src/lib/clients/mirrorNodeClient.ts	88.62% <79.16%> (-1.13%)	⬇️
.../lib/services/ethService/ethCommonService/index.ts	90.14% <70.58%> (-3.76%)	⬇️