Making TracingClientInterceptor.Builder constructor private (#47)

opentracing-contrib · Mar 5, 2020 · e6f7911 · e6f7911
1 parent d95b851
commit e6f7911
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/java/io/opentracing/contrib/grpc/TracingClientInterceptor.java b/src/main/java/io/opentracing/contrib/grpc/TracingClientInterceptor.java
@@ -332,7 +332,7 @@ public static class Builder {
     private Map<Class<?>, ClientCloseDecorator> clientCloseDecorators;
 
     /** Creates a Builder with GlobalTracer if present else NoopTracer. */
-    public Builder() {
+    private Builder() {
       this.tracer = GlobalTracer.isRegistered() ? GlobalTracer.get() : NoopTracerFactory.create();
       this.operationNameConstructor = OperationNameConstructor.DEFAULT;
       this.streaming = false;

diff --git a/src/test/java/io/opentracing/contrib/grpc/TracingClientInterceptorTest.java b/src/test/java/io/opentracing/contrib/grpc/TracingClientInterceptorTest.java
@@ -133,7 +133,7 @@ public void testTracedClientTwoInterceptors() {
   @Test
   public void testTracedClientWithVerbosity() {
     TracingClientInterceptor tracingInterceptor =
-        new TracingClientInterceptor.Builder().withTracer(clientTracer).withVerbosity().build();
+        TracingClientInterceptor.newBuilder().withTracer(clientTracer).withVerbosity().build();
     TracedClient client = new TracedClient(grpcServer.getChannel(), tracingInterceptor);
 
     assertEquals("call should complete successfully", "Hello world", client.greet().getMessage());