fix: fixes flaky CI tests in RPC Batch 2

[simzzz]

Description:
The beforeAll() block in the eth_getCode suite is timing out.

By optimizing the hook in the RPC API batch 2 and removing unnecessary and expensive calls, and then rerunning the job multiple times - now it fails much less often.

Related issue(s):

Fixes partially #3353

Notes for reviewer:

Checklist

[github-actions]

Test Results

 23 files  +  4  303 suites  +49   48m 4s ⏱️ + 11m 35s
620 tests + 10  612 ✅ +  8  4 💤 ±0  4 ❌ +2 
902 runs  +175  891 ✅ +172  6 💤 ±0  5 ❌ +3 

For more details on these failures, see this check.

Results for commit edba11b. ± Comparison against base commit ef2d8bb.

This pull request removes 2 and adds 12 tests. Note that renamed tests count towards both.

"before all" hook in "@erc20 Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests "before all" hook in "@erc20 Acceptance Tests"
should execute "web3_client_version" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests @release Hardcoded RPC Endpoints should execute "web3_client_version"

"after all" hook for "@release should execute "eth_estimateGas" for contract call, using a standard websocket" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-1 eth_estimateGas "after all" hook for "@release should execute "eth_estimateGas" for contract call, using a standard websocket"
"before all" hook for "Function calling HederaTokenService.isToken(token)" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-3 RPC Server Acceptance Tests eth_call with contract that calls precompiles "before all" hook for "Function calling HederaTokenService.isToken(token)"
@release Should return errors for blacklisted methods ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-3 RPC Server Acceptance Tests Batch Request Test Suite BATCH_REQUESTS_ENABLED = true @release Should return errors for blacklisted methods
Should allow different accounts associated in the same HbarSpendingPlan to contribute to the same budget ‑ 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 EXTENDED Tier Should allow different accounts associated in the same HbarSpendingPlan to contribute to the same budget
Should allow different accounts associated in the same HbarSpendingPlan to contribute to the same budget ‑ 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 Should allow different accounts associated in the same HbarSpendingPlan to contribute to the same budget
Should eventually exhaust the hbar limit for EXTENDED user and still allow another EXTENDED 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 EXTENDED Tier Should eventually exhaust the hbar limit for EXTENDED user and still allow another EXTENDED user to make calls
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 Should eventually exhaust the hbar limit for PRIVILEGED user and still allow another PRIVILEGED user to make calls
Should increase the amount spent of the spending plan by the transaction cost ‑ 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 EXTENDED Tier Should increase the amount spent of the spending plan by the transaction cost
Should increase the amount spent of the spending plan by the transaction cost ‑ 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 Should increase the amount spent of the spending plan by the transaction cost
should create a BASIC spending plan for a new user and use the same plan on second transaction and different plan on third transaction from another user ‑ 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 BASIC Tier should create a BASIC spending plan for a new user and use the same plan on second transaction and different plan on third transaction from another user
…

♻️ This comment has been updated with latest results.

[quiet-node]

Nice attempt! Would it be possible to also take this opportunity to fix the should execute "eth_sendRawTransaction" if the receiver's account has receiver_sig_required disabled' test? I’ve noticed it fails quite often. If you think it’s better addressed in a separate PR, that works too.

[simzzz]

@quiet-node Yes, I did debug that one as well but I still haven't determined an exact cause. I will get back to it in a separate PR

[konstantinabl]

a few comments

[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 SonarQube Cloud

[quiet-node]

LGTM thanks for looking into this definitely causes time and effort to re-run the CI

[quiet-node]

@simzzz woops Batch2 still fails pretty consistently

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.64%. Comparing base (4406824) to head (edba11b).
Report is 27 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3495      +/-   ##
==========================================
+ Coverage   85.17%   85.64%   +0.47%     
==========================================
  Files          69       75       +6     
  Lines        4734     4834     +100     
  Branches      999      998       -1     
==========================================
+ Hits         4032     4140     +108     
+ Misses        409      405       -4     
+ Partials      293      289       -4     

Flag	Coverage Δ
config-service	93.58% <ø> (-1.58%)	⬇️
relay	79.31% <ø> (-0.24%)	⬇️
server	85.15% <ø> (+1.55%)	⬆️
ws-server	36.31% <ø> (ø)

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

see 75 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.