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 2010/10/17 23:42:14 UTC

svn commit: r1023588 - /lucene/dev/trunk/solr/src/test/org/apache/solr/search/TestRangeQuery.java

Author: rmuir
Date: Sun Oct 17 21:42:13 2010
New Revision: 1023588

URL: http://svn.apache.org/viewvc?rev=1023588&view=rev
Log:
clean up resources in this test

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

Modified: lucene/dev/trunk/solr/src/test/org/apache/solr/search/TestRangeQuery.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/test/org/apache/solr/search/TestRangeQuery.java?rev=1023588&r1=1023587&r2=1023588&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/test/org/apache/solr/search/TestRangeQuery.java (original)
+++ lucene/dev/trunk/solr/src/test/org/apache/solr/search/TestRangeQuery.java Sun Oct 17 21:42:13 2010
@@ -263,7 +263,7 @@ public class TestRangeQuery extends Solr
       for (String q : qs) {
         // System.out.println("QUERY="+q);
         SolrQueryRequest req = req("q",q,"rows","1000");
-        SolrQueryResponse qr = h.queryAndResponse(handler, req("q",q,"rows","1000"));
+        SolrQueryResponse qr = h.queryAndResponse(handler, req);
         if (last != null) {
           // we only test if the same docs matched since some queries will include factors like idf, etc.
           sameDocs((DocSet)qr.getValues().get("response"), (DocSet)last.getValues().get("response"));
@@ -280,9 +280,7 @@ public class TestRangeQuery extends Solr
     assertEquals(a.size(), b.size());
     while (i.hasNext()) {
       int doc = i.nextDoc();
-      if (!b.exists(doc)) {
-        TestCase.fail("Missing doc " + doc);
-      }
+      assertTrue(b.exists(doc));
       // System.out.println("MATCH! " + doc);
     }
     return true;