Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add tests and coverage skeleton for RPC proxy #1200

Merged
merged 13 commits into from
Sep 2, 2024

Conversation

rodolfopietro97
Copy link
Collaborator

Description

Add missing tests for RPC proxy (parsing of configuration, parameters, ...)

@rodolfopietro97 rodolfopietro97 requested a review from a team as a code owner August 27, 2024 12:22
@rodolfopietro97 rodolfopietro97 linked an issue Aug 27, 2024 that may be closed by this pull request
@rodolfopietro97 rodolfopietro97 marked this pull request as draft August 27, 2024 12:23
Copy link

github-actions bot commented Aug 27, 2024

Test Coverage

Summary

Lines Statements Branches Functions
Coverage: 99%
99.42% (3782/3804) 97.73% (1036/1060) 99.86% (757/758)
Title Tests Skipped Failures Errors Time
core 548 0 💤 0 ❌ 0 🔥 1m 28s ⏱️
network 686 0 💤 0 ❌ 0 🔥 4m 9s ⏱️
errors 43 0 💤 0 ❌ 0 🔥 15.066s ⏱️

@rodolfopietro97 rodolfopietro97 marked this pull request as ready for review September 2, 2024 12:42
@victhorbi
Copy link
Collaborator

as anticipated by @rodolfopietro97 , the documentation for the CLI arguments is not available yet, and will be part of a separate PR.

@rodolfopietro97 rodolfopietro97 merged commit 40a2c37 into main Sep 2, 2024
10 checks passed
@rodolfopietro97 rodolfopietro97 deleted the 1197-rpc-proxy---add-coverage branch September 2, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RPC Proxy - Add coverage
3 participants