-
Notifications
You must be signed in to change notification settings - Fork 84
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
cmd: alpha test peers tests self #3052
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple comments
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3052 +/- ##
==========================================
+ Coverage 54.62% 54.89% +0.26%
==========================================
Files 207 209 +2
Lines 29158 29822 +664
==========================================
+ Hits 15928 16371 +443
- Misses 11388 11630 +242
+ Partials 1842 1821 -21 ☔ View full report in Codecov by Sentry. |
a86625c
to
661613e
Compare
661613e
to
88f05ed
Compare
Quality Gate passedIssues Measures |
Add alpha peers tests for the TCP node started. Currently only port opened is tested. Test for whether NAT is open is skipped for now as it makes more sense to be added to the peers' tests.
category: feature
ticket: #3076