You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by nf...@apache.org on 2023/06/06 18:03:56 UTC

[camel] branch main updated: CAMEL-18608: camel-jms - Keep only one instance of MyErrorLogger

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

nfilotto 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 328bccbcb7c CAMEL-18608: camel-jms - Keep only one instance of MyErrorLogger
328bccbcb7c is described below

commit 328bccbcb7c439c173e23d98bec18bd991b54267
Author: Nicolas Filotto <nf...@talend.com>
AuthorDate: Tue Jun 6 20:03:28 2023 +0200

    CAMEL-18608: camel-jms - Keep only one instance of MyErrorLogger
---
 .../org/apache/camel/component/jms/JmsTransferExceptionTest.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/components/camel-jms/src/test/java/org/apache/camel/component/jms/JmsTransferExceptionTest.java b/components/camel-jms/src/test/java/org/apache/camel/component/jms/JmsTransferExceptionTest.java
index e5a0c108f28..b3d0566e3e4 100644
--- a/components/camel-jms/src/test/java/org/apache/camel/component/jms/JmsTransferExceptionTest.java
+++ b/components/camel-jms/src/test/java/org/apache/camel/component/jms/JmsTransferExceptionTest.java
@@ -39,11 +39,11 @@ public class JmsTransferExceptionTest extends AbstractJMSTest {
     @Order(2)
     @RegisterExtension
     public static CamelContextExtension camelContextExtension = new DefaultCamelContextExtension();
+    private static final MyErrorLogger ERROR_LOGGER = new MyErrorLogger();
     private static int counter;
     protected CamelContext context;
     protected ProducerTemplate template;
     protected ConsumerTemplate consumer;
-    protected MyErrorLogger errorLogger = new MyErrorLogger();
 
     protected String getUri() {
         return "activemq:queue:JmsTransferExceptionTest?transferException=true";
@@ -81,7 +81,7 @@ public class JmsTransferExceptionTest extends AbstractJMSTest {
         assertEquals(0, e.getSuppressed().length);
 
         // and check what camel logged
-        Throwable t = errorLogger.getException();
+        Throwable t = ERROR_LOGGER.getException();
         assertNotNull(t);
         assertEquals(0, t.getSuppressed().length);
     }
@@ -96,7 +96,7 @@ public class JmsTransferExceptionTest extends AbstractJMSTest {
         return new RouteBuilder() {
             @Override
             public void configure() {
-                errorHandler(defaultErrorHandler().maximumRedeliveries(4).logger(errorLogger));
+                errorHandler(defaultErrorHandler().maximumRedeliveries(4).logger(ERROR_LOGGER));
 
                 from(getUri())
                         .process(exchange -> {