svn commit: r619219 - /maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java

[df...@apache.org]

Author: dfabulich
Date: Wed Feb  6 15:24:21 2008
New Revision: 619219

URL: http://svn.apache.org/viewvc?rev=619219&view=rev
Log:
Single class executor should start/stop suite so you get correct console reporting.  This begins to address SUREFIRE-446, but doesn't actually fix the bug (the result of this is you get an XML file with just one of the test results in it; the other test results are lost)

Modified:
    maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java

Modified: maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java?rev=619219&r1=619218&r2=619219&view=diff
==============================================================================
--- maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java (original)
+++ maven/surefire/trunk/surefire-providers/surefire-testng/src/main/java/org/apache/maven/surefire/testng/TestNGDirectoryTestSuite.java Wed Feb  6 15:24:21 2008
@@ -95,9 +95,13 @@
         {
             throw new TestSetFailedException( "Unable to find test set '" + testSetName + "' in suite" );
         }
+        
+        startTestSuite( reporterManager, this );
 
         TestNGExecutor.run( new Class[]{testSet.getTestClass()}, this.testSourceDirectory, this.options, this.version,
                             this.classifier, reporterManager, this, reportsDirectory );
+        
+        finishTestSuite( reporterManager, this );
     }
 
     public void execute( ReporterManager reporterManager, ClassLoader classLoader )