feat: enabled retrieval of network gasPrice in sendRawTransaction using MAPI instead of HAPI.

[quiet-node]

Description:

This PR enables sendRawTransaction to retrieve the current gasPrice of the network via the mirror node, instead of querying the consensus node for the fee schedule file. By switching to MAPI, the relay operator account can avoid incurring costs associated with MAPI queries.

Related issue(s):

Fixes #2979

Notes for reviewer:
This PR utilizes the gasPrice() method in the eth.ts module for sendRawTransaction to take advantage of querying the gasPrice through the mirror node. If that query fails, it falls back to querying the consensus node for the fee schedule using the getTinybarGasFee method in the sdkClient.

Checklist

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

[github-actions]

Tests

       3 files     274 suites   19s ⏱️
1 289 tests 1 288 ✔️ 1 💤 0 ❌
1 298 runs  1 297 ✔️ 1 💤 0 ❌

Results for commit f80b338.

♻️ This comment has been updated with latest results.

[github-actions]

Acceptance Tests

  17 files  225 suites   29m 34s ⏱️
601 tests 586 ✔️ 4 💤 11 ❌
734 runs  719 ✔️ 4 💤 11 ❌

Results for commit f80b338.

♻️ This comment has been updated with latest results.

[ebadiere]

LG.

[ebadiere]

LG

[sonarqubecloud]

Quality Gate passed

Issues
0 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 SonarCloud

[victor-yanev]

I'm not sure why we want to set the maxTransactionFee to MAX_GAS_PER_SEC. 🤔

Is this a realistic scenario for the relay operator to pay that much for a transaction?

[quiet-node]

I'm not sure why we want to set the maxTransactionFee to MAX_GAS_PER_SEC. 🤔

Is this a realistic scenario for the relay operator to pay that much for a transaction?

That's a valid concern. Please create a new ticket and move the conversation there. This PR focuses on the logic of utilizing MAPI instead of HAPI to get the gasPrice of the network.

[codecov]

Codecov Report

Attention: Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.39%. Comparing base (f55c304) to head (f80b338).
Report is 17 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/relay/src/lib/eth.ts	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2981      +/-   ##
==========================================
+ Coverage   89.54%   91.39%   +1.85%     
==========================================
  Files          56       59       +3     
  Lines        3796     3950     +154     
  Branches      775      809      +34     
==========================================
+ Hits         3399     3610     +211     
+ Misses        350      291      -59     
- Partials       47       49       +2     

Flag	Coverage Δ
relay	89.68% <93.33%> (ø)
server	88.15% <ø> (ø)
ws-server	100.00% <ø> (ø)

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

Files with missing lines	Coverage Δ
packages/relay/src/formatters.ts	78.76% <100.00%> (ø)
packages/relay/src/lib/clients/sdkClient.ts	68.61% <100.00%> (+5.85%)	⬆️
packages/relay/src/lib/constants.ts	100.00% <100.00%> (+9.30%)	⬆️
packages/relay/src/lib/precheck.ts	96.11% <100.00%> (+0.97%)	⬆️
packages/relay/src/lib/eth.ts	92.74% <83.33%> (+3.92%)	⬆️

... and 8 files with indirect coverage changes