maven-surefire-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From krosenv...@apache.org
Subject svn commit: r1060394 - /maven/surefire/trunk/surefire-providers/surefire-junit4/src/main/java/org/apache/maven/surefire/junit4/JUnit4Provider.java
Date Tue, 18 Jan 2011 14:36:37 GMT
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 )



Mime
View raw message