You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by pr...@apache.org on 2007/08/24 19:38:55 UTC

svn commit: r569463 - /geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java

Author: prasad
Date: Fri Aug 24 10:38:54 2007
New Revision: 569463

URL: http://svn.apache.org/viewvc?rev=569463&view=rev
Log:
* insert empty cells to account for failed testsuites.

Modified:
    geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java

Modified: geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java?rev=569463&r1=569462&r2=569463&view=diff
==============================================================================
--- geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java (original)
+++ geronimo/server/trunk/maven-plugins/testsuite-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/testsuite/ResultsSummaryMojo.java Fri Aug 24 10:38:54 2007
@@ -460,7 +460,7 @@
         int skippedNum = Integer.parseInt(skipped);
 
         //String pcent = tests + "/" + errors + "/" + failures + " (" + computePercentage(testsNum, errorsNum, failuresNum, skippedNum) + "%)";
-        String pcent = tests + "/" + errors + "/" + failures;
+        String pcent = tests + "/" + errors + "/" + failures; 
         return pcent;
     }
 
@@ -508,7 +508,16 @@
         Element tr = getElementById(table, "tr", suiteName);
 
         if ( tr != null ) {
-            Element td = document.createElement("TD");
+            // creating empty cells in the cols for the previous failed builds.
+            NodeList nodeList = tr.getElementsByTagName("td");
+            Element td;
+            for ( int i=nodeList.getLength()+1; i<cols; i++ ) {
+                td = document.createElement("TD");
+                td.setAttribute("class", "cell");                
+                tr.appendChild(td);
+            }
+
+            td = document.createElement("TD");
             td.setAttribute("class", "cell");
 
             Element anchor = document.createElement("a");