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 9b1c77277848b89a2e170651a333b045efc73d0d Author: Otavio Rodolfo Piske <angusyo...@gmail.com> AuthorDate: Tue Mar 8 14:41:06 2022 +0100 CAMEL-17763: cleaned up unused exceptions in camel-aws2-ses --- .../camel/component/aws2/ses/MockMessage.java | 84 +++++++++++----------- .../aws2/ses/SesComponentConfigurationTest.java | 8 +-- .../component/aws2/ses/SesComponentSpringTest.java | 16 ++--- .../camel/component/aws2/ses/SesComponentTest.java | 16 ++--- .../ses/SesComponentVerifierExtensionTest.java | 6 +- .../aws2/ses/integration/SesComponentManualIT.java | 8 +-- 6 files changed, 68 insertions(+), 70 deletions(-) diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/MockMessage.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/MockMessage.java index be12be6..e82e3e5 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/MockMessage.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/MockMessage.java @@ -16,7 +16,6 @@ */ package org.apache.camel.component.aws2.ses; -import java.io.IOException; import java.io.InputStream; import java.io.OutputStream; import java.util.Date; @@ -27,193 +26,192 @@ import javax.mail.Address; import javax.mail.Flags; import javax.mail.Header; import javax.mail.Message; -import javax.mail.MessagingException; import javax.mail.Multipart; public class MockMessage extends Message { @Override - public int getSize() throws MessagingException { + public int getSize() { return 0; } @Override - public int getLineCount() throws MessagingException { + public int getLineCount() { return 0; } @Override - public String getContentType() throws MessagingException { + public String getContentType() { return null; } @Override - public boolean isMimeType(String mimeType) throws MessagingException { + public boolean isMimeType(String mimeType) { return false; } @Override - public String getDisposition() throws MessagingException { + public String getDisposition() { return null; } @Override - public void setDisposition(String disposition) throws MessagingException { + public void setDisposition(String disposition) { } @Override - public String getDescription() throws MessagingException { + public String getDescription() { return null; } @Override - public void setDescription(String description) throws MessagingException { + public void setDescription(String description) { } @Override - public String getFileName() throws MessagingException { + public String getFileName() { return null; } @Override - public void setFileName(String filename) throws MessagingException { + public void setFileName(String filename) { } @Override - public InputStream getInputStream() throws IOException, MessagingException { + public InputStream getInputStream() { return null; } @Override - public DataHandler getDataHandler() throws MessagingException { + public DataHandler getDataHandler() { return null; } @Override - public Object getContent() throws IOException, MessagingException { + public Object getContent() { return null; } @Override - public void setDataHandler(DataHandler dh) throws MessagingException { + public void setDataHandler(DataHandler dh) { } @Override - public void setContent(Object obj, String type) throws MessagingException { + public void setContent(Object obj, String type) { } @Override - public void setText(String text) throws MessagingException { + public void setText(String text) { } @Override - public void setContent(Multipart mp) throws MessagingException { + public void setContent(Multipart mp) { } @Override - public void writeTo(OutputStream os) throws IOException, MessagingException { + public void writeTo(OutputStream os) { } @Override - public String[] getHeader(String headername) throws MessagingException { + public String[] getHeader(String headername) { return null; } @Override - public void setHeader(String headername, String headervalue) throws MessagingException { + public void setHeader(String headername, String headervalue) { } @Override - public void addHeader(String headername, String headervalue) throws MessagingException { + public void addHeader(String headername, String headervalue) { } @Override - public void removeHeader(String headername) throws MessagingException { + public void removeHeader(String headername) { } @Override - public Enumeration<Header> getAllHeaders() throws MessagingException { + public Enumeration<Header> getAllHeaders() { return null; } @Override - public Enumeration<Header> getMatchingHeaders(String[] headernames) throws MessagingException { + public Enumeration<Header> getMatchingHeaders(String[] headernames) { return null; } @Override - public Enumeration<Header> getNonMatchingHeaders(String[] headernames) throws MessagingException { + public Enumeration<Header> getNonMatchingHeaders(String[] headernames) { return null; } @Override - public Address[] getFrom() throws MessagingException { + public Address[] getFrom() { return null; } @Override - public void setFrom() throws MessagingException { + public void setFrom() { } @Override - public void setFrom(Address address) throws MessagingException { + public void setFrom(Address address) { } @Override - public void addFrom(Address[] addresses) throws MessagingException { + public void addFrom(Address[] addresses) { } @Override - public Address[] getRecipients(RecipientType type) throws MessagingException { + public Address[] getRecipients(RecipientType type) { return null; } @Override - public void setRecipients(RecipientType type, Address[] addresses) throws MessagingException { + public void setRecipients(RecipientType type, Address[] addresses) { } @Override - public void addRecipients(RecipientType type, Address[] addresses) throws MessagingException { + public void addRecipients(RecipientType type, Address[] addresses) { } @Override - public String getSubject() throws MessagingException { + public String getSubject() { return null; } @Override - public void setSubject(String subject) throws MessagingException { + public void setSubject(String subject) { } @Override - public Date getSentDate() throws MessagingException { + public Date getSentDate() { return null; } @Override - public void setSentDate(Date date) throws MessagingException { + public void setSentDate(Date date) { } @Override - public Date getReceivedDate() throws MessagingException { + public Date getReceivedDate() { return null; } @Override - public Flags getFlags() throws MessagingException { + public Flags getFlags() { return null; } @Override - public void setFlags(Flags flag, boolean set) throws MessagingException { + public void setFlags(Flags flag, boolean set) { } @Override - public Message reply(boolean replyToAll) throws MessagingException { + public Message reply(boolean replyToAll) { return null; } @Override - public void saveChanges() throws MessagingException { + public void saveChanges() { } } diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentConfigurationTest.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentConfigurationTest.java index ca12a91..732e40b 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentConfigurationTest.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentConfigurationTest.java @@ -136,7 +136,7 @@ public class SesComponentConfigurationTest extends CamelTestSupport { } @Test - public void createEndpointWithoutSourceName() throws Exception { + public void createEndpointWithoutSourceName() { Ses2Component component = context.getComponent("aws2-ses", Ses2Component.class); assertThrows(IllegalArgumentException.class, () -> { component.createEndpoint("aws2-ses:// "); @@ -144,7 +144,7 @@ public class SesComponentConfigurationTest extends CamelTestSupport { } @Test - public void createEndpointWithoutAmazonSESClientConfiguration() throws Exception { + public void createEndpointWithoutAmazonSESClientConfiguration() { Ses2Component component = context.getComponent("aws2-ses", Ses2Component.class); assertThrows(IllegalArgumentException.class, () -> { component.createEndpoint("aws2-ses://f...@example.com"); @@ -152,7 +152,7 @@ public class SesComponentConfigurationTest extends CamelTestSupport { } @Test - public void createEndpointWithoutAccessKeyConfiguration() throws Exception { + public void createEndpointWithoutAccessKeyConfiguration() { Ses2Component component = context.getComponent("aws2-ses", Ses2Component.class); assertThrows(IllegalArgumentException.class, () -> { component.createEndpoint("aws2-ses://f...@example.com?secretKey=yyy"); @@ -160,7 +160,7 @@ public class SesComponentConfigurationTest extends CamelTestSupport { } @Test - public void createEndpointWithoutSecretKeyConfiguration() throws Exception { + public void createEndpointWithoutSecretKeyConfiguration() { Ses2Component component = context.getComponent("aws2-ses", Ses2Component.class); assertThrows(IllegalArgumentException.class, () -> { component.createEndpoint("aws2-ses://f...@example.com?accessKey=xxx"); diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentSpringTest.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentSpringTest.java index a7549e6..dae9961 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentSpringTest.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentSpringTest.java @@ -44,10 +44,10 @@ public class SesComponentSpringTest extends CamelSpringTestSupport { } @Test - public void sendInOnlyMessageUsingUrlOptions() throws Exception { + public void sendInOnlyMessageUsingUrlOptions() { Exchange exchange = template.send("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); } }); @@ -61,10 +61,10 @@ public class SesComponentSpringTest extends CamelSpringTestSupport { } @Test - public void sendInOutMessageUsingUrlOptions() throws Exception { + public void sendInOutMessageUsingUrlOptions() { Exchange exchange = template.request("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); } }); @@ -73,11 +73,11 @@ public class SesComponentSpringTest extends CamelSpringTestSupport { } @Test - public void sendRawMessage() throws Exception { + public void sendRawMessage() { final MockMessage mess = new MockMessage(); Exchange exchange = template.request("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody(mess); } }); @@ -89,9 +89,9 @@ public class SesComponentSpringTest extends CamelSpringTestSupport { } @Test - public void sendMessageUsingMessageHeaders() throws Exception { + public void sendMessageUsingMessageHeaders() { Exchange exchange = template.send("direct:start", new Processor() { - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); exchange.getIn().setHeader(Ses2Constants.FROM, "anotherf...@example.com"); exchange.getIn().setHeader(Ses2Constants.TO, "another...@example.com, another...@example.com"); diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentTest.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentTest.java index 3d04d9c..edb1515 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentTest.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentTest.java @@ -35,10 +35,10 @@ public class SesComponentTest extends CamelTestSupport { private AmazonSESClientMock sesClient = new AmazonSESClientMock(); @Test - public void sendInOnlyMessageUsingUrlOptions() throws Exception { + public void sendInOnlyMessageUsingUrlOptions() { Exchange exchange = template.send("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); } }); @@ -59,10 +59,10 @@ public class SesComponentTest extends CamelTestSupport { } @Test - public void sendInOutMessageUsingUrlOptions() throws Exception { + public void sendInOutMessageUsingUrlOptions() { Exchange exchange = template.request("direct:start", new Processor() { @Override - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); } }); @@ -71,9 +71,9 @@ public class SesComponentTest extends CamelTestSupport { } @Test - public void sendMessageUsingMessageHeaders() throws Exception { + public void sendMessageUsingMessageHeaders() { Exchange exchange = template.send("direct:start", new Processor() { - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setBody("This is my message text."); exchange.getIn().setHeader(Ses2Constants.FROM, "anotherf...@example.com"); exchange.getIn().setHeader(Ses2Constants.TO, "another...@example.com, another...@example.com"); @@ -100,10 +100,10 @@ public class SesComponentTest 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("aws2-ses://f...@example.com" + "?to=t...@example.com,t...@example.com" + "&subject=Subject" + "&returnPath=bou...@example.com" + "&replyToAddresses=reply...@example.com,reply...@example.com" diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentVerifierExtensionTest.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentVerifierExtensionTest.java index aa28285..b11d04b 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentVerifierExtensionTest.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/SesComponentVerifierExtensionTest.java @@ -37,7 +37,7 @@ public class SesComponentVerifierExtensionTest extends CamelTestSupport { } @Test - public void testParameters() throws Exception { + public void testParameters() { Component component = context().getComponent("aws2-ses"); ComponentVerifierExtension verifier @@ -55,7 +55,7 @@ public class SesComponentVerifierExtensionTest extends CamelTestSupport { } @Test - public void testConnectivity() throws Exception { + public void testConnectivity() { Component component = context().getComponent("aws2-ses"); ComponentVerifierExtension verifier = component.getExtension(ComponentVerifierExtension.class).orElseThrow(IllegalStateException::new); @@ -72,7 +72,7 @@ public class SesComponentVerifierExtensionTest extends CamelTestSupport { } @Test - public void testConnectivityAndRegion() throws Exception { + public void testConnectivityAndRegion() { Component component = context().getComponent("aws2-ses"); ComponentVerifierExtension verifier = component.getExtension(ComponentVerifierExtension.class).orElseThrow(IllegalStateException::new); diff --git a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/integration/SesComponentManualIT.java b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/integration/SesComponentManualIT.java index 78f139b..4c9eb93 100644 --- a/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/integration/SesComponentManualIT.java +++ b/components/camel-aws/camel-aws2-ses/src/test/java/org/apache/camel/component/aws2/ses/integration/SesComponentManualIT.java @@ -36,9 +36,9 @@ import static org.junit.jupiter.api.Assertions.assertNotNull; public class SesComponentManualIT extends CamelTestSupport { @Test - public void sendUsingAccessKeyAndSecretKey() throws Exception { + public void sendUsingAccessKeyAndSecretKey() { Exchange exchange = template.send("direct:start", ExchangePattern.InOnly, new Processor() { - public void process(Exchange exchange) throws Exception { + public void process(Exchange exchange) { exchange.getIn().setHeader(Ses2Constants.SUBJECT, "This is my subject"); exchange.getIn().setHeader(Ses2Constants.TO, "t...@example.com"); exchange.getIn().setBody("This is my message text."); @@ -49,10 +49,10 @@ public class SesComponentManualIT 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("aws2-ses://f...@example.com?accessKey={{aws.access.key}}&secretKey={{aws.secret.key}}"); }