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 )