fix: added polling logic to ensure the retrieval of fully mature records from MN

[quiet-node]

Description:
This PR enhances the retry mechanisms in getContractResultWithRetry() and getContractResultsLogsWithRetry() by adding polling logic to ensure the retrieval of fully mature records. Additionally, the Relay now implements strict error handling to throw exceptions when immature records are encountered. This approach aims to prevent the Cannot read properties of null (reading 'xyz') error caused by improper handling of null checks.

Related issue(s):

Fixes #3366

Notes for reviewer:

Checklist

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

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Test Results

 20 files   -   7  267 suites   - 109   46m 30s ⏱️ - 52m 18s
612 tests  -   3  606 ✅ + 17  4 💤 +1  2 ❌  - 21 
689 runs   - 195  683 ✅  - 172  4 💤 ±0  2 ❌  - 23 

For more details on these failures, see this check.

Results for commit 256a28b. ± Comparison against base commit 16939ff.

This pull request removes 6 and adds 3 tests. Note that renamed tests count towards both.

"before all" hook in "@api-batch-2 RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before all" hook in "@api-batch-2 RPC Server Acceptance Tests"
"before all" hook in "@tokenmanagement HTS Precompile Token Management Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @tokenmanagement HTS Precompile Token Management Acceptance Tests "before all" hook in "@tokenmanagement HTS Precompile Token Management Acceptance Tests"
"before all" hook in "HBAR Rate Limit Tests" ‑ RPC Server Acceptance Tests Acceptance tests @hbarlimiter HBAR Limiter Acceptance Tests HBAR Rate Limit Tests "before all" hook in "HBAR Rate Limit Tests"
"before each" hook for "Should eventually exhaust the hbar limit for PRIVILEGED user and still allow another PRIVILEGED user to make calls" ‑ RPC Server Acceptance Tests Acceptance tests @hbarlimiter HBAR Limiter Acceptance Tests HBAR Rate Limit Tests HBAR Rate Limit For Different Spending Plan Tiers @hbarlimiter-batch2 Preconfigured Tiers PRIVILEGED Tier "before each" hook for "Should eventually exhaust the hbar limit for PRIVILEGED user and still allow another PRIVILEGED user to make calls"
"before each" hook for "from/to Addresses in transaction to a contract (deployed through the relay) are in evm and long-zero format" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before each" hook for "from/to Addresses in transaction to a contract (deployed through the relay) are in evm and long-zero format"
"before each" hook for "should execute "eth_getStorageAt" request to get current state changes" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before each" hook for "should execute "eth_getStorageAt" request to get current state changes"

from/to Addresses in transaction to a contract (deployed through HAPI tx) are in evm and long-zero format ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests Formats of addresses in Transaction and Receipt results from/to Addresses in transaction to a contract (deployed through HAPI tx) are in evm and long-zero format
from/to Addresses in transaction to a contract (deployed through the relay) are in evm and long-zero format ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests Formats of addresses in Transaction and Receipt results from/to Addresses in transaction to a contract (deployed through the relay) are in evm and long-zero format
from/to Addresses when transferring HTS tokens to the tokenAddress are in evm and long-zero format ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests Formats of addresses in Transaction and Receipt results from/to Addresses when transferring HTS tokens to the tokenAddress are in evm and long-zero format

♻️ This comment has been updated with latest results.

[natanasow]

LG

[codecov]

Codecov Report

Attention: Patch coverage is 82.22222% with 8 lines in your changes missing coverage. Please review.

Project coverage is 85.00%. Comparing base (16939ff) to head (256a28b).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/relay/src/lib/clients/mirrorNodeClient.ts	82.85%	3 Missing and 3 partials ⚠️
packages/relay/src/lib/eth.ts	75.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3368      +/-   ##
==========================================
+ Coverage   84.79%   85.00%   +0.21%     
==========================================
  Files          69       69              
  Lines        4721     4742      +21     
  Branches     1063     1067       +4     
==========================================
+ Hits         4003     4031      +28     
+ Misses        400      396       -4     
+ Partials      318      315       -3     

Flag	Coverage Δ
config-service	98.14% <ø> (ø)
relay	79.56% <82.22%> (+0.06%)	⬆️
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 Δ
.../lib/services/ethService/ethCommonService/index.ts	90.20% <100.00%> (+0.06%)	⬆️
packages/relay/src/lib/eth.ts	86.23% <75.00%> (+0.08%)	⬆️
packages/relay/src/lib/clients/mirrorNodeClient.ts	89.16% <82.85%> (+0.54%)	⬆️

... and 3 files with indirect coverage changes