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/04/13 08:36:23 UTC

[camel] 03/08: CAMEL-17763: cleaned up unused exceptions in camel-opentracing

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 3785e014e2230cecd3f0651e8c0a85a96bd5eb40
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Tue Apr 12 18:15:51 2022 +0200

    CAMEL-17763: cleaned up unused exceptions in camel-opentracing
---
 .../src/test/java/org/apache/camel/opentracing/ABCRouteTest.java  | 6 +++---
 .../org/apache/camel/opentracing/CamelOpenTracingTestSupport.java | 2 +-
 .../apache/camel/opentracing/ClientRecipientListRouteTest.java    | 6 +++---
 .../apache/camel/opentracing/CustomComponentNameRouteTest.java    | 6 +++---
 .../org/apache/camel/opentracing/EIPTracingActiveSpanTest.java    | 8 ++++----
 .../java/org/apache/camel/opentracing/EIPTracingRouteTest.java    | 6 +++---
 .../test/java/org/apache/camel/opentracing/MDCSupportTest.java    | 8 ++++----
 .../org/apache/camel/opentracing/MulticastParallelRouteTest.java  | 6 +++---
 .../java/org/apache/camel/opentracing/MulticastRouteTest.java     | 6 +++---
 .../camel/opentracing/OpentracingSpanCollectorInRegistryTest.java | 2 +-
 .../java/org/apache/camel/opentracing/RouteConcurrentTest.java    | 8 ++++----
 .../java/org/apache/camel/opentracing/SpanProcessorsTest.java     | 6 +++---
 .../camel/opentracing/SpringOpenTracingSimpleRouteTest.java       | 2 +-
 .../camel/opentracing/TracingClientRecipientListRouteTest.java    | 6 +++---
 .../camel/opentracing/TracingMulticastParallelRouteTest.java      | 6 +++---
 .../test/java/org/apache/camel/opentracing/TwoServiceTest.java    | 6 +++---
 .../org/apache/camel/opentracing/TwoServiceWithExcludeTest.java   | 6 +++---
 .../org/apache/camel/opentracing/integration/InterprocessIT.java  | 6 +++---
 18 files changed, 51 insertions(+), 51 deletions(-)

diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ABCRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ABCRouteTest.java
index 5074913f820..e4cb53f5b2d 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ABCRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ABCRouteTest.java
@@ -37,17 +37,17 @@ public class ABCRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CamelOpenTracingTestSupport.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CamelOpenTracingTestSupport.java
index 86589551896..223e0eeeaeb 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CamelOpenTracingTestSupport.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CamelOpenTracingTestSupport.java
@@ -51,7 +51,7 @@ public class CamelOpenTracingTestSupport extends CamelTestSupport {
     }
 
     @Override
