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 2014/01/31 15:58:21 UTC

svn commit: r1563142 - in /lucene/dev/trunk/solr/core/src/test/org/apache/solr: analytics/expression/ analytics/facet/ analytics/util/valuesource/ schema/

Author: yonik
Date: Fri Jan 31 14:58:21 2014
New Revision: 1563142

URL: http://svn.apache.org/r1563142
Log:
SOLR-5685: tests: add assertU to commit calls

Modified:
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/expression/ExpressionTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetExtrasTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/QueryFacetTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/util/valuesource/FunctionTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java
    lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesTest.java

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/expression/ExpressionTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/expression/ExpressionTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/expression/ExpressionTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/expression/ExpressionTest.java Fri Jan 31 14:58:21 2014
@@ -64,7 +64,7 @@ public class ExpressionTest extends Abst
           "double_dd", "" + d, "date_dtd", dt, "string_sd", s));
 
       if (usually()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
 

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetExtrasTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetExtrasTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetExtrasTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetExtrasTest.java Fri Jan 31 14:58:21 2014
@@ -99,7 +99,7 @@ public class FieldFacetExtrasTest extend
       }
       
       if (usually()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
     

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/FieldFacetTest.java Fri Jan 31 14:58:21 2014
@@ -383,7 +383,7 @@ public class FieldFacetTest extends Abst
       }
       
       if (usually()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
     

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/QueryFacetTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/QueryFacetTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/QueryFacetTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/facet/QueryFacetTest.java Fri Jan 31 14:58:21 2014
@@ -95,7 +95,7 @@ public class QueryFacetTest extends Abst
       }
       
       if (usually()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
     

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/util/valuesource/FunctionTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/util/valuesource/FunctionTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/util/valuesource/FunctionTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/analytics/util/valuesource/FunctionTest.java Fri Jan 31 14:58:21 2014
@@ -76,7 +76,7 @@ public class FunctionTest extends Abstra
       
       
       if (usually()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
     

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java Fri Jan 31 14:58:21 2014
@@ -45,7 +45,7 @@ public class DocValuesMultiTest extends 
 
   public void testDocValues() throws IOException {
     assertU(adoc("id", "1", "floatdv", "4.5", "intdv", "-1", "intdv", "3", "stringdv", "value1", "stringdv", "value2"));
-    commit();
+    assertU(commit());
     SolrCore core = h.getCoreInc();
     try {
       final RefCounted<SolrIndexSearcher> searcherRef = core.openNewSearcher(true, true);
@@ -150,7 +150,7 @@ public class DocValuesMultiTest extends 
     }
     for (int i = 0; i < 50; ++i) {
       if (rarely()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
       assertU(adoc("id", "1000" + i, "floatdv", "" + i, "intdv", "" + i, "doubledv", "" + i, "longdv", "" + i, "datedv", (1900+i) + "-12-31T23:59:59.999Z", "stringdv", "abc" + i));
     }

Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesTest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesTest.java?rev=1563142&r1=1563141&r2=1563142&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesTest.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/schema/DocValuesTest.java Fri Jan 31 14:58:21 2014
@@ -43,7 +43,7 @@ public class DocValuesTest extends SolrT
 
   public void testDocValues() throws IOException {
     assertU(adoc("id", "1"));
-    commit();
+    assertU(commit());
     SolrCore core = h.getCoreInc();
     try {
       final RefCounted<SolrIndexSearcher> searcherRef = core.openNewSearcher(true, true);
@@ -147,7 +147,7 @@ public class DocValuesTest extends SolrT
     }
     for (int i = 0; i < 50; ++i) {
       if (rarely()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
       assertU(adoc("id", "1000" + i, "floatdv", "" + i, "intdv", "" + i, "doubledv", "" + i, "longdv", "" + i, "datedv", (1900+i) + "-12-31T23:59:59.999Z", "stringdv", "abc" + i));
     }
@@ -192,7 +192,7 @@ public class DocValuesTest extends SolrT
     for (int i = 0; i < 50; ++i) {
       assertU(adoc("id", "1000" + i, "floatdv", "" + i%2, "intdv", "" + i%3, "doubledv", "" + i%4, "longdv", "" + i%5, "datedv", (1900+i%6) + "-12-31T23:59:59.999Z", "stringdv", "abc" + i%7));
       if (rarely()) {
-        commit(); // to have several segments
+        assertU(commit()); // to have several segments
       }
     }
     assertU(commit());