From 8f6bb178b6dd5917a6c2072158c7225b158fc881 Mon Sep 17 00:00:00 2001 From: Dave Borowitz Date: Wed, 16 Oct 2013 12:46:12 -0700 Subject: [PATCH] Add query tests for filtering/restarting paginated queries Change-Id: Ifd8950cb3a2ed72ef19d611f5e5d51f2a03ce097 --- .../change/AbstractQueryChangesTest.java | 28 +++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java b/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java index 692d845c2a..1e8b87a611 100644 --- a/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java +++ b/gerrit-server/src/test/java/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java @@ -540,6 +540,34 @@ public abstract class AbstractQueryChangesTest { assertResultEquals(change1, results.get(1)); } + @Test + public void filterOutMoreThanOnePageOfResults() throws Exception { + TestRepository repo = createProject("repo"); + Change change = newChange(repo, null, null, userId.get(), null).insert(); + int user2 = accountManager.authenticate(AuthRequest.forUser("anotheruser")) + .getAccountId().get(); + for (int i = 0; i < 5; i++) { + newChange(repo, null, null, user2, null).insert(); + } + + assertResultEquals(change, queryOne("status:new ownerin:Administrators")); + assertResultEquals(change, + queryOne("status:new ownerin:Administrators limit:2")); + } + + @Test + public void filterOutAllResults() throws Exception { + TestRepository repo = createProject("repo"); + int user2 = accountManager.authenticate(AuthRequest.forUser("anotheruser")) + .getAccountId().get(); + for (int i = 0; i < 5; i++) { + newChange(repo, null, null, user2, null).insert(); + } + + assertTrue(query("status:new ownerin:Administrators").isEmpty()); + assertTrue(query("status:new ownerin:Administrators limit:2").isEmpty()); + } + protected ChangeInserter newChange( TestRepository repo, @Nullable RevCommit commit, @Nullable String key, @Nullable Integer owner,