You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Arseniy Alekseyev (JIRA)" <ji...@codehaus.org> on 2010/11/09 13:11:04 UTC

[jira] Updated: (SUREFIRE-535) surefire tries to treat nested classes, even anonymous ones, as test classes?

     [ http://jira.codehaus.org/browse/SUREFIRE-535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Arseniy Alekseyev updated SUREFIRE-535:
---------------------------------------

    Attachment: patch.txt

I think the "surefire-junit" plugin does not exhibit this problem. The misbehaving one is "surefire-junit4". The problem is that it does not check whether the class has tests or not. I think that I have fixed it (patch attached). I've looked at the Eclipse's implementation of JUnit4 test runner and implemented test detection here in a similar way. This code is not covered by tests, actually there are none, but one my real-life project works with this. I've changed the dependency to Java 1.5 by the way, because JUnit4 depends on that and for convenience.

If someone authoritative approves this approach to the problem, I will cover this with unit tests.

Please review.

> surefire tries to treat nested classes, even anonymous ones, as test classes?
> -----------------------------------------------------------------------------
>
>                 Key: SUREFIRE-535
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-535
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: JUnit 3.x support
>    Affects Versions: 2.4.2
>            Reporter: Benson Margulies
>         Attachments: patch.txt
>
>
> Given a top-level test class that inherits from TestCase.
> If it, internally, declares an anonymous class, or even a named child class, surefire + junit tries to run it, and fails with a warning or an initializationError. This does not happen from the vanilla junit test runner.

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