[ http://jira.codehaus.org/browse/MSUREFIRE-23?page=comments#action_55488 ] 

Jesse Kuhnert commented on MSUREFIRE-23:
----------------------------------------

The reason why the reporting plugin needed work was because he really needed a 
better grouping construct. Class names and package names were being parsed out 
of attributes in a very unintuitive way, but I've already made all the code 
changes to the report plugin as well, just one tiny little javascript change to 
the output and I'll be done. Either way I'll post all of my patches to everyone 
tomorrow morning (e.s.t.) 

It was currently doing things like this in the xml output that was causing 
string index errors(while looking for class keyword):

<testsuite name="class org.foo.blah.ClassUnderTest" >

  <test name="fooMethod" />

</testsuite>

Which I've changed to something more universally doable in junit & testng:

<testsuite name="ClassUnderTest" group="org.foo.blah">

</testsuite>

I think the change was needed whether testng came into the picture or not, but 
if people don't like it there may be other options, I was just getting runtime 
errors or else I might've tried to avoid doing anything at all. 



> Support TestNG
> --------------
>
>          Key: MSUREFIRE-23
>          URL: http://jira.codehaus.org/browse/MSUREFIRE-23
>      Project: Maven 2.x Surefire Plugin
>         Type: New Feature

>     Reporter: Mike Perham

>
>
> Add support for running unit tests with TestNG.
> http://www.testng.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to