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

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

commit c3b284013ffddf5b30ad04d73dfba4b40a558dd9
Author: Guillaume Nodet <gno...@gmail.com>
AuthorDate: Fri Jul 24 10:48:08 2020 +0200

    [CAMEL-11807] Upgrade camel-spring-integration to junit5
---
 components/camel-spring-integration/pom.xml           |  6 +++---
 .../integration/SpringIntegrationMessageTest.java     |  4 ++--
 .../SpringIntegrationOneWayConsumerTest.java          |  4 ++--
 .../integration/SpringIntegrationProducerTest.java    | 10 ++++++----
 .../SpringIntegrationTwoWayConsumerTest.java          |  9 ++++++---
 .../integration/adapter/CamelSourceAdapterTest.java   | 11 +++++++----
 .../integration/adapter/CamelTargetAdapterTest.java   | 11 +++++++----
 .../spring/integration/adapter/ConfigurationTest.java | 19 ++++++++++---------
 8 files changed, 43 insertions(+), 31 deletions(-)

diff --git a/components/camel-spring-integration/pom.xml 
b/components/camel-spring-integration/pom.xml
index a4c1b39..88f4091 100644
--- a/components/camel-spring-integration/pom.xml
+++ b/components/camel-spring-integration/pom.xml
@@ -77,8 +77,8 @@
 
         <!-- testing -->
         <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
@@ -93,7 +93,7 @@
         </dependency>
         <dependency>
             <groupId>org.apache.camel</groupId>
