(camel) branch main updated: (chores): fix a misleading manual test in camel-core (#12478)

[orpiske]

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 aa9629936e1 (chores): fix a misleading manual test in camel-core 
(#12478)
aa9629936e1 is described below

commit aa9629936e19d13826f5a6da4cf9582e3faca277
Author: Otavio Rodolfo Piske <orpi...@users.noreply.github.com>
AuthorDate: Mon Dec 18 14:54:14 2023 -0300

    (chores): fix a misleading manual test in camel-core (#12478)
---
 ...eTest.java => SplitterParallelAggregateManualTest.java} | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git 
a/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
similarity index 91%
rename from 
core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateTest.java
rename to 
core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
index c53860f778a..3767bd01fd4 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
@@ -30,15 +30,10 @@ import org.apache.camel.Exchange;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.util.StopWatch;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.EnabledIfSystemProperty;
 
-import static org.junit.jupiter.api.Assumptions.assumeTrue;
-
-public class SplitterParallelAggregateTest extends ContextTestSupport {
-
-    // run this test manually as it takes some time to process, but shows that
-    // parallel aggregate can
-    // be faster when enabled.
-    private boolean enabled;
+@EnabledIfSystemProperty(named = "core.manual.tests", matches = "true", 
disabledReason = "Manual test")
+public class SplitterParallelAggregateManualTest extends ContextTestSupport {
 
     @Override
     protected RouteBuilder createRouteBuilder() throws Exception {
@@ -58,21 +53,18 @@ public class SplitterParallelAggregateTest extends 
ContextTestSupport {
 
     @Test
     public void test1() throws Exception {
-        assumeTrue(enabled);
         int numberOfRequests = 1;
         timeSplitRoutes(numberOfRequests);
     }
 
     @Test
     public void test2() throws Exception {
-        assumeTrue(enabled);
         int numberOfRequests = 2;
         timeSplitRoutes(numberOfRequests);
     }
 
     @Test
     public void test4() throws Exception {
-        assumeTrue(enabled);
         int numberOfRequests = 4;
         timeSplitRoutes(numberOfRequests);
     }