You are viewing a plain text version of this content. The canonical link for it is here.
Posted to surefire-commits@maven.apache.org by df...@apache.org on 2007/12/09 08:28:35 UTC
svn commit: r602647 -
/maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java
Author: dfabulich
Date: Sat Dec 8 23:28:35 2007
New Revision: 602647
URL: http://svn.apache.org/viewvc?rev=602647&view=rev
Log:
[SUREFIRE-398] Skipped tests don't call testStarting. Assume startTime = endTime if startTime is 0.
Modified:
maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java
Modified: maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java?rev=602647&r1=602646&r2=602647&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java (original)
+++ maven/surefire/trunk/surefire-api/src/main/java/org/apache/maven/surefire/report/AbstractReporter.java Sat Dec 8 23:28:35 2007
@@ -165,6 +165,12 @@
++completedCount;
endTime = System.currentTimeMillis();
+ // SUREFIRE-398 skipped tests call endTest without calling testStarting
+ // if startTime = 0, set it to endTime, so the diff will be 0
+ if ( startTime == 0 )
+ {
+ startTime = endTime;
+ }
}
// ----------------------------------------------------------------------