Ver Fonte

Enable running with indy for jodd-http-4.2 (#9898)

Lauri Tulmin há 1 ano atrás
pai
commit
a3b0e31b4b

+ 10 - 0
instrumentation/jodd-http-4.2/javaagent-unit-tests/build.gradle.kts

@@ -0,0 +1,10 @@
+plugins {
+  id("otel.java-conventions")
+}
+
+dependencies {
+  testImplementation("org.jodd:jodd-http:4.2.0")
+  testImplementation(project(":instrumentation:jodd-http-4.2:javaagent"))
+  testImplementation(project(":instrumentation-api"))
+  testImplementation(project(":instrumentation-api-semconv"))
+}

+ 0 - 0
instrumentation/jodd-http-4.2/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/joddhttp/v4_2/JoddHttpHttpAttributesGetterTest.java → instrumentation/jodd-http-4.2/javaagent-unit-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/joddhttp/v4_2/JoddHttpHttpAttributesGetterTest.java


+ 0 - 7
instrumentation/jodd-http-4.2/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/joddhttp/v4_2/JoddHttpInstrumentationModule.java

@@ -18,13 +18,6 @@ public class JoddHttpInstrumentationModule extends InstrumentationModule {
     super("jodd-http", "jodd-http-4.2");
   }
 
-  @Override
-  public boolean isIndyModule() {
-    // JoddHttpHttpAttributesGetterTest is not an agent test, with indy it can't access
-    // JoddHttpHttpAttributesGetter
-    return false;
-  }
-
   @Override
   public List<TypeInstrumentation> typeInstrumentations() {
     return Collections.singletonList(new JoddHttpInstrumentation());

+ 1 - 0
settings.gradle.kts

@@ -324,6 +324,7 @@ include(":instrumentation:jms:jms-common:javaagent-unit-tests")
 include(":instrumentation:jmx-metrics:javaagent")
 include(":instrumentation:jmx-metrics:library")
 include(":instrumentation:jodd-http-4.2:javaagent")
+include(":instrumentation:jodd-http-4.2:javaagent-unit-tests")
 include(":instrumentation:jsf:jsf-javax-common:javaagent")
 include(":instrumentation:jsf:jsf-javax-common:testing")
 include(":instrumentation:jsf:jsf-jakarta-common:javaagent")