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 7d7c9f1d6fff347f5a6890aa96cf354deb0b46bb Author: Otavio Rodolfo Piske <[email protected]> AuthorDate: Sat Jan 24 19:46:48 2026 +0000 CAMEL-21196: modernize exception-based assertions in camel-sjms2 Replace try-catch-fail patterns with simple throws in Sjms2EndpointTest.java: - testDefaultExchangePattern - testInOnlyExchangePattern - testInOutExchangePattern --- .../camel/component/sjms2/Sjms2EndpointTest.java | 37 +++++++--------------- 1 file changed, 12 insertions(+), 25 deletions(-) diff --git a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java index 86512e141f82..92e81cf566d4 100644 --- a/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java +++ b/components/camel-sjms2/src/test/java/org/apache/camel/component/sjms2/Sjms2EndpointTest.java @@ -30,7 +30,6 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertFalse; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.junit.jupiter.api.Assertions.assertTrue; -import static org.junit.jupiter.api.Assertions.fail; public class Sjms2EndpointTest extends CamelTestSupport { @@ -98,36 +97,24 @@ public class Sjms2EndpointTest extends CamelTestSupport { } @Test - public void testDefaultExchangePattern() { - try { - Sjms2Endpoint sjms = (Sjms2Endpoint) context.getEndpoint("sjms2:queue:test"); - assertNotNull(sjms); - assertEquals(ExchangePattern.InOnly, sjms.getExchangePattern()); - } catch (Exception e) { - fail("Exception thrown: " + e.getLocalizedMessage()); - } + public void testDefaultExchangePattern() throws Exception { + Sjms2Endpoint sjms = (Sjms2Endpoint) context.getEndpoint("sjms2:queue:test"); + assertNotNull(sjms); + assertEquals(ExchangePattern.InOnly, sjms.getExchangePattern()); } @Test - public void testInOnlyExchangePattern() { - try { - Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOnly); - assertNotNull(sjms); - assertEquals(ExchangePattern.InOnly, sjms.createExchange().getPattern()); - } catch (Exception e) { - fail("Exception thrown: " + e.getLocalizedMessage()); - } + public void testInOnlyExchangePattern() throws Exception { + Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOnly); + assertNotNull(sjms); + assertEquals(ExchangePattern.InOnly, sjms.createExchange().getPattern()); } @Test - public void testInOutExchangePattern() { - try { - Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOut); - assertNotNull(sjms); - assertEquals(ExchangePattern.InOut, sjms.createExchange().getPattern()); - } catch (Exception e) { - fail("Exception thrown: " + e.getLocalizedMessage()); - } + public void testInOutExchangePattern() throws Exception { + Endpoint sjms = context.getEndpoint("sjms2:queue:test?exchangePattern=" + ExchangePattern.InOut); + assertNotNull(sjms); + assertEquals(ExchangePattern.InOut, sjms.createExchange().getPattern()); } @Test
