(camel) 01/05: (chores) camel-core: correctly disabled manual tests

[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

commit f234169d47bdb44f412f059ed4970a4ad6f8459b
Author: Otavio Rodolfo Piske <angusyo...@gmail.com>
AuthorDate: Mon Apr 22 13:31:24 2024 +0200

    (chores) camel-core: correctly disabled manual tests
---
 .../apache/camel/impl/DefaultExecutorServiceManagerTest.java   |  2 +-
 .../camel/processor/RouteContextProcessorManualTest.java       |  6 +-----
 .../camel/processor/aggregator/AggregateTimeoutManualTest.java | 10 ++++------
 .../java/org/apache/camel/util/CaseInsensitiveMapTest.java     |  2 +-
 4 files changed, 7 insertions(+), 13 deletions(-)

diff --git 
a/core/camel-core/src/test/java/org/apache/camel/impl/DefaultExecutorServiceManagerTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/impl/DefaultExecutorServiceManagerTest.java
index d75d5152123..6f3bff5923a 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/impl/DefaultExecutorServiceManagerTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/impl/DefaultExecutorServiceManagerTest.java
@@ -534,7 +534,7 @@ public class DefaultExecutorServiceManagerTest extends 
ContextTestSupport {
     }
 
     @Disabled("This is a manual test, by looking at the logs")
-    public void xxxtestLongShutdownOfThreadPool() throws Exception {
+    public void testLongShutdownOfThreadPool() throws Exception {
         final CountDownLatch latch = new CountDownLatch(1);
         ExecutorService pool = 
context.getExecutorServiceManager().newSingleThreadExecutor(this, "Cool");
 
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/processor/RouteContextProcessorManualTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/processor/RouteContextProcessorManualTest.java
index e10f3b45630..1973d211a59 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/processor/RouteContextProcessorManualTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/processor/RouteContextProcessorManualTest.java
@@ -50,11 +50,7 @@ public class RouteContextProcessorManualTest extends 
ContextTestSupport {
     private static final int NUMBER_OF_MESSAGES = 10000;
 
     @Test
-    public void testForkAndJoin() {
-        // enable the other test method for manual testing
-    }
-
-    public void xxxTestForkAndJoin() throws InterruptedException {
+    public void testForkAndJoin() throws InterruptedException {
         MockEndpoint mock = getMockEndpoint("mock:result");
         mock.expectedMessageCount(NUMBER_OF_MESSAGES);
 
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutManualTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutManualTest.java
index 7521de18d9f..8ef06425c7b 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutManualTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutManualTest.java
@@ -20,21 +20,19 @@ import org.apache.camel.ContextTestSupport;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.mock.MockEndpoint;
 import org.apache.camel.processor.aggregate.UseLatestAggregationStrategy;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 /**
  * To be run manually when testing the timeout aggregator
  */
+@Disabled("This is a manual test")
 public class AggregateTimeoutManualTest extends ContextTestSupport {
 
     @Test
-    public void testDisabled() {
-        // noop
-    }
-
-    public void xxxtestAggregateTimeoutManual() throws Exception {
+    public void testAggregateTimeoutManual() throws Exception {
         MockEndpoint result = getMockEndpoint("mock:result");
-        // by default the use latest aggregation strategy
+        // by default, the use latest aggregation strategy
         result.expectedBodiesReceived("Message 1999");
         // should take at least 3 seconds to complete this one
         result.setResultMinimumWaitTime(2500);
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java
index 86733ac4ae6..68af18d0531 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java
@@ -503,7 +503,7 @@ public class CaseInsensitiveMapTest {
     }
 
     @Disabled("Manual test")
-    public void xxxTestCopyMapWithCamelHeadersTest() throws Exception {
+    public void testCopyMapWithCamelHeadersTest() throws Exception {
         Map<String, Object> map = new CaseInsensitiveMap();
         map.put("CamelA", "A");
         map.put("CamelB", "B");