This is an automated email from the ASF dual-hosted git repository.

gnodet pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new 5b3870618823 CAMEL-23427: Fix flaky SpanPropagationUpstreamTest in 
camel-telemetry-dev
5b3870618823 is described below

commit 5b3870618823109a0657a9b3a8047e7f813a9b78
Author: Guillaume Nodet <[email protected]>
AuthorDate: Wed May 6 10:27:39 2026 +0200

    CAMEL-23427: Fix flaky SpanPropagationUpstreamTest in camel-telemetry-dev
    
    Change clearLogTraces() from @AfterEach to @BeforeEach to ensure the trace 
log file is rolled over before each test runs, not after. With @AfterEach, the 
rollover happened before CamelTestSupport.tearDown() stopped the context (JUnit 
5 runs subclass @AfterEach before superclass), so any trace output during 
context shutdown would be written to the new clean log file and picked up by 
the next test as a stale trace.
---
 .../org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
 
b/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
index 56f63d67d5c3..0b7f67e96f3f 100644
--- 
a/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
+++ 
b/components/camel-telemetry-dev/src/test/java/org/apache/camel/telemetrydev/TelemetryDevTracerTestSupport.java
@@ -31,7 +31,7 @@ import org.apache.camel.test.junit6.ExchangeTestSupport;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.core.LoggerContext;
 import org.apache.logging.log4j.core.appender.RollingFileAppender;
-import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
 
 public class TelemetryDevTracerTestSupport extends ExchangeTestSupport {
 
@@ -61,7 +61,7 @@ public class TelemetryDevTracerTestSupport extends 
ExchangeTestSupport {
      * This one is required to rollover the log traces database file and make 
sure each test has its own
      * set of fresh data.
      */
-    @AfterEach
+    @BeforeEach
     public synchronized void clearLogTraces() throws IOException {
         final LoggerContext ctx = (LoggerContext) LogManager.getContext(false);
         RollingFileAppender appender = (RollingFileAppender) 
ctx.getConfiguration().getAppenders().get("file2");

Reply via email to