buildr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pekka Enberg (JIRA)" <j...@apache.org>
Subject [jira] Issue Comment Edited: (BUILDR-62) JUnit test filtering should allow custom filtering, and not test JUnit @Test annotation on class
Date Tue, 22 Apr 2008 06:33:21 GMT

    [ https://issues.apache.org/jira/browse/BUILDR-62?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12591218#action_12591218
] 

penberg edited comment on BUILDR-62 at 4/21/08 11:31 PM:
--------------------------------------------------------------

One of the biggest strengths of JDave is the fact that it doesn't _need_ any special tools
support so I consider dropping this from Buildr 1.3 a serious regression. Especially considering
how Buildr doesn't support JDave "natively" I strongly encourage you to merge this patch (or
some other version of the patch) as a stop-gap measure so that existing users can upgrade
to 1.3.

Furthermore, keep in mind that @RunWith _is not_ specific to JDave at all so you're potentially
breaking other working setups as well here.

      was (Author: penberg):
    One of the biggest strengths of JDave is the fact that it doesn't _need_ any special tools
support so I consider dropping this from Buildr 1.3 a serious regression. Especially considering
how Buildr doesn't support JDave "natively" I strongly encourage you to merge this patch (or
some other version of the patch) as a stop-gap measure so that existing users can upgrade
to 1.3.
  
> JUnit test filtering should allow custom filtering, and not test JUnit @Test annotation
on class
> ------------------------------------------------------------------------------------------------
>
>                 Key: BUILDR-62
>                 URL: https://issues.apache.org/jira/browse/BUILDR-62
>             Project: Buildr
>          Issue Type: Bug
>          Components: Test frameworks
>    Affects Versions: 1.3
>            Reporter: Timo Rantalaiho
>
> If I understand correctly, in test_frameworks.rb
>   http://svn.apache.org/repos/asf/incubator/buildr/trunk/lib/buildr/java/test_frameworks.rb
> JUnit tests are collected by taking 
> 1) everything that extends JUnit 3 TestCase
> 2) every class that is annotated with JUnit 4 @Test
> 3) every method that is annotated with JUnit 4 @Test
> 2) is clearly wrong, as @Test only applies to methods, not classes
>   http://junit.sourceforge.net/javadoc_40/org/junit/Test.html
> Also, I found no way to run JDave ( http://www.jdave.org/ ) specs or a custom suite with
the current 1.3 trunk (22 April 2008). In 1.2, this is very easy with Buildr::Java::JUnit::JUNIT_TESTS_PATTERN
filtering by test / spec class names. 
> Some kind of an abstraction for filtering the relevant test or spec classes is important
not only for custom test framework integration, but for custom suites in big projects. Running
a lot of tests is typically faster in a suite, and when you have different sets of tests (unit,
integration, acceptance, black-box against a deployed application, ...) you typically want
to specify which set to run. For example, the continuous integration server might only run
tests against the deployed application a couple of times a day, but unit tests all the time
for fast feedback. Different grouping methods of test frameworks are framework specific, and
rather than making Buildr support framework specific ways, it could operate on an extra abstraction
level.
> Filtering by class name has worked well elsewhere (ant, maven, buildr 1.2), so it would
be excellent if it would be included in 1.3 as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message