-
Notifications
You must be signed in to change notification settings - Fork 384
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
Add more observable counter tests #1818
Add more observable counter tests #1818
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1818 +/- ##
=======================================
- Coverage 73.6% 73.6% -0.1%
=======================================
Files 122 122
Lines 19529 19515 -14
=======================================
- Hits 14389 14376 -13
+ Misses 5140 5139 -1 ☔ View full report in Codecov by Sentry. |
// Run this test with stdout enabled to see output. | ||
// cargo test observable_counter_aggregation --features=metrics,testing -- --nocapture | ||
// cargo test observable_counter_aggregation_cumulative_non_zero_increment --features=testing -- --nocapture |
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.
nit - I know this is the existing comment. Do we know what output tester may want to see with nocapture
as mentioned in this comment. Not blocker, just curious.
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.
println() text is not captured in test output without this. It is used when debugging things quickly. Sometimes it is a lot easier that way, as opposed to running with RUST_FLAG to show stack when tests panic!
Demonstrating #1517
This adds more scenarios to improve coverage overall.
#1644 has the fix, but that is not yet merged as we are working through some major refactoring!