summaryrefslogtreecommitdiffstats
path: root/gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java
diff options
context:
space:
mode:
Diffstat (limited to 'gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java')
-rw-r--r--gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java262
1 files changed, 0 insertions, 262 deletions
diff --git a/gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java b/gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java
deleted file mode 100644
index 4a6663a95d..0000000000
--- a/gerrit-server/src/test/java/com/google/gerrit/server/index/change/ChangeIndexRewriterTest.java
+++ /dev/null
@@ -1,262 +0,0 @@
-// Copyright (C) 2013 The Android Open Source Project
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package com.google.gerrit.server.index.change;
-
-import static com.google.common.truth.Truth.assertThat;
-import static com.google.gerrit.common.data.GlobalCapability.DEFAULT_MAX_QUERY_LIMIT;
-import static com.google.gerrit.index.query.Predicate.and;
-import static com.google.gerrit.index.query.Predicate.or;
-import static com.google.gerrit.reviewdb.client.Change.Status.MERGED;
-import static com.google.gerrit.reviewdb.client.Change.Status.NEW;
-import static com.google.gerrit.server.index.change.IndexedChangeQuery.convertOptions;
-import static org.junit.Assert.assertEquals;
-
-import com.google.common.collect.ImmutableSet;
-import com.google.gerrit.index.IndexConfig;
-import com.google.gerrit.index.QueryOptions;
-import com.google.gerrit.index.query.Predicate;
-import com.google.gerrit.index.query.QueryParseException;
-import com.google.gerrit.reviewdb.client.Change;
-import com.google.gerrit.server.query.change.AndChangeSource;
-import com.google.gerrit.server.query.change.ChangeData;
-import com.google.gerrit.server.query.change.ChangeQueryBuilder;
-import com.google.gerrit.server.query.change.ChangeStatusPredicate;
-import com.google.gerrit.server.query.change.OrSource;
-import com.google.gerrit.testutil.GerritBaseTests;
-import java.util.Arrays;
-import java.util.EnumSet;
-import java.util.Set;
-import org.junit.Before;
-import org.junit.Test;
-
-public class ChangeIndexRewriterTest extends GerritBaseTests {
- private static final IndexConfig CONFIG = IndexConfig.createDefault();
-
- private FakeChangeIndex index;
- private ChangeIndexCollection indexes;
- private ChangeQueryBuilder queryBuilder;
- private ChangeIndexRewriter rewrite;
-
- @Before
- public void setUp() throws Exception {
- index = new FakeChangeIndex(FakeChangeIndex.V2);
- indexes = new ChangeIndexCollection();
- indexes.setSearchIndex(index);
- queryBuilder = new FakeQueryBuilder(indexes);
- rewrite = new ChangeIndexRewriter(indexes, IndexConfig.builder().maxTerms(3).build());
- }
-
- @Test
- public void indexPredicate() throws Exception {
- Predicate<ChangeData> in = parse("file:a");
- assertThat(rewrite(in)).isEqualTo(query(in));
- }
-
- @Test
- public void nonIndexPredicate() throws Exception {
- Predicate<ChangeData> in = parse("foo:a");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(AndChangeSource.class).isSameAs(out.getClass());
- assertThat(out.getChildren())
- .containsExactly(query(ChangeStatusPredicate.open()), in)
- .inOrder();
- }
-
- @Test
- public void indexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("file:a file:b");
- assertThat(rewrite(in)).isEqualTo(query(in));
- }
-
- @Test
- public void nonIndexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("foo:a OR foo:b");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(AndChangeSource.class).isSameAs(out.getClass());
- assertThat(out.getChildren())
- .containsExactly(query(ChangeStatusPredicate.open()), in)
- .inOrder();
- }
-
- @Test
- public void oneIndexPredicate() throws Exception {
- Predicate<ChangeData> in = parse("foo:a file:b");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(AndChangeSource.class).isSameAs(out.getClass());
- assertThat(out.getChildren()).containsExactly(query(in.getChild(1)), in.getChild(0)).inOrder();
- }
-
- @Test
- public void threeLevelTreeWithAllIndexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("-status:abandoned (file:a OR file:b)");
- assertThat(rewrite.rewrite(in, options(0, DEFAULT_MAX_QUERY_LIMIT))).isEqualTo(query(in));
- }
-
- @Test
- public void threeLevelTreeWithSomeIndexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("-foo:a (file:b OR file:c)");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(out.getClass()).isSameAs(AndChangeSource.class);
- assertThat(out.getChildren()).containsExactly(query(in.getChild(1)), in.getChild(0)).inOrder();
- }
-
- @Test
- public void multipleIndexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("file:a OR foo:b OR file:c OR foo:d");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(out.getClass()).isSameAs(OrSource.class);
- assertThat(out.getChildren())
- .containsExactly(query(or(in.getChild(0), in.getChild(2))), in.getChild(1), in.getChild(3))
- .inOrder();
- }
-
- @Test
- public void indexAndNonIndexPredicates() throws Exception {
- Predicate<ChangeData> in = parse("status:new bar:p file:a");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(AndChangeSource.class).isSameAs(out.getClass());
- assertThat(out.getChildren())
- .containsExactly(query(and(in.getChild(0), in.getChild(2))), in.getChild(1))
- .inOrder();
- }
-
- @Test
- public void duplicateCompoundNonIndexOnlyPredicates() throws Exception {
- Predicate<ChangeData> in = parse("status:new bar:p file:a");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(out.getClass()).isEqualTo(AndChangeSource.class);
- assertThat(out.getChildren())
- .containsExactly(query(and(in.getChild(0), in.getChild(2))), in.getChild(1))
- .inOrder();
- }
-
- @Test
- public void duplicateCompoundIndexOnlyPredicates() throws Exception {
- Predicate<ChangeData> in = parse("(status:new OR file:a) bar:p file:b");
- Predicate<ChangeData> out = rewrite(in);
- assertThat(out.getClass()).isEqualTo(AndChangeSource.class);
- assertThat(out.getChildren())
- .containsExactly(query(and(in.getChild(0), in.getChild(2))), in.getChild(1))
- .inOrder();
- }
-
- @Test
- public void optionsArgumentOverridesAllLimitPredicates() throws Exception {
- Predicate<ChangeData> in = parse("limit:1 file:a limit:3");
- Predicate<ChangeData> out = rewrite(in, options(0, 5));
- assertThat(out.getClass()).isEqualTo(AndChangeSource.class);
- assertThat(out.getChildren())
- .containsExactly(query(in.getChild(1), 5), parse("limit:5"), parse("limit:5"))
- .inOrder();
- }
-
- @Test
- public void startIncreasesLimitInQueryButNotPredicate() throws Exception {
- int n = 3;
- Predicate<ChangeData> f = parse("file:a");
- Predicate<ChangeData> l = parse("limit:" + n);
- Predicate<ChangeData> in = andSource(f, l);
- assertThat(rewrite.rewrite(in, options(0, n))).isEqualTo(andSource(query(f, 3), l));
- assertThat(rewrite.rewrite(in, options(1, n))).isEqualTo(andSource(query(f, 4), l));
- assertThat(rewrite.rewrite(in, options(2, n))).isEqualTo(andSource(query(f, 5), l));
- }
-
- @Test
- public void getPossibleStatus() throws Exception {
- Set<Change.Status> all = EnumSet.allOf(Change.Status.class);
- assertThat(status("file:a")).isEqualTo(all);
- assertThat(status("is:new")).containsExactly(NEW);
- assertThat(status("is:new OR is:merged")).containsExactly(NEW, MERGED);
- assertThat(status("is:new OR is:x")).isEqualTo(all);
-
- assertThat(status("is:new is:merged")).isEmpty();
- assertThat(status("(is:new) (is:merged)")).isEmpty();
- assertThat(status("(is:new) (is:merged)")).isEmpty();
- assertThat(status("is:new is:x")).containsExactly(NEW);
- }
-
- @Test
- public void unsupportedIndexOperator() throws Exception {
- Predicate<ChangeData> in = parse("status:merged file:a");
- assertThat(rewrite(in)).isEqualTo(query(in));
-
- indexes.setSearchIndex(new FakeChangeIndex(FakeChangeIndex.V1));
-
- exception.expect(QueryParseException.class);
- exception.expectMessage("Unsupported index predicate: file:a");
- rewrite(in);
- }
-
- @Test
- public void tooManyTerms() throws Exception {
- String q = "file:a OR file:b OR file:c";
- Predicate<ChangeData> in = parse(q);
- assertEquals(query(in), rewrite(in));
-
- exception.expect(QueryParseException.class);
- exception.expectMessage("too many terms in query");
- rewrite(parse(q + " OR file:d"));
- }
-
- @Test
- public void testConvertOptions() throws Exception {
- assertEquals(options(0, 3), convertOptions(options(0, 3)));
- assertEquals(options(0, 4), convertOptions(options(1, 3)));
- assertEquals(options(0, 5), convertOptions(options(2, 3)));
- }
-
- @Test
- public void addingStartToLimitDoesNotExceedBackendLimit() throws Exception {
- int max = CONFIG.maxLimit();
- assertEquals(options(0, max), convertOptions(options(0, max)));
- assertEquals(options(0, max), convertOptions(options(1, max)));
- assertEquals(options(0, max), convertOptions(options(1, max - 1)));
- assertEquals(options(0, max), convertOptions(options(2, max - 1)));
- }
-
- private Predicate<ChangeData> parse(String query) throws QueryParseException {
- return queryBuilder.parse(query);
- }
-
- @SafeVarargs
- private static AndChangeSource andSource(Predicate<ChangeData>... preds) {
- return new AndChangeSource(Arrays.asList(preds));
- }
-
- private Predicate<ChangeData> rewrite(Predicate<ChangeData> in) throws QueryParseException {
- return rewrite.rewrite(in, options(0, DEFAULT_MAX_QUERY_LIMIT));
- }
-
- private Predicate<ChangeData> rewrite(Predicate<ChangeData> in, QueryOptions opts)
- throws QueryParseException {
- return rewrite.rewrite(in, opts);
- }
-
- private IndexedChangeQuery query(Predicate<ChangeData> p) throws QueryParseException {
- return query(p, DEFAULT_MAX_QUERY_LIMIT);
- }
-
- private IndexedChangeQuery query(Predicate<ChangeData> p, int limit) throws QueryParseException {
- return new IndexedChangeQuery(index, p, options(0, limit));
- }
-
- private static QueryOptions options(int start, int limit) {
- return IndexedChangeQuery.createOptions(CONFIG, start, limit, ImmutableSet.<String>of());
- }
-
- private Set<Change.Status> status(String query) throws QueryParseException {
- return ChangeIndexRewriter.getPossibleStatus(parse(query));
- }
-}