-
Notifications
You must be signed in to change notification settings - Fork 36
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
Implement setParent() for SimpleTracer, implement assertions for parentId #676
Implement setParent() for SimpleTracer, implement assertions for parentId #676
Conversation
...ests/micrometer-tracing-test/src/main/java/io/micrometer/tracing/test/simple/SpanAssert.java
Outdated
Show resolved
Hide resolved
...ests/micrometer-tracing-test/src/main/java/io/micrometer/tracing/test/simple/SpanAssert.java
Outdated
Show resolved
Hide resolved
...ests/micrometer-tracing-test/src/main/java/io/micrometer/tracing/test/simple/SpanAssert.java
Outdated
Show resolved
Hide resolved
SimpleTracer simpleTracer = new SimpleTracer(); | ||
SimpleSpanBuilder builder = new SimpleSpanBuilder(simpleTracer); | ||
SimpleTraceContextBuilder ctxBuilder = new SimpleTraceContextBuilder(); | ||
TraceContext parentCtx = ctxBuilder.spanId("spam").traceId("bar").sampled(true).build(); |
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.
"span" or "spam"? :D
...micrometer-tracing-test/src/test/java/io/micrometer/tracing/test/simple/SpanAssertTests.java
Show resolved
Hide resolved
...micrometer-tracing-test/src/test/java/io/micrometer/tracing/test/simple/SpanAssertTests.java
Outdated
Show resolved
Hide resolved
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.
I accidentally submitted my previous review with approve, what I meant is "Request changes".
@jonatan-ivanov any more notes for this one? |
@jamesmoessis Thank you for the PR! |
Fixes #675
I've also added assertions for parent ID since they didn't exist before, and it was useful for testing the change to SimpleTracer.