Remove usage of groovy test classes (open-telemetry#13235)

Author: laurit

instrumentation/netty/netty-4.1/testing/src/main/java/io/opentelemetry/instrumentation/netty/v4_1/AbstractNetty41ClientTest.java

@@ -5,7 +5,6 @@
 
 package io.opentelemetry.instrumentation.netty.v4_1;
 
-import static io.opentelemetry.instrumentation.test.base.HttpClientTest.getPort;
 import static io.opentelemetry.semconv.ServerAttributes.SERVER_ADDRESS;
 import static io.opentelemetry.semconv.ServerAttributes.SERVER_PORT;
 
@@ -138,6 +137,18 @@ private static String getExpectedClientSpanName(URI uri, String method) {
     }
   }
 
+  private static int getPort(URI uri) {
+    if (uri.getPort() != -1) {
+      return uri.getPort();
+    } else if ("http".equals(uri.getScheme())) {
+      return 80;
+    } else if ("https".equals(uri.getScheme())) {
+      return 443;
+    } else {
+      throw new IllegalArgumentException("Unexpected uri: " + uri);
+    }
+  }
+
   @Test
   void closeChannel() throws ExecutionException, InterruptedException {
     String method = "GET";

instrumentation/spring/spring-webflux/spring-webflux-5.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/webflux/v5_0/server/base/package-info.java

@@ -162,18 +162,6 @@ abstract class HttpClientTest<REQUEST> extends InstrumentationSpecification {
     server.stop()
   }
 
-  static int getPort(URI uri) {
-    if (uri.port != -1) {
-      return uri.port
-    } else if (uri.scheme == "http") {
-      return 80
-    } else if (uri.scheme == "https") {
-      443
-    } else {
-      throw new IllegalArgumentException("Unexpected uri: $uri")
-    }
-  }
-
   def "basic GET request #path"() {
     expect:
     junitTest.successfulGetRequest(path)

testing-common/src/main/groovy/io/opentelemetry/instrumentation/test/base/HttpClientTest.groovy

@@ -121,8 +121,9 @@ URI resolveWithoutFragment(URI address) throws URISyntaxException {
 
   /**
    * Populates custom test attributes for the {@link
-   * io.opentelemetry.instrumentation.test.base.HttpServerTest#controller} span (which must be the
-   * current span when this is called) based on URL parameters. Required for {@link #INDEXED_CHILD}.
+   * io.opentelemetry.instrumentation.testing.junit.http.AbstractHttpServerTest#controller} span
+   * (which must be the current span when this is called) based on URL parameters. Required for
+   * {@link #INDEXED_CHILD}.
    */
   public void collectSpanAttributes(ServerEndpoint.UrlParameterProvider parameterProvider) {
     if (this == INDEXED_CHILD) {