Partially revert auditing bean registration changes.

Restoring bean registration while keeping the optional nature of AuditorAware in the Reactive-/AuditingHandler.

The changes done previously caused AOT code generation to fail with:
org.springframework.aot.generate.UnsupportedTypeValueCodeGenerationException: Code generation does not support org.springframework.beans.factory.config.AutowiredPropertyMarker

See: #3177

Author: christophstrobl

src/main/antora/modules/ROOT/pages/auditing.adoc

@@ -3,16 +3,13 @@
 
 [[auditing.basics]]
 == Basics
-
-Spring Data provides sophisticated support to transparently keep track of who created or changed an entity and when the change happened.
-To benefit from that functionality, you have to equip your entity classes with auditing metadata that can be defined either using annotations or by implementing an interface.
+Spring Data provides sophisticated support to transparently keep track of who created or changed an entity and when the change happened.To benefit from that functionality, you have to equip your entity classes with auditing metadata that can be defined either using annotations or by implementing an interface.
 Additionally, auditing has to be enabled either through Annotation configuration or XML configuration to register the required infrastructure components.
 Please refer to the store-specific section for configuration samples.
 
 [NOTE]
 ====
 Applications that only track creation and modification dates are not required do make their entities implement <<auditing.auditor-aware,`AuditorAware`>>.
-If no `AuditorAware` or `DateTimeProvider` bean is configured, `AuditingHandler` will use Spring's autowiring to detect a matching bean if beans of the corresponding type are available in the application context.
 ====
 
 [[auditing.annotations]]

src/main/java/org/springframework/data/auditing/config/AuditingBeanDefinitionRegistrarSupport.java

