-Dtest supports multiple test classes but not multiple test methods

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 2.12.1
Component/s: None
Labels:
None
Environment:

Hide
Apache Maven 3.0.2 (r1056850; 2011-01-08 19:58:10-0500)
Java version: 1.6.0_24, vendor: Apple Inc.
Java home: /System/Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home
Default locale: en_CA, platform encoding: MacRoman
OS name: "mac os x", version: "10.6.7", arch: "x86_64", family: "mac"

Show
Apache Maven 3.0.2 (r1056850; 2011-01-08 19:58:10-0500) Java version: 1.6.0_24, vendor: Apple Inc. Java home: /System/Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home Default locale: en_CA, platform encoding: MacRoman OS name: "mac os x", version: "10.6.7", arch: "x86_64", family: "mac"

Complexity:
Intermediate
Number of attachments :
6

Description

The -Dtest parameter is very handy for running a specific test class or test method. It also supports running multiple test classes. Unfortunately, it does not permit specifying running multiple test methods. It would be great if this were possible.

The examples below are from the Apache Commons project.

WORKS: Run multiple test classes:
mvn test -Dtest=ImmutablePairTest,StopWatchTest

WORKS: Run a specific test method:
mvn test -Dtest=ImmutablePairTest#testBasic

DOES NOT WORK:
mvn test -Dtest=StopWatchTest#testStopWatchSimple,StopWatchTest#testStopWatchSimpleGet
mvn test -Dtest=ImmutablePairTest#testBasic,StopWatchTest#testLang315

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

multipleMethods.patch

22/Feb/12 11:54 PM

10 kB

rainLee
multipleMethods-v2.patch

23/Feb/12 4:01 AM

5 kB

rainLee
multipleMethods-v3.patch

23/Feb/12 4:14 AM

14 kB

rainLee
multipleMethods-v4.patch

26/Feb/12 10:18 PM

14 kB

rainLee
SUREFIRE-745.patch

27/Feb/12 4:19 AM

18 kB

rainLee
SUREFIRE-745-v2.patch

27/Feb/12 11:45 PM

22 kB

rainLee

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

rainLee added a comment - 22/Feb/12 3:05 AM

i have fixed this issue
can i submit a patch?

Show

rainLee added a comment - 22/Feb/12 3:05 AM i have fixed this issue can i submit a patch?

Hide

Permalink

Olivier Lamy added a comment - 22/Feb/12 3:09 AM

sure
with it test integrated ?

Show

Olivier Lamy added a comment - 22/Feb/12 3:09 AM sure with it test integrated ?

Hide

Permalink

rainLee added a comment - 22/Feb/12 11:22 PM

my patch is created from version 2.8
should i created from trunk?

Show

rainLee added a comment - 22/Feb/12 11:22 PM my patch is created from version 2.8 should i created from trunk?

Hide

Permalink

rainLee added a comment - 22/Feb/12 11:54 PM - edited

submit my patch that created from version 2.8
and my patch support -Dtest=xxx.xxx.ATest#testMethod1#testMethod2,xxx.xxx.BTest#testMethod3

Show

rainLee added a comment - 22/Feb/12 11:54 PM - edited submit my patch that created from version 2.8 and my patch support -Dtest=xxx.xxx.ATest#testMethod1#testMethod2,xxx.xxx.BTest#testMethod3

Hide

Permalink

Kristian Rosenvold added a comment - 23/Feb/12 1:13 AM

Patches should always be created from trunk.

Show

Kristian Rosenvold added a comment - 23/Feb/12 1:13 AM Patches should always be created from trunk.

Hide

Permalink

rainLee added a comment - 23/Feb/12 4:01 AM

update my patch.
this patch is created from trunk

Show

rainLee added a comment - 23/Feb/12 4:01 AM update my patch. this patch is created from trunk

Hide

Permalink

rainLee added a comment - 23/Feb/12 4:14 AM

add integrated test

Show

rainLee added a comment - 23/Feb/12 4:14 AM add integrated test

Hide

Permalink

Robert Scholte added a comment - 23/Feb/12 11:56 AM

I noticed that mtClass#myMethod#myMethod2,secondClass#testMethod would become a valid expression.
IMO the # is a class-to-method separator. My suggestion would be to use + between methods:
mtClass#myMethod+myMethod2,secondClass#testMethod

Show

Robert Scholte added a comment - 23/Feb/12 11:56 AM I noticed that mtClass#myMethod#myMethod2,secondClass#testMethod would become a valid expression. IMO the # is a class-to-method separator. My suggestion would be to use + between methods: mtClass#myMethod+myMethod2,secondClass#testMethod

