Polish "Use Neo4jManagedTypes to populate the mapping context"

See gh-37574
This commit is contained in:
Andy Wilkinson 2023-09-27 09:32:53 +01:00
parent e0a5de01ca
commit 6b107530f5
2 changed files with 26 additions and 2 deletions

View File

@ -1,5 +1,5 @@
/*
* Copyright 2012-2022 the original author or authors.
* Copyright 2012-2023 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@ -81,7 +81,6 @@ public class Neo4jDataAutoConfiguration {
@Bean
@ConditionalOnMissingBean
public Neo4jMappingContext neo4jMappingContext(Neo4jManagedTypes managedTypes, Neo4jConversions neo4jConversions) {
Neo4jMappingContext context = new Neo4jMappingContext(neo4jConversions);
context.setManagedTypes(managedTypes);
return context;

View File

@ -29,6 +29,7 @@ import org.springframework.boot.autoconfigure.neo4j.Neo4jAutoConfiguration;
import org.springframework.boot.test.context.runner.ApplicationContextRunner;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.data.neo4j.aot.Neo4jManagedTypes;
import org.springframework.data.neo4j.core.DatabaseSelection;
import org.springframework.data.neo4j.core.DatabaseSelectionProvider;
import org.springframework.data.neo4j.core.Neo4jClient;
@ -37,6 +38,7 @@ import org.springframework.data.neo4j.core.Neo4jTemplate;
import org.springframework.data.neo4j.core.convert.Neo4jConversions;
import org.springframework.data.neo4j.core.mapping.Neo4jMappingContext;
import org.springframework.data.neo4j.core.transaction.Neo4jTransactionManager;
import org.springframework.test.util.ReflectionTestUtils;
import org.springframework.transaction.PlatformTransactionManager;
import org.springframework.transaction.ReactiveTransactionManager;
import org.springframework.transaction.TransactionManager;
@ -162,6 +164,29 @@ class Neo4jDataAutoConfigurationTests {
});
}
@Test
void shouldProvideManagedTypes() {
this.contextRunner.run((context) -> {
assertThat(context).hasSingleBean(Neo4jManagedTypes.class);
assertThat(context.getBean(Neo4jMappingContext.class))
.extracting((mappingContext) -> ReflectionTestUtils.getField(mappingContext, "managedTypes"))
.isEqualTo(context.getBean(Neo4jManagedTypes.class));
});
}
@Test
void shouldReuseExistingManagedTypes() {
Neo4jManagedTypes managedTypes = Neo4jManagedTypes.from();
this.contextRunner.withBean("customManagedTypes", Neo4jManagedTypes.class, () -> managedTypes)
.run((context) -> {
assertThat(context).hasSingleBean(Neo4jManagedTypes.class);
assertThat(context).doesNotHaveBean("neo4jManagedTypes");
assertThat(context.getBean(Neo4jMappingContext.class))
.extracting((mappingContext) -> ReflectionTestUtils.getField(mappingContext, "managedTypes"))
.isSameAs(managedTypes);
});
}
@Configuration(proxyBeanMethods = false)
static class CustomDatabaseSelectionProviderConfiguration {