[ http://jira.codehaus.org/browse/SUREFIRE-433?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_121224 ]
Dan Fabulich commented on SUREFIRE-433: --------------------------------------- @Benjamin: My point in the long post was that Surefire sometimes puts all of the test results into one file, but so does TestNG itself! That's supposed to be fine; TestNG can/should be allowed to do that, and we should be able to report on it correctly. surefire-report should be able to read that file and separate the tests out into their individual classes. > All tests run in a single execution are reported as coming from just one class > ------------------------------------------------------------------------------ > > Key: SUREFIRE-433 > URL: http://jira.codehaus.org/browse/SUREFIRE-433 > Project: Maven Surefire > Issue Type: Bug > Components: report plugin, xml generation > Affects Versions: 2.4 > Environment: Linux, maven 2.0.8, surefire 2.4, TestNg 5.7 > Reporter: Henrik Lynggaard > Attachments: bug-example.zip, junit4-dir-suite-it.patch, > testng-dir-suite-it.patch > > > When running a directory of Junit tests, surefire will generate a > TEST-(full-classname).xml per test class in the surefire-reports directory. > Each XML will contain a testcase for each method in the given class. > However when runnign TestNG test, surefire only generates one > Test-TestSuite.xml in the surefire-reports directory. This xml contains a > testcase per test method for all the classes in the given project. > This is a problem since it breaks the surefire-report. > - The report wil have a class section with one row, The class name in the row > will be TestSuite > - The report will have a package section with one row. The package name in > the row will be empty. ("TestSuite" contains no dot so no package) > - Proberly unrelated, but any < or > characters appers on the page as < > and > > I have looked a bit at the code and the problem appears to be that in > SureFire TestNG Runner(surefire-testng) the methods > - > org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.startTestSuite(...) > - > org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.finishTestSuite(...) > is only called once, and inside it uses a hardcoded "TestSuite" for the name. > I think it should be called for each class and passed the full classname as > name parameter. > I will attach a simple maven project that shows the problem. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira