From 9c20aff99dd8bb984d56bf0fa58a51d542548f8d Mon Sep 17 00:00:00 2001 From: Oliver Gierke Date: Thu, 26 Apr 2012 19:22:40 +0200 Subject: [PATCH] DATAJPA-201 - Test case showing handing in null for Pageable arguments works. --- .../data/jpa/repository/UserRepositoryTests.java | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java b/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java index 76317144f..5bca46c71 100644 --- a/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java +++ b/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java @@ -815,6 +815,22 @@ public class UserRepositoryTests { assertThat(result, hasItems(secondUser, thirdUser)); } + /** + * @see DATAJPA-201 + */ + public void allowsExecutingPageableMethodWithNullPageable() { + + flushTestUsers(); + + List users = repository.findByFirstname("Oliver", null); + assertThat(users.size(), is(1)); + assertThat(users, hasItem(firstUser)); + + Page page = repository.findByFirstnameIn(null, "Oliver"); + assertThat(page.getNumberOfElements(), is(1)); + assertThat(page.getContent(), hasItem(firstUser)); + } + protected void flushTestUsers() { firstUser = repository.save(firstUser);