Author: krosenvold
Date: Tue Jan 18 14:36:36 2011
New Revision: 1060394
URL: http://svn.apache.org/viewvc?rev=1060394&view=rev
Log:
[SUREFIRE-500] Ensured consistency in testset reporting
Modified:
maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java
Modified:
maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java
URL:
http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java?rev=1060394&r1=1060393&r2=1060394&view=diff
==============================================================================
---
maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java
(original)
+++
maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java
Tue Jan 18 14:36:36 2011
@@ -118,16 +118,18 @@ public class JUnit4Provider
}
catch ( TestSetFailedException e )
{
- throw e;
+ throw e;
}
catch ( Throwable e )
{
reporter.testError( new SimpleReportEntry( report.getSourceName(),
report.getName(),
- new
PojoStackTraceWriter( report.getSourceName(),
-
report.getName(), e ) ) );
+ new
PojoStackTraceWriter( report.getSourceName(),
+
report.getName(), e ) ) );
+ }
+ finally
+ {
+ reporter.testSetCompleted( report );
}
-
- reporter.testSetCompleted( report );
}
private RunNotifier getRunNotifer( RunListener main, List<RunListener>
others )