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 )


Reply via email to