[ https://issues.apache.org/jira/browse/SUREFIRE-1153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14490246#comment-14490246 ]
ASF GitHub Bot commented on SUREFIRE-1153: ------------------------------------------ GitHub user trohovsky opened a pull request: https://github.com/apache/maven-surefire/pull/88 SUREFIRE-1153 Take includes into account when using test/it.test propety You can merge this pull request into a Git repository by running: $ git pull https://github.com/trohovsky/maven-surefire SUREFIRE-1153 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-surefire/pull/88.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #88 ---- commit 5ac81fbdd9831a34ef086fec878deeb3b71da5aa Author: Tomas Rohovsky <troho...@redhat.com> Date: 2015-04-10T19:49:01Z SUREFIRE-1153 Take includes into account when using test/it.test propety ---- > Take includes into account when using test/it.test propety > ---------------------------------------------------------- > > Key: SUREFIRE-1153 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1153 > Project: Maven Surefire > Issue Type: Improvement > Affects Versions: 2.18.1 > Reporter: Tomas Rohovsky > Fix For: 2.19 > > Attachments: failsafe-test-parameter.zip > > > Currently test/it.test property overrides includes and excludes parameters. I > think that it would be better to take includes into account. Lets have this > configuration as an example: > {code} > <plugin> > <groupId>org.apache.maven.plugins</groupId> > <artifactId>maven-failsafe-plugin</artifactId> > <version>2.18.1</version> > <configuration> > <includes> > <include>**/*Test.java</include> > </includes> > </configuration> > <executions> > <execution> > <goals> > <goal>integration-test</goal> > <goal>verify</goal> > </goals> > </execution> > </executions> > </plugin> > {code} > If you execute the test with {{mvn verify -Dit.test=#testFailsafe}} you will > end up with: > {code} > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-failsafe-plugin:2.18.1:integration-test > (default) on project failsafe-test-parameter: No tests were executed! (Set > -DfailIfNoTests=false to ignore this error.) -> [Help 1] > {code} > Sure, you can specify the class name: {{mvn verify > -Dit.test=ExampleTest#testFailsafe}} and it will work. But it is more > convenient to not do that. > A reproducer is attached. -- This message was sent by Atlassian JIRA (v6.3.4#6332)