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/08 14:47:14 UTC
[camel] 09/09: CAMEL-17763: cleanup unused exceptions in camel-grok
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 d50894912740a177c573ccc07dc54bff96969c8e
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Fri Apr 8 15:48:58 2022 +0200
CAMEL-17763: cleanup unused exceptions in camel-grok
---
.../org/apache/camel/component/grok/GrokFileUnmarshalTest.java | 4 ++--
.../java/org/apache/camel/component/grok/GrokMarshalTest.java | 6 +++---
.../apache/camel/component/grok/GrokOptionalOptionsTest.java | 10 +++++-----
.../java/org/apache/camel/component/grok/GrokPatternsTest.java | 6 +++---
.../org/apache/camel/component/grok/GrokUnmarshalTest.java | 4 ++--
5 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokFileUnmarshalTest.java b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokFileUnmarshalTest.java
index ca3866f2567..91679fb6cc1 100644
--- a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokFileUnmarshalTest.java
+++ b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokFileUnmarshalTest.java
@@ -28,10 +28,10 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
public class GrokFileUnmarshalTest extends CamelTestSupport {
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
from("file:src/test/resources/org/apache/camel/component/grok/data?fileName=access_log&noop=true")
.unmarshal().grok("%{COMMONAPACHELOG}")
.to("mock:apachelog");
diff --git a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokMarshalTest.java b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokMarshalTest.java
index 3815f93c31a..eaf20aeefe4 100644
--- a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokMarshalTest.java
+++ b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokMarshalTest.java
@@ -27,10 +27,10 @@ import static org.junit.jupiter.api.Assertions.assertThrows;
public class GrokMarshalTest extends CamelTestSupport {
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
from("direct:direct")
.marshal().grok("%{IP:ip}")
.to("mock:mock");
@@ -40,7 +40,7 @@ public class GrokMarshalTest extends CamelTestSupport {
}
@Test
- public void testMarshalNotSupported() throws Exception {
+ public void testMarshalNotSupported() {
CamelExecutionException e = assertThrows(CamelExecutionException.class,
() -> template.sendBody("direct:direct", ""));
assertIsInstanceOf(UnsupportedOperationException.class, e.getCause());
diff --git a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokOptionalOptionsTest.java b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokOptionalOptionsTest.java
index 9336cc20112..eaa5a083693 100644
--- a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokOptionalOptionsTest.java
+++ b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokOptionalOptionsTest.java
@@ -36,10 +36,10 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
public class GrokOptionalOptionsTest extends CamelTestSupport {
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
DataFormat grokFlattenedTrue = new GrokDataFormat("%{INT:i} %{INT:i}")
.setFlattened(true);
DataFormat grokFlattenedFalse = new GrokDataFormat("%{INT:i} %{INT:i}")
@@ -68,7 +68,7 @@ public class GrokOptionalOptionsTest extends CamelTestSupport {
@Test
@SuppressWarnings("unchecked")
- public void testFlattened() throws Exception {
+ public void testFlattened() {
Map<String, Object> flattenedFalse = template.requestBody("direct:flattenedFalse", "123 456", Map.class);
assertNotNull(flattenedFalse);
assertTrue(flattenedFalse.containsKey("i"));
@@ -83,7 +83,7 @@ public class GrokOptionalOptionsTest extends CamelTestSupport {
@Test
@SuppressWarnings("unchecked")
- public void testNamedOnly() throws Exception {
+ public void testNamedOnly() {
Map<String, Object> namedOnlyTrue
= template.requestBody("direct:namedOnlyTrue", "https://github.com/apache/camel", Map.class);
assertNotNull(namedOnlyTrue);
@@ -103,7 +103,7 @@ public class GrokOptionalOptionsTest extends CamelTestSupport {
@Test
@SuppressWarnings("unchecked")
- public void testAllowMultipleMatchesPerLine() throws Exception {
+ public void testAllowMultipleMatchesPerLine() {
List<Map<String, Object>> allowMultipleMatchesPerLineTrue = template.requestBody(
"direct:allowMultipleMatchesPerLineTrue",
"1 2 \n 3",
diff --git a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokPatternsTest.java b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokPatternsTest.java
index 6d06655a7b9..d63c3d7ea2c 100644
--- a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokPatternsTest.java
+++ b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokPatternsTest.java
@@ -65,10 +65,10 @@ public class GrokPatternsTest extends CamelTestSupport {
}
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
}
};
}
@@ -78,7 +78,7 @@ public class GrokPatternsTest extends CamelTestSupport {
public void testPattern(String pattern, String input, Consumer<Map> expectedOutputTest) throws Exception {
context.addRoutes(new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
from("direct:input")
.unmarshal().grok(pattern);
}
diff --git a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokUnmarshalTest.java b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokUnmarshalTest.java
index d6d44346253..fc180c8f143 100644
--- a/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokUnmarshalTest.java
+++ b/components/camel-grok/src/test/java/org/apache/camel/component/grok/GrokUnmarshalTest.java
@@ -32,10 +32,10 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
public class GrokUnmarshalTest extends CamelTestSupport {
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
bindToRegistry("myCustomPatternBean", new GrokPattern("FOOBAR", "foo|bar"));
bindToRegistry("myAnotherCustomPatternBean",
new GrokPattern("FOOBAR_WITH_PREFIX_AND_SUFFIX", "-- %{FOOBAR}+ --"));