You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2011/12/14 02:09:16 UTC

svn commit: r1214013 - in /lucene/dev/branches/branch_3x: ./ lucene/ lucene/src/test/org/apache/lucene/search/TestSearchAfter.java

Author: rmuir
Date: Wed Dec 14 01:09:15 2011
New Revision: 1214013

URL: http://svn.apache.org/viewvc?rev=1214013&view=rev
Log:
increase test coverage

Modified:
    lucene/dev/branches/branch_3x/   (props changed)
    lucene/dev/branches/branch_3x/lucene/   (props changed)
    lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/search/TestSearchAfter.java

Modified: lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/search/TestSearchAfter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/search/TestSearchAfter.java?rev=1214013&r1=1214012&r2=1214013&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/search/TestSearchAfter.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/test/org/apache/lucene/search/TestSearchAfter.java Wed Dec 14 01:09:15 2011
@@ -61,15 +61,21 @@ public class TestSearchAfter extends Luc
   }
   
   public void testQueries() throws Exception {
-    Filter odd = new QueryWrapperFilter(new TermQuery(new Term("oddeven", "odd")));
-    assertQuery(new MatchAllDocsQuery(), null);
-    assertQuery(new TermQuery(new Term("english", "one")), null);
-    assertQuery(new MatchAllDocsQuery(), odd);
-    assertQuery(new TermQuery(new Term("english", "four")), odd);
-    BooleanQuery bq = new BooleanQuery();
-    bq.add(new TermQuery(new Term("english", "one")), BooleanClause.Occur.SHOULD);
-    bq.add(new TermQuery(new Term("oddeven", "even")), BooleanClause.Occur.SHOULD);
-    assertQuery(bq, null);
+    // because the first page has a null 'after', we get a normal collector.
+    // so we need to run the test a few times to ensure we will collect multiple
+    // pages.
+    int n = atLeast(10);
+    for (int i = 0; i < n; i++) {
+      Filter odd = new QueryWrapperFilter(new TermQuery(new Term("oddeven", "odd")));
+      assertQuery(new MatchAllDocsQuery(), null);
+      assertQuery(new TermQuery(new Term("english", "one")), null);
+      assertQuery(new MatchAllDocsQuery(), odd);
+      assertQuery(new TermQuery(new Term("english", "four")), odd);
+      BooleanQuery bq = new BooleanQuery();
+      bq.add(new TermQuery(new Term("english", "one")), BooleanClause.Occur.SHOULD);
+      bq.add(new TermQuery(new Term("oddeven", "even")), BooleanClause.Occur.SHOULD);
+      assertQuery(bq, null);
+    }
   }
   
   void assertQuery(Query query, Filter filter) throws Exception {