kotlinx + integration tests

Author: SylvainJuge

instrumentation/kotlinx-coroutines/kotlinx-coroutines-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/kotlinxcoroutines/instrumentationannotations/AnnotationInstrumentationModule.java

@@ -26,12 +26,6 @@ public AnnotationInstrumentationModule() {
         "opentelemetry-instrumentation-annotations");
   }
 
-  @Override
-  public boolean isIndyModule() {
-    // needs helper classes in the same class loader
-    return false;
-  }
-
   @Override
   public int order() {
     // Run first to ensure other automatic instrumentation is added after and therefore is executed

testing-common/integration-tests/build.gradle.kts

@@ -52,15 +52,13 @@ tasks {
       includeTestsMatching("InstrumentOldBytecode")
     }
     include("**/InstrumentOldBytecode.*")
-    jvmArgs("-Dotel.instrumentation.inline-ibm-resource-level.enabled=false")
   }
 
   val testInlineModuleOldBytecodeInstrumentation by registering(Test::class) {
     filter {
       includeTestsMatching("InstrumentOldBytecode")
     }
     include("**/InstrumentOldBytecode.*")
-    jvmArgs("-Dotel.instrumentation.indy-ibm-resource-level.enabled=false")
   }
 
   test {

testing-common/integration-tests/src/main/java/InlineIbmResourceLevelInstrumentationModule.java testing-common/integration-tests/src/main/java/IbmResourceLevelInstrumentationModule.java

@@ -11,13 +11,13 @@
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)
-public class InlineIbmResourceLevelInstrumentationModule extends InstrumentationModule {
-  public InlineIbmResourceLevelInstrumentationModule() {
-    super("inline-ibm-resource-level");
+public class IbmResourceLevelInstrumentationModule extends InstrumentationModule {
+  public IbmResourceLevelInstrumentationModule() {
+    super("ibm-resource-level");
   }
 
   @Override
   public List<TypeInstrumentation> typeInstrumentations() {
-    return singletonList(new InlineResourceLevelInstrumentation());
+    return singletonList(new ResourceLevelInstrumentation());
   }
 }

testing-common/integration-tests/src/main/java/IndyIbmResourceLevelInstrumentationModule.java

@@ -11,7 +11,7 @@
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
-public class IndyResourceLevelInstrumentation implements TypeInstrumentation {
+public class ResourceLevelInstrumentation implements TypeInstrumentation {
   @Override
   public ElementMatcher<TypeDescription> typeMatcher() {
     return named("com.ibm.as400.resource.ResourceLevel");
@@ -25,7 +25,7 @@ public void transform(TypeTransformer transformer) {
 
   @SuppressWarnings("unused")
   public static class ToStringAdvice {
-    @Advice.OnMethodExit(suppress = Throwable.class, inline = false)
+    @Advice.OnMethodExit(suppress = Throwable.class)
     @Advice.AssignReturned.ToReturned
     public static String toStringReplace() {
       return "instrumented";