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:20:38 UTC
svn commit: r1441831 -
/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java
Author: jpountz
Date: Sat Feb 2 23:20:38 2013
New Revision: 1441831
URL: http://svn.apache.org/viewvc?rev=1441831&view=rev
Log:
Replace "throw exception" with the more idiomatic Junit's "assertNull", following Uwe's advice.
Modified:
lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java
Modified: lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java?rev=1441831&r1=1441830&r2=1441831&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java (original)
+++ lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/index/BaseTermVectorsFormatTestCase.java Sat Feb 2 23:20:38 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());
}
}