Github user cbismuth commented on a diff in the pull request: https://github.com/apache/lucene-solr/pull/495#discussion_r232176947 --- Diff: lucene/core/src/test/org/apache/lucene/search/TestConstantScoreScorer.java --- @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You 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 org.apache.lucene.search; + +import java.io.IOException; + +import org.apache.lucene.document.Document; +import org.apache.lucene.document.Field; +import org.apache.lucene.index.IndexReader; +import org.apache.lucene.index.LeafReaderContext; +import org.apache.lucene.index.RandomIndexWriter; +import org.apache.lucene.index.Term; +import org.apache.lucene.store.Directory; +import org.apache.lucene.util.LuceneTestCase; + +import static org.apache.lucene.search.BooleanClause.Occur; +import static org.apache.lucene.search.DocIdSetIterator.NO_MORE_DOCS; +import static org.hamcrest.CoreMatchers.equalTo; +import static org.hamcrest.CoreMatchers.is; + +public class TestConstantScoreScorer extends LuceneTestCase { + private static final String FIELD = "f"; + private static final String[] VALUES = new String[]{"foo", "bar", "foo bar", "azerty"}; + + public void testBasics() throws Exception { + try (final TestConstantScoreScorerIndex index = new TestConstantScoreScorerIndex()) { + BooleanQuery query = new BooleanQuery.Builder() + .add(new ConstantScoreQuery(new TermQuery(new Term(FIELD, "foo"))), Occur.MUST) + .build(); + + final ConstantScoreScorer scorer = index.constantScoreScorer(query, 1f); + assertThat(scorer.iterator().nextDoc(), equalTo(0)); + assertThat(scorer.score(), is(equalTo(1f))); + assertThat(scorer.iterator().nextDoc(), equalTo(2)); + assertThat(scorer.score(), is(equalTo(1f))); + assertThat(scorer.iterator().nextDoc(), equalTo(NO_MORE_DOCS)); + } + } + + public void testWithMinCompetitiveScoreSet() throws Exception { + try (final TestConstantScoreScorerIndex index = new TestConstantScoreScorerIndex()) { + BooleanQuery query = new BooleanQuery.Builder() + .add(new ConstantScoreQuery(new TermQuery(new Term(FIELD, "foo"))), Occur.MUST) + .build(); + + final ConstantScoreScorer scorer = index.constantScoreScorer(query, 1f); + assertThat(scorer.iterator().nextDoc(), equalTo(0)); + assertThat(scorer.score(), is(equalTo(1f))); + scorer.setMinCompetitiveScore(2f); + assertThat(scorer.iterator().nextDoc(), equalTo(NO_MORE_DOCS)); + } + } + + static class TestConstantScoreScorerIndex implements AutoCloseable { + + private final Directory directory; + private final RandomIndexWriter writer; + private final IndexReader reader; + + TestConstantScoreScorerIndex() throws IOException { + directory = newDirectory(); + + writer = new RandomIndexWriter(random(), directory); + for (String VALUE : VALUES) { + Document doc = new Document(); + doc.add(newTextField(FIELD, VALUE, Field.Store.YES)); + writer.addDocument(doc); + } + reader = writer.getReader(); --- End diff -- Done in 8d7d5aefd477dc86a35cc3eb137a8a45534db285, thank you. Could you please explain me why docs have to be in a single segment to ensure this test passes?
--- --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org