Initialize auditing metadata for known persistent entities.

We're warming the cache for known entities.

Closes: #3441
Original Pull Request: #3442

Author: mp911de

src/main/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactory.java

@@ -71,6 +71,9 @@ public MappingAuditableBeanWrapperFactory(PersistentEntities entities) {
 
 		this.entities = entities;
 		this.metadataCache = new ConcurrentHashMap<>();
+
+		entities
+				.forEach(it -> entities.mapOnContext(it.getType(), (context, entity) -> getMetadata(context, it.getType())));
 	}
 
 	@Override
@@ -82,20 +85,26 @@ public <T> Optional<AuditableBeanWrapper<T>> getBeanWrapperFor(T source) {
 				return super.getBeanWrapperFor(source);
 			}
 
-			return entities.mapOnContext(it.getClass(), (context, entity) -> {
+			Class<?> entityClass = it.getClass();
+			return entities.mapOnContext(entityClass, (context, entity) -> {
 
-				MappingAuditingMetadata metadata = metadataCache.computeIfAbsent(it.getClass(),
-						key -> new MappingAuditingMetadata(context, it.getClass()));
+				MappingAuditingMetadata metadata = getMetadata(context, entityClass);
 
-				return Optional.<AuditableBeanWrapper<T>> ofNullable(metadata.isAuditable() //
-						? new MappingMetadataAuditableBeanWrapper<>(getConversionService(), entity.getPropertyPathAccessor(it),
-								metadata)
-						: null);
+				if (metadata.isAuditable()) {
+					return Optional.<AuditableBeanWrapper<T>> of(new MappingMetadataAuditableBeanWrapper<>(getConversionService(),
+							entity.getPropertyPathAccessor(it), metadata));
+				}
 
+				return Optional.<AuditableBeanWrapper<T>> empty();
 			}).orElseGet(() -> super.getBeanWrapperFor(source));
 		});
 	}
 
+	private MappingAuditingMetadata getMetadata(MappingContext<?, ? extends PersistentProperty<?>> context,
+			Class<?> entityClass) {
+		return metadataCache.computeIfAbsent(entityClass, key -> new MappingAuditingMetadata(context, entityClass));
+	}
+
 	/**
 	 * Captures {@link PersistentProperty} instances equipped with auditing annotations.
 	 *

src/test/java/org/springframework/data/auditing/IsNewAwareAuditingHandlerUnitTests.java

@@ -20,12 +20,12 @@
 
 import java.util.Optional;
 
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.junit.jupiter.MockitoExtension;
 import org.mockito.junit.jupiter.MockitoSettings;
 import org.mockito.quality.Strictness;
+
 import org.springframework.data.annotation.Id;
 import org.springframework.data.mapping.context.PersistentEntities;
 import org.springframework.data.mapping.context.SampleMappingContext;
@@ -34,24 +34,18 @@
  * Unit test for {@code AuditingHandler}.
  *
  * @author Oliver Gierke
+ * @author Mark Paluch
  * @since 1.5
  */
 @ExtendWith(MockitoExtension.class)
 @MockitoSettings(strictness = Strictness.LENIENT)
 class IsNewAwareAuditingHandlerUnitTests extends AuditingHandlerUnitTests {
 
-	SampleMappingContext mappingContext;
-
-	@BeforeEach
-	void init() {
-
-		this.mappingContext = new SampleMappingContext();
-		this.mappingContext.getPersistentEntity(AuditedUser.class);
-		this.mappingContext.afterPropertiesSet();
-	}
-
 	@Override
 	protected IsNewAwareAuditingHandler getHandler() {
+		SampleMappingContext mappingContext = new SampleMappingContext();
+		mappingContext.getPersistentEntity(AuditedUser.class);
+		mappingContext.afterPropertiesSet();
 		return new IsNewAwareAuditingHandler(PersistentEntities.of(mappingContext));
 	}
 

src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java

@@ -45,12 +45,14 @@
 import org.springframework.data.domain.Auditable;
 import org.springframework.data.mapping.context.PersistentEntities;
 import org.springframework.data.mapping.context.SampleMappingContext;
+import org.springframework.test.util.ReflectionTestUtils;
 
 /**
  * Unit tests for {@link MappingAuditableBeanWrapperFactory}.
  *
  * @author Oliver Gierke
  * @author Jens Schauder
+ * @author Mark Paluch
  * @since 1.8
  */
 class MappingAuditableBeanWrapperFactoryUnitTests {
@@ -69,6 +71,13 @@ void setUp() {
 		factory = new MappingAuditableBeanWrapperFactory(entities);
 	}
 
+	@Test // GH-3441
+	void cacheWarmedWithKnownPersistentEntities() {
+
+		Map<?, ?> metadataCache = (Map<?, ?>) ReflectionTestUtils.getField(factory, "metadataCache");
+		assertThat(metadataCache).hasSize(4);
+	}
+
 	@Test // DATACMNS-365
 	void discoversAuditingPropertyOnField() {