You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2010/10/09 14:48:13 UTC

svn commit: r1006146 - /lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java

Author: mikemccand
Date: Sat Oct  9 12:48:13 2010
New Revision: 1006146

URL: http://svn.apache.org/viewvc?rev=1006146&view=rev
Log:
don't double-parse query text in this test

Modified:
    lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java

Modified: lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java?rev=1006146&r1=1006145&r2=1006146&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java (original)
+++ lucene/dev/trunk/lucene/src/test/org/apache/lucene/search/TestBoolean2.java Sat Oct  9 12:48:13 2010
@@ -128,20 +128,19 @@ public class TestBoolean2 extends Lucene
 //System.out.println();
 //System.out.println("Query: " + queryText);
 
-    Query query1 = makeQuery(queryText);
+    Query query = makeQuery(queryText);
     TopScoreDocCollector collector = TopScoreDocCollector.create(1000, false);
-    searcher.search(query1, null, collector);
+    searcher.search(query, null, collector);
     ScoreDoc[] hits1 = collector.topDocs().scoreDocs;
-    
-    Query query2 = makeQuery(queryText); // there should be no need to parse again...
+
     collector = TopScoreDocCollector.create(1000, true);
-    searcher.search(query2, null, collector);
+    searcher.search(query, null, collector);
     ScoreDoc[] hits2 = collector.topDocs().scoreDocs; 
 
     assertEquals(mulFactor * collector.totalHits,
-                 bigSearcher.search(query1, 1).totalHits);
+                 bigSearcher.search(query, 1).totalHits);
       
-    CheckHits.checkHitsQuery(query2, hits1, hits2, expDocNrs);
+    CheckHits.checkHitsQuery(query, hits1, hits2, expDocNrs);
   }
 
   @Test