opendev/gerrit
Code Issues Proposed changes

AbstractQueryAccountsTest: Simplify some assertions

Browse Source 
Change-Id: Ie09af76192139a93818ea34fa2ebf17e2ebc95df
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2016-10-27 15:50:59 +02:00
parent e4e18f6d2b
commit 1677806af5
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-server/src/test/java/com/google/gerrit/server/query/account/AbstractQueryAccountsTest.java
View File

@@ -19,6 +19,7 @@ import static java.util.stream.Collectors.toList;
import static org.junit.Assert.fail;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Iterables;
import com.google.gerrit.extensions.api.GerritApi;
import com.google.gerrit.extensions.api.accounts.Accounts.QueryRequest;
import com.google.gerrit.extensions.client.ListAccountsOption;
@@ -281,10 +282,10 @@ public abstract class AbstractQueryAccountsTest extends GerritServerTests {
AccountInfo user3 = newAccountWithEmail("user3", "user3@" + domain);
List<AccountInfo> result = assertQuery(domain, user1, user2, user3);
assertThat(result.get(result.size() - 1)._moreAccounts).isNull();
assertThat(Iterables.getLast(result)._moreAccounts).isNull();
result = assertQuery(newQuery(domain).withLimit(2), user1, user2);
assertThat(result.get(result.size() - 1)._moreAccounts).isTrue();
assertThat(Iterables.getLast(result)._moreAccounts).isTrue();
}
@Test