[ http://jira.codehaus.org/browse/MSUREFIRE-131?page=all ]

Philip Gerlach updated MSUREFIRE-131:
-------------------------------------

    Attachment: commons-events-pom.xml

> Surefire-JUnit does not recognize "suite"-methods
> -------------------------------------------------
>
>          Key: MSUREFIRE-131
>          URL: http://jira.codehaus.org/browse/MSUREFIRE-131
>      Project: Maven 2.x Surefire Plugin
>         Type: Bug

>     Versions: 2.2
>     Reporter: Philip Gerlach
>  Attachments: commons-events-pom.xml, maven-surefire-junit-trunk-412516.patch
>
>
> Since Surefire-JUnit doesn't support JUnit4 yet, i tried to use a 
> "suite"-method like
> ------
> public static junit.framework.Test suite() {
>    return new junit.framework.JUnit4TestAdapter(Foo.class);
> }
> -----
> to run it, but Surefire-JUnit did not recognize these methods and treated 
> them like PojoTests what obviously lead to TestFailures.
> So I fetched the source code from the repository and searched for the 
> problem. I found two if-conditons in JUnitTestSet and JUnitDirectoryTestSuite 
> that did not test for the "suite"-mechanism, so I wrote a new static method 
> to test for this situation and integrated it in the if-conditions.
> Now the "suite"-methods work for my JUnit4 Tests and should do also for 
> others ;-)
> The patch is attached.
> P.S. Since this it is the first time, I'm trying to bugfix something for an 
> open source-project, please just let me know, if I have done something wrong 
> with this process.

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