From 50c44e301a03e10bf509b173733ea23e3d456454 Mon Sep 17 00:00:00 2001
From: tish <tish@candyttl.com>
Date: Thu, 25 Jan 2024 07:59:49 +0400
Subject: [PATCH 1/2] Prevent double registration of event publisher registrar

See gh-39297
---
 .../testcontainers/properties/TestcontainersPropertySource.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java b/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
index 1994303645e..d52cc0c8d51 100644
--- a/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
+++ b/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
@@ -115,7 +115,7 @@ public class TestcontainersPropertySource extends EnumerablePropertySource<Map<S
 		if (eventPublisher != null) {
 			propertySource.addEventPublisher(eventPublisher);
 		}
-		else if (registry != null) {
+		else if (registry != null && !registry.containsBeanDefinition(EventPublisherRegistrar.NAME)) {
 			registry.registerBeanDefinition(EventPublisherRegistrar.NAME, new RootBeanDefinition(
 					EventPublisherRegistrar.class, () -> new EventPublisherRegistrar(environment)));
 		}

From 61ca87f7a4960353c1e7699a9567c666fc6c2231 Mon Sep 17 00:00:00 2001
From: Andy Wilkinson <andy.wilkinson@broadcom.com>
Date: Tue, 30 Jan 2024 16:02:38 +0000
Subject: [PATCH 2/2] Polish "Prevent double registration of event publisher
 registrar"

See gh-39297
---
 .../TestcontainersPropertySource.java         |  2 +-
 .../TestcontainersPropertySourceTests.java    | 31 +++++++++++++++++--
 2 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java b/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
index d52cc0c8d51..10bcb2a9864 100644
--- a/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
+++ b/spring-boot-project/spring-boot-testcontainers/src/main/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySource.java
@@ -138,7 +138,7 @@ public class TestcontainersPropertySource extends EnumerablePropertySource<Map<S
 	 * to the {@link TestcontainersPropertySource}. This class is a
 	 * {@link BeanFactoryPostProcessor} so that it is initialized as early as possible.
 	 */
-	private static class EventPublisherRegistrar implements BeanFactoryPostProcessor, ApplicationEventPublisherAware {
+	static class EventPublisherRegistrar implements BeanFactoryPostProcessor, ApplicationEventPublisherAware {
 
 		static final String NAME = EventPublisherRegistrar.class.getName();
 
diff --git a/spring-boot-project/spring-boot-testcontainers/src/test/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySourceTests.java b/spring-boot-project/spring-boot-testcontainers/src/test/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySourceTests.java
index 4dce61ffc94..e11488b349c 100644
--- a/spring-boot-project/spring-boot-testcontainers/src/test/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySourceTests.java
+++ b/spring-boot-project/spring-boot-testcontainers/src/test/java/org/springframework/boot/testcontainers/properties/TestcontainersPropertySourceTests.java
@@ -23,6 +23,8 @@ import java.util.Map;
 import org.junit.jupiter.api.Test;
 
 import org.springframework.beans.factory.support.BeanDefinitionRegistry;
+import org.springframework.beans.factory.support.DefaultListableBeanFactory;
+import org.springframework.boot.testcontainers.properties.TestcontainersPropertySource.EventPublisherRegistrar;
 import org.springframework.context.ApplicationEvent;
 import org.springframework.context.support.GenericApplicationContext;
 import org.springframework.core.env.EnumerablePropertySource;
@@ -42,6 +44,13 @@ class TestcontainersPropertySourceTests {
 
 	private MockEnvironment environment = new MockEnvironment();
 
+	private GenericApplicationContext context = new GenericApplicationContext();
+
+	TestcontainersPropertySourceTests() {
+		((DefaultListableBeanFactory) this.context.getBeanFactory()).setAllowBeanDefinitionOverriding(false);
+		this.context.setEnvironment(this.environment);
+	}
+
 	@Test
 	void getPropertyWhenHasValueSupplierReturnsSuppliedValue() {
 		DynamicPropertyRegistry registry = TestcontainersPropertySource.attach(this.environment);
@@ -90,14 +99,14 @@ class TestcontainersPropertySourceTests {
 	}
 
 	@Test
-	void attachWhenNotAttachedAttaches() {
+	void attachToEnvironmentWhenNotAttachedAttaches() {
 		TestcontainersPropertySource.attach(this.environment);
 		PropertySource<?> propertySource = this.environment.getPropertySources().get(TestcontainersPropertySource.NAME);
 		assertThat(propertySource).isNotNull();
 	}
 
 	@Test
-	void attachWhenAlreadyAttachedReturnsExisting() {
+	void attachToEnvironmentWhenAlreadyAttachedReturnsExisting() {
 		DynamicPropertyRegistry r1 = TestcontainersPropertySource.attach(this.environment);
 		PropertySource<?> p1 = this.environment.getPropertySources().get(TestcontainersPropertySource.NAME);
 		DynamicPropertyRegistry r2 = TestcontainersPropertySource.attach(this.environment);
@@ -106,6 +115,24 @@ class TestcontainersPropertySourceTests {
 		assertThat(p1).isSameAs(p2);
 	}
 
+	@Test
+	void attachToEnvironmentAndContextWhenNotAttachedAttaches() {
+		TestcontainersPropertySource.attach(this.environment, this.context);
+		PropertySource<?> propertySource = this.environment.getPropertySources().get(TestcontainersPropertySource.NAME);
+		assertThat(propertySource).isNotNull();
+		assertThat(this.context.containsBean(EventPublisherRegistrar.NAME));
+	}
+
+	@Test
+	void attachToEnvironmentAndContextWhenAlreadyAttachedReturnsExisting() {
+		DynamicPropertyRegistry r1 = TestcontainersPropertySource.attach(this.environment, this.context);
+		PropertySource<?> p1 = this.environment.getPropertySources().get(TestcontainersPropertySource.NAME);
+		DynamicPropertyRegistry r2 = TestcontainersPropertySource.attach(this.environment, this.context);
+		PropertySource<?> p2 = this.environment.getPropertySources().get(TestcontainersPropertySource.NAME);
+		assertThat(r1).isSameAs(r2);
+		assertThat(p1).isSameAs(p2);
+	}
+
 	@Test
 	void getPropertyPublishesEvent() {
 		try (GenericApplicationContext applicationContext = new GenericApplicationContext()) {