Update logs TCK

eclipse · May 16, 2024 · 7670812 · 7670812
1 parent 0e6c69f
commit 7670812
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 8 deletions.
diff --git a/tck/logs/pom.xml b/tck/logs/pom.xml
@@ -50,11 +50,6 @@
             <groupId>io.opentelemetry</groupId>
             <artifactId>opentelemetry-sdk-extension-autoconfigure</artifactId>
         </dependency>
-        <dependency>
-            <groupId>jakarta.enterprise.concurrent</groupId>
-            <artifactId>jakarta.enterprise.concurrent-api</artifactId>
-            <version>3.0.3</version>
-        </dependency>        
         <dependency>
             <groupId>io.opentelemetry.instrumentation</groupId>
             <artifactId>opentelemetry-instrumentation-annotations</artifactId>

diff --git a/...rc/main/java/org/eclipse/microprofile/telemetry/logs/tck/application/LogAppenderTest.java b/...rc/main/java/org/eclipse/microprofile/telemetry/logs/tck/application/LogAppenderTest.java
@@ -20,6 +20,7 @@
 
 package org.eclipse.microprofile.telemetry.logs.tck.application;
 
+import java.io.File;
 import java.util.logging.Logger;
 
 import org.eclipse.microprofile.telemetry.logs.tck.TestLibraries;
@@ -31,6 +32,7 @@
 import org.jboss.shrinkwrap.api.asset.EmptyAsset;
 import org.jboss.shrinkwrap.api.asset.StringAsset;
 import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.shrinkwrap.resolver.api.maven.Maven;
 import org.slf4j.bridge.SLF4JBridgeHandler;
 import org.testng.Assert;
 import org.testng.annotations.Test;
@@ -43,14 +45,25 @@
 public class LogAppenderTest extends Arquillian {
     @Deployment
     public static WebArchive createDeployment() {
+        File[] slf4jFiles = Maven.resolver().resolve("org.slf4j:jul-to-slf4j:2.0.13").withTransitivity().asFile();
+        File[] telemetryLogbackFiles = Maven.resolver()
+                .resolve("io.opentelemetry.instrumentation:opentelemetry-logback-appender-1.0:2.2.0-alpha")
+                .withTransitivity().asFile();
+        File[] logbackFiles =
+                Maven.resolver().resolve("ch.qos.logback:logback-classic:1.5.5").withTransitivity().asFile();
+
         return ShrinkWrap.create(WebArchive.class)
                 .addClasses(InMemoryLogRecordExporter.class, InMemoryLogRecordExporterProvider.class)
                 .addAsLibrary(TestLibraries.AWAITILITY_LIB)
+                .addAsLibraries(slf4jFiles)
+                .addAsLibraries(telemetryLogbackFiles)
+                .addAsLibraries(logbackFiles)
                 .addAsServiceProvider(ConfigurableLogRecordExporterProvider.class,
                         InMemoryLogRecordExporterProvider.class)
                 .addAsResource(new StringAsset(
-                        "otel.sdk.disabled=false\notel.metrics.exporter=\notel.traces.exporter=none\notel.logs.exporter=in-memory"),
+                        "otel.sdk.disabled=false\notel.metrics.exporter=none\notel.traces.exporter=none\notel.logs.exporter=in-memory\notel.blrp.max.export.batch.size=1\notel.blrp.schedule.delay=30"),
                         "META-INF/microprofile-config.properties")
+                .addAsResource("logback.xml")
                 .addAsManifestResource(EmptyAsset.INSTANCE, "beans.xml");
     }
 
@@ -60,7 +73,7 @@ public static WebArchive createDeployment() {
     @Inject
     private InMemoryLogRecordExporter memoryExporter;
 
-    private static final java.util.logging.Logger julLogger = Logger.getLogger("jul-logger");
+    private static final Logger julLogger = Logger.getLogger("jul-logger");
 
     @Test
     void julTest() throws InterruptedException {
@@ -71,6 +84,8 @@ void julTest() throws InterruptedException {
         SLF4JBridgeHandler.removeHandlersForRootLogger();
         SLF4JBridgeHandler.install();
         julLogger.info("A JUL log message");
-        Assert.assertEquals(memoryExporter.getFinishedLogRecordItems(1).get(0).getBody(), "A JUL log message");
+
+        Assert.assertTrue(
+                memoryExporter.getFinishedLogRecordItems(1).get(0).getBody().toString().contains("A JUL log message"));
     }
 }
diff --git a/tck/logs/src/main/resources/logback.xml b/tck/logs/src/main/resources/logback.xml
@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<configuration>
+    <appender name="OpenTelemetry"
+              class="io.opentelemetry.instrumentation.logback.appender.v1_0.OpenTelemetryAppender">
+        <captureExperimentalAttributes>true</captureExperimentalAttributes>
+        <captureKeyValuePairAttributes>true</captureKeyValuePairAttributes>
+    </appender>
+    <root level="INFO">
+        <appender-ref ref="OpenTelemetry"/>
+    </root>
+</configuration>