-            <artifactId>camel-test-spring</artifactId>
+            <artifactId>camel-test-spring-junit5</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationMessageTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationMessageTest.java
index c029852..ddc64fb 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationMessageTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationMessageTest.java
@@ -18,10 +18,10 @@ package org.apache.camel.component.spring.integration;
 
 import org.apache.camel.CamelContext;
 import org.apache.camel.impl.DefaultCamelContext;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.springframework.messaging.support.MessageBuilder;
 
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 public class SpringIntegrationMessageTest {
 
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationOneWayConsumerTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationOneWayConsumerTest.java
index 9230d3d..f2683cb 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationOneWayConsumerTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationOneWayConsumerTest.java
@@ -17,8 +17,8 @@
 package org.apache.camel.component.spring.integration;
 
 import org.apache.camel.component.mock.MockEndpoint;
-import org.apache.camel.test.spring.CamelSpringTestSupport;
-import org.junit.Test;
+import org.apache.camel.test.spring.junit5.CamelSpringTestSupport;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 import org.springframework.messaging.MessageChannel;
 import org.springframework.messaging.support.GenericMessage;
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationProducerTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationProducerTest.java
index bdcd823..bcc0119 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationProducerTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationProducerTest.java
@@ -16,17 +16,19 @@
  */
 package org.apache.camel.component.spring.integration;
 
-import org.apache.camel.test.spring.CamelSpringTestSupport;
-import org.junit.Test;
+import org.apache.camel.test.spring.junit5.CamelSpringTestSupport;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
 public class SpringIntegrationProducerTest extends CamelSpringTestSupport {
 
     @Test
     public void testSendingTwoWayMessage() throws Exception {
         String result = template.requestBody("direct:twowayMessage", "Willem", 
String.class);
 
-        assertEquals("Can't get the right response", result, "Hello Willem");
+        assertEquals(result, "Hello Willem", "Can't get the right response");
     }
 
     @Test
@@ -34,7 +36,7 @@ public class SpringIntegrationProducerTest extends 
CamelSpringTestSupport {
         template.sendBody("direct:onewayMessage", "Greet");
 
         HelloWorldService service = getMandatoryBean(HelloWorldService.class, 
"helloService");
-        assertEquals("We should call the service", service.getGreetName(), 
"Greet");        
+        assertEquals(service.getGreetName(), "Greet", "We should call the 
service");
     }
 
     @Override
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationTwoWayConsumerTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationTwoWayConsumerTest.java
index 65c7f7c..354abf5 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationTwoWayConsumerTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/SpringIntegrationTwoWayConsumerTest.java
@@ -21,8 +21,8 @@ import java.util.Map;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
-import org.apache.camel.test.spring.CamelSpringTestSupport;
-import org.junit.Test;
+import org.apache.camel.test.spring.junit5.CamelSpringTestSupport;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 import org.springframework.integration.channel.DirectChannel;
 import org.springframework.messaging.Message;
@@ -31,6 +31,9 @@ import org.springframework.messaging.MessageHandler;
 import org.springframework.messaging.MessageHeaders;
 import org.springframework.messaging.support.GenericMessage;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
 public class SpringIntegrationTwoWayConsumerTest extends 
CamelSpringTestSupport {
 
     private static final String MESSAGE_BODY = "Request message";
@@ -47,7 +50,7 @@ public class SpringIntegrationTwoWayConsumerTest extends 
CamelSpringTestSupport
         responseChannel.subscribe(new MessageHandler() {
             public void handleMessage(Message<?> message) {
                 latch.countDown();
-                assertEquals("Get the wrong result", MESSAGE_BODY + " is 
processed",  message.getPayload());
+                assertEquals(MESSAGE_BODY + " is processed",  
message.getPayload(), "Get the wrong result");
                 assertEquals("Done",  message.getHeaders().get("Status"));
             }             
         });
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelSourceAdapterTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelSourceAdapterTest.java
index dc09fa0..445928b 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelSourceAdapterTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelSourceAdapterTest.java
@@ -19,13 +19,16 @@ package 
org.apache.camel.component.spring.integration.adapter;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
-import org.apache.camel.test.spring.CamelSpringTestSupport;
-import org.junit.Test;
+import org.apache.camel.test.spring.junit5.CamelSpringTestSupport;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 import org.springframework.integration.channel.DirectChannel;
 import org.springframework.messaging.Message;
 import org.springframework.messaging.MessageHandler;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
 public class CamelSourceAdapterTest extends CamelSpringTestSupport {
 
     @Test
@@ -35,7 +38,7 @@ public class CamelSourceAdapterTest extends 
CamelSpringTestSupport {
         channelA.subscribe(new MessageHandler() {
             public void handleMessage(Message<?> message) {
                 latch.countDown();
-                assertEquals("We should get the message from channelA", 
message.getPayload(), "Willem");             
+                assertEquals("Willem", message.getPayload(), "We should get 
the message from channelA");
             }            
         });
 
@@ -48,7 +51,7 @@ public class CamelSourceAdapterTest extends 
CamelSpringTestSupport {
     public void testSendingTwoWayMessage() throws Exception {
         String result = template.requestBody("direct:TwoWay", "Willem", 
String.class);
 
-        assertEquals("Can't get the right response", result, "Hello Willem");
+        assertEquals("Hello Willem", result, "Can't get the right response");
     }
 
     @Override
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelTargetAdapterTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelTargetAdapterTest.java
index bebacf5..e29402b 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelTargetAdapterTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/CamelTargetAdapterTest.java
@@ -22,8 +22,8 @@ import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.camel.component.mock.MockEndpoint;
-import org.apache.camel.test.spring.CamelSpringTestSupport;
-import org.junit.Test;
+import org.apache.camel.test.spring.junit5.CamelSpringTestSupport;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 import org.springframework.integration.channel.DirectChannel;
 import org.springframework.messaging.Message;
@@ -32,6 +32,9 @@ import org.springframework.messaging.MessageHandler;
 import org.springframework.messaging.MessageHeaders;
 import org.springframework.messaging.support.GenericMessage;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
 public class CamelTargetAdapterTest extends CamelSpringTestSupport {
 
     private static final String MESSAGE_BODY = "hello world";
@@ -57,7 +60,7 @@ public class CamelTargetAdapterTest extends 
CamelSpringTestSupport {
         responseChannel.subscribe(new MessageHandler() {
             public void handleMessage(Message<?> message) {
                 latch.countDown();
-                assertEquals("Get the wrong result", MESSAGE_BODY + " is 
processed",  message.getPayload());
+                assertEquals(MESSAGE_BODY + " is processed",  
message.getPayload(), "Get the wrong result");
             }            
         });
 
@@ -77,7 +80,7 @@ public class CamelTargetAdapterTest extends 
CamelSpringTestSupport {
         responseChannel.subscribe(new MessageHandler() {
             public void handleMessage(Message<?> message) {
                 latch.countDown();
-                assertEquals("Get the wrong result", MESSAGE_BODY + " is 
processed",  message.getPayload());
+                assertEquals(MESSAGE_BODY + " is processed",  
message.getPayload(), "Get the wrong result");
             }            
         });
 
diff --git 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/ConfigurationTest.java
 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/ConfigurationTest.java
index bdd805f..7a09702 100644
--- 
a/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/ConfigurationTest.java
+++ 
b/components/camel-spring-integration/src/test/java/org/apache/camel/component/spring/integration/adapter/ConfigurationTest.java
@@ -17,20 +17,21 @@
 package org.apache.camel.component.spring.integration.adapter;
 
 import org.apache.camel.util.IOHelper;
-import org.junit.After;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Test;
 import org.springframework.context.support.AbstractXmlApplicationContext;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 public class ConfigurationTest {
 
     private AbstractXmlApplicationContext context;
 
-    @After
+    @AfterEach
     public void tearDown() {
         IOHelper.close(context);
     }
@@ -44,11 +45,11 @@ public class ConfigurationTest {
 
         assertNotNull(camelSourceA);
         assertTrue(camelSourceA.getChannel().toString().contains("channelA"));
-        assertEquals("ExpectReply should be false ", 
camelSourceA.isExpectReply(), false);
+        assertFalse(camelSourceA.isExpectReply(), "ExpectReply should be 
false");
         CamelSourceAdapter camelSourceB = context.getBean("camelSourceB", 
CamelSourceAdapter.class);
         assertNotNull(camelSourceB);
         assertTrue(camelSourceB.getChannel().toString().contains("channelB"));
-        assertEquals("ExpectReply should be true ", 
camelSourceB.isExpectReply(), true);
+        assertTrue(camelSourceB.isExpectReply(), "ExpectReply should be true");
     }
 
     @Test
@@ -59,7 +60,7 @@ public class ConfigurationTest {
         CamelTargetAdapter camelTargetA = context.getBean("camelTargetA", 
CamelTargetAdapter.class);
 
         assertNotNull(camelTargetA);
-        assertEquals("Subscript the wrong CamelEndpointUri", 
camelTargetA.getCamelEndpointUri(), "direct:EndpointA");
+        assertEquals("direct:EndpointA", camelTargetA.getCamelEndpointUri(), 
"Subscript the wrong CamelEndpointUri");
         CamelTargetAdapter camelTargetB = context.getBean("camelTargetB", 
CamelTargetAdapter.class);
         assertNotNull(camelTargetB);
         
assertTrue(camelTargetB.getReplyChannel().toString().contains("channelC"));

Reply via email to