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 57b92f04ce1fe6b559fa0790d7c6b413a7e8ea96 Author: Otavio Rodolfo Piske <[email protected]> AuthorDate: Thu Mar 10 15:43:27 2022 +0100 CAMEL-17763: cleaned up unused exceptions in camel-solr --- .../java/org/apache/camel/component/solr/InitSolrEndpointTest.java | 6 +++--- .../test/java/org/apache/camel/component/solr/MapBodyManualIT.java | 6 +++--- .../org/apache/camel/component/solr/SolrComponentTestSupport.java | 4 ++-- .../java/org/apache/camel/component/solr/SolrServerMissingTest.java | 6 +++--- .../test/java/org/apache/camel/component/solr/SolrTestSupport.java | 2 +- .../org/apache/camel/component/solr/SolrUpdateAutocommitTest.java | 4 ++-- .../test/java/org/apache/camel/component/solr/SolrUpdateTest.java | 4 ++-- .../camel/component/solr/integration/SolrCloudProducerIT.java | 4 ++-- 8 files changed, 18 insertions(+), 18 deletions(-) diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/InitSolrEndpointTest.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/InitSolrEndpointTest.java index d35d096..34ffb6c 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/InitSolrEndpointTest.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/InitSolrEndpointTest.java @@ -41,7 +41,7 @@ public class InitSolrEndpointTest extends SolrTestSupport { private String solrUrl = "solr://localhost:" + getPort() + "/solr"; @Test - public void endpointCreatedCorrectlyWithAllOptions() throws Exception { + public void endpointCreatedCorrectlyWithAllOptions() { HttpClient httpClient = HttpClientUtil.createClient(new ModifiableSolrParams()); context.getRegistry().bind("http", httpClient); SolrClient httpSolrClient = new HttpSolrClient.Builder(solrUrl).build(); @@ -53,7 +53,7 @@ public class InitSolrEndpointTest extends SolrTestSupport { } @Test - public void streamingEndpointCreatedCorrectly() throws Exception { + public void streamingEndpointCreatedCorrectly() { SolrEndpoint solrEndpoint = context.getEndpoint(solrUrl, SolrEndpoint.class); assertNotNull(solrEndpoint); assertEquals(SolrConstants.DEFUALT_STREAMING_QUEUE_SIZE, solrEndpoint.getSolrConfiguration().getStreamingQueueSize(), @@ -64,7 +64,7 @@ public class InitSolrEndpointTest extends SolrTestSupport { } @Test - public void wrongURLFormatFailsEndpointCreation() throws Exception { + public void wrongURLFormatFailsEndpointCreation() { assertThrows(ResolveEndpointFailedException.class, () -> context.getEndpoint("solr://localhost:x99/solr")); } diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/MapBodyManualIT.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/MapBodyManualIT.java index 41388a3..4a73534 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/MapBodyManualIT.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/MapBodyManualIT.java @@ -47,7 +47,7 @@ public class MapBodyManualIT extends CamelTestSupport { template.send("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { Map<String, String> map = new HashMap<>(); map.put("id", "0553579923"); map.put("cat", "Test"); @@ -66,10 +66,10 @@ public class MapBodyManualIT extends CamelTestSupport { } @Override - protected RouteBuilder createRouteBuilder() throws Exception { + protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { @Override - public void configure() throws Exception { + public void configure() { from("direct:start").to("{{solr.address}}").to("mock:result"); } }; diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrComponentTestSupport.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrComponentTestSupport.java index a7494eb..30ef7fb 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrComponentTestSupport.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrComponentTestSupport.java @@ -99,10 +99,10 @@ public abstract class SolrComponentTestSupport extends SolrTestSupport { } @Override - protected RouteBuilder createRouteBuilder() throws Exception { + protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { @Override - public void configure() throws Exception { + public void configure() { from("direct:start").to(solrRouteUri()); from("direct:splitThenCommit") .split(body()) diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrServerMissingTest.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrServerMissingTest.java index 95c593d..40c1451 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrServerMissingTest.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrServerMissingTest.java @@ -25,7 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue; public class SolrServerMissingTest extends SolrTestSupport { @Test - public void indexSingleDocumentToNonexistentServer() throws Exception { + public void indexSingleDocumentToNonexistentServer() { Exchange exchange = createExchangeWithBody(null); exchange.getIn().setHeader(SolrConstants.OPERATION, SolrConstants.OPERATION_INSERT); exchange.getIn().setHeader("SolrField.id", "MA147LL/A"); @@ -36,10 +36,10 @@ public class SolrServerMissingTest extends SolrTestSupport { } @Override - protected RouteBuilder createRouteBuilder() throws Exception { + protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { @Override - public void configure() throws Exception { + public void configure() { from("direct:start").to("solr://localhost:" + getPort() + "/missingSolr"); } }; diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrTestSupport.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrTestSupport.java index 60f310b..0130f7d 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrTestSupport.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrTestSupport.java @@ -26,7 +26,7 @@ public abstract class SolrTestSupport extends CamelTestSupport { private static int httpsPort; @BeforeAll - public static void initPort() throws Exception { + public static void initPort() { port = AvailablePortFinder.getNextAvailable(); httpsPort = AvailablePortFinder.getNextAvailable(); //TODO : add test for TLSv1.3 diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateAutocommitTest.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateAutocommitTest.java index 5ae96fb..303d4e8 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateAutocommitTest.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateAutocommitTest.java @@ -145,7 +145,7 @@ public class SolrUpdateAutocommitTest extends SolrComponentTestSupport { } @Test - public void caughtSolrExceptionIsHandledElegantly() throws Exception { + public void caughtSolrExceptionIsHandledElegantly() { Exchange exchange = createExchangeWithBody(null); exchange.getIn().setHeader(SolrConstants.OPERATION, SolrConstants.OPERATION_INSERT); exchange.getIn().setHeader("SolrField.name", "Missing required field throws exception."); @@ -245,7 +245,7 @@ public class SolrUpdateAutocommitTest extends SolrComponentTestSupport { } @Test - public void queryDocumentsToCSVUpdateHandlerWithoutParameters() throws Exception { + public void queryDocumentsToCSVUpdateHandlerWithoutParameters() { solrEndpoint.setRequestHandler("/update/csv"); Exchange exchange = createExchangeWithBody(new File("src/test/resources/data/books.csv")); diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateTest.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateTest.java index 0241d77..8e2b857 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateTest.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/SolrUpdateTest.java @@ -154,7 +154,7 @@ public class SolrUpdateTest extends SolrComponentTestSupport { } @Test - public void caughtSolrExceptionIsHandledElegantly() throws Exception { + public void caughtSolrExceptionIsHandledElegantly() { Exchange exchange = createExchangeWithBody(null); exchange.getIn().setHeader(SolrConstants.OPERATION, SolrConstants.OPERATION_INSERT); exchange.getIn().setHeader("SolrField.name", "Missing required field throws exception."); @@ -264,7 +264,7 @@ public class SolrUpdateTest extends SolrComponentTestSupport { } @Test - public void queryDocumentsToCSVUpdateHandlerWithoutParameters() throws Exception { + public void queryDocumentsToCSVUpdateHandlerWithoutParameters() { solrEndpoint.setRequestHandler("/update/csv"); Exchange exchange = createExchangeWithBody(new File("src/test/resources/data/books.csv")); diff --git a/components/camel-solr/src/test/java/org/apache/camel/component/solr/integration/SolrCloudProducerIT.java b/components/camel-solr/src/test/java/org/apache/camel/component/solr/integration/SolrCloudProducerIT.java index 51c5bb3..56fe5d5 100644 --- a/components/camel-solr/src/test/java/org/apache/camel/component/solr/integration/SolrCloudProducerIT.java +++ b/components/camel-solr/src/test/java/org/apache/camel/component/solr/integration/SolrCloudProducerIT.java @@ -30,10 +30,10 @@ public class SolrCloudProducerIT extends SolrCloudITSupport { } @Override - protected RouteBuilder createRouteBuilder() throws Exception { + protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { @Override - public void configure() throws Exception { + public void configure() { from("direct:send").to(solrEndpointUri); } };
