Document findBy(Pageable) for Firestore (#410)

This is a substitute for `findAll(Pageable)`. Related to #409.
GoogleCloudPlatform · Mar 25, 2021 · f33286c · f33286c
1 parent bb71964
commit f33286c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/...restore/src/test/java/com/google/cloud/spring/data/firestore/entities/UserRepository.java b/...restore/src/test/java/com/google/cloud/spring/data/firestore/entities/UserRepository.java
@@ -32,6 +32,8 @@
  */
 //tag::repository[]
 public interface UserRepository extends FirestoreReactiveRepository<User> {
+	Flux<User> findBy(Pageable pageable);
+
 	Flux<User> findByAge(Integer age);
 
 	Flux<User> findByAgeNot(Integer age);

diff --git a/...t/java/com/google/cloud/spring/data/firestore/it/FirestoreRepositoryIntegrationTests.java b/...t/java/com/google/cloud/spring/data/firestore/it/FirestoreRepositoryIntegrationTests.java
@@ -157,6 +157,7 @@ public void partTreeRepositoryMethodTest() {
 		this.userRepository.saveAll(users).blockLast();
 
 		assertThat(this.userRepository.count().block()).isEqualTo(2);
+		assertThat(this.userRepository.findBy(PageRequest.of(0, 10)).collectList().block()).containsExactly(u1, u2);
 		assertThat(this.userRepository.findByAge(22).collectList().block()).containsExactly(u1);
 		assertThat(this.userRepository.findByAgeNot(22).collectList().block()).containsExactly(u2);
 		assertThat(this.userRepository.findByHomeAddressCountry("USA").collectList().block()).containsExactly(u1);