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 15:09:01 UTC
[camel] 03/10: CAMEL-17763: cleanup unused exceptions in camel-zipfile
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 b428cdc55959f315f9fc17682962244bbfb739b1
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Fri Apr 8 15:57:52 2022 +0200
CAMEL-17763: cleanup unused exceptions in camel-zipfile
---
.../camel/dataformat/zipfile/ZipFileDataFormatTest.java | 12 ++++++------
.../dataformat/zipfile/ZipFileIteratorDataFormatTest.java | 4 ++--
.../dataformat/zipfile/ZipFileMultipleFilesSplitterTest.java | 4 ++--
.../camel/dataformat/zipfile/ZipFileSplitAndDeleteTest.java | 4 ++--
.../dataformat/zipfile/ZipFileSplitIteratorCorruptTest.java | 4 ++--
.../camel/dataformat/zipfile/ZipFileSplitOneFileTest.java | 4 ++--
.../camel/dataformat/zipfile/ZipSplitterRouteIssueTest.java | 4 ++--
.../camel/dataformat/zipfile/ZipSplitterRouteTest.java | 4 ++--
.../zipfile/AggregationStrategyWithFilenameHeaderTest.java | 4 ++--
.../zipfile/AggregationStrategyWithPreservationTest.java | 4 ++--
.../zipfile/ZipAggregationStrategyEmptyFileTest.java | 4 ++--
.../aggregate/zipfile/ZipAggregationStrategyTest.java | 4 ++--
12 files changed, 28 insertions(+), 28 deletions(-)
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileDataFormatTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileDataFormatTest.java
index 0a04867b1bf..a36044f6ab2 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileDataFormatTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileDataFormatTest.java
@@ -145,7 +145,7 @@ public class ZipFileDataFormatTest extends CamelTestSupport {
}
@Test
- public void testUnzipWithEmptyDirectorySupported() throws Exception {
+ public void testUnzipWithEmptyDirectorySupported() {
deleteDirectory(new File("hello_out"));
zip.setUsingIterator(true);
zip.setAllowEmptyDirectory(true);
@@ -155,7 +155,7 @@ public class ZipFileDataFormatTest extends CamelTestSupport {
}
@Test
- public void testUnzipWithEmptyDirectoryUnsupported() throws Exception {
+ public void testUnzipWithEmptyDirectoryUnsupported() {
deleteDirectory(new File("hello_out"));
zip.setUsingIterator(true);
zip.setAllowEmptyDirectory(false);
@@ -165,7 +165,7 @@ public class ZipFileDataFormatTest extends CamelTestSupport {
}
@Test
- public void testUnzipWithCorruptedZipFile() throws Exception {
+ public void testUnzipWithCorruptedZipFile() {
deleteDirectory(new File("hello_out"));
assertThrows(CamelExecutionException.class,
@@ -252,7 +252,7 @@ public class ZipFileDataFormatTest extends CamelTestSupport {
}
@Test
- public void testUnzipMaxDecompressedSize() throws Exception {
+ public void testUnzipMaxDecompressedSize() {
// We are only allowing 10 bytes to be decompressed, so we expect an error
assertThrows(CamelExecutionException.class,
() -> template.sendBody("direct:unzipMaxDecompressedSize", getZippedText("file")));
@@ -289,10 +289,10 @@ public class ZipFileDataFormatTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
interceptSendToEndpoint("file:*").to("mock:intercepted");
zip = new ZipFileDataFormat();
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileIteratorDataFormatTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileIteratorDataFormatTest.java
index 8c01bbdf449..8f39d94549b 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileIteratorDataFormatTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileIteratorDataFormatTest.java
@@ -48,10 +48,10 @@ public class ZipFileIteratorDataFormatTest 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:zip")
.setHeader(Exchange.FILE_NAME, constant("report.txt"))
.marshal().zipFile()
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileMultipleFilesSplitterTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileMultipleFilesSplitterTest.java
index 4585d0bd3c2..528a00d24db 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileMultipleFilesSplitterTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileMultipleFilesSplitterTest.java
@@ -39,10 +39,10 @@ public class ZipFileMultipleFilesSplitterTest extends ZipSplitterRouteTest {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
// Unzip file and Split it according to FileEntry
ZipFileDataFormat zipFile = new ZipFileDataFormat();
zipFile.setUsingIterator(true);
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitAndDeleteTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitAndDeleteTest.java
index 22264cbcc42..4496867b885 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitAndDeleteTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitAndDeleteTest.java
@@ -75,10 +75,10 @@ public class ZipFileSplitAndDeleteTest extends CamelTestSupport {
}
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
ZipFileDataFormat dataFormat = new ZipFileDataFormat();
dataFormat.setUsingIterator(true);
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitIteratorCorruptTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitIteratorCorruptTest.java
index efc8430b6d4..857b7a9504f 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitIteratorCorruptTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitIteratorCorruptTest.java
@@ -37,10 +37,10 @@ public class ZipFileSplitIteratorCorruptTest extends CamelTestSupport {
}
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
ZipFileDataFormat zf = new ZipFileDataFormat();
zf.setUsingIterator(true);
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitOneFileTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitOneFileTest.java
index d28260527d1..9e3685b4f35 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitOneFileTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipFileSplitOneFileTest.java
@@ -54,10 +54,10 @@ public class ZipFileSplitOneFileTest extends CamelTestSupport {
}
@Override
- protected RoutesBuilder createRouteBuilder() throws Exception {
+ protected RoutesBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
ZipFileDataFormat zf = new ZipFileDataFormat();
zf.setUsingIterator(true);
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteIssueTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteIssueTest.java
index 44f9b601a71..2eb0e74eba3 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteIssueTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteIssueTest.java
@@ -55,10 +55,10 @@ public class ZipSplitterRouteIssueTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
errorHandler(deadLetterChannel("mock:errors"));
from("direct:decompressFiles")
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteTest.java
index 4ef96308937..69e5f828acd 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/dataformat/zipfile/ZipSplitterRouteTest.java
@@ -31,10 +31,10 @@ public class ZipSplitterRouteTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
// Unzip file and Split it according to FileEntry
from("file:src/test/resources/org/apache/camel/dataformat/zipfile/data?delay=1000&noop=true")
.log("Start processing big file: ${header.CamelFileName}")
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithFilenameHeaderTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithFilenameHeaderTest.java
index 3924c9b9bb4..38901a0b9e7 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithFilenameHeaderTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithFilenameHeaderTest.java
@@ -80,10 +80,10 @@ public class AggregationStrategyWithFilenameHeaderTest 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")
.aggregate(new ZipAggregationStrategy(false, true))
.constant(true)
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithPreservationTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithPreservationTest.java
index 0c1bdabb8b0..22c45b25f0b 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithPreservationTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/AggregationStrategyWithPreservationTest.java
@@ -84,10 +84,10 @@ public class AggregationStrategyWithPreservationTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
// Unzip file and Split it according to FileEntry
from("file:src/test/resources/org/apache/camel/aggregate/zipfile/data?delay=1000&noop=true&recursive=true")
.aggregate(new ZipAggregationStrategy(true, true))
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyEmptyFileTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyEmptyFileTest.java
index e8a4b206481..6a9b585775b 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyEmptyFileTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyEmptyFileTest.java
@@ -79,10 +79,10 @@ public class ZipAggregationStrategyEmptyFileTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
from("file:target/foo")
.aggregate(new ZipAggregationStrategy())
.constant(true)
diff --git a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyTest.java b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyTest.java
index d87c70cb15c..8310ea1a538 100644
--- a/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyTest.java
+++ b/components/camel-zipfile/src/test/java/org/apache/camel/processor/aggregate/zipfile/ZipAggregationStrategyTest.java
@@ -73,10 +73,10 @@ public class ZipAggregationStrategyTest extends CamelTestSupport {
}
@Override
- protected RouteBuilder createRouteBuilder() throws Exception {
+ protected RouteBuilder createRouteBuilder() {
return new RouteBuilder() {
@Override
- public void configure() throws Exception {
+ public void configure() {
// Unzip file and Split it according to FileEntry
from("file:src/test/resources/org/apache/camel/aggregate/zipfile/data?delay=1000&noop=true")
.setHeader("foo", constant("bar"))