This is an automated email from the ASF dual-hosted git repository.
davsclaus 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 a0382c2a75ed CAMEL-25820: LogConfigurationTest starts without the log
directory property that another test may have left (#28146)
a0382c2a75ed is described below
commit a0382c2a75ed2c5213d34f726d568ecc1275832f
Author: Claus Ibsen <[email protected]>
AuthorDate: Tue Oct 20 09:25:28 2026 +0200
CAMEL-25820: LogConfigurationTest starts without the log directory property
that another test may have left (#28146)
The tests of camel-jbang-core run in reused forks. A test of a command that
configured the logging leaves
camel.jbang.logDir set, and when
withoutTheDirectoryOfTheCliTheLogFileIsInTheCamelDirectoryOfTheUser ran after
it in
the same fork, the log file was in that test's directory. Each test now
starts without the property.
Claude-Session: https://claude.ai/code/session_01STT6whBgK1AqsSsUKrnE8m
Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]>
---
.../org/apache/camel/dsl/jbang/core/common/LogConfigurationTest.java | 2 ++
1 file changed, 2 insertions(+)
diff --git
a/dsl/camel-jbang/camel-jbang-core/src/test/java/org/apache/camel/dsl/jbang/core/common/LogConfigurationTest.java
b/dsl/camel-jbang/camel-jbang-core/src/test/java/org/apache/camel/dsl/jbang/core/common/LogConfigurationTest.java
index e391bb5aa0b6..b06d0d01f867 100644
---
a/dsl/camel-jbang/camel-jbang-core/src/test/java/org/apache/camel/dsl/jbang/core/common/LogConfigurationTest.java
+++
b/dsl/camel-jbang/camel-jbang-core/src/test/java/org/apache/camel/dsl/jbang/core/common/LogConfigurationTest.java
@@ -52,6 +52,8 @@ class LogConfigurationTest {
void pid() {
// as the CLI sets it
System.setProperty("pid", PID);
+ // a test of a command that configured the logging before, in the same
JVM, leaves it set
+ System.clearProperty(RuntimeUtil.LOG_DIR);
}
@AfterEach