fix: Bug due to validations of eth_call

[victor-yanev]

Description:

This PR modifies the object validation logic in order to support the deletion of unknown properties.

• Adds check for deleteUnknownProperties in object schema
• Calls deleteUnknownProperties method if the schema property is set

Related issue(s):

Fixes #2827

Checklist

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

[github-actions]

🚨 Memory Leak Detected 🚨

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

Details

📊 Memory Leak Detection Report 📊

GC Type: Scavenge
Cost: 5,372.42 ms

Heap Statistics (before vs after executing the test):

• Total Heap Size: increased with 1.69 MB
• Total Heap Size Executable: no changes
• Total Physical Size: increased with 1.69 MB
• Total Available Size: increased with 9.63 MB
• Total Global Handles Size: no changes
• Used Global Handles Size: decreased with 352.00 bytes
• Used Heap Size: decreased with 9.30 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 3.23 MB
  • Space Available Size: decreased with 1.49 MB
  • Physical Space Size: increased with 1.84 MB

• Large Object Space:

  • Space Size: increased with 282.62 KB
  • Space Used Size: increased with 277.54 KB
  • Space Available Size: no changes
  • Physical Space Size: increased with 282.62 KB

Recommendations

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

[github-actions]

Acceptance Tests

  21 files  264 suites   33m 20s ⏱️
612 tests 588 ✔️ 4 💤 20 ❌
860 runs  834 ✔️ 4 💤 22 ❌

Results for commit 0f5e932.

♻️ This comment has been updated with latest results.

[sonarcloud]

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

[github-actions]

Tests

       3 files     204 suites   58s ⏱️
1 038 tests 1 037 ✔️ 1 💤 0 ❌
1 050 runs  1 049 ✔️ 1 💤 0 ❌

Results for commit 0f5e932.

[ebadiere]

LG.

[acuarica]

lg

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.19%. Comparing base (d75c277) to head (0f5e932).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2843      +/-   ##
==========================================
+ Coverage   79.76%   81.19%   +1.42%     
==========================================
  Files          30       46      +16     
  Lines        2718     3403     +685     
  Branches      586      717     +131     
==========================================
+ Hits         2168     2763     +595     
- Misses        356      409      +53     
- Partials      194      231      +37     

Flag	Coverage Δ
relay	80.90% <ø> (+1.14%)	⬆️
server	81.19% <100.00%> (?)
ws-server	97.87% <ø> (?)

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

Files	Coverage Δ
packages/server/src/validator/objectTypes.ts	90.24% <100.00%> (ø)

... and 16 files with indirect coverage changes