This is an automated email from the ASF dual-hosted git repository. gnodet pushed a commit to branch CAMEL-23214-port-misc in repository https://gitbox.apache.org/repos/asf/camel.git
commit 4e4ea7c57671d8bbcf32544af3b5fd3ea39c5bfe Author: Guillaume Nodet <[email protected]> AuthorDate: Fri Mar 20 07:31:45 2026 +0100 CAMEL-23214: Fix Stomp tests passing Port object to format strings Use .getPort() instead of passing Port object directly to fromF/toF, which fails with IllegalFormatConversion when using %d format specifier. Co-Authored-By: Claude Opus 4.6 <[email protected]> --- .../camel/component/stomp/StompConsumerHeaderFilterStrategyTest.java | 4 ++-- .../test/java/org/apache/camel/component/stomp/StompConsumerTest.java | 2 +- .../java/org/apache/camel/component/stomp/StompConsumerUriTest.java | 2 +- .../org/apache/camel/component/stomp/StompGlobalSslConsumerTest.java | 2 +- .../test/java/org/apache/camel/component/stomp/StompProducerTest.java | 2 +- .../java/org/apache/camel/component/stomp/StompSslConsumerTest.java | 2 +- .../java/org/apache/camel/component/stomp/StompSslProducerTest.java | 2 +- 7 files changed, 8 insertions(+), 8 deletions(-) diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerHeaderFilterStrategyTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerHeaderFilterStrategyTest.java index a9d7a5ab0b84..9c4b1df21ef4 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerHeaderFilterStrategyTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerHeaderFilterStrategyTest.java @@ -72,8 +72,8 @@ public class StompConsumerHeaderFilterStrategyTest extends StompBaseTest { protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { public void configure() { - fromF("stomp:test?brokerURL=tcp://localhost:%s&headerFilterStrategy=#customHeaderFilterStrategy", - servicePort) + fromF("stomp:test?brokerURL=tcp://localhost:%d&headerFilterStrategy=#customHeaderFilterStrategy", + servicePort.getPort()) .id("headerFilterStrategyRoute") .transform(body().convertToString()) .to("mock:result"); diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerTest.java index 931830be3cfc..ea3378cff7d8 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerTest.java @@ -61,7 +61,7 @@ public class StompConsumerTest extends StompBaseTest { protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { public void configure() { - fromF("stomp:test?brokerURL=tcp://localhost:%s", servicePort) + fromF("stomp:test?brokerURL=tcp://localhost:%d", servicePort.getPort()) .transform(body().convertToString()) .to("mock:result"); } diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerUriTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerUriTest.java index a487c0495a47..498889f08196 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerUriTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompConsumerUriTest.java @@ -61,7 +61,7 @@ public class StompConsumerUriTest extends StompBaseTest { protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { public void configure() { - fromF("stomp:test?brokerURL=tcp://localhost:%d", servicePort) + fromF("stomp:test?brokerURL=tcp://localhost:%d", servicePort.getPort()) .transform(body().convertToString()) .to("mock:result"); } diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompGlobalSslConsumerTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompGlobalSslConsumerTest.java index a064c5da4a08..69cda338269c 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompGlobalSslConsumerTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompGlobalSslConsumerTest.java @@ -42,7 +42,7 @@ public class StompGlobalSslConsumerTest extends StompConsumerTest { protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { public void configure() { - fromF("stomp:test?brokerURL=ssl://localhost:%d", sslServicePort) + fromF("stomp:test?brokerURL=ssl://localhost:%d", sslServicePort.getPort()) .transform(body().convertToString()) .to("mock:result"); } diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompProducerTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompProducerTest.java index 8d6e00a59bb6..ecd64a680c75 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompProducerTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompProducerTest.java @@ -94,7 +94,7 @@ public class StompProducerTest extends StompBaseTest { protected RouteBuilder createRouteBuilder() { return new RouteBuilder() { public void configure() { - from("direct:foo").toF("stomp:test?brokerURL=tcp://localhost:%s", servicePort); + from("direct:foo").toF("stomp:test?brokerURL=tcp://localhost:%d", servicePort.getPort()); } }; } diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslConsumerTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslConsumerTest.java index 14a26b48c5c4..a380601490e9 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslConsumerTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslConsumerTest.java @@ -32,7 +32,7 @@ public class StompSslConsumerTest extends StompConsumerTest { return new RouteBuilder() { public void configure() { fromF("stomp:test?brokerURL=ssl://localhost:%d&sslContextParameters=#sslContextParameters", - sslServicePort) + sslServicePort.getPort()) .transform(body().convertToString()) .to("mock:result"); } diff --git a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslProducerTest.java b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslProducerTest.java index 801ad99e846d..e8062c3de92e 100644 --- a/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslProducerTest.java +++ b/components/camel-stomp/src/test/java/org/apache/camel/component/stomp/StompSslProducerTest.java @@ -31,7 +31,7 @@ public class StompSslProducerTest extends StompProducerTest { public void configure() { from("direct:foo") .toF("stomp:test?brokerURL=ssl://localhost:%d&sslContextParameters=#sslContextParameters", - sslServicePort); + sslServicePort.getPort()); } }; }
