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

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


The following commit(s) were added to refs/heads/master by this push:
     new 6ce8cb9  Fixed test
6ce8cb9 is described below

commit 6ce8cb971ba4311b6f658a4d54f7180bdcaa10da
Author: Claus Ibsen <claus.ib...@gmail.com>
AuthorDate: Sun Mar 29 17:00:55 2020 +0200

    Fixed test
---
 .../apache/camel/component/jgroups/CamelJGroupsReceiver.java  |  7 ++-----
 .../org/apache/camel/component/jgroups/JGroupsComponent.java  |  4 +++-
 .../org/apache/camel/component/jgroups/JGroupsEndpoint.java   | 11 +++++++----
 .../camel/component/jgroups/JGroupsClusterRouteTest.java      |  2 --
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/CamelJGroupsReceiver.java
 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/CamelJGroupsReceiver.java
index 67c7817..9fcdef8 100644
--- 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/CamelJGroupsReceiver.java
+++ 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/CamelJGroupsReceiver.java
@@ -72,11 +72,8 @@ public class CamelJGroupsReceiver extends ReceiverAdapter {
         Exchange exchange = endpoint.createExchange(message);
         try {
             LOG.debug("Processing message: {}", message);
-            processor.process(exchange, new AsyncCallback() {
-                @Override
-                public void done(boolean doneSync) {
-                    // noop
-                }
+            processor.process(exchange, doneSync -> {
+                // noop
             });
         } catch (Exception e) {
             throw new JGroupsException("Error in consumer while dispatching 
exchange containing message " + message, e);
diff --git 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsComponent.java
 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsComponent.java
index e86886c..cff59d3 100644
--- 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsComponent.java
+++ 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsComponent.java
@@ -42,7 +42,9 @@ public class JGroupsComponent extends DefaultComponent {
 
     @Override
     protected Endpoint createEndpoint(String uri, String clusterName, 
Map<String, Object> parameters) throws Exception {
-        return new JGroupsEndpoint(uri, this, channel, clusterName, 
channelProperties, enableViewMessages);
+        JGroupsEndpoint endpoint = new JGroupsEndpoint(uri, this, channel, 
clusterName, channelProperties, enableViewMessages);
+        setProperties(endpoint, parameters);
+        return endpoint;
     }
 
     public JChannel getChannel() {
diff --git 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java
 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java
index a1409ed..60406b9 100644
--- 
a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java
+++ 
b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java
@@ -73,7 +73,9 @@ public class JGroupsEndpoint extends DefaultEndpoint {
 
     @Override
     public Consumer createConsumer(Processor processor) throws Exception {
-        return new JGroupsConsumer(this, processor, resolvedChannel, 
clusterName);
+        JGroupsConsumer consumer = new JGroupsConsumer(this, processor, 
resolvedChannel, clusterName);
+        configureConsumer(consumer);
+        return consumer;
     }
 
     public Exchange createExchange(Message message) {
@@ -106,8 +108,10 @@ public class JGroupsEndpoint extends DefaultEndpoint {
 
     @Override
     protected void doStop() throws Exception {
-        LOG.trace("Closing JGroups Channel {}", getEndpointUri());
-        resolvedChannel.close();
+        if (resolvedChannel != null) {
+            LOG.trace("Closing JGroups Channel {}", getEndpointUri());
+            resolvedChannel.close();
+        }
         super.doStop();
     }
 
@@ -123,7 +127,6 @@ public class JGroupsEndpoint extends DefaultEndpoint {
 
     /**
      * Connect shared channel, called by producer and consumer.
-     * @throws Exception
      */
     public void connect() throws Exception {
         connectCount.incrementAndGet();
diff --git 
a/components/camel-jgroups/src/test/java/org/apache/camel/component/jgroups/JGroupsClusterRouteTest.java
 
b/components/camel-jgroups/src/test/java/org/apache/camel/component/jgroups/JGroupsClusterRouteTest.java
index 218c511..d33a6b3 100644
--- 
a/components/camel-jgroups/src/test/java/org/apache/camel/component/jgroups/JGroupsClusterRouteTest.java
+++ 
b/components/camel-jgroups/src/test/java/org/apache/camel/component/jgroups/JGroupsClusterRouteTest.java
@@ -107,9 +107,7 @@ public class JGroupsClusterRouteTest extends Assert {
 
         expectMasterIsNot(firstCamelContext);
         firstCamelContext.stop();
-        assertMasterIsNot(firstCamelContext);
 
-        expectMasterIsNot(firstCamelContext);
         firstCamelContext.start();
         assertMasterIsNot(firstCamelContext);
 

Reply via email to