GitHub user jon-bell opened a pull request:
https://github.com/apache/maven-surefire/pull/161
SUREFIRE-1396: Provider class path is incorrect for custom provider in
Failsafe
Hi,
When using a custom Surefire provider with Surefire (not Failsafe), the
"provider clas
Github user jon-bell commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/161#discussion_r130447222
--- Diff:
surefire-integration-tests/src/test/resources/surefire-1396-pluggableproviders-classpath-provider/pom.xml
---
@@ -0,0 +1,61
Github user jon-bell commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/161#discussion_r130446456
--- Diff:
surefire-integration-tests/src/test/resources/surefire-1396-pluggableproviders-classpath-provider/pom.xml
---
@@ -0,0 +1,61
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
@Tibor17 It seems like it had worked for me because I had previously
executed a `mvn install` - I see on a clean maven repo the problem.
It looks like the `SurefireLauncher` does
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
Thanks - I made the fixes you suggested above, and confirmed that the build
passed on `mvn clean install -P run-its`
---
If your project is set up for it, you can reply to this email
Github user jon-bell commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/161#discussion_r130497860
--- Diff:
surefire-integration-tests/src/test/resources/surefire-1396-pluggableproviders-classpath-provider/pom.xml
---
@@ -0,0 +1,61
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
Done
---
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
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
@Tibor17 Let me know if there is anything else I can do on this.
Thanks
Jonathan
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
@Tibor17 I have rebased master on top of my branch, it is again up to date.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jon-bell commented on the issue:
https://github.com/apache/maven-surefire/pull/161
@Tibor17 Is there anything else I can do to help get this merged?
---
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
10 matches
Mail list logo