Remove some references to groovy (#13099)

open-telemetry · Jan 24, 2025 · 3620bab · 3620bab
1 parent 9e70f93
commit 3620bab
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 9 deletions.
diff --git a/.../test/java/io/opentelemetry/javaagent/instrumentation/jaxws/jws/v1_1/WebServiceClass.java b/.../test/java/io/opentelemetry/javaagent/instrumentation/jaxws/jws/v1_1/WebServiceClass.java
@@ -7,7 +7,6 @@
 
 import javax.jws.WebService;
 
-// This is pure java to not have any groovy generated public method surprises
 @WebService
 public class WebServiceClass {
   public void doSomethingPublic() {}

diff --git a/...a/io/opentelemetry/javaagent/instrumentation/spring/batch/v3_0/jsr/TestItemProcessor.java b/...a/io/opentelemetry/javaagent/instrumentation/spring/batch/v3_0/jsr/TestItemProcessor.java
@@ -6,11 +6,10 @@
 package io.opentelemetry.javaagent.instrumentation.spring.batch.v3_0.jsr;
 
 import javax.batch.api.chunk.ItemProcessor;
-import org.codehaus.groovy.runtime.DefaultGroovyMethods;
 
 class TestItemProcessor implements ItemProcessor {
   @Override
   public Object processItem(Object item) {
-    return Integer.parseInt(DefaultGroovyMethods.asType(item, String.class));
+    return Integer.parseInt((String) item);
   }
 }
diff --git a/...java/io/opentelemetry/javaagent/instrumentation/spring/batch/v3_0/jsr/TestItemWriter.java b/...java/io/opentelemetry/javaagent/instrumentation/spring/batch/v3_0/jsr/TestItemWriter.java
@@ -9,7 +9,6 @@
 import java.util.ArrayList;
 import java.util.List;
 import javax.batch.api.chunk.ItemWriter;
-import org.codehaus.groovy.runtime.DefaultGroovyMethods;
 
 class TestItemWriter implements ItemWriter {
   private final List<Integer> items = new ArrayList<>();
@@ -23,7 +22,7 @@ public void close() {}
   @Override
   public void writeItems(List<Object> items) {
     for (Object item : items) {
-      this.items.add(DefaultGroovyMethods.asType(item, Integer.class));
+      this.items.add((Integer) item);
     }
   }
 

diff --git a/...g/spring-webflux/spring-webflux-5.0/javaagent/src/test/java/server/RedirectComponent.java b/...g/spring-webflux/spring-webflux-5.0/javaagent/src/test/java/server/RedirectComponent.java
@@ -14,7 +14,6 @@
 import org.springframework.web.reactive.function.server.RouterFunction;
 import org.springframework.web.reactive.function.server.ServerResponse;
 
-// Need to keep this in Java because groovy creates crazy proxies around lambdas
 @Component
 public class RedirectComponent {
   @Bean

diff --git a/instrumentation/vertx/vertx-web-3.0/javaagent/build.gradle.kts b/instrumentation/vertx/vertx-web-3.0/javaagent/build.gradle.kts
@@ -53,9 +53,6 @@ tasks {
     named("test") {
       enabled = false
     }
-    named("compileTestGroovy") {
-      enabled = false
-    }
   }
 
   named("latestDepTest") {