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 support for skip pattern #86
Conversation
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
/** | ||
* @author Pavol Loffay | ||
*/ | ||
public class OpenTracingBaseTests extends Arquillian { |
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.
The OpentracingClientTests
is quite big I have introduced this base class with some common methods. Different features can be tested in separate test classes.
@pilhuhn @fmhwong @objectiser could you please review? |
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
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.
LGTM - only thing not sure whether some of the headers should be updated to 2018 after the refactoring.
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
thanks @objectiser, I have changed year in license headers in new files to 2018. There is a license check and it was passing even with 2017 so I guess it's fine. |
@@ -74,50 +55,7 @@ | |||
* Opentracing TCK tests. | |||
* @author <a href="mailto:steve.m.fontes@gmail.com">Steve Fontes</a> | |||
*/ | |||
public class OpentracingClientTests extends Arquillian { |
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 have renamed it to OpenTracing
LGTM |
Resolves: #16
Example implementation: smallrye/smallrye-opentracing#23. The implementation itself is in https://github.com/opentracing-contrib/java-jaxrs/blob/master/opentracing-jaxrs2/src/main/java/io/opentracing/contrib/jaxrs2/server/ServerTracingFilter.java#L128
Signed-off-by: Pavol Loffay ploffay@redhat.com