Hide

Permalink

rainLee added a comment - 23/Feb/12 9:28 PM

should I take Robert's opinion?

Show

rainLee added a comment - 23/Feb/12 9:28 PM should I take Robert's opinion?

Hide

Permalink

Olivier Lamy added a comment - 26/Feb/12 11:36 AM

agree with Robert.
BTW mtClass#myMethod* must still work and execute both myMethod AND myMethod2.
I think there is an integration test for that.

Show

Olivier Lamy added a comment - 26/Feb/12 11:36 AM agree with Robert. BTW mtClass#myMethod* must still work and execute both myMethod AND myMethod2. I think there is an integration test for that.

Hide

Permalink

rainLee added a comment - 26/Feb/12 8:03 PM

yeah, Olivier.
mtClass#myMethod* does still work and the integration test is passed

ok, i'll take Robert's opinion and refresh the patch later

Show

rainLee added a comment - 26/Feb/12 8:03 PM yeah, Olivier. mtClass#myMethod* does still work and the integration test is passed ok, i'll take Robert's opinion and refresh the patch later

Hide

Permalink

rainLee added a comment - 26/Feb/12 10:18 PM

change method-method separator from "#" to "+"

Show

rainLee added a comment - 26/Feb/12 10:18 PM change method-method separator from "#" to "+"

Hide

Permalink

Olivier Lamy added a comment - 27/Feb/12 1:44 AM

can you generate a patch from top level using svn diff > ~~SUREFIRE-745~~.patch
It looks you generate this with eclipse which means I cannot apply it using cli (as I use intellij )
And sorry to bother you .

Show

Olivier Lamy added a comment - 27/Feb/12 1:44 AM can you generate a patch from top level using svn diff > SUREFIRE-745 .patch It looks you generate this with eclipse which means I cannot apply it using cli (as I use intellij ) And sorry to bother you .

Hide

Permalink

rainLee added a comment - 27/Feb/12 4:19 AM

sure.
sorry to make you wield.cause this is my first time submit a patch to the community.
I should have known this before i submit.
thanks for reminding me

Show

rainLee added a comment - 27/Feb/12 4:19 AM sure. sorry to make you wield.cause this is my first time submit a patch to the community. I should have known this before i submit. thanks for reminding me

Hide

Permalink

Olivier Lamy added a comment - 27/Feb/12 4:30 AM

why all of those changes:

   <parent>
     <groupId>org.apache.maven.surefire</groupId>
     <artifactId>surefire-providers</artifactId>
-    <version>2.8</version>
+    <version>2.8-itest</version>
     <relativePath>../pom.xml</relativePath>
   </parent>

Show

Olivier Lamy added a comment - 27/Feb/12 4:30 AM why all of those changes: <parent> <groupId>org.apache.maven.surefire</groupId> <artifactId>surefire-providers</artifactId> - <version>2.8</version> + <version>2.8-itest</version> <relativePath>../pom.xml</relativePath> </parent>

Hide

Permalink

rainLee added a comment - 27/Feb/12 4:32 AM

oh,sorry,i have already used this feature in my own maven repository,so i changed the version.
i'll revert this soon

Show

rainLee added a comment - 27/Feb/12 4:32 AM oh,sorry,i have already used this feature in my own maven repository,so i changed the version. i'll revert this soon

Hide

Permalink

Olivier Lamy added a comment - 27/Feb/12 4:59 AM

As I see only it test for junit 4.4: does that work with all junit 4.x versions ?

Show

Olivier Lamy added a comment - 27/Feb/12 4:59 AM As I see only it test for junit 4.4: does that work with all junit 4.x versions ?

Hide

Permalink

rainLee added a comment - 27/Feb/12 11:45 PM

update patch
remove pom modification
add junit48 integration test

Show

rainLee added a comment - 27/Feb/12 11:45 PM update patch remove pom modification add junit48 integration test

Hide

Permalink

Olivier Lamy added a comment - 28/Feb/12 4:47 AM - edited

fixed r1294568.
Thanks !

Show

Olivier Lamy added a comment - 28/Feb/12 4:47 AM - edited fixed r1294568. Thanks !

Hide

Permalink

rainLee added a comment - 28/Feb/12 8:10 PM

my pleasure

Show

rainLee added a comment - 28/Feb/12 8:10 PM my pleasure

People

Assignee:

Olivier Lamy

Reporter:

reid holmes

Vote (3)

Watch (3)

Dates

Created:

14/Jun/11 12:54 PM

Updated:

28/Feb/12 8:10 PM

Resolved:

28/Feb/12 4:47 AM