Skip to content

Commit 1f321c4

Browse files
committed
Polish "Add service connection for Docker Compose and Testcontainers Artemis"
See gh-39311
1 parent f15cd93 commit 1f321c4

File tree

6 files changed

+28
-7
lines changed

6 files changed

+28
-7
lines changed

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/jms/artemis/ArtemisConnectionDetails.java

+16
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,28 @@
2626
*/
2727
public interface ArtemisConnectionDetails extends ConnectionDetails {
2828

29+
/**
30+
* Artemis deployment mode, auto-detected by default.
31+
* @return the Artemis deployment mode, auto-detected by default
32+
*/
2933
ArtemisMode getMode();
3034

35+
/**
36+
* Artemis broker url.
37+
* @return the Artemis broker url
38+
*/
3139
String getBrokerUrl();
3240

41+
/**
42+
* Login user of the broker.
43+
* @return the login user of the broker
44+
*/
3345
String getUser();
3446

47+
/**
48+
* Login password of the broker.
49+
* @return the login password of the broker
50+
*/
3551
String getPassword();
3652

3753
}

spring-boot-project/spring-boot-docker-compose/src/main/java/org/springframework/boot/docker/compose/service/connection/activemq/ArtemisDockerComposeConnectionDetailsFactory.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,15 @@
2727
* {@link ArtemisConnectionDetails} for an {@code artemis} service.
2828
*
2929
* @author Eddú Meléndez
30+
* @author Moritz Halbritter
3031
*/
3132
class ArtemisDockerComposeConnectionDetailsFactory
3233
extends DockerComposeConnectionDetailsFactory<ArtemisConnectionDetails> {
3334

3435
private static final int ACTIVEMQ_PORT = 61616;
3536

3637
protected ArtemisDockerComposeConnectionDetailsFactory() {
37-
super("apache/activemq-classic");
38+
super("apache/activemq-artemis");
3839
}
3940

4041
@Override

spring-boot-project/spring-boot-docker-compose/src/main/java/org/springframework/boot/docker/compose/service/connection/activemq/ArtemisEnvironment.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
* Artemis environment details.
2323
*
2424
* @author Eddú Meléndez
25+
* @author Moritz Halbritter
2526
*/
2627
class ArtemisEnvironment {
2728

@@ -30,8 +31,8 @@ class ArtemisEnvironment {
3031
private final String password;
3132

3233
ArtemisEnvironment(Map<String, String> env) {
33-
this.user = env.get("ACTIVEMQ_CONNECTION_USER");
34-
this.password = env.get("ACTIVEMQ_CONNECTION_PASSWORD");
34+
this.user = env.get("ARTEMIS_USER");
35+
this.password = env.get("ARTEMIS_PASSWORD");
3536
}
3637

3738
String getUser() {

spring-boot-project/spring-boot-docker-compose/src/test/java/org/springframework/boot/docker/compose/service/connection/activemq/ArtemisEnvironmentTests.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ void getUserWhenHasNoActiveMqUser() {
3838

3939
@Test
4040
void getUserWhenHasActiveMqUser() {
41-
ArtemisEnvironment environment = new ArtemisEnvironment(Map.of("ACTIVEMQ_CONNECTION_USER", "me"));
41+
ArtemisEnvironment environment = new ArtemisEnvironment(Map.of("ARTEMIS_USER", "me"));
4242
assertThat(environment.getUser()).isEqualTo("me");
4343
}
4444

@@ -50,7 +50,7 @@ void getPasswordWhenHasNoActiveMqPassword() {
5050

5151
@Test
5252
void getPasswordWhenHasActiveMqPassword() {
53-
ArtemisEnvironment environment = new ArtemisEnvironment(Map.of("ACTIVEMQ_CONNECTION_PASSWORD", "secret"));
53+
ArtemisEnvironment environment = new ArtemisEnvironment(Map.of("ARTEMIS_PASSWORD", "secret"));
5454
assertThat(environment.getPassword()).isEqualTo("secret");
5555
}
5656

spring-boot-project/spring-boot-docker-compose/src/test/resources/org/springframework/boot/docker/compose/service/connection/activemq/artemis-compose.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@ services:
44
ports:
55
- '61616'
66
environment:
7-
ACTIVEMQ_CONNECTION_USER: 'root'
8-
ACTIVEMQ_CONNECTION_PASSWORD: 'secret'
7+
ARTEMIS_USER: 'root'
8+
ARTEMIS_PASSWORD: 'secret'

spring-boot-project/spring-boot-docs/src/docs/asciidoc/features/docker-compose.adoc

+3
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,9 @@ The following service connections are currently supported:
7272
| `ActiveMQConnectionDetails`
7373
| Containers named "symptoma/activemq"
7474

75+
| `ArtemisConnectionDetails`
76+
| Containers named "apache/activemq-artemis"
77+
7578
| `CassandraConnectionDetails`
7679
| Containers named "cassandra"
7780

0 commit comments

Comments
 (0)