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 2011/06/14 21:43:05 UTC
svn commit: r1135764 -
/lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
Author: mikemccand
Date: Tue Jun 14 19:43:04 2011
New Revision: 1135764
URL: http://svn.apache.org/viewvc?rev=1135764&view=rev
Log:
add comment to clarify confusing test
Modified:
lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
Modified: lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java?rev=1135764&r1=1135763&r2=1135764&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java (original)
+++ lucene/dev/trunk/lucene/src/test/org/apache/lucene/index/TestIndexWriter.java Tue Jun 14 19:43:04 2011
@@ -1375,6 +1375,11 @@ public class TestIndexWriter extends Luc
IndexReader r2 = r.reopen();
assertTrue(r != r2);
files = Arrays.asList(dir.listAll());
+
+ // NOTE: here we rely on "Windows" behavior, ie, even
+ // though IW wanted to delete _0.cfs since it was
+ // optimized away, because we have a reader open
+ // against this file, it should still be here:
assertTrue(files.contains("_0.cfs"));
// optimize created this
//assertTrue(files.contains("_2.cfs"));