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 9960826227a (chores) camel-test-junit5: code cleanups: (#11024)
9960826227a is described below

commit 9960826227add52af8fb8403592c050169898e9b
Author: Otavio Rodolfo Piske <[email protected]>
AuthorDate: Mon Aug 7 16:05:24 2023 +0200

    (chores) camel-test-junit5: code cleanups: (#11024)
    
    - simplified assertion
    - simplify other complex operations
---
 .../main/java/org/apache/camel/test/CamelRouteCoverageDumper.java    | 2 +-
 .../org/apache/camel/test/junit5/params/ParameterizedExtension.java  | 5 ++---
 .../org/apache/camel/test/junit5/patterns/AsyncSendMockTest.java     | 5 +++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/CamelRouteCoverageDumper.java
 
b/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/CamelRouteCoverageDumper.java
index 9849935cd83..ad534686597 100644
--- 
a/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/CamelRouteCoverageDumper.java
+++ 
b/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/CamelRouteCoverageDumper.java
@@ -191,7 +191,7 @@ public class CamelRouteCoverageDumper {
                 .append(contextExchangesTotal).append("\n");
 
         if (!uncoveredRoutes.isEmpty()) {
-            builder.append("\t\tUncovered routes: 
").append(uncoveredRoutes.stream().collect(Collectors.joining(", ")))
+            builder.append("\t\tUncovered routes: ").append(String.join(", ", 
uncoveredRoutes))
                     .append("\n");
         }
 
diff --git 
a/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/junit5/params/ParameterizedExtension.java
 
b/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/junit5/params/ParameterizedExtension.java
index ac61a0ed9df..cda04cfc491 100644
--- 
a/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/junit5/params/ParameterizedExtension.java
+++ 
b/components/camel-test/camel-test-junit5/src/main/java/org/apache/camel/test/junit5/params/ParameterizedExtension.java
@@ -19,7 +19,6 @@ package org.apache.camel.test.junit5.params;
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
-import java.util.Arrays;
 import java.util.Comparator;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -116,7 +115,7 @@ public class ParameterizedExtension implements 
TestTemplateInvocationContextProv
 
         @Override
         public java.util.List<Extension> getAdditionalExtensions() {
-            return Arrays.asList(
+            return List.of(
                     (TestInstancePostProcessor) this::postProcessTestInstance);
         }
 
@@ -127,7 +126,7 @@ public class ParameterizedExtension implements 
TestTemplateInvocationContextProv
                     .flatMap(Stream::of)
                     .filter(f -> isAnnotated(f, Parameter.class))
                     .sorted(Comparator.comparing(f -> (Integer) 
f.getAnnotation(Parameter.class).value()))
-                    .collect(Collectors.toList());
+                    .toList();
             if (params.length != fields.size()) {
                 throw new TestInstantiationException(
                         "Expected " + fields.size() + " parameters bug got " + 
params.length + " when instantiating "
diff --git 
a/components/camel-test/camel-test-junit5/src/test/java/org/apache/camel/test/junit5/patterns/AsyncSendMockTest.java
 
b/components/camel-test/camel-test-junit5/src/test/java/org/apache/camel/test/junit5/patterns/AsyncSendMockTest.java
index 10b0b91aaae..c2ee57c0a01 100644
--- 
a/components/camel-test/camel-test-junit5/src/test/java/org/apache/camel/test/junit5/patterns/AsyncSendMockTest.java
+++ 
b/components/camel-test/camel-test-junit5/src/test/java/org/apache/camel/test/junit5/patterns/AsyncSendMockTest.java
@@ -24,6 +24,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.fail;
 
 public class AsyncSendMockTest extends CamelTestSupport {
     private static final Logger LOG = 
LoggerFactory.getLogger(AsyncSendMockTest.class);
@@ -34,7 +35,7 @@ public class AsyncSendMockTest extends CamelTestSupport {
     }
 
     @Test
-    public void testmakeAsyncApiCall() {
+    public void testMakeAsyncApiCall() {
         try {
             
getMockEndpoint("mock:seda:start").expectedHeaderReceived("username", 
"admin123");
             getMockEndpoint("mock:seda:start").expectedBodiesReceived("Hello");
@@ -46,7 +47,7 @@ public class AsyncSendMockTest extends CamelTestSupport {
             MockEndpoint.assertIsSatisfied(context);
         } catch (Exception e) {
             LOG.warn("Failed to make async call to api: {}", e.getMessage(), 
e);
-            assertTrue(false, "Failed to make async call to api");
+            fail("Failed to make async call to api");
         }
     }
 }

Reply via email to