test: added HBAR Limiter e2e tests for multiple users across different tiers

hashgraph / hedera-json-rpc-relay

Implementation of Ethereum JSON-RPC APIs for Hedera

Apache License 2.0

68 stars 72 forks source link

test: added HBAR Limiter e2e tests for multiple users across different tiers #3163

Closed quiet-node closed 2 weeks ago

quiet-node commented 3 weeks ago

Description:

This PR adds HBAR Limiter e2e tests for multiple users across different tiers: BASIC, EXTENDED, and PRIVILEGED.
With the new implementation of HbarLimitService enabled, the HBAR Limiter test has become highly flaky. Since adding expenses and updating spending plans are executed asynchronously, and the suite includes exhaustion tests with numerous consecutive calls to the network, the test client sometimes fails to retrieve the correct amounts. The tests are flaky to the extent that they either fail or only pass after multiple retries. To address this, this PR includes measures to stabilize the tests by:
- Adding polling methods to verify the remaining HBARs and amounts spent by hbarSpendingPlans.
- Increasing tolerance in verifyRemainingLimit().
- Adding a 3000ms wait time after each test case to allow asynchronous expense updates to complete in the background.

Fixes #3068 Fixes #3188

Notes for reviewer:

Checklist

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

github-actions[bot] commented 3 weeks ago

Test Results

20 files - 3 280 suites - 8 31m 28s :stopwatch: - 5m 9s 608 tests ± 0 600 :white_check_mark: +7 4 :zzz: ±0 4 :x: - 7 773 runs - 16 765 :white_check_mark: - 7 4 :zzz: - 2 4 :x: - 7

For more details on these failures, see this check.

Results for commit 6e267f56. ± Comparison against base commit 6606c385.

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

