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 2011/11/02 17:11:58 UTC
svn commit: r1196664 - in
/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene: ./
index/ index/codecs/pulsing/ util/fst/
Author: rmuir
Date: Wed Nov 2 16:11:57 2011
New Revision: 1196664
URL: http://svn.apache.org/viewvc?rev=1196664&view=rev
Log:
LUCENE-3490: clear up some obselete nocommits
Modified:
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/TestExternalCodecs.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestAddIndexes.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestCodecs.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestDocTermOrds.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/Test10KPulsings.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/TestPulsingReuse.java
lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/util/fst/TestFSTs.java
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/TestExternalCodecs.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/TestExternalCodecs.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/TestExternalCodecs.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/TestExternalCodecs.java Wed Nov 2 16:11:57 2011
@@ -34,8 +34,6 @@ import org.apache.lucene.util.Bits;
/* Intentionally outside of oal.index to verify fully
external codecs work fine */
-// nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-// automagically
public class TestExternalCodecs extends LuceneTestCase {
private static final class CustomPerFieldCodec extends Lucene40Codec {
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestAddIndexes.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestAddIndexes.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestAddIndexes.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestAddIndexes.java Wed Nov 2 16:11:57 2011
@@ -55,8 +55,6 @@ import org.apache.lucene.store.RAMDirect
import org.apache.lucene.util.LuceneTestCase;
import org.apache.lucene.util._TestUtil;
-//nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-//automagically
public class TestAddIndexes extends LuceneTestCase {
public void testSimpleCase() throws IOException {
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestCodecs.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestCodecs.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestCodecs.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestCodecs.java Wed Nov 2 16:11:57 2011
@@ -66,8 +66,6 @@ import org.junit.BeforeClass;
// - skipTo(term)
// - skipTo(doc)
-//nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-//automagically
public class TestCodecs extends LuceneTestCase {
private static String[] fieldNames = new String[] {"one", "two", "three", "four"};
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestDocTermOrds.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestDocTermOrds.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestDocTermOrds.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/TestDocTermOrds.java Wed Nov 2 16:11:57 2011
@@ -58,8 +58,6 @@ import org.apache.lucene.util._TestUtil;
// - test w/ cutoff
// - crank docs way up so we get some merging sometimes
-//nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-//automagically
public class TestDocTermOrds extends LuceneTestCase {
public void testSimple() throws Exception {
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/Test10KPulsings.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/Test10KPulsings.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/Test10KPulsings.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/Test10KPulsings.java Wed Nov 2 16:11:57 2011
@@ -49,8 +49,6 @@ import org.junit.Ignore;
*
* @lucene.experimental
*/
-//nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-//automagically
public class Test10KPulsings extends LuceneTestCase {
public void test10kPulsed() throws Exception {
// we always run this test with pulsing codec.
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/TestPulsingReuse.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/TestPulsingReuse.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/TestPulsingReuse.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/index/codecs/pulsing/TestPulsingReuse.java Wed Nov 2 16:11:57 2011
@@ -40,8 +40,6 @@ import org.apache.lucene.util._TestUtil;
/**
* Tests that pulsing codec reuses its enums and wrapped enums
*/
-//nocommit: add any custom codecs here to test-framework so they can be 'loaded'
-//automagically
public class TestPulsingReuse extends LuceneTestCase {
// TODO: this is a basic test. this thing is complicated, add more
public void testSophisticatedReuse() throws Exception {
Modified: lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/util/fst/TestFSTs.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/util/fst/TestFSTs.java?rev=1196664&r1=1196663&r2=1196664&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/util/fst/TestFSTs.java (original)
+++ lucene/dev/branches/lucene2621/lucene/src/test/org/apache/lucene/util/fst/TestFSTs.java Wed Nov 2 16:11:57 2011
@@ -1014,7 +1014,7 @@ public class TestFSTs extends LuceneTest
// file, up until a time limit
public void testRealTerms() throws Exception {
- // nocommit: is this necessary? we use the annotation...
+ // TODO: is this necessary? we use the annotation...
final String defaultFormat = _TestUtil.getPostingsFormat("abracadabra");
if (defaultFormat.equals("SimpleText") || defaultFormat.equals("Memory")) {
// no