search

apache / camel-quarkus

Apache Camel Quarkus
https://camel.apache.org
Apache License 2.0
257 stars 191 forks source link

Use an Artemis container instead of `quarkus-test-artemis` in `camel-quarkus-integration-test-jms-artemis-client` #6364

Closed ppalaga closed 2 months ago

ppalaga commented 2 months ago

Description

... to improve the robustness and lower the maintenance overhead.

ppalaga commented 2 months ago

cc @jamesnetherton

jamesnetherton commented 2 months ago

Would be good to also apply this to the jms-jpa example if possible:

https://github.com/apache/camel-quarkus-examples/blob/8e3f6067ca6639fda54092366356b1a4d67c96a9/jms-jpa/pom.xml#L139-L143

zhfeng commented 2 months ago

It could impact the CustomArtemisTest https://github.com/apache/camel-quarkus/blob/ffb13f065c66f45f4a30a09a2f6a546c5ea3a8f0/integration-tests/jms-artemis-client/src/main/java/org/apache/camel/quarkus/component/jms/artemis/it/CustomConnectionFactory.java#L30-L35

jamesnetherton commented 2 months ago

The main part is done in #6390. I'll fix up the jms-jpa example after camel-quarkus-main branch is reset for 3.15.

jamesnetherton commented 2 months ago

https://github.com/apache/camel-quarkus-examples/pull/258 completes this task.