You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2023/08/08 19:00:01 UTC

[camel] branch main updated: (chores) Adjusted camel-route-parser to use log markers (#11051)

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

orpiske 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 bb6bf3cf0a5 (chores) Adjusted camel-route-parser to use log markers (#11051)
bb6bf3cf0a5 is described below

commit bb6bf3cf0a531b9f81f585db7a8c02d058393ac2
Author: Otavio Rodolfo Piske <or...@users.noreply.github.com>
AuthorDate: Tue Aug 8 20:59:53 2023 +0200

    (chores) Adjusted camel-route-parser to use log markers (#11051)
---
 .../camel/parser/java/RoasterCSimpleRouteBuilderConfigureTest.java    | 4 ++--
 .../parser/java/RoasterConcatFieldRouteBuilderConfigureTest.java      | 4 ++--
 .../camel/parser/java/RoasterFieldRouteBuilderConfigureTest.java      | 4 ++--
 .../test/java/org/apache/camel/parser/java/RoasterJavaDslTest.java    | 2 +-
 .../org/apache/camel/parser/java/RoasterJavaDslTwoRoutesTest.java     | 4 ++--
 .../org/apache/camel/parser/java/RoasterJavaPartialRouteDslTest.java  | 2 +-
 .../camel/parser/java/RoasterMethodCallRouteBuilderConfigureTest.java | 4 ++--
 .../java/RoasterMyFieldMethodCallRouteBuilderConfigureTest.java       | 4 ++--
 .../apache/camel/parser/java/RoasterMyLocalAddRouteBuilderTest.java   | 4 ++--
 .../test/java/org/apache/camel/parser/java/RoasterMyNettyTest.java    | 4 ++--
 .../parser/java/RoasterNewLineConstRouteBuilderConfigureTest.java     | 4 ++--
 .../camel/parser/java/RoasterNewLineRouteBuilderConfigureTest.java    | 4 ++--
 .../camel/parser/java/RoasterRouteBuilderCamelTestSupportTest.java    | 4 ++--
 .../apache/camel/parser/java/RoasterRouteBuilderConfigureTest.java    | 4 ++--
 .../org/apache/camel/parser/java/RoasterRouteBuilderEmptyUriTest.java | 4 ++--
 .../java/RoasterRouteDuplicateIdBuilderDCamelTestSupportTest.java     | 4 ++--
 .../java/org/apache/camel/parser/java/RoasterSimpleProcessorTest.java | 4 ++--
 .../camel/parser/java/RoasterSimpleRouteBuilderConfigureTest.java     | 4 ++--
 .../test/java/org/apache/camel/parser/java/RoasterSimpleToDTest.java  | 4 ++--
 .../test/java/org/apache/camel/parser/java/RoasterSimpleToFTest.java  | 4 ++--
 .../java/org/apache/camel/parser/java/RoasterSplitTokenizeTest.java   | 4 ++--
 .../java/org/apache/camel/parser/xml/FindElementInRoutesTest.java     | 2 +-
 .../src/test/java/org/apache/camel/parser/xml/XmlParseTreeTest.java   | 2 +-
 .../java/org/apache/camel/parser/xml/XmlWithPrefixParseTreeTest.java  | 2 +-
 24 files changed, 43 insertions(+), 43 deletions(-)

diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterCSimpleRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterCSimpleRouteBuilderConfigureTest.java
index 8d030fc370b..16e16482e37 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterCSimpleRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterCSimpleRouteBuilderConfigureTest.java
@@ -44,8 +44,8 @@ public class RoasterCSimpleRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelLanguageExpressions(method, "csimple");
         for (ParserResult csimple : list) {
-            LOG.info("CSimple: " + csimple.getElement());
-            LOG.info("   Line: " + findLineNumber(csimple.getPosition()));
+            LOG.info("CSimple: {}", csimple.getElement());
+            LOG.info("   Line: {}", findLineNumber(csimple.getPosition()));
         }
         assertEquals("${body} > 99", list.get(0).getElement());
         assertEquals(true, list.get(0).getPredicate());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterConcatFieldRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterConcatFieldRouteBuilderConfigureTest.java
index 51f0697bf82..d153430abd3 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterConcatFieldRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterConcatFieldRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterConcatFieldRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("ftp:localhost:{{ftpPort}}/myapp?password=admin&username=admin", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("log:b", list.get(0).getElement());
     }
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterFieldRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterFieldRouteBuilderConfigureTest.java
index b01383505e0..522485cd6d9 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterFieldRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterFieldRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterFieldRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("timer:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("file:output?fileExist=Override", list.get(0).getElement());
         assertEquals("log:b", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTest.java
index 38b0ca345a9..236a08e9472 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTest.java
@@ -58,7 +58,7 @@ public class RoasterJavaDslTest extends CamelTestSupport {
         assertEquals("28", list.get(0).getLineNumberEnd());
 
         String tree = details.dump(0);
-        LOG.info("\n" + tree);
+        LOG.info("\n{}", tree);
 
         assertTrue(tree.contains("28\tfrom"));
         assertTrue(tree.contains("29\t  log"));
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTwoRoutesTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTwoRoutesTest.java
index b302f384a0d..fd22fa985aa 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTwoRoutesTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaDslTwoRoutesTest.java
@@ -63,10 +63,10 @@ public class RoasterJavaDslTwoRoutesTest extends CamelTestSupport {
         assertEquals("org.apache.camel.parser.java.TwoRoutesRouteBuilder", details2.getClassName());
 
         String tree = details.dump(0);
-        LOG.info("\n" + tree);
+        LOG.info("\n{}", tree);
 
         String tree2 = details2.dump(0);
-        LOG.info("\n" + tree2);
+        LOG.info("\n{}", tree2);
 
         assertTrue(tree.contains("25\tfrom"));
         assertTrue(tree.contains("26\t  log"));
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaPartialRouteDslTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaPartialRouteDslTest.java
index b6d07a177dc..215fc7d0e37 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaPartialRouteDslTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterJavaPartialRouteDslTest.java
@@ -50,7 +50,7 @@ class RoasterJavaPartialRouteDslTest extends CamelTestSupport {
         assertEquals("25", list.get(0).getLineNumberEnd());
 
         String tree = details.dump(0);
-        LOG.info("\n" + tree);
+        LOG.info("\n{}", tree);
 
         assertTrue(tree.contains("25\tfrom"));
     }
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMethodCallRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMethodCallRouteBuilderConfigureTest.java
index 2e1e42aff75..a3b5a372284 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMethodCallRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMethodCallRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterMethodCallRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("timer:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("file:output?fileExist={{whatToDoWhenExists}}", list.get(0).getElement());
         assertEquals("log:b", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyFieldMethodCallRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyFieldMethodCallRouteBuilderConfigureTest.java
index b074e258ac1..9d572b53ec2 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyFieldMethodCallRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyFieldMethodCallRouteBuilderConfigureTest.java
@@ -42,14 +42,14 @@ public class RoasterMyFieldMethodCallRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("netty-http:http://0.0.0.0:{{port}}/foo", list.get(0).getElement());
         assertEquals("netty-http:http://0.0.0.0:{{getNextPort}}/bar", list.get(1).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("mock:input1", list.get(0).getElement());
         assertEquals("netty-http:http://0.0.0.0:{{getNextPort}}/bar", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyLocalAddRouteBuilderTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyLocalAddRouteBuilderTest.java
index b5225995b30..387cc772139 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyLocalAddRouteBuilderTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyLocalAddRouteBuilderTest.java
@@ -48,12 +48,12 @@ public class RoasterMyLocalAddRouteBuilderTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
     }
 
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyNettyTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyNettyTest.java
index e08e61ee106..105a8a23594 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyNettyTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterMyNettyTest.java
@@ -42,14 +42,14 @@ public class RoasterMyNettyTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("netty-http:http://0.0.0.0:{{port}}/foo", list.get(0).getElement());
         assertEquals("netty-http:http://0.0.0.0:{{getNextPort}}/bar", list.get(1).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("mock:input1", list.get(0).getElement());
         assertEquals("netty-http:http://0.0.0.0:{{getNextPort}}/bar", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineConstRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineConstRouteBuilderConfigureTest.java
index 7b9f4c19b33..a81f9742705 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineConstRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineConstRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterNewLineConstRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("timer:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("file:output?fileExist=Append&chmod=777&allowNullBody=true", list.get(0).getElement());
         assertEquals("log:b", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineRouteBuilderConfigureTest.java
index 0009256ecce..75a0bc94d09 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterNewLineRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterNewLineRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("timer:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("file:output?fileExist=Append&chmod=777&allowNullBody=true", list.get(0).getElement());
         assertEquals("log:b", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderCamelTestSupportTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderCamelTestSupportTest.java
index 8bf3b9420c5..4eeaedd182e 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderCamelTestSupportTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderCamelTestSupportTest.java
@@ -42,13 +42,13 @@ public class RoasterRouteBuilderCamelTestSupportTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("mock:foo", list.get(0).getElement());
     }
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderConfigureTest.java
index 3d92f778315..897030a4d00 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderConfigureTest.java
@@ -42,13 +42,13 @@ public class RoasterRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("timer:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("log:a", list.get(0).getElement());
         assertEquals("mock:tap", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderEmptyUriTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderEmptyUriTest.java
index dd8de71cfb0..630dc56f5d2 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderEmptyUriTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteBuilderEmptyUriTest.java
@@ -43,14 +43,14 @@ public class RoasterRouteBuilderEmptyUriTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:foo", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, false);
         assertEquals(1, list.size());
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
             assertFalse(result.isParsed(), "Should be invalid");
         }
     }
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteDuplicateIdBuilderDCamelTestSupportTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteDuplicateIdBuilderDCamelTestSupportTest.java
index 98022881bf7..d7a397f4e1e 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteDuplicateIdBuilderDCamelTestSupportTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterRouteDuplicateIdBuilderDCamelTestSupportTest.java
@@ -42,13 +42,13 @@ public class RoasterRouteDuplicateIdBuilderDCamelTestSupportTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, false);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals(3, list.size());
 
         list = CamelJavaParserHelper.parseCamelRouteIds(method);
         for (ParserResult result : list) {
-            LOG.info("Route id: " + result.getElement());
+            LOG.info("Route id: {}", result.getElement());
         }
         assertEquals(3, list.size());
         assertEquals("foo", list.get(0).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleProcessorTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleProcessorTest.java
index 92e0e90fd7d..9171cc4e50c 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleProcessorTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleProcessorTest.java
@@ -50,13 +50,13 @@ public class RoasterSimpleProcessorTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:start", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals(0, list.size());
 
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleRouteBuilderConfigureTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleRouteBuilderConfigureTest.java
index b3fd65b070e..b1ce39d4419 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleRouteBuilderConfigureTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleRouteBuilderConfigureTest.java
@@ -44,8 +44,8 @@ public class RoasterSimpleRouteBuilderConfigureTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelLanguageExpressions(method, "simple");
         for (ParserResult simple : list) {
-            LOG.info("Simple: " + simple.getElement());
-            LOG.info("  Line: " + findLineNumber(simple.getPosition()));
+            LOG.info("Simple: {}", simple.getElement());
+            LOG.info("  Line: {}", findLineNumber(simple.getPosition()));
         }
         assertEquals("${body} > 100", list.get(0).getElement());
         assertEquals(true, list.get(0).getPredicate());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToDTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToDTest.java
index e3e577530da..338bb242e31 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToDTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToDTest.java
@@ -50,13 +50,13 @@ public class RoasterSimpleToDTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:start", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("toD", list.get(0).getNode());
         assertEquals("log:a", list.get(0).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToFTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToFTest.java
index 8b52230eb02..3523e1977ca 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToFTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSimpleToFTest.java
@@ -50,13 +50,13 @@ public class RoasterSimpleToFTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:start", list.get(0).getElement());
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("toF", list.get(0).getNode());
         assertEquals("log:a?level={{%s}}", list.get(0).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSplitTokenizeTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSplitTokenizeTest.java
index fb351bdeaba..8091fd51bc5 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSplitTokenizeTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/java/RoasterSplitTokenizeTest.java
@@ -50,7 +50,7 @@ public class RoasterSplitTokenizeTest {
 
         List<ParserResult> list = CamelJavaParserHelper.parseCamelConsumerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Consumer: " + result.getElement());
+            LOG.info("Consumer: {}", result.getElement());
         }
         assertEquals("direct:a", list.get(0).getElement());
         assertEquals("direct:b", list.get(1).getElement());
@@ -61,7 +61,7 @@ public class RoasterSplitTokenizeTest {
 
         list = CamelJavaParserHelper.parseCamelProducerUris(method, true, true);
         for (ParserResult result : list) {
-            LOG.info("Producer: " + result.getElement());
+            LOG.info("Producer: {}", result.getElement());
         }
         assertEquals("mock:split", list.get(0).getElement());
         assertEquals("mock:split", list.get(1).getElement());
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/FindElementInRoutesTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/FindElementInRoutesTest.java
index 92260a33741..38807091135 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/FindElementInRoutesTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/FindElementInRoutesTest.java
@@ -39,7 +39,7 @@ public class FindElementInRoutesTest {
         Element element = CamelXmlHelper.getSelectedCamelElementNode(key, is);
         assertNotNull(element, "Could not find Element for key " + key);
 
-        LOG.info("Found element " + element.getTagName());
+        LOG.info("Found element {}", element.getTagName());
     }
 
 }
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlParseTreeTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlParseTreeTest.java
index d47d72f2940..e3827fe6eb8 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlParseTreeTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlParseTreeTest.java
@@ -56,7 +56,7 @@ public class XmlParseTreeTest {
         assertNull(details.getClassName());
 
         String tree = details.dump(0);
-        LOG.info("\n" + tree);
+        LOG.info("\n{}", tree);
 
         assertTrue(tree.contains("32\tfrom"));
         assertTrue(tree.contains("35\t  transform"));
diff --git a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlWithPrefixParseTreeTest.java b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlWithPrefixParseTreeTest.java
index 69e12c115eb..9cc46c3ace8 100644
--- a/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlWithPrefixParseTreeTest.java
+++ b/catalog/camel-route-parser/src/test/java/org/apache/camel/parser/xml/XmlWithPrefixParseTreeTest.java
@@ -57,7 +57,7 @@ public class XmlWithPrefixParseTreeTest {
         assertNull(details.getClassName());
 
         String tree = details.dump(0);
-        LOG.info("\n" + tree);
+        LOG.info("\n{}", tree);
 
         assertTrue(tree.contains("34\tfrom"));
         assertTrue(tree.contains("36\t  transform"));