You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2022/03/10 14:52:28 UTC

[camel] 01/02: CAMEL-17763: cleaned up unused exceptions in camel-sjms2

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 3de7b649d1a8f34227de0adae44a4a9cbd419a83
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Thu Mar 10 14:53:26 2022 +0100

    CAMEL-17763: cleaned up unused exceptions in camel-sjms2
---
 .../component/sjms2/SimpleJms2ComponentTest.java   |  4 ++--
 .../component/sjms2/Sjms2ComponentRestartTest.java |  4 ++--
 .../sjms2/Sjms2EndpointNameOverrideTest.java       |  6 +++---
 .../camel/component/sjms2/Sjms2EndpointTest.java   | 22 +++++++++++-----------
 .../consumer/InOnlyTopicDurableConsumerTest.java   |  4 ++--
 .../consumer/InOnlyTopicSharedConsumerTest.java    |  2 +-
 .../sjms2/producer/InOnlyQueueProducerTest.java    |  2 +-
 .../sjms2/producer/InOnlyTopicProducerTest.java    |  5 ++---
 .../sjms2/producer/Sjms2ToDSendDynamicTest.java    |  6 +++---
 9 files changed, 27 insertions(+), 28 deletions(-)

diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/SimpleJms2ComponentTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/SimpleJms2ComponentTest.java
index 0d386cb..7d97228 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/SimpleJms2ComponentTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/SimpleJms2ComponentTest.java
@@ -26,13 +26,13 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
 public class SimpleJms2ComponentTest extends CamelTestSupport {
 
     @Test
-    public void testHelloWorld() throws Exception {
+    public void testHelloWorld() {
         Sjms2Component component = context.getComponent("sjms2", Sjms2Component.class);
         assertNotNull(component);
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             public void configure() {
                 ActiveMQConnectionFactory connectionFactory
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2ComponentRestartTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2ComponentRestartTest.java
index fb943f6..daf192e 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2ComponentRestartTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2ComponentRestartTest.java
@@ -37,7 +37,7 @@ public class Sjms2ComponentRestartTest extends CamelTestSupport {
     public void testRestartWithStopStart() throws Exception {
         RouteBuilder routeBuilder = new RouteBuilder(context) {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("sjms2:queue:test").to("mock:test");
             }
         };
@@ -73,7 +73,7 @@ public class Sjms2ComponentRestartTest extends CamelTestSupport {
     public void testRestartWithSuspendResume() throws Exception {
         RouteBuilder routeBuilder = new RouteBuilder(context) {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("sjms2:queue:test").to("mock:test");
             }
         };
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointNameOverrideTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointNameOverrideTest.java
index 9fd58c3..c009e61 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointNameOverrideTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointNameOverrideTest.java
@@ -37,7 +37,7 @@ public class Sjms2EndpointNameOverrideTest extends CamelTestSupport {
     }
 
     @Test
-    public void testDefaults() throws Exception {
+    public void testDefaults() {
         Endpoint endpoint = context.getEndpoint(BEAN_NAME + ":test");
         assertNotNull(endpoint);
         assertTrue(endpoint instanceof Sjms2Endpoint);
@@ -47,7 +47,7 @@ public class Sjms2EndpointNameOverrideTest extends CamelTestSupport {
     }
 
     @Test
-    public void testQueueEndpoint() throws Exception {
+    public void testQueueEndpoint() {
         Endpoint sjms = context.getEndpoint(BEAN_NAME + ":queue:test");
         assertNotNull(sjms);
         assertTrue(sjms instanceof Sjms2Endpoint);
@@ -55,7 +55,7 @@ public class Sjms2EndpointNameOverrideTest extends CamelTestSupport {
     }
 
     @Test
-    public void testTopicEndpoint() throws Exception {
+    public void testTopicEndpoint() {
         Endpoint sjms = context.getEndpoint(BEAN_NAME + ":topic:test");
         assertNotNull(sjms);
         assertTrue(sjms instanceof Sjms2Endpoint);
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java
index 844f5c0..8f03013 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java
@@ -33,7 +33,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testDefaults() throws Exception {
+    public void testDefaults() {
         Endpoint endpoint = context.getEndpoint("sjms2:test");
         assertNotNull(endpoint);
         assertTrue(endpoint instanceof Sjms2Endpoint);
@@ -43,7 +43,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testQueueEndpoint() throws Exception {
+    public void testQueueEndpoint() {
         Endpoint sjms = context.getEndpoint("sjms2:queue:test");
         assertNotNull(sjms);
         assertEquals("sjms2://queue:test", sjms.getEndpointUri());
@@ -51,7 +51,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testJndiStyleEndpointName() throws Exception {
+    public void testJndiStyleEndpointName() {
         Sjms2Endpoint sjms = context.getEndpoint("sjms2:/jms/test/hov.t1.dev:topic", Sjms2Endpoint.class);
         assertNotNull(sjms);
         assertFalse(sjms.isTopic());
@@ -59,7 +59,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testSetTransacted() throws Exception {
+    public void testSetTransacted() {
         Endpoint endpoint = context.getEndpoint("sjms2:queue:test?transacted=true");
         assertNotNull(endpoint);
         assertTrue(endpoint instanceof Sjms2Endpoint);
@@ -68,7 +68,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testAsyncProducer() throws Exception {
+    public void testAsyncProducer() {
         Endpoint endpoint = context.getEndpoint("sjms2:queue:test?synchronous=true");
         assertNotNull(endpoint);
         assertTrue(endpoint instanceof Sjms2Endpoint);
@@ -77,7 +77,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testReplyTo() throws Exception {
+    public void testReplyTo() {
         String replyTo = "reply.to.queue";
         Endpoint endpoint = context.getEndpoint("sjms2:queue:test?replyTo=" + replyTo);
         assertNotNull(endpoint);
@@ -88,7 +88,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testDefaultExchangePattern() throws Exception {
+    public void testDefaultExchangePattern() {
         try {
             Sjms2Endpoint sjms = (Sjms2Endpoint) context.getEndpoint("sjms2:queue:test");
             assertNotNull(sjms);
@@ -99,7 +99,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testInOnlyExchangePattern() throws Exception {
+    public void testInOnlyExchangePattern() {
         try {
             Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOnly);
             assertNotNull(sjms);
@@ -110,7 +110,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testInOutExchangePattern() throws Exception {
+    public void testInOutExchangePattern() {
         try {
             Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOut);
             assertNotNull(sjms);
@@ -121,7 +121,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testReplyToAndMEPMatch() throws Exception {
+    public void testReplyToAndMEPMatch() {
         String replyTo = "reply.to.queue";
         Endpoint endpoint = context
                 .getEndpoint("sjms2:queue:test?replyTo=" + replyTo + "&exchangePattern=" + ExchangePattern.InOut);
@@ -133,7 +133,7 @@ public class Sjms2EndpointTest extends CamelTestSupport {
     }
 
     @Test
-    public void testDestinationName() throws Exception {
+    public void testDestinationName() {
         Endpoint endpoint = context.getEndpoint("sjms2:queue:test?synchronous=true");
         assertNotNull(endpoint);
         assertTrue(endpoint instanceof Sjms2Endpoint);
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicDurableConsumerTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicDurableConsumerTest.java
index b38747c..7454750 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicDurableConsumerTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicDurableConsumerTest.java
@@ -64,10 +64,10 @@ public class InOnlyTopicDurableConsumerTest extends Jms2TestSupport {
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("sjms2:topic:foo?durableSubscriptionName=bar1")
                         .to("mock:result");
 
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicSharedConsumerTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicSharedConsumerTest.java
index ba2d897..1993ec7 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicSharedConsumerTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/consumer/InOnlyTopicSharedConsumerTest.java
@@ -48,7 +48,7 @@ public class InOnlyTopicSharedConsumerTest extends Jms2TestSupport {
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             public void configure() {
                 from("direct:start")
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyQueueProducerTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyQueueProducerTest.java
index 68e7ee6..6d4a5bc 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyQueueProducerTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyQueueProducerTest.java
@@ -85,7 +85,7 @@ public class InOnlyQueueProducerTest extends Jms2TestSupport {
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             public void configure() {
                 from("direct:start")
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyTopicProducerTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyTopicProducerTest.java
index 9369beb..f1511d3 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyTopicProducerTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/InOnlyTopicProducerTest.java
@@ -67,13 +67,12 @@ public class InOnlyTopicProducerTest extends Jms2TestSupport {
     }
 
     /**
-     * @see              org.apache.camel.test.junit5.CamelTestSupport#createRouteBuilder()
+     * @see    org.apache.camel.test.junit5.CamelTestSupport#createRouteBuilder()
      *
      * @return
-     * @throws Exception
      */
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             public void configure() {
                 from("direct:start")
diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/Sjms2ToDSendDynamicTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/Sjms2ToDSendDynamicTest.java
index 8499cdc..a223641 100644
--- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/Sjms2ToDSendDynamicTest.java
+++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/producer/Sjms2ToDSendDynamicTest.java
@@ -25,7 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 public class Sjms2ToDSendDynamicTest extends Jms2TestSupport {
 
     @Test
-    public void testToD() throws Exception {
+    public void testToD() {
         template.sendBodyAndHeader("direct:start", "Hello bar", "where", "bar");
         template.sendBodyAndHeader("direct:start", "Hello beer", "where", "beer");
 
@@ -41,10 +41,10 @@ public class Sjms2ToDSendDynamicTest extends Jms2TestSupport {
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 // route message dynamic using toD
                 from("direct:start").toD("sjms2:queue:${header.where}");
             }