@@ -116,23 +116,19 @@ protected BeanDefinitionBuilder getAuditHandlerBeanDefinitionBuilder(AuditingCon
 	protected BeanDefinitionBuilder configureDefaultAuditHandlerAttributes(AuditingConfiguration configuration,
 			BeanDefinitionBuilder builder) {
 
+		builder.setAutowireMode(AbstractBeanDefinition.AUTOWIRE_BY_TYPE);
+
 		if (StringUtils.hasText(configuration.getAuditorAwareRef())) {
 			builder.addPropertyValue(AUDITOR_AWARE,
 					createLazyInitTargetSourceBeanDefinition(configuration.getAuditorAwareRef()));
 		}
-		else {
-			builder.addAutowiredProperty(AUDITOR_AWARE);
-		}
 
 		builder.addPropertyValue(SET_DATES, configuration.isSetDates());
 		builder.addPropertyValue(MODIFY_ON_CREATE, configuration.isModifyOnCreate());
 
 		if (StringUtils.hasText(configuration.getDateTimeProviderRef())) {
 			builder.addPropertyReference(DATE_TIME_PROVIDER, configuration.getDateTimeProviderRef());
 		}
-		else {
-			builder.addAutowiredProperty(DATE_TIME_PROVIDER);
-		}
 
 		builder.setRole(AbstractBeanDefinition.ROLE_INFRASTRUCTURE);
 

src/test/java/org/springframework/data/auditing/config/AuditingBeanDefinitionRegistrarSupportUnitTests.java

@@ -20,23 +20,19 @@
 import static org.mockito.Mockito.*;
 
 import java.lang.annotation.Annotation;
-import java.util.Optional;
 
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
 
-import org.springframework.beans.factory.config.AutowiredPropertyMarker;
 import org.springframework.beans.factory.config.BeanDefinition;
 import org.springframework.beans.factory.support.AbstractBeanDefinition;
 import org.springframework.beans.factory.support.BeanDefinitionBuilder;
 import org.springframework.beans.factory.support.BeanDefinitionRegistry;
-import org.springframework.context.support.GenericApplicationContext;
 import org.springframework.core.type.AnnotationMetadata;
 import org.springframework.data.auditing.AuditingHandler;
 import org.springframework.data.auditing.EnableAuditing;
-import org.springframework.data.mapping.context.PersistentEntities;
 
 /**
  * Unit tests for {@link AuditingBeanDefinitionRegistrarSupport}.
@@ -46,7 +42,6 @@
  * @author Oliver Gierke
  * @author Francisco Soler
  * @author Jaeyeon Kim
- * @author Mark Paluch
  */
 @ExtendWith(MockitoExtension.class)
 class AuditingBeanDefinitionRegistrarSupportUnitTests {
@@ -82,28 +77,13 @@ void rejectsNullRegistry() {
 				.isThrownBy(() -> registrar.registerBeanDefinitions(metadata, null));
 	}
 
-	@Test // GH-3177
+	@Test // DATACMNS-3177
 	void setsAuditorAwareAndDateTimeProviderIfConfigured() {
 
-		AuditingConfiguration configuration = new DummyAuditingBeanDefinitionRegistrarSupport().getConfiguration(null);
-
-		BeanDefinitionBuilder builder = BeanDefinitionBuilder.rootBeanDefinition(AuditingHandler.class);
-		DummyAuditingBeanDefinitionRegistrarSupport registrar = new DummyAuditingBeanDefinitionRegistrarSupport();
-
-		BeanDefinitionBuilder result = registrar.configureDefaultAuditHandlerAttributes(configuration, builder);
-		AbstractBeanDefinition beanDefinition = result.getBeanDefinition();
-
-		assertThat(beanDefinition.getPropertyValues().contains("auditorAware")).isTrue();
-		assertThat(beanDefinition.getPropertyValues().contains("dateTimeProvider")).isTrue();
-	}
-
-	@Test // GH-3177
-	void doesNotSetAuditorAwareAndDateTimeProviderIfNotConfigured() {
-
 		AuditingConfiguration configuration = new AuditingConfiguration() {
 			@Override
 			public String getAuditorAwareRef() {
-				return "";
+				return "auditorAwareBean";
 			}
 
 			@Override
@@ -113,7 +93,7 @@ public boolean isSetDates() {
 
 			@Override
 			public String getDateTimeProviderRef() {
-				return "";
+				return "dateTimeProviderBean";
 			}
 
 			@Override
@@ -128,12 +108,13 @@ public boolean isModifyOnCreate() {
 		BeanDefinitionBuilder result = registrar.configureDefaultAuditHandlerAttributes(configuration, builder);
 		AbstractBeanDefinition beanDefinition = result.getBeanDefinition();
 
-		assertThat(beanDefinition.getPropertyValues().get("auditorAware")).isEqualTo(AutowiredPropertyMarker.INSTANCE);
-		assertThat(beanDefinition.getPropertyValues().get("dateTimeProvider")).isEqualTo(AutowiredPropertyMarker.INSTANCE);
+		assertThat(beanDefinition.getAutowireMode()).isEqualTo(AbstractBeanDefinition.AUTOWIRE_BY_TYPE);
+		assertThat(beanDefinition.getPropertyValues().contains("auditorAware")).isTrue();
+		assertThat(beanDefinition.getPropertyValues().contains("dateTimeProvider")).isTrue();
 	}
 
-	@Test // GH-3177
-	void optionalAutowiringShouldConsiderOptionalProperties() {
+	@Test // DATACMNS-3177
+	void doesNotSetAuditorAwareAndDateTimeProviderIfNotConfigured() {
 
 		AuditingConfiguration configuration = new AuditingConfiguration() {
 			@Override
@@ -152,24 +133,18 @@ public String getDateTimeProviderRef() {
 			}
 
 			@Override
-			public boolean isModifyOnCreate() {
-				return true;
-			}
+			public boolean isModifyOnCreate() { return true; }
 		};
 
 		BeanDefinitionBuilder builder = BeanDefinitionBuilder.rootBeanDefinition(AuditingHandler.class);
-		builder.addConstructorArgValue(PersistentEntities.of());
 		DummyAuditingBeanDefinitionRegistrarSupport registrar = new DummyAuditingBeanDefinitionRegistrarSupport();
 
 		BeanDefinitionBuilder result = registrar.configureDefaultAuditHandlerAttributes(configuration, builder);
+		AbstractBeanDefinition beanDefinition = result.getBeanDefinition();
 
-		GenericApplicationContext context = new GenericApplicationContext();
-		context.registerBeanDefinition("auditingHandler", result.getBeanDefinition());
-		context.refresh();
-
-		AuditingHandler handler = context.getBean(AuditingHandler.class);
-
-		assertThat(handler).extracting("auditorAware").isEqualTo(Optional.empty());
+		assertThat(beanDefinition.getAutowireMode()).isEqualTo(AbstractBeanDefinition.AUTOWIRE_BY_TYPE);
+		assertThat(beanDefinition.getPropertyValues().contains("auditorAware")).isFalse();
+		assertThat(beanDefinition.getPropertyValues().contains("dateTimeProvider")).isFalse();
 	}
 
 	static class SampleConfig {}