PpagingAndSortingRepository split, builds cleanly.

Author: schauder

src/main/java/org/springframework/data/repository/PagingAndSortingRepository.java

@@ -29,7 +29,7 @@
  * @see Page
  */
 @NoRepositoryBean
-public interface PagingAndSortingRepository<T, ID> extends CrudRepository<T, ID> {
+public interface PagingAndSortingRepository<T, ID> extends Repository<T, ID> {
 
 	/**
 	 * Returns all entities sorted by the given options.

src/main/java/org/springframework/data/repository/support/DefaultRepositoryInvokerFactory.java

@@ -93,10 +93,7 @@ private RepositoryInvoker prepareInvokers(Class<?> domainType) {
 	@SuppressWarnings("unchecked")
 	protected RepositoryInvoker createInvoker(RepositoryInformation information, Object repository) {
 
-		if (repository instanceof PagingAndSortingRepository) {
-			return new PagingAndSortingRepositoryInvoker((PagingAndSortingRepository<Object, Object>) repository, information,
-					conversionService);
-		} else if (repository instanceof CrudRepository) {
+		if (repository instanceof CrudRepository) {
 			return new CrudRepositoryInvoker((CrudRepository<Object, Object>) repository, information, conversionService);
 		} else {
 			return new ReflectionRepositoryInvoker(repository, information, conversionService);

src/main/java/org/springframework/data/repository/support/PagingAndSortingRepositoryInvoker.java

@@ -27,6 +27,7 @@
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.querydsl.User;
+import org.springframework.data.repository.CrudRepository;
 import org.springframework.data.repository.PagingAndSortingRepository;
 import org.springframework.data.repository.Repository;
 import org.springframework.data.repository.core.RepositoryMetadata;
@@ -95,7 +96,7 @@ void handlesGenericTypeInReturnedCollectionCorrectly() throws SecurityException,
 	@Test // DATACMNS-471
 	void detectsArrayReturnTypeCorrectly() throws Exception {
 
-		RepositoryMetadata metadata = new DefaultRepositoryMetadata(PagedRepository.class);
+		RepositoryMetadata metadata = new DefaultRepositoryMetadata(CompletePageableAndSortingRepository.class);
 		var method = PagedRepository.class.getMethod("returnsArray");
 
 		assertThat(metadata.getReturnedDomainClass(method)).isEqualTo(User.class);
@@ -169,4 +170,8 @@ abstract class Container implements Iterable<Element> {}
 	interface ContainerRepository extends Repository<Container, Long> {
 		Container someMethod();
 	}
+
+	interface CompletePageableAndSortingRepository extends PagingAndSortingRepository<Container, Long> {
+	}
+
 }

src/test/java/org/springframework/data/repository/core/support/AbstractRepositoryMetadataUnitTests.java

@@ -197,7 +197,7 @@ interface Domain {}
 
 	interface DomainCrudRepository extends CrudRepository<Domain, Long> {}
 
-	interface DomainPagingAndSortingRepository extends PagingAndSortingRepository<Domain, Long> {}
+	interface DomainPagingAndSortingRepository extends PagingAndSortingRepository<Domain, Long>, CrudRepository<Domain, Long> {}
 
 	interface RepositoryWithCustomSave extends Repository<Domain, Serializable> {
 

src/test/java/org/springframework/data/repository/core/support/DefaultCrudMethodsUnitTests.java

@@ -111,27 +111,11 @@ void considersIntermediateMethodsAsFinderMethods() {
 		assertThat(information.hasCustomMethod()).isFalse();
 	}
 
-	@Test
-	void discoversIntermediateMethodsAsBackingMethods() throws NoSuchMethodException, SecurityException {
-
-		var metadata = new DefaultRepositoryMetadata(CustomRepository.class);
-		var information = new DefaultRepositoryInformation(metadata,
-				PagingAndSortingRepository.class, RepositoryComposition.empty());
-
-		var method = CustomRepository.class.getMethod("findAll", Pageable.class);
-		assertThat(information.isBaseClassMethod(method)).isTrue();
-
-		method = getMethodFrom(CustomRepository.class, "existsById");
-
-		assertThat(information.isBaseClassMethod(method)).isTrue();
-		assertThat(information.getQueryMethods()).isEmpty();
-	}
-
 	@Test // DATACMNS-151
 	void doesNotConsiderManuallyDefinedSaveMethodAQueryMethod() {
 
 		RepositoryMetadata metadata = new DefaultRepositoryMetadata(CustomRepository.class);
-		RepositoryInformation information = new DefaultRepositoryInformation(metadata, PagingAndSortingRepository.class,
+		RepositoryInformation information = new DefaultRepositoryInformation(metadata, CompletePageableAndSortingRepository.class,
 				RepositoryComposition.empty());
 
 		assertThat(information.getQueryMethods()).isEmpty();
@@ -427,4 +411,8 @@ public Sample save(Sample entity) {
 			return entity;
 		}
 	}
+
+	interface CompletePageableAndSortingRepository<T, ID> extends CrudRepository<T, ID>, PagingAndSortingRepository<T, ID> {
+	}
+
 }

src/test/java/org/springframework/data/repository/core/support/DefaultRepositoryInformationUnitTests.java

@@ -51,6 +51,7 @@
 import org.springframework.data.domain.Sort;
 import org.springframework.data.projection.ProjectionFactory;
 import org.springframework.data.querydsl.QuerydslPredicateExecutor;
+import org.springframework.data.repository.CrudRepository;
 import org.springframework.data.repository.PagingAndSortingRepository;
 import org.springframework.data.repository.Repository;
 import org.springframework.data.repository.RepositoryDefinition;
@@ -90,7 +91,7 @@ class RepositoryFactorySupportUnitTests {
 
 	DummyRepositoryFactory factory;
 
-	@Mock PagingAndSortingRepository<Object, Object> backingRepo;
+	@Mock CrudRepository<Object, Object> backingRepo;
 	@Mock ObjectRepositoryCustom customImplementation;
 
 	@Mock MyQueryCreationListener listener;
@@ -167,12 +168,11 @@ void invokesCustomMethodCompositionMethodIfItRedeclaresACRUDOne() {
 	void createsRepositoryInstanceWithCustomIntermediateRepository() {
 
 		var repository = factory.getRepository(CustomRepository.class);
-		Pageable pageable = PageRequest.of(0, 10);
 
-		when(backingRepo.findAll(pageable)).thenReturn(new PageImpl<>(Collections.emptyList()));
-		repository.findAll(pageable);
+		when(backingRepo.findAll()).thenReturn(new PageImpl<>(Collections.emptyList()));
+		repository.findAll();
 
-		verify(backingRepo, times(1)).findAll(pageable);
+		verify(backingRepo, times(1)).findAll();
 	}
 
 	@Test

src/test/java/org/springframework/data/repository/core/support/RepositoryFactorySupportUnitTests.java

@@ -153,7 +153,7 @@ interface OrderRepository extends CrudRepository<Order, Long> {
 
 	static class Person {}
 
-	interface PersonRepository extends PagingAndSortingRepository<Person, Long> {
+	interface PersonRepository extends PagingAndSortingRepository<Person, Long>, CrudRepository<Person, Long> {
 
 		Page<Person> findByFirstName(@Param("firstName") String firstName, Pageable pageable);
 

src/test/java/org/springframework/data/repository/support/CrudRepositoryInvokerUnitTests.java

@@ -93,7 +93,6 @@ void createsCrudRepositoryInvokerForRepositoryExtendingCrudRepository() {
 		var invoker = factory.getInvokerFor(User.class);
 
 		assertThat(invoker)//
-				.isInstanceOf(CrudRepositoryInvoker.class)//
-				.isNotInstanceOf(PagingAndSortingRepositoryInvoker.class);
+				.isInstanceOf(CrudRepositoryInvoker.class);
 	}
 }