You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by yo...@apache.org on 2010/10/17 19:58:36 UTC

svn commit: r1023536 - /lucene/dev/trunk/solr/src/test/org/apache/solr/BasicFunctionalityTest.java

Author: yonik
Date: Sun Oct 17 17:58:36 2010
New Revision: 1023536

URL: http://svn.apache.org/viewvc?rev=1023536&view=rev
Log:
tests: fix resource leak

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

Modified: lucene/dev/trunk/solr/src/test/org/apache/solr/BasicFunctionalityTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/test/org/apache/solr/BasicFunctionalityTest.java?rev=1023536&r1=1023535&r2=1023536&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/test/org/apache/solr/BasicFunctionalityTest.java (original)
+++ lucene/dev/trunk/solr/src/test/org/apache/solr/BasicFunctionalityTest.java Sun Oct 17 17:58:36 2010
@@ -303,11 +303,13 @@ public class BasicFunctionalityTest exte
     rsp.add("\"quoted\"", "\"value\"");
 
     StringWriter writer = new StringWriter(32000);
-    XMLWriter.writeResponse(writer,req("foo"),rsp);
+    SolrQueryRequest req = req("foo");
+    XMLWriter.writeResponse(writer,req,rsp);
 
     DocumentBuilder builder = DocumentBuilderFactory.newInstance().newDocumentBuilder();
     builder.parse(new ByteArrayInputStream
                   (writer.toString().getBytes("UTF-8")));
+    req.close();
   }
 
   @Test
@@ -327,6 +329,7 @@ public class BasicFunctionalityTest exte
     assertEquals(2, arrayParams.length);
     assertEquals("array", arrayParams[0]);
     assertEquals("value", arrayParams[1]);
+    req.close();
   }
 
   @Test
@@ -442,7 +445,7 @@ public class BasicFunctionalityTest exte
     assertEquals("SSS", p.get("ss"));
     assertEquals("XXX", p.get("xx"));
 
-    
+    req.close();
   }
 
   @Test