``` "before all" hook for "@release should execute "eth_getTransactionCount" primary" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-3 RPC Server Acceptance Tests eth_getTransactionCount "before all" hook for "@release should execute "eth_getTransactionCount" primary" "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)" "before all" hook in "Debug API Test Suite" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-3 RPC Server Acceptance Tests Debug API Test Suite "before all" hook in "Debug API Test Suite" "before all" hook in "RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-1 RPC Server Acceptance Tests RPC Server Acceptance Tests "before all" hook in "RPC Server Acceptance Tests" Should eventually exhaust the hbar limit for EXTENDED 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 Preconfigured Tiers EXTENDED Tier Should eventually exhaust the hbar limit for EXTENDED user Should eventually exhaust the hbar limit for PRIVILEGED 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 Preconfigured Tiers PRIVILEGED Tier Should eventually exhaust the hbar limit for PRIVILEGED user ``` ``` "after all" hook in "@web-socket-batch-2 eth_getTransactionByHash" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-2 eth_getTransactionByHash "after all" hook in "@web-socket-batch-2 eth_getTransactionByHash" "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" 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 eventually exhaust the total HBAR limits after many large contract deployments by different tiered users ‑ 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 Multiple users with different tiers Should eventually exhaust the total HBAR limits after many large contract deployments by different tiered users should individually update amountSpents of different spending plans ‑ 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 Multiple users with different tiers should individually update amountSpents of different spending plans ```

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute "net_peerCount". This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 32,912.4 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 425.24 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 849.62 KB
Heap Size Limit: no changes
Malloced Memory: increased with 64.88 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 577.12 KB
- Space Available Size: increased with 577.12 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute "eth_sendTransaction". This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 28,550.9 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 99.97 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 961.62 KB
Heap Size Limit: no changes
Malloced Memory: increased with 59.46 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 579.34 KB
- Space Available Size: increased with 1.61 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should not execute batch request when disabled. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 27,607 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 644.02 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 456.01 KB
Heap Size Limit: no changes
Malloced Memory: increased with 65.34 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 362.83 KB
- Space Available Size: increased with 362.83 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates parameter 1 is valid block number. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 29,336.9 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 505.94 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 696.58 KB
Heap Size Limit: no changes
Malloced Memory: increased with 7.42 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 494.15 KB
- Space Available Size: increased with 1.53 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates parameter 0 exists. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 29,951.3 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 877.34 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 631.26 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 550.07 KB
- Space Available Size: increased with 550.07 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates Transactionvalueparam is hex. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 25,241.1 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 551.88 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 553.82 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 478.84 KB
- Space Available Size: increased with 1.51 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates parameter 2 is array. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 29,381.3 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 1.06 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 737.10 KB
Heap Size Limit: no changes
Malloced Memory: increased with 156.26 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 474.54 KB
- Space Available Size: increased with 474.54 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates parameter 0 is valid block tag. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 26,410.8 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: decreased with 1.22 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 518.06 KB
Heap Size Limit: no changes
Malloced Memory: decreased with 139.37 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 450.22 KB
- Space Available Size: increased with 1.48 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute HTTP OPTIONS cors preflight check. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 30,350.6 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 1.51 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 390.90 KB
Heap Size Limit: no changes
Malloced Memory: increased with 132.57 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 226.28 KB
- Space Available Size: increased with 226.28 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute metrics collection. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: Scavenge Cost: 3,503.09 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 579.62 KB
Total Global Handles Size: no changes
Used Global Handles Size: no changes
Used Heap Size: decreased with 601.13 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 716.70 KB
- Space Available Size: increased with 1.75 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute with valid hash, no TracerType and no TracerConfig. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 25,784.9 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.31 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 536.58 KB
Total Available Size: decreased with 850.50 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 681.90 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 455.29 KB
- Space Available Size: increased with 455.29 KB
- Physical Space Size: increased with 274.43 KB
Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: decreased with 1.26 KB
- Space Available Size: increased with 5.44 KB
- Physical Space Size: increased with 262.14 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute with empty TracerConfigWrapper.tracerConfig. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 36,856.8 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.31 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 536.58 KB
Total Available Size: decreased with 2.54 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 677.61 KB
Heap Size Limit: no changes
Malloced Memory: increased with 33.35 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 471.98 KB
- Space Available Size: increased with 471.98 KB
- Physical Space Size: increased with 274.43 KB
Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: decreased with 7.73 KB
- Space Available Size: decreased with 1.68 MB
- Physical Space Size: increased with 262.14 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should fail with valid hash, valid tracer type and invalid tracer configuration. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 30,592.1 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 471.68 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 970.45 KB
Heap Size Limit: no changes
Malloced Memory: increased with 7.42 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 672.38 KB
- Space Available Size: increased with 1.70 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error if address is other type. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 26,699.3 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 916.05 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 133.42 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 108.24 KB
- Space Available Size: increased with 108.24 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws error if block number is decimal. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 36,173.9 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 260.06 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 284.95 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 77.77 KB
- Space Available Size: increased with 1.11 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error if the param is not an Object. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 26,561.6 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 885.52 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 164.70 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 116.42 KB
- Space Available Size: increased with 116.42 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled does not throw an error for input: {"tracer":"opcodeLogger","tracerConfig":{"enableMemory":true,"disableStack":false,"disableStorage":true,"unknownParam":true}}. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 26,361.7 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 125.22 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 144.00 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 54.70 KB
- Space Available Size: increased with 1.09 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error if Filter Object param contains unexpected param. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 40,680.5 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.05 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 274.43 KB
Total Available Size: decreased with 809.86 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 244.67 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 64.19 KB
- Space Available Size: increased with 64.19 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should return true when REQUEST_ID_IS_OPTIONAL is set to true. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 38,114.1 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 124.81 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 123.74 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 80.06 KB
- Space Available Size: increased with 1.11 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions[bot] commented 3 weeks ago

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates enforcement of request id. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact Cost: 20,627.2 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.46 MB
Total Heap Size Executable: no changes
Total Physical Size: decreased with 348.16 KB
Total Available Size: decreased with 13.76 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 3.45 MB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

Old Space:
- Space Size: increased with 1.84 MB
- Space Used Size: increased with 2.07 MB
- Space Available Size: decreased with 16.55 MB
- Physical Space Size: increased with 1.84 MB
Large Object Space:
- Space Size: increased with 835.58 KB
- Space Used Size: increased with 813.50 KB
- Space Available Size: no changes
- Physical Space Size: increased with 835.58 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

sonarcloud[bot] commented 2 weeks ago

Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarCloud

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.39%. Comparing base (6606c38) to head (6e267f5).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #3163 +/- ## ======================================= Coverage 83.39% 83.39% ======================================= Files 66 66 Lines 4283 4283 Branches 835 835 ======================================= Hits 3572 3572 Misses 471 471 Partials 240 240 ``` | [Flag](https://app.codecov.io/gh/hashgraph/hedera-json-rpc-relay/pull/3163/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | Coverage Δ | | |---|---|---| | [config-service](https://app.codecov.io/gh/hashgraph/hedera-json-rpc-relay/pull/3163/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | `98.14% <ø> (ø)` | | | [relay](https://app.codecov.io/gh/hashgraph/hedera-json-rpc-relay/pull/3163/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | `85.60% <ø> (ø)` | | | [server](https://app.codecov.io/gh/hashgraph/hedera-json-rpc-relay/pull/3163/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | `83.52% <ø> (ø)` | | | [ws-server](https://app.codecov.io/gh/hashgraph/hedera-json-rpc-relay/pull/3163/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | `36.87% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph#carryforward-flags-in-the-pull-request-comment) to find out more.