[ 
http://jira.codehaus.org/browse/SUREFIRE-452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_123174
 ] 

Erez Nahir commented on SUREFIRE-452:
-------------------------------------

Yep, I agree.
But in our configuration, we are not using suite.xml. So, I did not even tried 
to fix it.
I'll try to take a look, but I don't think my current fix should not be in, 
just because suite.xml is broken.

> suitename always = "TestSuite" even if <suitename> defined as property
> ----------------------------------------------------------------------
>
>                 Key: SUREFIRE-452
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-452
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: TestNG support
>    Affects Versions: 2.4, 2.4.1
>         Environment: Maven version: 2.0.8
> Java version: 1.5.0_14
> OS name: "windows xp" version: "5.1" arch: "x86" Family: "windows"
> and also on Linux
>            Reporter: Erez Nahir
>             Fix For: 2.x
>
>         Attachments: TestNGDirectoryTestSuite.java, 
> TestNGDirectoryTestSuite.java
>
>
> Using surefire and testng with no testng.xml but with <cofiguration>, the 
> generated standard output xml always have suite name="TestSuite".
> This is an issue while using CruiseControl and JUnitReport to generate a 
> JUnit report.
> Here is the configuration in my pom file:
>                 <plugin>
>                     <groupId>org.apache.maven.plugins</groupId>
>                     <artifactId>maven-surefire-plugin</artifactId>
>                     <version>2.4.1</version>
>                     <configuration>
>                         <testFailureIgnore>true</testFailureIgnore>
>                         <groups>fast</groups>
>                         <excludedgroups>broken</excludedgroups>
>                         <properties>
>                             <property>
>                                     <name>suitename</name>
>                                     <value>${artifactId}</value>
>                              </property>
>                             <property>
>                                     <name>testname</name>
>                                     <value>${artifactId}</value>
>                              </property>                             
>                         </properties>
>                     </configuration>
>                 </plugin>
> There is an attached patch too.

-- 
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

        

Reply via email to