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 2012/10/09 20:28:39 UTC

svn commit: r1396162 - in /lucene/dev/branches/branch_4x: ./ lucene/ lucene/codecs/ lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java lucene/core/ lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java

Author: rmuir
Date: Tue Oct  9 18:28:38 2012
New Revision: 1396162

URL: http://svn.apache.org/viewvc?rev=1396162&view=rev
Log:
LUCENE-4469: fix these tests to actually fail if checkindex fails

Modified:
    lucene/dev/branches/branch_4x/   (props changed)
    lucene/dev/branches/branch_4x/lucene/   (props changed)
    lucene/dev/branches/branch_4x/lucene/codecs/   (props changed)
    lucene/dev/branches/branch_4x/lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java
    lucene/dev/branches/branch_4x/lucene/core/   (props changed)
    lucene/dev/branches/branch_4x/lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java

Modified: lucene/dev/branches/branch_4x/lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java?rev=1396162&r1=1396161&r2=1396162&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java (original)
+++ lucene/dev/branches/branch_4x/lucene/codecs/src/test/org/apache/lucene/codecs/pulsing/TestPulsingReuse.java Tue Oct  9 18:28:38 2012
@@ -85,7 +85,6 @@ public class TestPulsingReuse extends Lu
     // we always run this test with pulsing codec.
     Codec cp = _TestUtil.alwaysPostingsFormat(new NestedPulsingPostingsFormat());
     BaseDirectoryWrapper dir = newDirectory();
-    dir.setCheckIndexOnClose(false); // will do this ourselves, custom codec
     RandomIndexWriter iw = new RandomIndexWriter(random(), dir, 
         newIndexWriterConfig(TEST_VERSION_CURRENT, new MockAnalyzer(random())).setCodec(cp));
     Document doc = new Document();
@@ -119,8 +118,6 @@ public class TestPulsingReuse extends Lu
     assertEquals(4, allEnums.size());
     
     ir.close();
-    CheckIndex ci = new CheckIndex(dir);
-    ci.checkIndex(null);
     dir.close();
   }
 }

Modified: lucene/dev/branches/branch_4x/lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java?rev=1396162&r1=1396161&r2=1396162&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java (original)
+++ lucene/dev/branches/branch_4x/lucene/core/src/test/org/apache/lucene/index/Test2BPostings.java Tue Oct  9 18:28:38 2012
@@ -48,7 +48,6 @@ public class Test2BPostings extends Luce
     if (dir instanceof MockDirectoryWrapper) {
       ((MockDirectoryWrapper)dir).setThrottling(MockDirectoryWrapper.Throttling.NEVER);
     }
-    dir.setCheckIndexOnClose(false); // don't double-checkindex
     
     IndexWriter w = new IndexWriter(dir,
         new IndexWriterConfig(TEST_VERSION_CURRENT, new MockAnalyzer(random()))
@@ -80,11 +79,6 @@ public class Test2BPostings extends Luce
     }
     w.forceMerge(1);
     w.close();
-    CheckIndex ci = new CheckIndex(dir);
-    if (VERBOSE) {
-      ci.setInfoStream(System.out);
-    }
-    ci.checkIndex();
     dir.close();
   }