svn commit: r649322 - /maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java

[bentmann]

Author: bentmann
Date: Thu Apr 17 15:53:05 2008
New Revision: 649322

URL: http://svn.apache.org/viewvc?rev=649322&view=rev
Log:
o Properly marked section boundaries (compare 
AbstractMavenReportRenderer.startSection())

Modified:
    
maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java

Modified: 
maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java
URL: 
http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java?rev=649322&r1=649321&r2=649322&view=diff
==============================================================================
--- 
maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java
 (original)
+++ 
maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java
 Thu Apr 17 15:53:05 2008
@@ -86,9 +86,11 @@
 
         sink.body();
 
+        sink.section1();
         sink.sectionTitle1();
         sink.text( bundle.getString( "report.surefire.header" ) );
         sink.sectionTitle1_();
+        sink.section1_();
 
         constructSummarySection( bundle, sink );
 
@@ -122,6 +124,7 @@
     {
         Map summary = report.getSummary( testSuites );
 
+        sink.section1();
         sink.sectionTitle1();
 
         sinkAnchor( sink, "Summary" );
@@ -175,12 +178,15 @@
         sink.rawText( bundle.getString( "report.surefire.text.note1" ) );
 
         sinkLineBreak( sink );
+
+        sink.section1_();
     }
 
     private void constructPackagesSection( ResourceBundle bundle, Sink sink, 
Map suitePackages )
     {
         NumberFormat numberFormat = report.getNumberFormat();
 
+        sink.section1();
         sink.sectionTitle1();
 
         sinkAnchor( sink, "Package_List" );
@@ -258,6 +264,7 @@
 
             Iterator suiteIterator = testSuiteList.iterator();
 
+            sink.section2();
             sink.sectionTitle2();
 
             sinkAnchor( sink, packageName );
@@ -344,15 +351,20 @@
             }
 
             sink.table_();
+
+            sink.section2_();
         }
 
         sinkLineBreak( sink );
+
+        sink.section1_();
     }
 
     private void constructTestCasesSection( ResourceBundle bundle, Sink sink )
     {
         NumberFormat numberFormat = report.getNumberFormat();
 
+        sink.section1();
         sink.sectionTitle1();
 
         sinkAnchor( sink, "Test_Cases" );
@@ -375,6 +387,7 @@
             {
                 ListIterator caseIterator = testCases.listIterator();
 
+                sink.section2();
                 sink.sectionTitle2();
 
                 sinkAnchor( sink, suite.getPackageName() + suite.getName() );
@@ -483,10 +496,14 @@
                 }
 
                 sink.table_();
+
+                sink.section2_();
             }
         }
 
         sinkLineBreak( sink );
+
+        sink.section1_();
     }
 
     private void constructFailureDetails( Sink sink, ResourceBundle bundle, 
List failureList )
@@ -495,6 +512,7 @@
 
         if ( failIter != null )
         {
+            sink.section1();
             sink.sectionTitle1();
 
             sinkAnchor( sink, "Failure_Details" );
@@ -601,6 +619,8 @@
         }
 
         sinkLineBreak( sink );
+
+        sink.section1_();
     }
 
     private String getErrorLineNumber( String className, String source )