-
Notifications
You must be signed in to change notification settings - Fork 849
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
OpenTracing Shim: Handle unsupported types when setting Attributes. #4939
OpenTracing Shim: Handle unsupported types when setting Attributes. #4939
Conversation
This happens through: * SpanBuilder.withTag() * Span.setTag()
Codecov ReportBase: 90.96% // Head: 91.01% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #4939 +/- ##
============================================
+ Coverage 90.96% 91.01% +0.04%
- Complexity 4838 4841 +3
============================================
Files 546 546
Lines 14439 14442 +3
Branches 1395 1395
============================================
+ Hits 13135 13144 +9
+ Misses 901 893 -8
- Partials 403 405 +2
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 at Codecov. |
SpanData spanData = ((ReadableSpan) span.getSpan()).toSpanData(); | ||
assertThat(spanData.getAttributes().size()).isEqualTo(1); | ||
assertThat(spanData.getAttributes().get(AttributeKey.stringKey("foo"))) | ||
.isEqualTo(BigInteger.TEN.toString()); |
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.
Do you think there's any value in being explicit about the expected string? I.e. .isEqualTo("10")
?
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.
Assertions now compare against literals directly ;)
…pen-telemetry#4939) * OpenTracing Shim: Handle unsupported types when setting Attributes. This happens through: * SpanBuilder.withTag() * Span.setTag() * Simplify BigInteger conversion assertions.
Fixes #4908
This happens via: