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 )


Reply via email to