You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2007/09/12 14:50:07 UTC
svn commit: r574924 -
/maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java
Author: evenisse
Date: Wed Sep 12 05:50:06 2007
New Revision: 574924
URL: http://svn.apache.org/viewvc?rev=574924&view=rev
Log:
Don't try to parse surefire reports directory if it doesn't exist
Modified:
maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java
Modified: maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java?rev=574924&r1=574923&r2=574924&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java (original)
+++ maven/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/SurefireReportAction.java Wed Sep 12 05:50:06 2007
@@ -22,7 +22,6 @@
import org.apache.maven.continuum.ContinuumException;
import org.apache.maven.continuum.configuration.ConfigurationException;
import org.apache.maven.continuum.model.project.Project;
-import org.apache.maven.continuum.model.scm.TestResult;
import org.apache.maven.continuum.web.exception.AuthorizationRequiredException;
import org.codehaus.plexus.util.DirectoryScanner;
import org.codehaus.plexus.util.StringUtils;
@@ -83,7 +82,12 @@
File reportsDirectory = getContinuum().getConfiguration().getTestReportsDirectory( buildId, projectId );
- parseReports( reportsDirectory );
+ testSuites = new ArrayList();
+
+ if ( reportsDirectory != null && reportsDirectory.exists() )
+ {
+ parseReports( reportsDirectory );
+ }
getSummary( testSuites );
@@ -97,8 +101,6 @@
{
String[] xmlReportFiles = getIncludedFiles( reportsDirectory, "*.xml", "*.txt" );
- testSuites = new ArrayList();
-
for ( int index = 0; index < xmlReportFiles.length; index++ )
{
ReportTestSuite testSuite = new ReportTestSuite();
@@ -138,7 +140,7 @@
for ( Iterator suites = suiteList.iterator(); suites.hasNext(); )
{
-
+
ReportTestSuite suite = (ReportTestSuite) suites.next();
totalTests += suite.getNumberOfTests();