You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by jp...@apache.org on 2013/02/03 00:29:31 UTC

svn commit: r1441835 - in /lucene/dev/branches/branch_4x: ./ lucene/ lucene/test-framework/ lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java

Author: jpountz
Date: Sat Feb  2 23:29:31 2013
New Revision: 1441835

URL: http://svn.apache.org/viewvc?rev=1441835&view=rev
Log:
Replace "throw exception" with the more idiomatic Junit's "assertNull", following Uwe's advice.

Modified:
    lucene/dev/branches/branch_4x/   (props changed)
    lucene/dev/branches/branch_4x/lucene/   (props changed)
    lucene/dev/branches/branch_4x/lucene/test-framework/   (props changed)
    lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java

Modified: lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java?rev=1441835&r1=1441834&r2=1441835&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java (original)
+++ lucene/dev/branches/branch_4x/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java Sat Feb  2 23:29:31 2013
@@ -721,9 +721,7 @@ public abstract class BaseTermVectorsFor
       reader.close();
       writer.close();
       dir.close();
-      if (exception.get() != null) {
-        throw new RuntimeException("One thread threw an exception", exception.get());
-      }
+      assertNull("One thread threw an exception", exception.get());
     }
   }