-
Notifications
You must be signed in to change notification settings - Fork 19
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
test for broker-tcp and nmq-mqtt #515
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #515 +/- ##
==========================================
+ Coverage 51.27% 53.73% +2.45%
==========================================
Files 120 120
Lines 36422 36416 -6
==========================================
+ Hits 18677 19569 +892
+ Misses 17745 16847 -898 see 24 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
6444e08
to
f0af841
Compare
tests/trantest.h
Outdated
} | ||
if (nng_ctx_open(&work->ctx, tt->repsock) != 0) { | ||
printf("ctx_open fail\n"); | ||
} |
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.
Using assert or something else instead of IF or PRINTF ?
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.
got it
d8f145f
to
b0b6476
Compare
b0b6476
to
ca5538c
Compare
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.
Pretty good for 1st time
No description provided.