You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by us...@apache.org on 2014/03/28 00:49:03 UTC

svn commit: r1582549 - /lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java

Author: uschindler
Date: Thu Mar 27 23:49:03 2014
New Revision: 1582549

URL: http://svn.apache.org/r1582549
Log:
SOLR-5914: Fix missing close on Reader

Modified:
    lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java

Modified: lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java?rev=1582549&r1=1582548&r2=1582549&view=diff
==============================================================================
--- lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java (original)
+++ lucene/dev/trunk/solr/solrj/src/test/org/apache/solr/common/util/ContentStreamTest.java Thu Mar 27 23:49:03 2014
@@ -58,12 +58,14 @@ public class ContentStreamTest extends S
     FileInputStream fis = new FileInputStream(file);
     InputStreamReader isr = new InputStreamReader(
         new FileInputStream(file), "UTF-8");
+    Reader r = stream.getReader();
     try {
       assertEquals(file.length(), stream.getSize().intValue());
       assertTrue(IOUtils.contentEquals(fis, s));
-      assertTrue(IOUtils.contentEquals(isr, stream.getReader()));
+      assertTrue(IOUtils.contentEquals(isr, r));
     } finally {
       s.close();
+      r.close();
       isr.close();
       fis.close();
     }