Update

Author: heyams

instrumentation/play/play-ws/play-ws-1.0/javaagent/src/test/groovy/PlayWsClientTest.groovy

@@ -0,0 +1,19 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+package io.opentelemetry.javaagent.instrumentation.playws.v1_0;
+
+import io.opentelemetry.javaagent.instrumentation.playws.PlayJavaStreamedWsClientBaseTest;
+import io.opentelemetry.javaagent.instrumentation.playws.PlayJavaWsClientBaseTest;
+import io.opentelemetry.javaagent.instrumentation.playws.PlayScalaStreamedWsClientBaseTest;
+import io.opentelemetry.javaagent.instrumentation.playws.PlayScalaWsClientBaseTest;
+
+class PlayJavaWsClientTest extends PlayJavaWsClientBaseTest {}
+
+class PlayJavaStreamedWsClientTest extends PlayJavaStreamedWsClientBaseTest {}
+
+class PlayScalaWsClientTest extends PlayScalaWsClientBaseTest {}
+
+class PlayScalaStreamedWsClientTest extends PlayScalaStreamedWsClientBaseTest {}

instrumentation/play/play-ws/play-ws-1.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/playws/v1_0/PlayWsClientTest.java

@@ -112,7 +112,7 @@ protected void configure(HttpClientTestOptions.Builder optionsBuilder) {
         });
   }
 
-  private AsyncHttpClient createClient(boolean readTimeout) {
+  private static AsyncHttpClient createClient(boolean readTimeout) {
     DefaultAsyncHttpClientConfig.Builder builder =
         new DefaultAsyncHttpClientConfig.Builder()
             .setMaxRequestRetry(0)
@@ -128,31 +128,31 @@ private AsyncHttpClient createClient(boolean readTimeout) {
     AsyncHttpClientConfig asyncHttpClientConfig = builder.build();
     return new DefaultAsyncHttpClient(asyncHttpClientConfig);
   }
-}
 
-class CustomNameResolver extends InetNameResolver {
+  private static class CustomNameResolver extends InetNameResolver {
 
-  public CustomNameResolver(EventExecutor executor) {
-    super(executor);
-  }
+    public CustomNameResolver(EventExecutor executor) {
+      super(executor);
+    }
 
-  @Override
-  protected void doResolve(String inetHost, Promise<InetAddress> promise) throws Exception {
-    try {
-      promise.setSuccess(InetAddress.getByName(inetHost));
-    } catch (UnknownHostException exception) {
-      promise.setFailure(exception);
+    @Override
+    protected void doResolve(String inetHost, Promise<InetAddress> promise) throws Exception {
+      try {
+        promise.setSuccess(InetAddress.getByName(inetHost));
+      } catch (UnknownHostException exception) {
+        promise.setFailure(exception);
+      }
     }
-  }
 
-  @Override
-  protected void doResolveAll(String inetHost, Promise<List<InetAddress>> promise)
-      throws Exception {
-    try {
-      // default implementation calls InetAddress.getAllByName
-      promise.setSuccess(Collections.singletonList(InetAddress.getByName(inetHost)));
-    } catch (UnknownHostException exception) {
-      promise.setFailure(exception);
+    @Override
+    protected void doResolveAll(String inetHost, Promise<List<InetAddress>> promise)
+        throws Exception {
+      try {
+        // default implementation calls InetAddress.getAllByName
+        promise.setSuccess(Collections.singletonList(InetAddress.getByName(inetHost)));
+      } catch (UnknownHostException exception) {
+        promise.setFailure(exception);
+      }
     }
   }
 }