-    protected void doPostSetup() throws Exception {
+    protected void doPostSetup() {
         tracer.reset();
     }
 
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ClientRecipientListRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ClientRecipientListRouteTest.java
index 6c508a58900..7a9a2ed7b6c 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ClientRecipientListRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/ClientRecipientListRouteTest.java
@@ -37,17 +37,17 @@ public class ClientRecipientListRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").recipientList(constant("seda:a,seda:b,seda:c")).routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CustomComponentNameRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CustomComponentNameRouteTest.java
index 4e1f4f223d1..cafd243fddb 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CustomComponentNameRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/CustomComponentNameRouteTest.java
@@ -37,17 +37,17 @@ public class CustomComponentNameRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 context.addComponent("myseda", context.getComponent("seda"));
 
                 from("direct:start").to("myseda:a").routeId("start");
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingActiveSpanTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingActiveSpanTest.java
index a443f0a441c..cc27211f152 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingActiveSpanTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingActiveSpanTest.java
@@ -48,7 +48,7 @@ public class EIPTracingActiveSpanTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
@@ -60,13 +60,13 @@ public class EIPTracingActiveSpanTest extends CamelOpenTracingTestSupport {
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").routeId("start").process(new Processor() {
                     @Override
-                    public void process(Exchange exchange) throws Exception {
+                    public void process(Exchange exchange) {
                         // here you can use GlobalTracer if it's in your classpath or
                         // use the exchange context to look for a tracer in the registry
                         GlobalTracer.get().buildSpan("using-active-span")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingRouteTest.java
index bb30a3a6f3d..26139553cf5 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/EIPTracingRouteTest.java
@@ -58,7 +58,7 @@ public class EIPTracingRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
@@ -70,10 +70,10 @@ public class EIPTracingRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").routeId("start")
                         .to("seda:a").id("direct-to");
 
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MDCSupportTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MDCSupportTest.java
index 19fa1f9e521..5e7bd534c04 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MDCSupportTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MDCSupportTest.java
@@ -46,22 +46,22 @@ public class MDCSupportTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").routeId("start");
 
                 from("seda:a").routeId("a")
                         .process(new Processor() {
-                            public void process(Exchange exchange) throws Exception {
+                            public void process(Exchange exchange) {
                                 assertNotNull(MDC.get("traceId"));
                                 assertNotNull(MDC.get("spanId"));
                             }
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastParallelRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastParallelRouteTest.java
index 9712b171d14..ce4e9005b36 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastParallelRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastParallelRouteTest.java
@@ -37,17 +37,17 @@ public class MulticastParallelRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify(true);
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastRouteTest.java
index 1270e27e9cf..cea08e1cf77 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/MulticastRouteTest.java
@@ -37,17 +37,17 @@ public class MulticastRouteTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/OpentracingSpanCollectorInRegistryTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/OpentracingSpanCollectorInRegistryTest.java
index 1e2d879fb92..baa2091dcb5 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/OpentracingSpanCollectorInRegistryTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/OpentracingSpanCollectorInRegistryTest.java
@@ -49,7 +49,7 @@ public class OpentracingSpanCollectorInRegistryTest extends CamelTestSupport {
     }
 
     @Test
-    public void testZipkinConfiguration() throws Exception {
+    public void testZipkinConfiguration() {
         assertNotNull(openTracing.getTracer());
         assertTrue(openTracing.getTracer() instanceof NoopTracer);
     }
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/RouteConcurrentTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/RouteConcurrentTest.java
index 5c1efcb1c83..8176c8244a4 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/RouteConcurrentTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/RouteConcurrentTest.java
@@ -38,7 +38,7 @@ public class RouteConcurrentTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testSingleInvocationsOfRoute() throws Exception {
+    public void testSingleInvocationsOfRoute() {
         NotifyBuilder notify = new NotifyBuilder(context).whenDone(2).create();
 
         template.sendBody("seda:foo", "Hello World");
@@ -49,7 +49,7 @@ public class RouteConcurrentTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testConcurrentInvocationsOfRoute() throws Exception {
+    public void testConcurrentInvocationsOfRoute() {
         NotifyBuilder notify = new NotifyBuilder(context).whenDone(10).create();
 
         for (int i = 0; i < 5; i++) {
@@ -62,10 +62,10 @@ public class RouteConcurrentTest extends CamelOpenTracingTestSupport {
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("seda:foo?concurrentConsumers=5").routeId("foo")
                         .log("routing at ${routeId}")
                         .delay(simple("${random(1000,2000)}"))
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpanProcessorsTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpanProcessorsTest.java
index 6748e69f9c3..f29fc9af53d 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpanProcessorsTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpanProcessorsTest.java
@@ -43,7 +43,7 @@ public class SpanProcessorsTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         Exchange result = template.request("direct:start",
                 exchange -> {
                     exchange.getIn().setBody("Hello");
@@ -55,10 +55,10 @@ public class SpanProcessorsTest extends CamelOpenTracingTestSupport {
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpringOpenTracingSimpleRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpringOpenTracingSimpleRouteTest.java
index b5c62e3b5a0..28e9c7a9f11 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpringOpenTracingSimpleRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/SpringOpenTracingSimpleRouteTest.java
@@ -36,7 +36,7 @@ public class SpringOpenTracingSimpleRouteTest extends CamelSpringTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         NotifyBuilder notify = new NotifyBuilder(context).whenDone(5).create();
 
         for (int i = 0; i < 5; i++) {
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingClientRecipientListRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingClientRecipientListRouteTest.java
index 7c143aa23c7..12d973517f7 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingClientRecipientListRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingClientRecipientListRouteTest.java
@@ -50,17 +50,17 @@ public class TracingClientRecipientListRouteTest extends CamelOpenTracingTestSup
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").recipientList(constant("seda:a,seda:b,seda:c")).id("direct-recipientList-1")
                         .routeId("start");
 
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingMulticastParallelRouteTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingMulticastParallelRouteTest.java
index ea6c6500435..2a4a11d408d 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingMulticastParallelRouteTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TracingMulticastParallelRouteTest.java
@@ -58,17 +58,17 @@ public class TracingMulticastParallelRouteTest extends CamelOpenTracingTestSuppo
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:start", "Hello");
 
         verify(true);
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:start").to("seda:a").id("direct-to-1").routeId("start");
 
                 from("seda:a").routeId("a")
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceTest.java
index f8fe2e7ea20..8e6aa82a25e 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceTest.java
@@ -33,17 +33,17 @@ public class TwoServiceTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:ServiceA", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:ServiceA")
                         .log("ServiceA has been called")
                         .delay(simple("${random(1000,2000)}"))
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceWithExcludeTest.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceWithExcludeTest.java
index b64196f4b6d..e6774208973 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceWithExcludeTest.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/TwoServiceWithExcludeTest.java
@@ -39,17 +39,17 @@ public class TwoServiceWithExcludeTest extends CamelOpenTracingTestSupport {
     }
 
     @Test
-    public void testRoute() throws Exception {
+    public void testRoute() {
         template.requestBody("direct:ServiceA", "Hello");
 
         verify();
     }
 
     @Override
-    protected RoutesBuilder createRouteBuilder() throws Exception {
+    protected RoutesBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 from("direct:ServiceA")
                         .log("ServiceA has been called")
                         .delay(simple("${random(1000,2000)}"))
diff --git a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/integration/InterprocessIT.java b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/integration/InterprocessIT.java
index 6379b5c423d..9f372d3d119 100644
--- a/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/integration/InterprocessIT.java
+++ b/components/camel-opentracing/src/test/java/org/apache/camel/opentracing/integration/InterprocessIT.java
@@ -70,12 +70,12 @@ public class InterprocessIT extends CamelOpenTracingTestSupport {
     }
 
     @Override
-    protected RouteBuilder createRouteBuilder() throws Exception {
+    protected RouteBuilder createRouteBuilder() {
         return new RouteBuilder() {
             @Override
-            public void configure() throws Exception {
+            public void configure() {
                 Processor serviceProc = new Processor() {
-                    public void process(Exchange exchange) throws Exception {
+                    public void process(Exchange exchange) {
                         // get the request URL and copy it to the request body
                         String uri = exchange.getIn().getHeader(Exchange.HTTP_URI, String.class);
                         exchange.getMessage().setBody(uri);