You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Mike Perham <Mi...@webifysolutions.com> on 2006/05/17 18:32:30 UTC

surefire patch needs review

I have fixed http://jira.codehaus.org/browse/MSUREFIRE-113 but don't
have commit privileges.  This is a major regression versus 2.1.3 and for
those of us who run Junit tests via suites a blocker.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: surefire patch needs review

Posted by Brett Porter <br...@apache.org>.
It looks fine, but incomplete. I'd forgotten about this better base 
class, and also referenced TestCase in the TestNG provider. If you can 
update that as well, then it should be fine.

I've given you commit rights on surefire.

Cheers,
Brett

Mike Perham wrote:
> I have fixed http://jira.codehaus.org/browse/MSUREFIRE-113 but don't
> have commit privileges.  This is a major regression versus 2.1.3 and for
> those of us who run Junit tests via suites a blocker.
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
> 


-- 
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org