You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by mgreau <gi...@git.apache.org> on 2017/04/11 13:42:05 UTC
[GitHub] maven-surefire pull request #146: SUREFIRE-1330: Fix module name in surefire...
GitHub user mgreau opened a pull request:
https://github.com/apache/maven-surefire/pull/146
SUREFIRE-1330: Fix module name in surefire-providers reactor
This PR fixes the JUnit5 module name `surefire-junit-platform` in the `surefire-providers` POM reactor.
<details>
<summary>Build logs results with and without the fix</summary>
Indeed, without this PR I can't build the project on this branch:
```
\u256d\u2500mgreau@mbpro ~/GIT/mgreau/maven-surefire \u20393.0-rc1\u203a
\u2570\u2500$ mvn clean install
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[ERROR] Child module /Users/mgreau/Dev/Sources/eXo/all_git_repos/mgreau/maven-surefire/surefire-providers/surefire-junit5 of /Users/mgreau/Dev/Sources/eXo/all_git_repos/mgreau/maven-surefire/surefire-providers/pom.xml does not exist @
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR] The project org.apache.maven.surefire:surefire-providers:2.19.2-SNAPSHOT (/Users/mgreau/Dev/Sources/eXo/all_git_repos/mgreau/maven-surefire/surefire-providers/pom.xml) has 1 error
[ERROR] Child module /Users/mgreau/Dev/Sources/eXo/all_git_repos/mgreau/maven-surefire/surefire-providers/surefire-junit5 of /Users/mgreau/Dev/Sources/eXo/all_git_repos/mgreau/maven-surefire/surefire-providers/pom.xml does not exist
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
```
With this PR, I have a Build Failure because of the module `Maven Surefire Integration Tests` but the reactor is OK:
```
Tests run: 701, Failures: 36, Errors: 498, Skipped: 133
[INFO]
[INFO] --- maven-checkstyle-plugin:2.15:check (checkstyle-check) @ surefire-integration-tests ---
[INFO]
[INFO] --- maven-failsafe-plugin:2.12.4:verify (default) @ surefire-integration-tests ---
[INFO] Failsafe report directory: /Users/mgreau/Dev/Sources/eXo/all_git_repos/3party/maven-surefire/surefire-integration-tests/target/failsafe-reports
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Maven Surefire .............................. SUCCESS [ 3.863 s]
[INFO] ShadeFire JUnit3 Provider .......................... SUCCESS [ 3.156 s]
[INFO] SureFire API ....................................... SUCCESS [ 4.231 s]
[INFO] SureFire Booter .................................... SUCCESS [ 1.602 s]
[INFO] Maven Surefire Test-Grouping Support ............... SUCCESS [ 1.314 s]
[INFO] SureFire Providers ................................. SUCCESS [ 0.516 s]
[INFO] Shared JUnit3 Provider Code ........................ SUCCESS [ 0.799 s]
[INFO] Shared Java 5 Provider Base ........................ SUCCESS [ 1.042 s]
[INFO] Shared JUnit4 Provider Code ........................ SUCCESS [ 1.136 s]
[INFO] Shared JUnit48 Provider Code ....................... SUCCESS [ 1.686 s]
[INFO] SureFire JUnit Runner .............................. SUCCESS [ 1.018 s]
[INFO] SureFire JUnit4 Runner ............................. SUCCESS [ 1.400 s]
[INFO] Maven Surefire Common .............................. SUCCESS [ 4.381 s]
[INFO] SureFire JUnitCore Runner .......................... SUCCESS [ 49.355 s]
[INFO] SureFire JUnit Platform Runner ..................... SUCCESS [ 1.624 s]
[INFO] SureFire TestNG Utils .............................. SUCCESS [ 0.909 s]
[INFO] SureFire TestNG Runner ............................. SUCCESS [ 1.702 s]
[INFO] Surefire Report Parser ............................. SUCCESS [ 1.346 s]
[INFO] Maven Surefire Plugin .............................. SUCCESS [ 3.826 s]
[INFO] Maven Failsafe Plugin .............................. SUCCESS [ 2.514 s]
[INFO] Maven Surefire Report Plugin ....................... SUCCESS [ 7.545 s]
[INFO] Maven Surefire Integration Test Setup .............. SUCCESS [ 34.920 s]
[INFO] Maven Surefire Integration Tests ................... FAILURE [23:41 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 25:51 min
[INFO] Finished at: 2017-04-11T15:34:37+02:00
[INFO] Final Memory: 85M/1284M
[INFO] ------------------------------------------------------------------------
```
</details>
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mgreau/maven-surefire SUREFIRE-1330
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/maven-surefire/pull/146.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 #146
----
commit 70c8843e935882b0a78916894cb9f042964f2717
Author: Maxime Gr�au <mg...@exoplatform.com>
Date: 2017-04-11T13:32:10Z
SUREFIRE-1330: Fix module name in surefire-providers reactor
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by Tibor17 <gi...@git.apache.org>.
Github user Tibor17 commented on the issue:
https://github.com/apache/maven-surefire/pull/146
Not on the same version. It's not legal to introduce incomplete feature. The acceptable way would be to introduce junit5 in 3.0-alpha-1.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire pull request #146: SUREFIRE-1330: Fix module name in surefire...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/maven-surefire/pull/146
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by britter <gi...@git.apache.org>.
Github user britter commented on the issue:
https://github.com/apache/maven-surefire/pull/146
@Tibor17 the provider is already part of the 3.0-rc1 branch. Please see #142. This just fixes an incorrect module name.
> It's not legal to introduce incomplete feature.
Since we have not added the `ProviderInfo` for JUnit 5 in `AbstractSurefireMojo`, the provider is effectively not usable. So we can think of this as a feature toggle. The code does not hurt anybody, but it is much easier to work on it, when it is already part of the 3.0 line.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by Tibor17 <gi...@git.apache.org>.
Github user Tibor17 commented on the issue:
https://github.com/apache/maven-surefire/pull/146
@britter
@mgreau
Done. https://git-wip-us.apache.org/repos/asf?p=maven-surefire.git;a=commit;h=70c8843e935882b0a78916894cb9f042964f2717
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by Tibor17 <gi...@git.apache.org>.
Github user Tibor17 commented on the issue:
https://github.com/apache/maven-surefire/pull/146
@mgreau
@britter
Hey guys, let's utilize your power and complete `AbstractSurefireMojo` with `ProviderInfo`, some tests are still in progress.
Meanwhile Apache Maven will complete 2.20.1. The reason is the stability of this plugin. It is more important to release 2.20.1. Maybe 2.20.1 would split again as it was with 2.20 and the release will get faster with faster user response.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by Tibor17 <gi...@git.apache.org>.
Github user Tibor17 commented on the issue:
https://github.com/apache/maven-surefire/pull/146
@mgreau
The time has not yet come for 3.0 and junit5.
We are in progress. Both branches are not mergable. I should cherry pick changes and fix a failing IT in 3.0-rc1. First of all we have to wait for the current Vote in mailing list until Version 2.20 is ready to go out. Then we will integrate support for Jigsaw Java 9 and fixes for blocker issues.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by mgreau <gi...@git.apache.org>.
Github user mgreau commented on the issue:
https://github.com/apache/maven-surefire/pull/146
Hi,
Ok, thanks for the information.
This PR just fixes an oversight when #142 was merged on the 3.0-rc1 branch, nothing more.
> Then we will integrate support for Jigsaw Java 9 and fixes for blocker issues.
Do you mean that JUnit5 support and Jigsaw Java 9 support will be done on the same version?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by britter <gi...@git.apache.org>.
Github user britter commented on the issue:
https://github.com/apache/maven-surefire/pull/146
@Tibor17 I'm currently working on a `JUnitPlattformProviderInfo`. I have some issues with the ITs, which I will bring up on the ML.
In the mean time, I need you to merge this PR. The build in 3.0-rc1 branch is currently broken if we don't fix this, since the surefire-provider project references an unknown module. This is because the name of the module is wrong in surefire-provider pom.xml. So PLEASE, merge this!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] maven-surefire issue #146: SUREFIRE-1330: Fix module name in surefire-provid...
Posted by britter <gi...@git.apache.org>.
Github user britter commented on the issue:
https://github.com/apache/maven-surefire/pull/146
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org