From ab4b1c4e6114cbcadf17c9079e5a70dc75efe144 Mon Sep 17 00:00:00 2001 From: Gerrit Meier Date: Tue, 24 Nov 2020 12:19:02 +0100 Subject: [PATCH 1/2] Detect Persistent and RelationshipProperties with Neo4j See gh-24239 --- .../neo4j/Neo4jDataAutoConfiguration.java | 5 ++- .../Neo4jDataAutoConfigurationTests.java | 23 +++++++++++++ ...iveRepositoriesAutoConfigurationTests.java | 22 +++++++++++++ .../data/neo4j/scan/AnnotatedWithNode.java | 33 +++++++++++++++++++ .../neo4j/scan/AnnotatedWithPersistent.java | 33 +++++++++++++++++++ .../AnnotatedWithRelationshipProperties.java | 27 +++++++++++++++ .../data/neo4j/scan/NotAnnotatedEntity.java | 31 +++++++++++++++++ 7 files changed, 173 insertions(+), 1 deletion(-) create mode 100644 spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java create mode 100644 spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java create mode 100644 spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java create mode 100644 spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java diff --git a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfiguration.java b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfiguration.java index 204b3962000b..965f06e84a5a 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfiguration.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfiguration.java @@ -35,6 +35,7 @@ import org.springframework.context.ApplicationContext; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; +import org.springframework.data.annotation.Persistent; import org.springframework.data.neo4j.core.DatabaseSelectionProvider; import org.springframework.data.neo4j.core.Neo4jClient; import org.springframework.data.neo4j.core.Neo4jOperations; @@ -42,6 +43,7 @@ import org.springframework.data.neo4j.core.convert.Neo4jConversions; import org.springframework.data.neo4j.core.mapping.Neo4jMappingContext; import org.springframework.data.neo4j.core.schema.Node; +import org.springframework.data.neo4j.core.schema.RelationshipProperties; import org.springframework.data.neo4j.core.transaction.Neo4jTransactionManager; import org.springframework.data.neo4j.repository.config.Neo4jRepositoryConfigurationExtension; import org.springframework.transaction.PlatformTransactionManager; @@ -76,7 +78,8 @@ public Neo4jConversions neo4jConversions() { @ConditionalOnMissingBean public Neo4jMappingContext neo4jMappingContext(ApplicationContext applicationContext, Neo4jConversions neo4jConversions) throws ClassNotFoundException { - Set> initialEntityClasses = new EntityScanner(applicationContext).scan(Node.class); + Set> initialEntityClasses = new EntityScanner(applicationContext).scan(Node.class, Persistent.class, + RelationshipProperties.class); Neo4jMappingContext context = new Neo4jMappingContext(neo4jConversions); context.setInitialEntitySet(initialEntityClasses); return context; diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java index 5206a3ca5ada..9b145f0cc864 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java @@ -19,6 +19,11 @@ import org.junit.jupiter.api.Test; import org.springframework.boot.autoconfigure.AutoConfigurations; +import org.springframework.boot.autoconfigure.TestAutoConfigurationPackage; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithNode; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithPersistent; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithRelationshipProperties; +import org.springframework.boot.autoconfigure.data.neo4j.scan.NotAnnotatedEntity; import org.springframework.boot.autoconfigure.neo4j.Neo4jAutoConfiguration; import org.springframework.boot.test.context.runner.ApplicationContextRunner; import org.springframework.context.annotation.Bean; @@ -29,6 +34,7 @@ import org.springframework.data.neo4j.core.Neo4jOperations; 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.transaction.PlatformTransactionManager; import org.springframework.transaction.ReactiveTransactionManager; @@ -137,6 +143,17 @@ void shouldReuseExistingTransactionManager() { .hasBean("myCustomTransactionManager")); } + @Test + void shouldFilterInitialEntityScanWithKnownAnnotations() { + this.contextRunner.withUserConfiguration(PackageConfig.class).run((context) -> { + Neo4jMappingContext mappingContext = context.getBean(Neo4jMappingContext.class); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithNode.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithPersistent.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithRelationshipProperties.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(NotAnnotatedEntity.class)).isFalse(); + }); + } + @Configuration(proxyBeanMethods = false) static class CustomDatabaseSelectionProviderConfiguration { @@ -147,4 +164,10 @@ DatabaseSelectionProvider databaseSelectionProvider() { } + @Configuration(proxyBeanMethods = false) + @TestAutoConfigurationPackage(AnnotatedWithPersistent.class) + static class PackageConfig { + + } + } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java index 66afb613f5e3..2895ff0288c6 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java @@ -26,9 +26,14 @@ import org.springframework.boot.autoconfigure.data.neo4j.city.ReactiveCityRepository; import org.springframework.boot.autoconfigure.data.neo4j.country.CountryRepository; import org.springframework.boot.autoconfigure.data.neo4j.country.ReactiveCountryRepository; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithNode; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithPersistent; +import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithRelationshipProperties; +import org.springframework.boot.autoconfigure.data.neo4j.scan.NotAnnotatedEntity; import org.springframework.boot.test.context.runner.ApplicationContextRunner; import org.springframework.context.annotation.Configuration; import org.springframework.data.neo4j.core.ReactiveNeo4jTemplate; +import org.springframework.data.neo4j.core.mapping.Neo4jMappingContext; import org.springframework.data.neo4j.repository.ReactiveNeo4jRepository; import org.springframework.data.neo4j.repository.config.EnableReactiveNeo4jRepositories; @@ -84,6 +89,17 @@ void shouldRespectAtEnableReactiveNeo4jRepositories() { .hasSingleBean(ReactiveCountryRepository.class)); } + @Test + void shouldFilterInitialEntityScanWithKnownAnnotations() { + this.contextRunner.withUserConfiguration(PackageConfig.class).run((context) -> { + Neo4jMappingContext mappingContext = context.getBean(Neo4jMappingContext.class); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithNode.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithPersistent.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithRelationshipProperties.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(NotAnnotatedEntity.class)).isFalse(); + }); + } + @Configuration(proxyBeanMethods = false) @TestAutoConfigurationPackage(City.class) static class TestConfiguration { @@ -109,4 +125,10 @@ static class WithCustomReactiveRepositoryScan { } + @Configuration(proxyBeanMethods = false) + @TestAutoConfigurationPackage(AnnotatedWithPersistent.class) + static class PackageConfig { + + } + } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java new file mode 100644 index 000000000000..9c44e5479615 --- /dev/null +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java @@ -0,0 +1,33 @@ +/* + * Copyright 2012-2020 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. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.autoconfigure.data.neo4j.scan; + +import org.springframework.data.neo4j.core.schema.GeneratedValue; +import org.springframework.data.neo4j.core.schema.Id; +import org.springframework.data.neo4j.core.schema.Node; + +/** + * @author Gerrit Meier + */ +@Node +public class AnnotatedWithNode { + + @Id + @GeneratedValue + private Long id; + +} diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java new file mode 100644 index 000000000000..f987df09ab65 --- /dev/null +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java @@ -0,0 +1,33 @@ +/* + * Copyright 2012-2020 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. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.autoconfigure.data.neo4j.scan; + +import org.springframework.data.annotation.Persistent; +import org.springframework.data.neo4j.core.schema.GeneratedValue; +import org.springframework.data.neo4j.core.schema.Id; + +/** + * @author Gerrit Meier + */ +@Persistent +public class AnnotatedWithPersistent { + + @Id + @GeneratedValue + private Long id; + +} diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java new file mode 100644 index 000000000000..062e5bf15a19 --- /dev/null +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java @@ -0,0 +1,27 @@ +/* + * Copyright 2012-2020 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. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.autoconfigure.data.neo4j.scan; + +import org.springframework.data.neo4j.core.schema.RelationshipProperties; + +/** + * @author Gerrit Meier + */ +@RelationshipProperties +public class AnnotatedWithRelationshipProperties { + +} diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java new file mode 100644 index 000000000000..dd78d4cfe129 --- /dev/null +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java @@ -0,0 +1,31 @@ +/* + * Copyright 2012-2020 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. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.autoconfigure.data.neo4j.scan; + +import org.springframework.data.neo4j.core.schema.GeneratedValue; +import org.springframework.data.neo4j.core.schema.Id; + +/** + * @author Gerrit Meier + */ +public class NotAnnotatedEntity { + + @Id + @GeneratedValue + private Long id; + +} From 0691ba600f3509f30a55b589de29e486b7e70dba Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Tue, 22 Dec 2020 12:30:48 +0100 Subject: [PATCH 2/2] Polish "Detect Persistent and RelationshipProperties with Neo4j" See gh-24239 --- .../Neo4jDataAutoConfigurationTests.java | 22 +++++++++--------- ...o4jReactiveDataAutoConfigurationTests.java | 23 +++++++++++++++++++ ...iveRepositoriesAutoConfigurationTests.java | 22 ------------------ .../{AnnotatedWithNode.java => TestNode.java} | 5 +--- ...tatedEntity.java => TestNonAnnotated.java} | 5 +--- ...ithPersistent.java => TestPersistent.java} | 5 +--- ...s.java => TestRelationshipProperties.java} | 5 +--- 7 files changed, 38 insertions(+), 49 deletions(-) rename spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/{AnnotatedWithNode.java => TestNode.java} (93%) rename spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/{NotAnnotatedEntity.java => TestNonAnnotated.java} (92%) rename spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/{AnnotatedWithPersistent.java => TestPersistent.java} (92%) rename spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/{AnnotatedWithRelationshipProperties.java => TestRelationshipProperties.java} (90%) diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java index 9b145f0cc864..8fd1314d2b6e 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jDataAutoConfigurationTests.java @@ -20,10 +20,10 @@ import org.springframework.boot.autoconfigure.AutoConfigurations; import org.springframework.boot.autoconfigure.TestAutoConfigurationPackage; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithNode; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithPersistent; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithRelationshipProperties; -import org.springframework.boot.autoconfigure.data.neo4j.scan.NotAnnotatedEntity; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestNode; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestNonAnnotated; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestPersistent; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestRelationshipProperties; import org.springframework.boot.autoconfigure.neo4j.Neo4jAutoConfiguration; import org.springframework.boot.test.context.runner.ApplicationContextRunner; import org.springframework.context.annotation.Bean; @@ -145,12 +145,12 @@ void shouldReuseExistingTransactionManager() { @Test void shouldFilterInitialEntityScanWithKnownAnnotations() { - this.contextRunner.withUserConfiguration(PackageConfig.class).run((context) -> { + this.contextRunner.withUserConfiguration(EntityScanConfig.class).run((context) -> { Neo4jMappingContext mappingContext = context.getBean(Neo4jMappingContext.class); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithNode.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithPersistent.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithRelationshipProperties.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(NotAnnotatedEntity.class)).isFalse(); + assertThat(mappingContext.hasPersistentEntityFor(TestNode.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestPersistent.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestRelationshipProperties.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestNonAnnotated.class)).isFalse(); }); } @@ -165,8 +165,8 @@ DatabaseSelectionProvider databaseSelectionProvider() { } @Configuration(proxyBeanMethods = false) - @TestAutoConfigurationPackage(AnnotatedWithPersistent.class) - static class PackageConfig { + @TestAutoConfigurationPackage(TestPersistent.class) + static class EntityScanConfig { } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveDataAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveDataAutoConfigurationTests.java index 69096db96252..a107a8128118 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveDataAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveDataAutoConfigurationTests.java @@ -21,6 +21,11 @@ import reactor.test.StepVerifier; import org.springframework.boot.autoconfigure.AutoConfigurations; +import org.springframework.boot.autoconfigure.TestAutoConfigurationPackage; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestNode; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestNonAnnotated; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestPersistent; +import org.springframework.boot.autoconfigure.data.neo4j.scan.TestRelationshipProperties; import org.springframework.boot.autoconfigure.neo4j.Neo4jAutoConfiguration; import org.springframework.boot.test.context.runner.ApplicationContextRunner; import org.springframework.context.annotation.Bean; @@ -30,6 +35,7 @@ import org.springframework.data.neo4j.core.ReactiveNeo4jClient; import org.springframework.data.neo4j.core.ReactiveNeo4jOperations; import org.springframework.data.neo4j.core.ReactiveNeo4jTemplate; +import org.springframework.data.neo4j.core.mapping.Neo4jMappingContext; import org.springframework.transaction.ReactiveTransactionManager; import org.springframework.transaction.TransactionManager; @@ -121,6 +127,17 @@ void shouldUseExistingReactiveTransactionManager() { .hasSingleBean(TransactionManager.class)); } + @Test + void shouldFilterInitialEntityScanWithKnownAnnotations() { + this.contextRunner.withUserConfiguration(EntityScanConfig.class).run((context) -> { + Neo4jMappingContext mappingContext = context.getBean(Neo4jMappingContext.class); + assertThat(mappingContext.hasPersistentEntityFor(TestNode.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestPersistent.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestRelationshipProperties.class)).isTrue(); + assertThat(mappingContext.hasPersistentEntityFor(TestNonAnnotated.class)).isFalse(); + }); + } + @Configuration(proxyBeanMethods = false) static class CustomReactiveDatabaseSelectionProviderConfiguration { @@ -131,4 +148,10 @@ ReactiveDatabaseSelectionProvider databaseNameProvider() { } + @Configuration(proxyBeanMethods = false) + @TestAutoConfigurationPackage(TestPersistent.class) + static class EntityScanConfig { + + } + } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java index 2895ff0288c6..66afb613f5e3 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/Neo4jReactiveRepositoriesAutoConfigurationTests.java @@ -26,14 +26,9 @@ import org.springframework.boot.autoconfigure.data.neo4j.city.ReactiveCityRepository; import org.springframework.boot.autoconfigure.data.neo4j.country.CountryRepository; import org.springframework.boot.autoconfigure.data.neo4j.country.ReactiveCountryRepository; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithNode; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithPersistent; -import org.springframework.boot.autoconfigure.data.neo4j.scan.AnnotatedWithRelationshipProperties; -import org.springframework.boot.autoconfigure.data.neo4j.scan.NotAnnotatedEntity; import org.springframework.boot.test.context.runner.ApplicationContextRunner; import org.springframework.context.annotation.Configuration; import org.springframework.data.neo4j.core.ReactiveNeo4jTemplate; -import org.springframework.data.neo4j.core.mapping.Neo4jMappingContext; import org.springframework.data.neo4j.repository.ReactiveNeo4jRepository; import org.springframework.data.neo4j.repository.config.EnableReactiveNeo4jRepositories; @@ -89,17 +84,6 @@ void shouldRespectAtEnableReactiveNeo4jRepositories() { .hasSingleBean(ReactiveCountryRepository.class)); } - @Test - void shouldFilterInitialEntityScanWithKnownAnnotations() { - this.contextRunner.withUserConfiguration(PackageConfig.class).run((context) -> { - Neo4jMappingContext mappingContext = context.getBean(Neo4jMappingContext.class); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithNode.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithPersistent.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(AnnotatedWithRelationshipProperties.class)).isTrue(); - assertThat(mappingContext.hasPersistentEntityFor(NotAnnotatedEntity.class)).isFalse(); - }); - } - @Configuration(proxyBeanMethods = false) @TestAutoConfigurationPackage(City.class) static class TestConfiguration { @@ -125,10 +109,4 @@ static class WithCustomReactiveRepositoryScan { } - @Configuration(proxyBeanMethods = false) - @TestAutoConfigurationPackage(AnnotatedWithPersistent.class) - static class PackageConfig { - - } - } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNode.java similarity index 93% rename from spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java rename to spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNode.java index 9c44e5479615..50293f98b833 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithNode.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNode.java @@ -20,11 +20,8 @@ import org.springframework.data.neo4j.core.schema.Id; import org.springframework.data.neo4j.core.schema.Node; -/** - * @author Gerrit Meier - */ @Node -public class AnnotatedWithNode { +public class TestNode { @Id @GeneratedValue diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNonAnnotated.java similarity index 92% rename from spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java rename to spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNonAnnotated.java index dd78d4cfe129..c754c3877503 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/NotAnnotatedEntity.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestNonAnnotated.java @@ -19,10 +19,7 @@ import org.springframework.data.neo4j.core.schema.GeneratedValue; import org.springframework.data.neo4j.core.schema.Id; -/** - * @author Gerrit Meier - */ -public class NotAnnotatedEntity { +public class TestNonAnnotated { @Id @GeneratedValue diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestPersistent.java similarity index 92% rename from spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java rename to spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestPersistent.java index f987df09ab65..54aee30b7b14 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithPersistent.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestPersistent.java @@ -20,11 +20,8 @@ import org.springframework.data.neo4j.core.schema.GeneratedValue; import org.springframework.data.neo4j.core.schema.Id; -/** - * @author Gerrit Meier - */ @Persistent -public class AnnotatedWithPersistent { +public class TestPersistent { @Id @GeneratedValue diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestRelationshipProperties.java similarity index 90% rename from spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java rename to spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestRelationshipProperties.java index 062e5bf15a19..dbd848694283 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/AnnotatedWithRelationshipProperties.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/data/neo4j/scan/TestRelationshipProperties.java @@ -18,10 +18,7 @@ import org.springframework.data.neo4j.core.schema.RelationshipProperties; -/** - * @author Gerrit Meier - */ @RelationshipProperties -public class AnnotatedWithRelationshipProperties { +public class TestRelationshipProperties { }