Re: API for listening to generic test events

2019-05-16 Thread Benedikt Ritter
Hello,

Am Do., 16. Mai 2019 um 12:36 Uhr schrieb Tibor Digana <
tibordig...@apache.org>:

> Is it what Marc Philipp was asking me on the chat?
> The purpose looked like a top secret ;-) but it was not because Mark needed
> it for testing of some internal report.
> Now it looks like Junit5 wants to remove the Surefire report out been dead
> and move these features into Junit5.
>

My request is unrelated to JUnit 5 or how the test report is currently
created.


>
> The RunListener could be always added. So it exists in JUnit4 and TestNG
> provider for the purposes like custom report but not the standard report.


> We agreed with JUnit5 developers in OpenTes4J to propose a specification
> for a new standard report file.
> This means the runners (e.g. JUnit or TestNG) should only run the tests and
> Maven should run the build including creation of the project site reports.
> The JUnit5 must not create report file because Maven controls all regarding
> when/where and how to save and distribute the reports in the site.
> Additionally, since the Maven is part of build system it may easily
> integrate reports to CI systems, but JUnit5 must never do this; otherwise
> it would be *chaos of responsibilities*.
> Currently the IDEs handle events programatically from JUnit4 and then they
> create GUI report, so the file report is useless for such GUI.
>
> The Surefire has several kinds of reports:
> + Testset Info
> + Console Report
> + XML test report
>
> Having the test report in JUnit5 or OpenTes4J would be architecture
> misunderstanding of responsibilities.
> It is a perfect understanding of responsibilities when we introduce
> extensions of these reports in Surefire and we do so as we proposed
> together with Kristian Rosenvold and Andreas Gudian.
> The same should happen with the new report standard (not only JUnit5)
> initiated activities by JUni5 developers.
> This way we will be fully consistent and users may expect clear
> responsibility segregation between CI/Maven/JUnit5.
> So I expect designing a specification of test report by OpenTes4J
> contributors so that the tools, like Surefire etc, can internally implement
> it according to their programming limitations and technologies they use.
>
> Benedikt, you can provide PR of course. It's quite simple to do - only
> these two simple lines as follows (motivated by what exists in
> JUnitCoreProvider).
>
> https://github.com/apache/maven-surefire/blob/master/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreProvider.java#L101
>
> https://github.com/apache/maven-surefire/blob/master/surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/JUnitCoreProvider.java#L158
>
>
Yes, I know about this, but I'm talking
about org.apache.maven.surefire.report.RunListener and not about test
framework specific listeners.


> So it's quite easy because it is these two lines to add in
> JUnitPlatformProvider. Please improve the coverage with new unit tests and
> integration tests.
> Thx.


>
>
>
>
> On Thu, May 16, 2019 at 11:01 AM Benedikt Ritter 
> wrote:
>
> > Hello,
> >
> > I'm currently working on a Maven extension that needs to be notified of
> > what happened during test execution in Surefire. Currently it is only
> > possible to register test framework specific listeners via the
> 
> > setting [1]. However there is also the org.apache.maven.surefire.report.
> > RunListener interface [2]. I would like to provide my own implementation
> > for this interface, but it doesn't seem to be possible. I'm willing to
> > provide a PR to add this functionality if there are chances to get it
> > merged.
> >
> > WDYT?
> >
> > Benedikt
> >
> > [1]
> >
> >
> https://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#properties
> > [2]
> >
> >
> https://github.com/apache/maven-surefire/blob/master/surefire-api/src/main/java/org/apache/maven/surefire/report/RunListener.java
> >
>


API for listening to generic test events

2019-05-16 Thread Benedikt Ritter
Hello,

I'm currently working on a Maven extension that needs to be notified of
what happened during test execution in Surefire. Currently it is only
possible to register test framework specific listeners via the 
setting [1]. However there is also the org.apache.maven.surefire.report.
RunListener interface [2]. I would like to provide my own implementation
for this interface, but it doesn't seem to be possible. I'm willing to
provide a PR to add this functionality if there are chances to get it
merged.

WDYT?

Benedikt

[1]
https://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#properties
[2]
https://github.com/apache/maven-surefire/blob/master/surefire-api/src/main/java/org/apache/maven/surefire/report/RunListener.java


Re: [VOTE] Release Apache Maven Surefire Plugin version 2.22.2

2019-04-28 Thread Benedikt Ritter
Hello,

Am Sa., 27. Apr. 2019 um 18:04 Uhr schrieb Enrico Olivelli <
eolive...@gmail.com>:

> Hi,
>
> We solved 1 issue:
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12345433&styleName=Text&projectId=12317927&Create=Create&atl_token=A5KQ-2QAV-T4JA-FDED%7C568123541489d632001256d1b9872c5af17721f9%7Clout
>
> There are still a couple of issues left in JIRA:
>
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20SUREFIRE%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
>
> Staging repo:
> https://repository.apache.org/content/repositories/maven-1500/
>
> https://repository.apache.org/content/repositories/maven-1500/org/apache/maven/surefire/surefire/2.22.2/surefire-2.22.2-source-release.zip
>
> Source release checksum(s):
> surefire-2.22.2-source-release.zip
> sha512:
> de47c5f8343bc531fd78704e7cee35d3aa7496c8c631ba6ac8d6b3ee75d2ccaebf2f985b295b7ae5f44dc440c3a1ffa33e2993e0270690cdff8cd2f3c6cac3e8
>
> Staging site:
> https://maven.apache.org/surefire-archives/surefire-LATEST/
>
> Guide to testing staged releases:
> https://maven.apache.org/guides/development/guide-testing-releases.html
>
> Vote open for at least 72 hours.
>
> [ ] +1
> [ ] +0
> [ ] -1
>

The proposed artifacts passed all tests in our internal integration test
suite.

+1 (non-binding)

Benedikt


>
> Enrico Olivelli
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: Surefire maintenance release

2019-04-26 Thread Benedikt Ritter
Am Do., 25. Apr. 2019 um 22:08 Uhr schrieb Enrico Olivelli <
eolive...@gmail.com>:

> FYI
> I have staged the release artifacts at
> https://repository.apache.org/content/repositories/maven-1500/
> and created the tag
>
> but I have to do some burocratic steps in JIRA before sending the
> official [VOTE] email
> I have sent other emails on this mailing list in order to complete my task
>
> @Stephane Nicoll if you want to try out the artifacts you are welcome.
> Anyway I will expect an official +1 on the VOTE thread
>
> I am sorry that the procedure takes so long but the staging proceure
> (mvn release:prepare...release:perform) must run all of the Its and it
> longs 2h
>

Helllo Enrico,

the staged artifacts passed our internal test suite.

Regards,
Benedikt


>
> Enrico
>
> Il giorno gio 25 apr 2019 alle ore 10:25 Enrico Olivelli
>  ha scritto:
> >
> > Staging the artifact now.
> > It will take the whole day I guess
> >
> > Enrico
> >
> > Il mer 24 apr 2019, 13:21 Enrico Olivelli  ha
> scritto:
> >>
> >> Il giorno mer 24 apr 2019 alle ore 12:30 Tibor Digana
> >>  ha scritto:
> >> >
> >> > What a test has failed?
> >> > In this CI job all tests have passed successfully and the job is
> "blue".
> >>
> >> You are right !
> >> My browser should have get messed somehow
> >>
> >> So we are good to go
> >> sorry for beeing so late
> >>
> >> Enrico
> >>
> >> >
> >> > On Wed, Apr 24, 2019 at 8:28 AM Enrico Olivelli 
> wrote:
> >> >
> >> > > I am sorry,
> >> > > I had other priorities, this task is not still complete.
> >> > >
> >> > > Tests are still failing and this is weird because I think I saw
> them green.
> >> > >
> >> > > This is the link to the job
> >> > >
> >> > >
> >> > >
> https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven-surefire/job/release%252F2.22.2/
> >> > >
> >> > > Enrico
> >> > >
> >> > > Il gio 11 apr 2019, 08:11 Christian Stein  ha
> scritto:
> >> > >
> >> > > > On Thu, Apr 11, 2019 at 8:09 AM Enrico Olivelli <
> eolive...@gmail.com>
> >> > > > wrote:
> >> > > >
> >> > > > > This is the final branch from which I will cut the release.
> >> > > > > https://github.com/apache/maven-surefire/tree/release/2.22.2
> >> > > > >
> >> > > > > Re-launched Jenkins to check for the last time:
> >> > > > >
> >> > > > >
> >> > > >
> >> > >
> https://builds.apache.org/job/maven-box/job/maven-surefire/job/release%252F2.22.2/
> >> > > > >
> >> > > > >
> >> > > > Go, Jenkins, go! ;-)
> >> > > >
> >> > >
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: [MJAVADOC] Where can I find the latest SNAPSHOT?

2019-04-25 Thread Benedikt Ritter
Am Do., 25. Apr. 2019 um 10:23 Uhr schrieb Olivier Lamy :

> Hi
> 3.1.1-SNAPSHOT just deployed. (
>
> https://repository.apache.org/content/repositories/snapshots/org/apache/maven/plugins/maven-javadoc-plugin/3.1.1-SNAPSHOT/maven-javadoc-plugin-3.1.1-20190425.073608-1.jar
> )
> hth
> Olivier
>

Cool, thank you!


>
> On Thu, 25 Apr 2019 at 17:04, Benedikt Ritter  wrote:
>
> > Hi,
> >
> > I'd like to try my work project with the latest SNAPSHOT of the
> > maven-javadoc-plugin. The repository has 3.1.1-SNAPSHOT as latest version
> > [1] but the ASF snapshot repository only has 3.0.2-SNAPSHOT [2]. Any
> chance
> > of getting 3.1.1-SNAPSHOT into the snapshot repo?
> >
> > Thanks!
> > Benedikt
> >
> > [1]
> https://github.com/apache/maven-javadoc-plugin/blob/master/pom.xml#L33
> > [2]
> >
> >
> https://repository.apache.org/content/repositories/snapshots/org/apache/maven/plugins/maven-javadoc-plugin/
> >
>
>
> --
> Olivier Lamy
> http://twitter.com/olamy | http://linkedin.com/in/olamy
>


[MJAVADOC] Where can I find the latest SNAPSHOT?

2019-04-25 Thread Benedikt Ritter
Hi,

I'd like to try my work project with the latest SNAPSHOT of the
maven-javadoc-plugin. The repository has 3.1.1-SNAPSHOT as latest version
[1] but the ASF snapshot repository only has 3.0.2-SNAPSHOT [2]. Any chance
of getting 3.1.1-SNAPSHOT into the snapshot repo?

Thanks!
Benedikt

[1] https://github.com/apache/maven-javadoc-plugin/blob/master/pom.xml#L33
[2]
https://repository.apache.org/content/repositories/snapshots/org/apache/maven/plugins/maven-javadoc-plugin/


Re: Summary of meeting about Maven performance improvements

2019-04-24 Thread Benedikt Ritter
Am Mi., 24. Apr. 2019 um 10:50 Uhr schrieb Benedikt Ritter <
brit...@apache.org>:

> Hello,
>
> this is a summary of a video conference call that happened yesterday
> (April 24).
>

Sorry, actually yesterday was April 23... :o)


>
> Topic:
> Discussion about performance improvements that have been proposed by
> Stefan Oehme, namely:
>
> - [MNG-6638] - Prevent reparsing POMs in MavenMetadataSource (
> https://github.com/apache/maven/pull/244)
> - [MNG-6633] - Reduce memory usage of excludes (
> https://github.com/apache/maven/pull/243) - Speed up project discovery (
> https://github.com/apache/maven/pull/242) - Make location handling more
> memory efficient (https://github.com/codehaus-plexus/modello/pull/31)
>
> The goal of this call was to give some more insights into how Stefan found
> the improvements and to better understand what is missing before these
> changes be merged.
>
> Attendees of the call:
> - Benedikt Ritter (Gradle Inc.)
> - Stefan Oehme (Gradle Inc.)
> - Robert Scholte (Apache Maven Team)
> - Hervé Boutemy (Apache Maven Team; joined about half an hour after the
> call started)
>
> Summary:
>
> Stefan gave some insights into how he discovered bottlenecks in Maven:
>
>-
>
>One of our customers has a huge Maven build:
>-
>
>   Lots of sub projects (2000)
>   -
>
>   Lots of entries in dependency management (4000)
>   -
>
>   Results in a lot of garbage collection
>   -
>
>Problems discovered in that build:
>-
>
>   Re-parsing project POMs during dependency resolution
>   -
>
>   Model objects are too large because of location tracking
>   -
>
>   Low-level bottlenecks in project discovery (especially version
>   parsing)
>   -
>
>Customer now has a Maven fork with the proposed changes included:
>-
>
>   1h 50min, 12GB RAM without changes
>   -
>
>   45min, 8GB RAM with changes
>
>
> Robert:
>
>-
>
>How to ensure that improvements are not broken?
>-
>
>No answer to how to test this
>
>
> Stefan gave some insights into how performance testing works in the Gradle
> project:
>
>-
>
>Build has a project generator
>-
>
>Create different projects in different shapes (e.g. lots of
>subprojects, deeply nested projects) during the build
>-
>
>Download old Gradle version and run the build on generated projects
>-
>
>Run build again with current Gradle version
>-
>
>Compare results
>-
>
>use statistic methods to filter out variance
>-
>
>Downside to this approach is that it requires a lot of computing
>resources
>
> More information can be found on GitHub:
> https://github.com/gradle/gradle/tree/master/subprojects/performance
> The corresponding TeamCity build can be found here:
> https://builds.gradle.org/viewLog.html?buildId=22179604&buildTypeId=Gradle_Check_PerformanceExperimentCoordinator&tab=report_project941_Performance&branch_Gradle_Check_Stage_ReadyforRelease=master
>  (use
> "Login as guest" to view)
>
> Robert:
>
>-
>
>What about measuring performance using instruction calls?
>
>
> Stefan:
>
>-
>
>The performance improvements we found were mostly about garbage being
>created
>-
>
>Measuring using instruction calls is interesting
>-
>
>... but it is also very machine dependent
>
>
> Robert:
>
>-
>
>We need to find out who is interested in these kind improvements
>inside the Maven community.
>-
>
>Build a community of people who would like to work on these kind of
>things.
>
>
> Stefan:
>
>-
>
>It's easy to get started. We just used open source tools:
>-
>
>We used async-profiler for measuring things (
>https://github.com/jvm-profiling-tools/async-profiler)
>-
>
>Heap dumps for analyzing memory usage
>
> To get started with performance tests in the maven project:
>
>-
>
>Start with only a few test projects
>-
>
>The Gradle generator is Apache License v2 and can be used as a
>starting point to generate a big maven project
>
>
> Hervé:
>
>-
>
>PRs should be merged soon
>-
>
>Discussion need to be resolved
>-
>
>Why was the PR not merged after the discussion and resolving all
>issues with the code?
>-
>
>Hervé will take care that the changes are merged soon
>
>
> Thank you!
> Benedikt
>


Summary of meeting about Maven performance improvements

2019-04-24 Thread Benedikt Ritter
Hello,

this is a summary of a video conference call that happened yesterday (April
24).

Topic:
Discussion about performance improvements that have been proposed by Stefan
Oehme, namely:

- [MNG-6638] - Prevent reparsing POMs in MavenMetadataSource (
https://github.com/apache/maven/pull/244)
- [MNG-6633] - Reduce memory usage of excludes (
https://github.com/apache/maven/pull/243) - Speed up project discovery (
https://github.com/apache/maven/pull/242) - Make location handling more
memory efficient (https://github.com/codehaus-plexus/modello/pull/31)

The goal of this call was to give some more insights into how Stefan found
the improvements and to better understand what is missing before these
changes be merged.

Attendees of the call:
- Benedikt Ritter (Gradle Inc.)
- Stefan Oehme (Gradle Inc.)
- Robert Scholte (Apache Maven Team)
- Hervé Boutemy (Apache Maven Team; joined about half an hour after the
call started)

Summary:

Stefan gave some insights into how he discovered bottlenecks in Maven:

   -

   One of our customers has a huge Maven build:
   -

  Lots of sub projects (2000)
  -

  Lots of entries in dependency management (4000)
  -

  Results in a lot of garbage collection
  -

   Problems discovered in that build:
   -

  Re-parsing project POMs during dependency resolution
  -

  Model objects are too large because of location tracking
  -

  Low-level bottlenecks in project discovery (especially version
  parsing)
  -

   Customer now has a Maven fork with the proposed changes included:
   -

  1h 50min, 12GB RAM without changes
  -

  45min, 8GB RAM with changes


Robert:

   -

   How to ensure that improvements are not broken?
   -

   No answer to how to test this


Stefan gave some insights into how performance testing works in the Gradle
project:

   -

   Build has a project generator
   -

   Create different projects in different shapes (e.g. lots of subprojects,
   deeply nested projects) during the build
   -

   Download old Gradle version and run the build on generated projects
   -

   Run build again with current Gradle version
   -

   Compare results
   -

   use statistic methods to filter out variance
   -

   Downside to this approach is that it requires a lot of computing
   resources

More information can be found on GitHub:
https://github.com/gradle/gradle/tree/master/subprojects/performance
The corresponding TeamCity build can be found here:
https://builds.gradle.org/viewLog.html?buildId=22179604&buildTypeId=Gradle_Check_PerformanceExperimentCoordinator&tab=report_project941_Performance&branch_Gradle_Check_Stage_ReadyforRelease=master
(use
"Login as guest" to view)

Robert:

   -

   What about measuring performance using instruction calls?


Stefan:

   -

   The performance improvements we found were mostly about garbage being
   created
   -

   Measuring using instruction calls is interesting
   -

   ... but it is also very machine dependent


Robert:

   -

   We need to find out who is interested in these kind improvements inside
   the Maven community.
   -

   Build a community of people who would like to work on these kind of
   things.


Stefan:

   -

   It's easy to get started. We just used open source tools:
   -

   We used async-profiler for measuring things (
   https://github.com/jvm-profiling-tools/async-profiler)
   -

   Heap dumps for analyzing memory usage

To get started with performance tests in the maven project:

   -

   Start with only a few test projects
   -

   The Gradle generator is Apache License v2 and can be used as a starting
   point to generate a big maven project


Hervé:

   -

   PRs should be merged soon
   -

   Discussion need to be resolved
   -

   Why was the PR not merged after the discussion and resolving all issues
   with the code?
   -

   Hervé will take care that the changes are merged soon


Thank you!
Benedikt


Re: [SUREFIRE] maven-metadata.xml does not point to latest snapshot

2019-02-14 Thread Benedikt Ritter
Am Do., 14. Feb. 2019 um 14:22 Uhr schrieb Tibor Digana <
tibordig...@apache.org>:

> Benedikt, it does not mean that something with public IP is for public use.
> We made only a compromise for some users to verify some functionality.
>

Please read the information about Apache repositories:
https://www.apache.org/dev/repository-faq.html


>
> On Thu, Feb 14, 2019 at 11:42 AM Benedikt Ritter 
> wrote:
>
> > Am Mi., 13. Feb. 2019 um 17:48 Uhr schrieb Tibor Digana <
> > tibordig...@apache.org>:
> >
> > > Is it so a big problem to keep both versions? This is our internal
> Nexus
> > > server anyway.
> > >
> >
> > From my PoV the metadata is incorrect. Latest should point to the latest
> > available build. At the moment it does not. The Snapshot repository is
> not
> > an internal Nexus server. It is a public artifact repository for trying
> out
> > new things. I think the Maven project should get its own metadata
> right...
> > Why are you resisting? Do you see problems when updating master to
> > 3.0.0-SNAPSHOT?
> >
> > Benedikt
> >
> >
> > >
> > > On Wed, Feb 13, 2019 at 1:27 PM Benedikt Ritter 
> > > wrote:
> > >
> > > > Hello,
> > > >
> > > > Am Di., 12. Feb. 2019 um 08:39 Uhr schrieb Benedikt Ritter <
> > > > brit...@apache.org>:
> > > >
> > > > > Hello,
> > > > >
> > > > > Am Mo., 11. Feb. 2019 um 18:39 Uhr schrieb Enrico Olivelli <
> > > > > eolive...@gmail.com>:
> > > > >
> > > > >> In maven coordinates 3.0.0 is different from 3.0.0-M4 (you already
> > > > knew),
> > > > >> so they can coexist.
> > > > >>
> > > > >
> > > > > Yes, but the question is, what is latest available snapshot? The
> > > > > 3.0.0-SNAPSHOT has been deployed sometime last June.
> > > > >
> > > > >
> > > > >>
> > > > >> How are you referring to surefire in your project?
> > > > >>
> > > > >
> > > > > We have some end to end tests that determine the latest snapshots
> of
> > > some
> > > > > plugins so we can run tests against them. We do this by analyzing
> the
> > > > > maven-metadata.xml so we don't have to hard code all the versions.
> > > > >
> > > > >
> > > > >>
> > > > >> Btw I will check and try to deploy current master
> > > > >>
> > > > >
> > > > > I checked, it didn't help. Maybe the master branch should have
> > > > > 3.0.0-SNAPSHOT as version and only be updated to a milestone
> version
> > > for
> > > > a
> > > > > milestone release.
> > > > >
> > > >
> > > > the problem is still present. I suspect maven can not deal with
> > milestone
> > > > snapshot version. My suggestion is to change the version on the
> master
> > > > branch to 3.0.0-SNAPSHOT and create milestone releases without having
> > an
> > > > explicit milestone snapshot first. This would fix the metadata in
> that
> > > the
> > > > latest version would again be the latest deployment from the master
> > > branch.
> > > >
> > > > Regards,
> > > > Benedikt
> > > >
> > > >
> > > > >
> > > > > Thank you!
> > > > > Benedikt
> > > > >
> > > > >
> > > > >>
> > > > >> Enrico
> > > > >>
> > > > >> Il giorno lun 11 feb 2019, 14:59 Benedikt Ritter <
> > brit...@apache.org>
> > > > ha
> > > > >> scritto:
> > > > >>
> > > > >> > Hi,
> > > > >> >
> > > > >> > I just realized that the maven-metadata.xml for Surefire in the
> > > > snapshot
> > > > >> > repository [1] does not point to the latest Surefire snapshot.
> It
> > > says
> > > > >> that
> > > > >> > 3.0.0-SNAPSHOT is the latest version but 3.0.0-M4-SNAPSHOT is
> > > actually
> > > > >> the
> > > > >> > latest available snapshot. Is it possible to correct this?
> > > > >> >
> > > > >> > Thanks,
> > > > >> > Benedikt
> > > > >> >
> > > > >> > [1]
> > > > >> >
> > > > >> >
> > > > >>
> > > >
> > >
> >
> https://repository.apache.org/content/groups/snapshots/org/apache/maven/plugins/maven-surefire-plugin/maven-metadata.xml
> > > > >> >
> > > > >>
> > > > >
> > > >
> > >
> >
>


Re: [SUREFIRE] maven-metadata.xml does not point to latest snapshot

2019-02-14 Thread Benedikt Ritter
Am Mi., 13. Feb. 2019 um 17:48 Uhr schrieb Tibor Digana <
tibordig...@apache.org>:

> Is it so a big problem to keep both versions? This is our internal Nexus
> server anyway.
>

>From my PoV the metadata is incorrect. Latest should point to the latest
available build. At the moment it does not. The Snapshot repository is not
an internal Nexus server. It is a public artifact repository for trying out
new things. I think the Maven project should get its own metadata right...
Why are you resisting? Do you see problems when updating master to
3.0.0-SNAPSHOT?

Benedikt


>
> On Wed, Feb 13, 2019 at 1:27 PM Benedikt Ritter 
> wrote:
>
> > Hello,
> >
> > Am Di., 12. Feb. 2019 um 08:39 Uhr schrieb Benedikt Ritter <
> > brit...@apache.org>:
> >
> > > Hello,
> > >
> > > Am Mo., 11. Feb. 2019 um 18:39 Uhr schrieb Enrico Olivelli <
> > > eolive...@gmail.com>:
> > >
> > >> In maven coordinates 3.0.0 is different from 3.0.0-M4 (you already
> > knew),
> > >> so they can coexist.
> > >>
> > >
> > > Yes, but the question is, what is latest available snapshot? The
> > > 3.0.0-SNAPSHOT has been deployed sometime last June.
> > >
> > >
> > >>
> > >> How are you referring to surefire in your project?
> > >>
> > >
> > > We have some end to end tests that determine the latest snapshots of
> some
> > > plugins so we can run tests against them. We do this by analyzing the
> > > maven-metadata.xml so we don't have to hard code all the versions.
> > >
> > >
> > >>
> > >> Btw I will check and try to deploy current master
> > >>
> > >
> > > I checked, it didn't help. Maybe the master branch should have
> > > 3.0.0-SNAPSHOT as version and only be updated to a milestone version
> for
> > a
> > > milestone release.
> > >
> >
> > the problem is still present. I suspect maven can not deal with milestone
> > snapshot version. My suggestion is to change the version on the master
> > branch to 3.0.0-SNAPSHOT and create milestone releases without having an
> > explicit milestone snapshot first. This would fix the metadata in that
> the
> > latest version would again be the latest deployment from the master
> branch.
> >
> > Regards,
> > Benedikt
> >
> >
> > >
> > > Thank you!
> > > Benedikt
> > >
> > >
> > >>
> > >> Enrico
> > >>
> > >> Il giorno lun 11 feb 2019, 14:59 Benedikt Ritter 
> > ha
> > >> scritto:
> > >>
> > >> > Hi,
> > >> >
> > >> > I just realized that the maven-metadata.xml for Surefire in the
> > snapshot
> > >> > repository [1] does not point to the latest Surefire snapshot. It
> says
> > >> that
> > >> > 3.0.0-SNAPSHOT is the latest version but 3.0.0-M4-SNAPSHOT is
> actually
> > >> the
> > >> > latest available snapshot. Is it possible to correct this?
> > >> >
> > >> > Thanks,
> > >> > Benedikt
> > >> >
> > >> > [1]
> > >> >
> > >> >
> > >>
> >
> https://repository.apache.org/content/groups/snapshots/org/apache/maven/plugins/maven-surefire-plugin/maven-metadata.xml
> > >> >
> > >>
> > >
> >
>


Re: [SUREFIRE] maven-metadata.xml does not point to latest snapshot

2019-02-13 Thread Benedikt Ritter
Hello,

Am Di., 12. Feb. 2019 um 08:39 Uhr schrieb Benedikt Ritter <
brit...@apache.org>:

> Hello,
>
> Am Mo., 11. Feb. 2019 um 18:39 Uhr schrieb Enrico Olivelli <
> eolive...@gmail.com>:
>
>> In maven coordinates 3.0.0 is different from 3.0.0-M4 (you already knew),
>> so they can coexist.
>>
>
> Yes, but the question is, what is latest available snapshot? The
> 3.0.0-SNAPSHOT has been deployed sometime last June.
>
>
>>
>> How are you referring to surefire in your project?
>>
>
> We have some end to end tests that determine the latest snapshots of some
> plugins so we can run tests against them. We do this by analyzing the
> maven-metadata.xml so we don't have to hard code all the versions.
>
>
>>
>> Btw I will check and try to deploy current master
>>
>
> I checked, it didn't help. Maybe the master branch should have
> 3.0.0-SNAPSHOT as version and only be updated to a milestone version for a
> milestone release.
>

the problem is still present. I suspect maven can not deal with milestone
snapshot version. My suggestion is to change the version on the master
branch to 3.0.0-SNAPSHOT and create milestone releases without having an
explicit milestone snapshot first. This would fix the metadata in that the
latest version would again be the latest deployment from the master branch.

Regards,
Benedikt


>
> Thank you!
> Benedikt
>
>
>>
>> Enrico
>>
>> Il giorno lun 11 feb 2019, 14:59 Benedikt Ritter  ha
>> scritto:
>>
>> > Hi,
>> >
>> > I just realized that the maven-metadata.xml for Surefire in the snapshot
>> > repository [1] does not point to the latest Surefire snapshot. It says
>> that
>> > 3.0.0-SNAPSHOT is the latest version but 3.0.0-M4-SNAPSHOT is actually
>> the
>> > latest available snapshot. Is it possible to correct this?
>> >
>> > Thanks,
>> > Benedikt
>> >
>> > [1]
>> >
>> >
>> https://repository.apache.org/content/groups/snapshots/org/apache/maven/plugins/maven-surefire-plugin/maven-metadata.xml
>> >
>>
>


Re: [SUREFIRE] maven-metadata.xml does not point to latest snapshot

2019-02-11 Thread Benedikt Ritter
Hello,

Am Mo., 11. Feb. 2019 um 18:39 Uhr schrieb Enrico Olivelli <
eolive...@gmail.com>:

> In maven coordinates 3.0.0 is different from 3.0.0-M4 (you already knew),
> so they can coexist.
>

Yes, but the question is, what is latest available snapshot? The
3.0.0-SNAPSHOT has been deployed sometime last June.


>
> How are you referring to surefire in your project?
>

We have some end to end tests that determine the latest snapshots of some
plugins so we can run tests against them. We do this by analyzing the
maven-metadata.xml so we don't have to hard code all the versions.


>
> Btw I will check and try to deploy current master
>

I checked, it didn't help. Maybe the master branch should have
3.0.0-SNAPSHOT as version and only be updated to a milestone version for a
milestone release.

Thank you!
Benedikt


>
> Enrico
>
> Il giorno lun 11 feb 2019, 14:59 Benedikt Ritter  ha
> scritto:
>
> > Hi,
> >
> > I just realized that the maven-metadata.xml for Surefire in the snapshot
> > repository [1] does not point to the latest Surefire snapshot. It says
> that
> > 3.0.0-SNAPSHOT is the latest version but 3.0.0-M4-SNAPSHOT is actually
> the
> > latest available snapshot. Is it possible to correct this?
> >
> > Thanks,
> > Benedikt
> >
> > [1]
> >
> >
> https://repository.apache.org/content/groups/snapshots/org/apache/maven/plugins/maven-surefire-plugin/maven-metadata.xml
> >
>


[SUREFIRE] maven-metadata.xml does not point to latest snapshot

2019-02-11 Thread Benedikt Ritter
Hi,

I just realized that the maven-metadata.xml for Surefire in the snapshot
repository [1] does not point to the latest Surefire snapshot. It says that
3.0.0-SNAPSHOT is the latest version but 3.0.0-M4-SNAPSHOT is actually the
latest available snapshot. Is it possible to correct this?

Thanks,
Benedikt

[1]
https://repository.apache.org/content/groups/snapshots/org/apache/maven/plugins/maven-surefire-plugin/maven-metadata.xml


Re: Maven Surefire and JUnit 5

2017-10-28 Thread Benedikt Ritter
Hello Tibor,

did you have some time to do the rebase and help me get the build to work in 
the junit5 branch?

Regards,
Benedikt

> Am 18.10.2017 um 09:13 schrieb Benedikt Ritter :
> 
> Hello Tibor,
> 
> Sorry for the daily I somehow missed this mail.
> 
>> Am 15.10.2017 um 13:07 schrieb Tibor Digana :
>> 
>> Hi Benedikt,
>> 
>> I am fine now after my illness, thx.
>> I am finishing branch SUREFIRE-1262_2. Just to add few more tests and then
>> I will cut a release. I want to add a fix for SUREFIRE-1374 which would be
>> fast to do.
>> Then I am ready for you and JUnit5.
>> 
>> I have questions regarding our JUnit 5 Surefire Provider implemented in
>> Surefire.
>> 
>> Is it implemented exactly in the same like the origin by junit5 team?
>> I am asking because I saw the implementation made by junit5 team and it is
>> not like our *JUnitCoreProvider*. This means that they do not map method
>> <-> Thread in *RunListener *and thus the provider would not associate test
>> report logs and Thread properly and logs will be mixed.
> 
> We imported the code from the JUnit project some while ago. They have in the 
> mean time updated their provider code. They are planning to bring those 
> changes back to surefire, once we get of the ground with the implementation 
> in surefire. So if you’re seeing difference in the implementation in surefire 
> and in the implementation in JUnit, that must have been introduced after we 
> imported the code.
> 
>> 
>> Next question is regarding the feature re-run. It also exists in
>> *JUnitCoreProvider
>> *and *JUnit4Provider*. Does it exist in our provider too.
> 
> I don’t know, need to check the code.
> 
>> Also we need to have a feature where JUnit execution is stopped which can
>> be configured by *skipAfterFailureCount *in POM. We have implementation in
>> Surefire provider:
>> 
>> notifier.asFailFast( isFailFast() );
> 
> We have to figure out, how this is supported by JUnit 5.
> 
>> 
>> Regarding logger in forked provider, which has to do with parallel exec, I
>> think the logs go to the dump file instead of reports file, because STDOUT
>> is not wrapped in JUnit5's implementation and this should be called:
>> 
>> startCapture( listener )
>> 
>> 
>> See my questions in https://github.com/xwiki/xwiki-platform/commit/
>> 5258a22301977a7d1ee7276cdd81af7641f4f5ac
>> 
>> Do we have integration tests for these features?
> 
> We only have very limited integration tests yet. I’m planning to write more 
> integration tests and work on the structure of the integration test project, 
> once the build does work again. Currently I’m unable to exec an integration 
> test on the junit5 branch. Once this is possible again, we can write the 
> missing tests.
> 
> Thank you!
> Benedikt
> 
>> 
>> 
>> Cheers
>> Tibor
>> 
>> 
>> 
>> On Sat, Sep 30, 2017 at 10:34 AM, Benedikt Ritter 
>> wrote:
>> 
>>> Hello,
>>> 
>>> for over a year now I’m trying to help getting JUnit 5 support into Maven
>>> Surefire. This has been hard since Tibor seems to be the only one
>>> maintaining Maven Surefire and he had to come with other things.
>>> 
>>> For this reason I’d like to ask other Maven maintainers to help with the
>>> JUnit 5 support. I’m happy to do the work, but I’m constantly blocked by
>>> obscure build failures which I’m unable to resolve myself or by lack of
>>> code review und merge of changes.
>>> 
>>> - Work on JUnit5 support is currently done in the junit5 branch.
>>> - I have drafted a Provider Lookup implementation in the junit5 branch,
>>> but I don’t know whether it works because I can’t get the integration tests
>>> running
>>> - There is an open PR to merge the master branch back into junit5 branch,
>>> but it has build failures I don’t understand [1]
>>> 
>>> Please help!
>>> Cheers,
>>> Benedikt
>>> 
>>> [1] https://github.com/apache/maven-surefire/pull/165
>>> 
>>> 
>>> -
>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>>> For additional commands, e-mail: dev-h...@maven.apache.org
>>> 
>>> 
> 
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: Maven Surefire and JUnit 5

2017-10-18 Thread Benedikt Ritter
Hello Tibor,

Sorry for the daily I somehow missed this mail.

> Am 15.10.2017 um 13:07 schrieb Tibor Digana :
> 
> Hi Benedikt,
> 
> I am fine now after my illness, thx.
> I am finishing branch SUREFIRE-1262_2. Just to add few more tests and then
> I will cut a release. I want to add a fix for SUREFIRE-1374 which would be
> fast to do.
> Then I am ready for you and JUnit5.
> 
> I have questions regarding our JUnit 5 Surefire Provider implemented in
> Surefire.
> 
> Is it implemented exactly in the same like the origin by junit5 team?
> I am asking because I saw the implementation made by junit5 team and it is
> not like our *JUnitCoreProvider*. This means that they do not map method
> <-> Thread in *RunListener *and thus the provider would not associate test
> report logs and Thread properly and logs will be mixed.

We imported the code from the JUnit project some while ago. They have in the 
mean time updated their provider code. They are planning to bring those changes 
back to surefire, once we get of the ground with the implementation in 
surefire. So if you’re seeing difference in the implementation in surefire and 
in the implementation in JUnit, that must have been introduced after we 
imported the code.

> 
> Next question is regarding the feature re-run. It also exists in
> *JUnitCoreProvider
> *and *JUnit4Provider*. Does it exist in our provider too.

I don’t know, need to check the code.

> Also we need to have a feature where JUnit execution is stopped which can
> be configured by *skipAfterFailureCount *in POM. We have implementation in
> Surefire provider:
> 
> notifier.asFailFast( isFailFast() );

We have to figure out, how this is supported by JUnit 5.

> 
> Regarding logger in forked provider, which has to do with parallel exec, I
> think the logs go to the dump file instead of reports file, because STDOUT
> is not wrapped in JUnit5's implementation and this should be called:
> 
> startCapture( listener )
> 
> 
> See my questions in https://github.com/xwiki/xwiki-platform/commit/
> 5258a22301977a7d1ee7276cdd81af7641f4f5ac
> 
> Do we have integration tests for these features?

We only have very limited integration tests yet. I’m planning to write more 
integration tests and work on the structure of the integration test project, 
once the build does work again. Currently I’m unable to exec an integration 
test on the junit5 branch. Once this is possible again, we can write the 
missing tests.

Thank you!
Benedikt

> 
> 
> Cheers
> Tibor
> 
> 
> 
> On Sat, Sep 30, 2017 at 10:34 AM, Benedikt Ritter 
> wrote:
> 
>> Hello,
>> 
>> for over a year now I’m trying to help getting JUnit 5 support into Maven
>> Surefire. This has been hard since Tibor seems to be the only one
>> maintaining Maven Surefire and he had to come with other things.
>> 
>> For this reason I’d like to ask other Maven maintainers to help with the
>> JUnit 5 support. I’m happy to do the work, but I’m constantly blocked by
>> obscure build failures which I’m unable to resolve myself or by lack of
>> code review und merge of changes.
>> 
>> - Work on JUnit5 support is currently done in the junit5 branch.
>> - I have drafted a Provider Lookup implementation in the junit5 branch,
>> but I don’t know whether it works because I can’t get the integration tests
>> running
>> - There is an open PR to merge the master branch back into junit5 branch,
>> but it has build failures I don’t understand [1]
>> 
>> Please help!
>> Cheers,
>> Benedikt
>> 
>> [1] https://github.com/apache/maven-surefire/pull/165
>> 
>> 
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>> For additional commands, e-mail: dev-h...@maven.apache.org
>> 
>> 


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-10-14 Thread Benedikt Ritter
Hello,

> Am 13.10.2017 um 21:32 schrieb Tibor Digana :
> 
> This exception is expected.
> Use pluginGroup in settings.xml or try this:
> org.codehaus.mojo:tidy:1.0.0:check
> or
> org.codehaus.mojo:tidy-maven-plugin:1.0.0:check
> 
> I don't know what dummy.jar has to do with tidy plugin, it's internal code
> in surefire due to M2 API and tidy should not be influenced by.

There are to distinct issues:

1) surefire does some trickery which can not be resolved by the dependency 
plugin.
2) the tidy plugin does not work in offline mode, even if dependencies have 
been downloaded before. Why do I have to define the plugin explicitly to work 
in offline mode? It does work in online mode.

Regards,
Benedikt

> 
> Cheers
> Tibor
> 
> On Fri, Oct 13, 2017 at 6:36 PM, Benedikt Ritter  wrote:
> 
>> Hello,
>> 
>>> Am 08.10.2017 um 15:54 schrieb Hervé BOUTEMY :
>>> 
>>> Le dimanche 8 octobre 2017, 15:37:54 CEST Benedikt Ritter a écrit :
>>>> Hello Hervé
>>>> 
>>>>> Then I added a pluginManagement section to select version 3.0.2 and
>> re-ran
>>>>> the test: you'll see the output is completely different.
>>>>> 
>>>>> And there is no issue any more
>>>> 
>>>> Thank you so much, you took the time to investigate this issue! Really
>> much
>>>> appreciated. Now I wonder, why Maven uses an outdated version of the
>>>> dependency plugin. Is this a problem with the super pom?
>>> this is a choice to keep stability:
>>> if one downloads dependencies using Maven 3.0.5 then a few days later
>> builds
>>> offline with Maven 3.5, he does not have any issue (and blame
>> dependency-plugin)
>>> 
>>> remember the good practice: define your plugin versions, either in your
>> pom or
>>> by using a parent that does the job
>>> 
>>>> 
>>>> Furthermore I’ve noticed, that mvn -o test still does not work, because
>> some
>>>> surefire dependencies are missing:
>>>> 
>>>> [ERROR] Failed to execute goal
>>>> org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test
>> (default-test)
>>>> on project dependency-plugin-bug: Unable to generate classpath:
>>>> org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException:
>>>> Missing: [ERROR] --
>>>> [ERROR] 1) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
>>>> [ERROR]
>>>> [ERROR] Try downloading the file manually from the project website.
>>>> [ERROR]
>>>> [ERROR] Then, install it using the command:
>>>> [ERROR] mvn install:install-file -DgroupId=org.apache.maven.surefire
>>>> -DartifactId=surefire-junit3 -Dversion=2.12.4 -Dpackaging=jar
>>>> -Dfile=/path/to/file [ERROR]
>>>> [ERROR] Alternatively, if you host your own repository you can deploy
>> the
>>>> file there: [ERROR] mvn deploy:deploy-file
>>>> -DgroupId=org.apache.maven.surefire -DartifactId=surefire-junit3
>>>> -Dversion=2.12.4 -Dpackaging=jar -Dfile=/path/to/file -Durl=[url]
>>>> -DrepositoryId=[id] [ERROR]
>>>> [ERROR] Path to dependency:
>>>> [ERROR] 1) dummy:dummy:jar:1.0
>>>> [ERROR] 2) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
>>>> [ERROR]
>>>> [ERROR] --
>>>> [ERROR] 1 required artifact is missing.
>>>> [ERROR]
>>>> [ERROR] for artifact:
>>>> [ERROR] dummy:dummy:jar:1.0
>>>> [ERROR]
>>>> [ERROR] from the specified remote repositories:
>>>> [ERROR] central (https://repo.maven.apache.org/maven2, releases=true,
>>>> snapshots=false) [ERROR] -> [Help 1]
>>>> [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/MojoExecutionException
>>>> 
>>>> Since there are only a few dependencies missing, I will run test not in
>>>> offline modus on my CI server. But I wonder whether this is a bug.
>>> I'll have a look at this one and report
>> 
>> I made another interesting observation:
>> 
>> mvn tidy:check works, while mvn -o tidy:check does not work. It f

Re: dependency:go-offline broken?

2017-10-13 Thread Benedikt Ritter
Hello,

> Am 08.10.2017 um 15:54 schrieb Hervé BOUTEMY :
> 
> Le dimanche 8 octobre 2017, 15:37:54 CEST Benedikt Ritter a écrit :
>> Hello Hervé
>> 
>>> Then I added a pluginManagement section to select version 3.0.2 and re-ran
>>> the test: you'll see the output is completely different.
>>> 
>>> And there is no issue any more
>> 
>> Thank you so much, you took the time to investigate this issue! Really much
>> appreciated. Now I wonder, why Maven uses an outdated version of the
>> dependency plugin. Is this a problem with the super pom?
> this is a choice to keep stability:
> if one downloads dependencies using Maven 3.0.5 then a few days later builds 
> offline with Maven 3.5, he does not have any issue (and blame 
> dependency-plugin)
> 
> remember the good practice: define your plugin versions, either in your pom 
> or 
> by using a parent that does the job
> 
>> 
>> Furthermore I’ve noticed, that mvn -o test still does not work, because some
>> surefire dependencies are missing:
>> 
>> [ERROR] Failed to execute goal
>> org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test)
>> on project dependency-plugin-bug: Unable to generate classpath:
>> org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException:
>> Missing: [ERROR] --
>> [ERROR] 1) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
>> [ERROR]
>> [ERROR] Try downloading the file manually from the project website.
>> [ERROR]
>> [ERROR] Then, install it using the command:
>> [ERROR] mvn install:install-file -DgroupId=org.apache.maven.surefire
>> -DartifactId=surefire-junit3 -Dversion=2.12.4 -Dpackaging=jar
>> -Dfile=/path/to/file [ERROR]
>> [ERROR] Alternatively, if you host your own repository you can deploy the
>> file there: [ERROR] mvn deploy:deploy-file
>> -DgroupId=org.apache.maven.surefire -DartifactId=surefire-junit3
>> -Dversion=2.12.4 -Dpackaging=jar -Dfile=/path/to/file -Durl=[url]
>> -DrepositoryId=[id] [ERROR]
>> [ERROR] Path to dependency:
>> [ERROR] 1) dummy:dummy:jar:1.0
>> [ERROR] 2) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
>> [ERROR]
>> [ERROR] --
>> [ERROR] 1 required artifact is missing.
>> [ERROR]
>> [ERROR] for artifact:
>> [ERROR] dummy:dummy:jar:1.0
>> [ERROR]
>> [ERROR] from the specified remote repositories:
>> [ERROR] central (https://repo.maven.apache.org/maven2, releases=true,
>> snapshots=false) [ERROR] -> [Help 1]
>> [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/MojoExecutionException
>> 
>> Since there are only a few dependencies missing, I will run test not in
>> offline modus on my CI server. But I wonder whether this is a bug.
> I'll have a look at this one and report

I made another interesting observation:

mvn tidy:check works, while mvn -o tidy:check does not work. It fails with:

[INFO] 
[INFO] BUILD FAILURE
[INFO] 
[INFO] Total time: 0.640 s
[INFO] Finished at: 2017-10-13T18:34:32+02:00
[INFO] Final Memory: 13M/309M
[INFO] 
[ERROR] No plugin found for prefix 'tidy' in the current project and in the 
plugin groups [org.apache.maven.plugins, org.codehaus.mojo] available from the 
repositories [local (/Users/bene/workspace/github/dependency-plugin-bug/.m2), 
central (https://repo.maven.apache.org/maven2)] -> [Help 1]
[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/NoPluginFoundForPrefixException

Cheers,
Benedikt

> 
> Regards,
> 
> Hervé
> 
>> 
>> Regards,
>> Benedikt
>> 
>>> Regards,
>>> 
>>> Hervé
>>> 
>>> Le mercredi 20 septembre 2017, 22:48:15 CEST Benedikt Ritter a écrit :
>>>> Hi,
>>>> 
>>>> as far as I understand it should be possible

Re: Maven Surefire and JUnit 5

2017-10-08 Thread Benedikt Ritter
Hello Tibor

> Am 06.10.2017 um 01:18 schrieb Tibor Digana :
> 
> Hi Benedikt,
> 
> Would you agree with this plan.
> Since we try to release version 2.21.0 with Jigsaw modularity which is Java 9 
> related feature, we can make the same compromise with JUnit5 in next version 
> 2.22.0. Altough Surifire is compiled with javac -source 1.6 -target 1.6, and 
> JUnit 5/Java 1.8 provider is standalone jar file which does not force the 
> plugin itself to load Java 8 classes from the provider, we can freely work on 
> JUnit 5 provider after the version 2.21.0.Jigsaw has been released. I guess I 
> will start the release Vote next week and then we can pickup your commits 
> from the branch junit5, squash them into one single commit and rebase on the 
> top of future master/HEAD.
> I believe you want to merge some more fixes from JUnit team afterwards and 
> maybe to add some more tests.
> 
> What do you think, would it be possible for you?

This would be so great! I’m currently really blocked by the fact, that I can’t 
run the surefire build without errors. If you could help fix this, maybe by 
rebasing everything, this would be very much appreciated.

Looking forward to hearing from you!
Regards,
Benedikt

> 
> Cheers
> Tibor
> 
> On Sun, Oct 1, 2017 at 5:12 PM, Enrico Olivelli  wrote:
> Sorry
> I wanted to reply to another message from Benedikt
> Enrico
> 
> Il dom 1 ott 2017, 16:17 Karl Heinz Marbaise  ha scritto:
> 
> > Hi Enrico,
> >
> > On 01/10/17 16:00, Enrico Olivelli wrote:
> > > Benedikt,
> > > did you try to update all of your maven plugins to latest version?
> > > Can you share some stacktrace? This will give a first hint without having
> > > to build jUDDI
> >
> > This is the wrong mailing list...Users list was the subject about jUDDI
> > ;-)..
> >
> > Not related to Surefire and JUnit 5 ...
> >
> > Kind regards
> > Karl Heinz Marbaise
> > > Cheers
> > >
> > > Enrico
> > >
> > > Il sab 30 set 2017, 10:34 Benedikt Ritter  ha
> > scritto:
> > >
> > >> Hello,
> > >>
> > >> for over a year now I’m trying to help getting JUnit 5 support into
> > Maven
> > >> Surefire. This has been hard since Tibor seems to be the only one
> > >> maintaining Maven Surefire and he had to come with other things.
> > >>
> > >> For this reason I’d like to ask other Maven maintainers to help with the
> > >> JUnit 5 support. I’m happy to do the work, but I’m constantly blocked by
> > >> obscure build failures which I’m unable to resolve myself or by lack of
> > >> code review und merge of changes.
> > >>
> > >> - Work on JUnit5 support is currently done in the junit5 branch.
> > >> - I have drafted a Provider Lookup implementation in the junit5 branch,
> > >> but I don’t know whether it works because I can’t get the integration
> > tests
> > >> running
> > >> - There is an open PR to merge the master branch back into junit5
> > branch,
> > >> but it has build failures I don’t understand [1]
> > >>
> > >> Please help!
> > >> Cheers,
> > >> Benedikt
> >
> --
> 
> 
> -- Enrico Olivelli
> 
> 
> 
> -- 
> Cheers
> Tibor


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-10-08 Thread Benedikt Ritter
solver.MultipleArtifactsNotFoundException: Missing:
[ERROR] --
[ERROR] 1) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
[ERROR]
[ERROR] Try downloading the file manually from the project website.
[ERROR]
[ERROR] Then, install it using the command:
[ERROR] mvn install:install-file -DgroupId=org.apache.maven.surefire 
-DartifactId=surefire-junit3 -Dversion=2.12.4 -Dpackaging=jar 
-Dfile=/path/to/file
[ERROR]
[ERROR] Alternatively, if you host your own repository you can deploy the file 
there:
[ERROR] mvn deploy:deploy-file -DgroupId=org.apache.maven.surefire 
-DartifactId=surefire-junit3 -Dversion=2.12.4 -Dpackaging=jar 
-Dfile=/path/to/file -Durl=[url] -DrepositoryId=[id]
[ERROR]
[ERROR] Path to dependency:
[ERROR] 1) dummy:dummy:jar:1.0
[ERROR] 2) org.apache.maven.surefire:surefire-junit3:jar:2.12.4
[ERROR]
[ERROR] --
[ERROR] 1 required artifact is missing.
[ERROR]
[ERROR] for artifact:
[ERROR] dummy:dummy:jar:1.0
[ERROR]
[ERROR] from the specified remote repositories:
[ERROR] central (https://repo.maven.apache.org/maven2, releases=true, 
snapshots=false)
[ERROR] -> [Help 1]
[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/MojoExecutionException

Since there are only a few dependencies missing, I will run test not in offline 
modus on my CI server. But I wonder whether this is a bug.

Regards,
Benedikt

> 
> Regards,
> 
> Hervé
> 
> Le mercredi 20 septembre 2017, 22:48:15 CEST Benedikt Ritter a écrit :
>> Hi,
>> 
>> as far as I understand it should be possible to call mvn
>> dependency:go-offline and from there on work in offline mode (mvn -o). 
>> I’ve put a minimal example together [1] that demonstrates that this
>> currently does not work. Am I missing anything?
>> 
>> Thank you!
>> Benedikt
>> 
>> [1] https://github.com/britter/dependency-plugin-bug
>> 
>> 
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>> For additional commands, e-mail: dev-h...@maven.apache.org
> 
> 
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
> 


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-10-02 Thread Benedikt Ritter
Hello Laird

> Am 02.10.2017 um 18:42 schrieb Laird Nelson :
> 
> On Wed, Sep 27, 2017 at 4:53 AM Benedikt Ritter  wrote:
> 
>> I have a CI Build in GitLab, where I want to define a job that downloads
>> all the artifacts which are needed for the different lifecycle phases and
>> plugin executions as well as all project dependencies. I only want to
>> download all the stuff once and then copy the artifacts between the build
>> stages.
> 
> 
> Have you looked at the cache directive (
> https://docs.gitlab.com/ee/ci/yaml/#cache) and a custom settings.xml with a
>  element instead of using artifacts?

There is no need for a custom settings.xml, since you can set the local 
repository using MAVEN_OPTS. This can be done with a top level variables 
declaration in .gitlab-ci.yaml.

The cache directive won’t help you here, since it will cache dependencies 
between pipeline runs. I don’t think this is a good idea. I like my CI builds 
to run from a clean environment. However for one pipeline run, I want to 
download all dependencies upfront. But this does not seem to be possible.

Cheers,
Benedikt

> 
> Best,
> Laird


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-10-02 Thread Benedikt Ritter
Hello Alex,

> Am 01.10.2017 um 18:55 schrieb Alex O'Ree :
> 
> This is what I use to get artifacts before dropping offline (it may or
> may not help your scenario)
> 
> mvn dependency:resolve -Dclassifier=javadoc dependency:sources
> dependency:resolve-plugins -fn
> 
> This gets sources, javadocs and plugins. This works in most cases,
> however if you're using ant to download stuff or any other cases
> where's there's declared dependencies that are only resolved by
> running a specific phase or plugin, then you'll have to figure those
> out, then put them in dependency management and the above should work

I gave this a try:

export MAVEN_OPTS=-Dmaven.repo.local=/path/to/my/project.m2

mvn dependency:resolve dependency:resolve-plugins
mvn -o findbugs:check

[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ 
tourenplanung-backend ---
[WARNING] The POM for org.apache.maven:maven-project:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven:maven-core:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven:maven-artifact:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven:maven-settings:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven:maven-model:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven:maven-monitor:jar:2.0.6 is missing, no 
dependency information available
[WARNING] The POM for org.apache.maven.shared:maven-filtering:jar:1.1 is 
missing, no dependency information available
[WARNING] The POM for org.codehaus.plexus:plexus-interpolation:jar:1.13 is 
missing, no dependency information available
[INFO] 
[INFO] BUILD FAILURE
[INFO] 
[INFO] Total time: 1.482 s
[INFO] Finished at: 2017-10-02T18:30:37+02:00
[INFO] Final Memory: 22M/309M
[INFO] 
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-resources-plugin:2.6:resources 
(default-resources) on project my-project: Execution default-resources of goal 
org.apache.maven.plugins:maven-resources-plugin:2.6:resources failed: Plugin 
org.apache.maven.plugins:maven-resources-plugin:2.6 or one of its dependencies 
could not be resolved: Cannot access central 
(https://repo.maven.apache.org/maven2) in offline mode and the artifact 
classworlds:classworlds:jar:1.1-alpha-2 has not been downloaded from it before. 
-> [Help 1]
[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/PluginResolutionException

So this still doesn’t work…

Cheers,
Benedikt

> 
> 
> On Wed, Sep 27, 2017 at 5:59 PM, Brian Fox  wrote:
>> No, I don't think it will work. What you really need is a repository
>> manager to cache things locally for you. That's the recommended solution
>> any time you feel like you're going to be downloading components more than
>> once...which is really in every instance.
>> 
>> On Wed, Sep 27, 2017 at 7:53 AM, Benedikt Ritter  wrote:
>> 
>>> Hello Brain,
>>> 
>>>> Am 26.09.2017 um 23:10 schrieb Brian Fox :
>>>> 
>>>> On Mon, Sep 25, 2017 at 2:10 PM, Benedikt Ritter 
>>> wrote:
>>>> 
>>>>> Hello Brian,
>>>>> 
>>>>>> Am 20.09.2017 um 23:16 schrieb Brian Fox :
>>>>>> 
>>>>>> It's been a really long time, but I recall that there were issues
>>> getting
>>>>>> the dependencies of plugins bound to the lifecycle. This looks to be
>>> the
>>>>>> same problem. I think the documentation talked about a way to do this
>>>>>> effectively.
>>>>> 
>>>>> I’ve looked through the documentation of the dependency plugin, but I
>>> can
>>>>> not find anything about this. Do you recall where you found that
>>>>> documentation?
>>>>> 
>>>>> 
>>>> Ha no, since I wrote that goal 10 years ago I'm scratching the depth of
>>> my
>>>> memory. Maybe I just intended to describe it ;-)
>>> 
>>> Thank you for your response. Maybe I’ll just give a little more context
>>> about what I’m trying to achieve. Maybe I’

Maven Surefire and JUnit 5

2017-09-30 Thread Benedikt Ritter
Hello,

for over a year now I’m trying to help getting JUnit 5 support into Maven 
Surefire. This has been hard since Tibor seems to be the only one maintaining 
Maven Surefire and he had to come with other things.

For this reason I’d like to ask other Maven maintainers to help with the JUnit 
5 support. I’m happy to do the work, but I’m constantly blocked by obscure 
build failures which I’m unable to resolve myself or by lack of code review und 
merge of changes.

- Work on JUnit5 support is currently done in the junit5 branch.
- I have drafted a Provider Lookup implementation in the junit5 branch, but I 
don’t know whether it works because I can’t get the integration tests running
- There is an open PR to merge the master branch back into junit5 branch, but 
it has build failures I don’t understand [1]

Please help!
Cheers,
Benedikt

[1] https://github.com/apache/maven-surefire/pull/165


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-09-27 Thread Benedikt Ritter
Hello Brain,

> Am 26.09.2017 um 23:10 schrieb Brian Fox :
> 
> On Mon, Sep 25, 2017 at 2:10 PM, Benedikt Ritter  wrote:
> 
>> Hello Brian,
>> 
>>> Am 20.09.2017 um 23:16 schrieb Brian Fox :
>>> 
>>> It's been a really long time, but I recall that there were issues getting
>>> the dependencies of plugins bound to the lifecycle. This looks to be the
>>> same problem. I think the documentation talked about a way to do this
>>> effectively.
>> 
>> I’ve looked through the documentation of the dependency plugin, but I can
>> not find anything about this. Do you recall where you found that
>> documentation?
>> 
>> 
> Ha no, since I wrote that goal 10 years ago I'm scratching the depth of my
> memory. Maybe I just intended to describe it ;-)

Thank you for your response. Maybe I’ll just give a little more context about 
what I’m trying to achieve. Maybe I’m just doing the wrong thing:

I have a CI Build in GitLab, where I want to define a job that downloads all 
the artifacts which are needed for the different lifecycle phases and plugin 
executions as well as all project dependencies. I only want to download all the 
stuff once and then copy the artifacts between the build stages. I thought this 
would be possible with dependency:go-offline so that later steps can be run 
with mvn -o.
Am I missing something or is this simply not possible at the moment?

Regards,
Benedikt

> 
> 
>> Regards,
>> Benedikt
>> 
>>> 
>>> On Wed, Sep 20, 2017 at 4:48 PM, Benedikt Ritter 
>> wrote:
>>> 
>>>> Hi,
>>>> 
>>>> as far as I understand it should be possible to call mvn
>>>> dependency:go-offline and from there on work in offline mode (mvn -o).
>>>> I’ve put a minimal example together [1] that demonstrates that this
>>>> currently does not work. Am I missing anything?
>>>> 
>>>> Thank you!
>>>> Benedikt
>>>> 
>>>> [1] https://github.com/britter/dependency-plugin-bug
>>>> 
>>>> 
>>>> -
>>>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>>>> For additional commands, e-mail: dev-h...@maven.apache.org
>>>> 
>>>> 
>> 
>> 
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>> For additional commands, e-mail: dev-h...@maven.apache.org
>> 
>> 


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: dependency:go-offline broken?

2017-09-25 Thread Benedikt Ritter
Hello Brian,

> Am 20.09.2017 um 23:16 schrieb Brian Fox :
> 
> It's been a really long time, but I recall that there were issues getting
> the dependencies of plugins bound to the lifecycle. This looks to be the
> same problem. I think the documentation talked about a way to do this
> effectively.

I’ve looked through the documentation of the dependency plugin, but I can not 
find anything about this. Do you recall where you found that documentation?

Regards,
Benedikt

> 
> On Wed, Sep 20, 2017 at 4:48 PM, Benedikt Ritter  wrote:
> 
>> Hi,
>> 
>> as far as I understand it should be possible to call mvn
>> dependency:go-offline and from there on work in offline mode (mvn -o).
>> I’ve put a minimal example together [1] that demonstrates that this
>> currently does not work. Am I missing anything?
>> 
>> Thank you!
>> Benedikt
>> 
>> [1] https://github.com/britter/dependency-plugin-bug
>> 
>> 
>> -
>> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
>> For additional commands, e-mail: dev-h...@maven.apache.org
>> 
>> 


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



dependency:go-offline broken?

2017-09-20 Thread Benedikt Ritter
Hi,

as far as I understand it should be possible to call mvn dependency:go-offline 
and from there on work in offline mode (mvn -o).  I’ve put a minimal example 
together [1] that demonstrates that this currently does not work. Am I missing 
anything?

Thank you!
Benedikt

[1] https://github.com/britter/dependency-plugin-bug


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: [SUREFIRE] JUnit 5 support - current state and next steps

2017-06-11 Thread Benedikt Ritter
Hello Tibor,

thank you for sharing your thoughts and thank you for the review.

> Am 10.06.2017 um 12:48 schrieb Tibor Digana :
> 
> Both branches are messy.
> 3.0-rc1 is pretty old and not working properly because one IT fails.
> I wanted to continue on 3.0 two years ago but could not because the plugin
> was unstable and the 3.0 Extensions was undefined. The way to have
> extensions is clear to me now. Currently now the plugin is able to work
> with Maven 3 so the stability has higher preference. So I wanted release
> 2.20.1 by the end of this week and then 2.20.2 with JDK 9. After this we
> have nothing to fix in stability.
> 
> The JUnit 5 should go to one branch. I do not know why I was so liberal to
> accept pushing JUnit 5 provider to 3.0-rc1 branch. Anyway I do not want to
> push branches junit5 and 3.0-rc1 directly to master now because it is
> technically impossible. Instead I would like to create a patch from
> 3.0-rc1, test it, apply the patch in another branch on the top of master
> HEAD and commit consistent single commit to new branch 3.0-alpha1 and then
> JUnit 5 patch to another branch. These branches will be used for code
> review before pushing directly to master. This is usual process. Meanwhile
> the branches will be in progress there will be no other activity due to
> these are very big and merge conflicts should be avoided.
> 
> So I would propose committing all code related to JUnit 5 to branch junit5
> include changes in AbstractSurefireMojo and add ITs which are necessary.
> At the time when we prepare branch for code review we need to have tests to
> make sure the code is not risky.
> 

Okay, so let me propose this:

I will create a new PR targeted at the junit5 branch, where I will cherry-pick 
to commits with the provider code. Then I create another PR to remove the 
provider code vom 3.0-rc1 branch again. This way we have the junit 5 code in a 
separate branch and can decide where we want to merge it to.

Regards,
Benedikt

> 
> On Sat, Jun 10, 2017 at 11:24 AM, Tibor Digana-2 [via Maven] <
> ml+s40175n5909755...@n5.nabble.com> wrote:
> 
>> Pls give me time to read it.
>> 
>> On Sat, Jun 10, 2017 at 11:06 AM, Benedikt Ritter <[hidden email]
>> <http:///user/SendEmail.jtp?type=node&node=5909755&i=0>>
>> wrote:
>> 
>>> Hey guys,
>>> 
>>> any thoughts on this?
>>> 
>>> Benedikt
>>> 
>>> Benedikt Ritter <[hidden email]
>> <http:///user/SendEmail.jtp?type=node&node=5909755&i=1>> schrieb am Do.
>> 8. Juni 2017 um 15:16:
>>> 
>>>> Hello,
>>>> 
>>>> first of all, I’d like to apologize for not being very active over the
>>>> past few months. I’ve been busy at work and there was ApacheCON… so
>> you
>>>> know how it is :-)
>>>> 
>>>> I’d like to take some time to review where we’re standing with JUnit 5
>>>> support and what we our next steps will be. Currently the whole thing
>> is
>>> a
>>>> little bit messed up and I’m pretty much to blame for this:
>>>> 
>>>> - we have a junit5 branch, where I started to implement some
>> integration
>>>> tests for JUnit 5 support. There are no code changes to Surefire
>> itself
>>> in
>>>> this branch. It just tests that specifying the provider explicitly
>> does
>>>> work as shown in the JUnit docs.
>>>> - then we have 3.0-rc1. We have merged the Provider code from the
>> JUnit
>>>> team into this branch. But we don’t have any tests there.
>>>> - I’ve created a new PR to get work started on a ProviderInfo
>>>> implementation to enable automatic provider lookup [1]. This way users
>>>> don’t need to specify the provider explicitly anymore.
>>>> 
>>>> So what should be our next steps?
>>>> 
>>>> I think we should merge the junit5 branch into the 3.0-rc1 branch, so
>>> that
>>>> we have the existing integration tests we already implemented in place
>>> for
>>>> getting the work on the ProviderInfo started.
>>>> Then we will need some time to clean up the integration test project.
>> I
>>>> think it need to be restructured a little bit to make it easier to
>>>> understand and make it possible to run tests against several JUnit
>>> versions.
>>>> In the end we should be able to verify that all existing JUnit 4 also
>>> work
>>>> with the JUnit 5 provider.
>>>> 
>>>> @Tibor: Can you merge the junit5 branch to 3.0-rc1 branch? O

Re: [SUREFIRE] JUnit 5 support - current state and next steps

2017-06-10 Thread Benedikt Ritter
Hey guys,

any thoughts on this?

Benedikt

Benedikt Ritter  schrieb am Do. 8. Juni 2017 um 15:16:

> Hello,
>
> first of all, I’d like to apologize for not being very active over the
> past few months. I’ve been busy at work and there was ApacheCON… so you
> know how it is :-)
>
> I’d like to take some time to review where we’re standing with JUnit 5
> support and what we our next steps will be. Currently the whole thing is a
> little bit messed up and I’m pretty much to blame for this:
>
> - we have a junit5 branch, where I started to implement some integration
> tests for JUnit 5 support. There are no code changes to Surefire itself in
> this branch. It just tests that specifying the provider explicitly does
> work as shown in the JUnit docs.
> - then we have 3.0-rc1. We have merged the Provider code from the JUnit
> team into this branch. But we don’t have any tests there.
> - I’ve created a new PR to get work started on a ProviderInfo
> implementation to enable automatic provider lookup [1]. This way users
> don’t need to specify the provider explicitly anymore.
>
> So what should be our next steps?
>
> I think we should merge the junit5 branch into the 3.0-rc1 branch, so that
> we have the existing integration tests we already implemented in place for
> getting the work on the ProviderInfo started.
> Then we will need some time to clean up the integration test project. I
> think it need to be restructured a little bit to make it easier to
> understand and make it possible to run tests against several JUnit versions.
> In the end we should be able to verify that all existing JUnit 4 also work
> with the JUnit 5 provider.
>
> @Tibor: Can you merge the junit5 branch to 3.0-rc1 branch? Or should I
> create a PR for this?
>
> Best regards,
> Benedikt
>
> [1] https://github.com/apache/maven-surefire/pull/153
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


[SUREFIRE] JUnit 5 support - current state and next steps

2017-06-08 Thread Benedikt Ritter
Hello,

first of all, I’d like to apologize for not being very active over the past few 
months. I’ve been busy at work and there was ApacheCON… so you know how it is 
:-)

I’d like to take some time to review where we’re standing with JUnit 5 support 
and what we our next steps will be. Currently the whole thing is a little bit 
messed up and I’m pretty much to blame for this:

- we have a junit5 branch, where I started to implement some integration tests 
for JUnit 5 support. There are no code changes to Surefire itself in this 
branch. It just tests that specifying the provider explicitly does work as 
shown in the JUnit docs.
- then we have 3.0-rc1. We have merged the Provider code from the JUnit team 
into this branch. But we don’t have any tests there.
- I’ve created a new PR to get work started on a ProviderInfo implementation to 
enable automatic provider lookup [1]. This way users don’t need to specify the 
provider explicitly anymore.

So what should be our next steps?

I think we should merge the junit5 branch into the 3.0-rc1 branch, so that we 
have the existing integration tests we already implemented in place for getting 
the work on the ProviderInfo started.
Then we will need some time to clean up the integration test project. I think 
it need to be restructured a little bit to make it easier to understand and 
make it possible to run tests against several JUnit versions.
In the end we should be able to verify that all existing JUnit 4 also work with 
the JUnit 5 provider.

@Tibor: Can you merge the junit5 branch to 3.0-rc1 branch? Or should I create a 
PR for this?

Best regards,
Benedikt

[1] https://github.com/apache/maven-surefire/pull/153
-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[SUREFIRE] JUnit 5 surefire provider code donation

2017-02-05 Thread Benedikt Ritter
Hi all,

I’d like to give you an update about our progress with regards to the surefire 
support of JUnit 5. I didn’t have much time to work on this over the past 
couple of month, so please forgive me that it was quite quiet around this topic 
lately.

I’ve talked to Tibor about the next steps in IRC. We’ve agreed to target the 
3.0 release with JUnit 5 support. So we won’t work on JUnit 5 support for the 
2.x line. Furthermore we agreed that I ask the JUnit team to officially donate 
their provider code to the maven project so that we can start working on the 
integration.

Next I talked to Marc Philipp from the JUnit team and asked him what might be 
the best way for them to donate the code. We agreed that they will provide 
their code unchanged. I’ve created SUREFIRE-1330 [1] and Marc already uploaded 
a src archive of the provider code. The provider code is ALv2 license and all 
contributors agreed to the terms of the license [2]. So I think we can now 
start work on integrating the provider code.

Best regards,
Benedikt

[1] https://issues.apache.org/jira/browse/SUREFIRE-1330
[2] https://github.com/junit-team/junit5/issues/541


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: Lets talk about our changes openly. maven-surefire git commit: [SUREFIRE-1324] Surefire incorrectly suppresses exceptions when closing resources.

2017-01-02 Thread Benedikt Ritter
Hello,

Christian Schulte  schrieb am Di., 3. Jan. 2017 um 02:57 Uhr:

> Am 01/02/17 um 21:01 schrieb Benson Margulies:
> > On Mon, Jan 2, 2017 at 1:38 PM, Hervé BOUTEMY 
> wrote:
> >
> >> Christian,
> >>
> >> Please read Tibor's concerns:
> >> - big change,
> >> - near release (with parallel branches for JUnit 5)
> >> And I'll add: noise, like addition of final, reordering of imports,
> >> addition/
> >> suppression of empty lines
> >>
> >> Please follow Tibor's request, which tries to be as kind as possible:
> >>> So now please revert last change [1] and let's start from the ground.
> >>> We should again learn from the beginning and start communicating in the
> >>> community; otherwise this is the end of the project.
> >>
> >
> > In my uninteresting opinion, Tibor should formally veto the commit.
> That's
> > not un-nice, it's the way to express, clearly and crisply, his view that
> > the change is not acceptable without further discussion and refinement.
> On
> > the other hand, I don't think that the remarks about 'playgrounds' or
> > 'sandboxes' are appropriate or respectful. We should all assume good
> intent
> > and professional intent. We are running a CTR system here, and so we
> should
> > expect something like this to happen from time to time, where someone
> > commits with the best of intentions and someone else feels strongly that
> > more work is needed.
> >
> > Once Tibor has vetoed, Christian should revert, and then the process
> should
> > unfold from there.
> >
>
> This part of the commit does not correct any exception suppressed
> incorrectly, it really fixes a resource leak (lacking calls to close).
>

I think everybody should calm down for a moment and remember that we're all
just trying to create some great software here :-)

At the commons project we usually inform the mailing list when we start to
work on components we don't work on a regular basis. Maybe this is the kind
of transparency Tibor was asking about.
I understand Tibor's point about the surefire codebase. I suspect he feels
pretty much left alone. This is what I observed when I was working on the
JUnit 5 integration.
On the other hand I understand Christians motivation to try to make things
better.

Regards,
Benedikt


>
> <
> https://git-wip-us.apache.org/repos/asf?p=maven-surefire.git;a=commitdiff;h=e5a6b9c8d4f514100a01dea2acf1fb059e294968#patch11
> >
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


[SUREFIRE] Bring more order in the integration tests project?

2016-12-10 Thread Benedikt Ritter
Hello,

at the moment the integration test project has a directory containing
almost all integration tests. The Jira issue related tests are located in a
sub package. The test/resources directory contains all the test projects
with no further ordering. This makes it hard to navigate through the
project.
I propose to bring more order to the integration test project, by grouping
tests into sub packages (e.g. junit, testng, concurrent, ...). Further more
the structure should be reflected in the src/test/resources directory so it
becomes easier to find related test projects.

Thoughts?
Benedikt


Re: [SUREFIRE] Issues with classpath and module path

2016-11-24 Thread Benedikt Ritter
Hello,

Robert Scholte  schrieb am Do., 24. Nov. 2016 um
20:44 Uhr:

> Hi Benedikt,
>
> I noticed a new thread on the jigsaw mailinglist[1].
> It is close related to the issue I have: you don't want to "patch" java
> with arguments just to allow split packages for certain modules. I really
> hope that by chaining classloaders we can prevent this.
>

Thank you Robert. I've also forwarded this to the JUnit team.

Benedikt


>
> Robert
>
> [1]
> http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-November/010233.html
>
> On Tue, 22 Nov 2016 20:51:44 +0100, Benedikt Ritter 
> wrote:
>
> > Hello again,
> >
> > Benedikt Ritter  schrieb am Fr., 18. Nov. 2016 um
> > 08:40 Uhr:
> >
> >> Hi,
> >>
> >> I had the pleasure to meet Robert Scholte and Herve Boutemy here at
> >> ApacheCon in Seville. We talked al little bit about the progress of the
> >> JUnit 5 integration. This let us to create two sample projects which may
> >> indicate problems both within surefire and within JUnit. We're still
> >> investigating whether this are really issues. I'm in contact with the
> >> JUnit
> >> team to find out what they might need to fix and I'm reaching out to the
> >> Maven community to find out what we need to fix.
> >>
> >> The first sample project is https://github.com/britter/junit-cl
> >> It uses the configuration described at
> >>
> http://junit.org/junit5/docs/snapshot/user-guide/#running-tests-build-maven-engines-configure
> >> to
> >> configure the JUnit Jupiter test engine to run some tests. What we want
> >> to
> >> find out is, whether the test class can see the classes of the test
> >> engine
> >> (which they should not) and whether the production classes can see the
> >> test
> >> classes or the test engine classes (which they should not). This is what
> >> Robert called the "classpath problem", because everything in mangled
> >> together in one classpath. This probably needs to be fixed inside JUnit.
> >> Unfortunately currently the configuration described in the JUnit docs
> >> does
> >> not work. So instead of configuring the jupiter engine, it just reports
> >> that it can not find any engines.
> >>
> >
> > I've fixed the problem. The build now works an it fails for the right
> > reason :-)
> >
> > I tried to explain this problem to Marc Philipp from JUnit but I failed
> > to
> > understand why this is an issue at all. I mean, okay it may be cleaner if
> > we had separate classloaders, but what would be a really error scenario a
> > user might encounter when the same Classloader loads production and test
> > classes?
> >
> >
> >>
> >> The second example is https://github.com/britter/junit-java9 and it is
> >> about JUnit and Java 9 modules. Robert was sure that there is a problem
> >> with modules and that it needs to be fixed in surefire. However we did
> >> not
> >> get that example working at all, so Robert decided to investigate this
> >> some
> >> more when he is back home.
> >> Here is what we expect to fail: Suppose we have a package com.example
> >> with
> >> some classes. Then we add some tests in src/test/java in the same
> >> package.
> >> When we compile this and execute the tests, Java 9 should report that
> >> there
> >> are several jars providing classes for the com.example package.
> >>
> >
> > I've also applied the fix from above to the java9 example.
> >
> > Marc Philipp and I agreed upon tackling the Java 9 issue from both (maven
> > and junit) sites. He will talk to Sam Brannen from the Spring project and
> > ask them how they do reflection in Java 9. Furthermore he things about
> > writing to the OpenJDK Mailing list.
> > On the Maven site, Robert wanted to take a look into this. @Robert: Did
> > you
> > have some time to investigate this some more?
> >
> > Regards,
> > Benedikt
> >
> >
> >>
> >> Hope I was able to give you a heads up on what we were talking about :-)
> >>
> >> Regards,
> >> Benedikt
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: [SUREFIRE] Issues with classpath and module path

2016-11-22 Thread Benedikt Ritter
Hello again,

Benedikt Ritter  schrieb am Fr., 18. Nov. 2016 um
08:40 Uhr:

> Hi,
>
> I had the pleasure to meet Robert Scholte and Herve Boutemy here at
> ApacheCon in Seville. We talked al little bit about the progress of the
> JUnit 5 integration. This let us to create two sample projects which may
> indicate problems both within surefire and within JUnit. We're still
> investigating whether this are really issues. I'm in contact with the JUnit
> team to find out what they might need to fix and I'm reaching out to the
> Maven community to find out what we need to fix.
>
> The first sample project is https://github.com/britter/junit-cl
> It uses the configuration described at
> http://junit.org/junit5/docs/snapshot/user-guide/#running-tests-build-maven-engines-configure
>  to
> configure the JUnit Jupiter test engine to run some tests. What we want to
> find out is, whether the test class can see the classes of the test engine
> (which they should not) and whether the production classes can see the test
> classes or the test engine classes (which they should not). This is what
> Robert called the "classpath problem", because everything in mangled
> together in one classpath. This probably needs to be fixed inside JUnit.
> Unfortunately currently the configuration described in the JUnit docs does
> not work. So instead of configuring the jupiter engine, it just reports
> that it can not find any engines.
>

I've fixed the problem. The build now works an it fails for the right
reason :-)

I tried to explain this problem to Marc Philipp from JUnit but I failed to
understand why this is an issue at all. I mean, okay it may be cleaner if
we had separate classloaders, but what would be a really error scenario a
user might encounter when the same Classloader loads production and test
classes?


>
> The second example is https://github.com/britter/junit-java9 and it is
> about JUnit and Java 9 modules. Robert was sure that there is a problem
> with modules and that it needs to be fixed in surefire. However we did not
> get that example working at all, so Robert decided to investigate this some
> more when he is back home.
> Here is what we expect to fail: Suppose we have a package com.example with
> some classes. Then we add some tests in src/test/java in the same package.
> When we compile this and execute the tests, Java 9 should report that there
> are several jars providing classes for the com.example package.
>

I've also applied the fix from above to the java9 example.

Marc Philipp and I agreed upon tackling the Java 9 issue from both (maven
and junit) sites. He will talk to Sam Brannen from the Spring project and
ask them how they do reflection in Java 9. Furthermore he things about
writing to the OpenJDK Mailing list.
On the Maven site, Robert wanted to take a look into this. @Robert: Did you
have some time to investigate this some more?

Regards,
Benedikt


>
> Hope I was able to give you a heads up on what we were talking about :-)
>
> Regards,
> Benedikt
>


[SUREFIRE] Issues with classpath and module path

2016-11-17 Thread Benedikt Ritter
Hi,

I had the pleasure to meet Robert Scholte and Herve Boutemy here at
ApacheCon in Seville. We talked al little bit about the progress of the
JUnit 5 integration. This let us to create two sample projects which may
indicate problems both within surefire and within JUnit. We're still
investigating whether this are really issues. I'm in contact with the JUnit
team to find out what they might need to fix and I'm reaching out to the
Maven community to find out what we need to fix.

The first sample project is https://github.com/britter/junit-cl
It uses the configuration described at
http://junit.org/junit5/docs/snapshot/user-guide/#running-tests-build-maven-engines-configure
to
configure the JUnit Jupiter test engine to run some tests. What we want to
find out is, whether the test class can see the classes of the test engine
(which they should not) and whether the production classes can see the test
classes or the test engine classes (which they should not). This is what
Robert called the "classpath problem", because everything in mangled
together in one classpath. This probably needs to be fixed inside JUnit.
Unfortunately currently the configuration described in the JUnit docs does
not work. So instead of configuring the jupiter engine, it just reports
that it can not find any engines.

The second example is https://github.com/britter/junit-java9 and it is
about JUnit and Java 9 modules. Robert was sure that there is a problem
with modules and that it needs to be fixed in surefire. However we did not
get that example working at all, so Robert decided to investigate this some
more when he is back home.
Here is what we expect to fail: Suppose we have a package com.example with
some classes. Then we add some tests in src/test/java in the same package.
When we compile this and execute the tests, Java 9 should report that there
are several jars providing classes for the com.example package.

Hope I was able to give you a heads up on what we were talking about :-)

Regards,
Benedikt


[SUREFIRE] Incorporating JUnit 5 surefire provider

2016-10-19 Thread Benedikt Ritter
Hello,

as you might know, JUnit is EPL licensed. However there is currently an
effort going on to relicense the surefire provider under ALv2 [1]. It looks
like all contributors have accepted the relicensing, so we can incorporate
the provider code soon. Would this be okay for the maven community or do we
need to talk to the legal team to make this waterproof?

Regards,
Benedikt

[1] https://github.com/junit-team/junit5/issues/541


Re: [SUREFIRE] JUnit 5 support - how to move things forward?

2016-10-17 Thread Benedikt Ritter
FYI: the JUnit team is working on relicensing their provider code under AL
2.0 so there should not be a problem for us to accept their contribution.

https://github.com/junit-team/junit5/issues/541

BR,
Benedikt
Benedikt Ritter  schrieb am Di. 4. Okt. 2016 um 19:41:

> Hello Tibor,
>
> Tibor Digana  schrieb am Di., 4. Okt. 2016 um
> 02:14 Uhr:
>
> +1 to adopt JUnit 5 provider to Maven Surefire git branch.
> You would have to extend AbstractSurefireMojo.java with JUnit 5 provider in
> classpath builder.
>
>
> I see, we would have to implement a ProviderInfo for JUnit 5. I think I
> can work on thins.
>
>
> Do we have to keep using other license?
>
>
> JUnit is EPL. I talked to Marc and they asked all contributors to
> explicitly state that they are donating the code under the terms of EPL.
> I'm not sure how we would handle it. If they explicitly contribute the
> code, it should be no problem. Or do we have to talk to legal?
>
>
> Can we ask the JUnit team to continuously migrate stable changes to
> Surefire?
>
>
> I think there should be only one home for the provider. Either it is
> maven-surefire or it's junit but not both. So if we decide to take over the
> provider they will probably stop maintaining their own implementation
> because it is not their scope. I've volunteered to help maintaining the
> code since I'd like to get involved in the maven project and in junit. If
> that's not enough, we have to find a different solution.
>
> Regards,
> Benedikt
>
>
>
>
> On Mon, Oct 3, 2016 at 7:39 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5882104...@n5.nabble.com> wrote:
>
> > Hi,
> >
> > now that we have a separate branch for the JUnit 5 support in the
> surefire
> > repo, I'm asking myself how to much things forward. I've added some
> > additional IT implementations in my GitHub fork, but they all fail
> because
> > the 5.0.0-M2 release of junit-surefire-provider does not implement the
> > desired features.
> >
> > At this point I'm pretty much blocked: I can not pick up the latest
> > changes
> > to the JUnit 5 provider, because the JUnit team has not released it. The
> > JUnit team does not push the development of the provider further, since
> > they don't have integration tests...
> > Right now I think it would be best to start implementing a JUnit 5
> > provider
> > ourself in the junit5 branch, so we can add the missing features and have
> > it ready when JUnit 5 reaches GA.
> >
> > Thoughts?
> >
> > Benedikt
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/SUREFIRE-JUnit-5-support-
> > how-to-move-things-forward-tp5882104.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Re-SUREFIRE-JUnit-5-support-how-to-move-things-forward-tp5882138.html
> Sent from the Maven Developers mailing list archive at Nabble.com.
>
>


Re: [SUREFIRE] Parameterized Tests for Junit 4 and Junit 5? (Was: [SUREFIRE] JUnit 5 support - how to move things forward?)

2016-10-12 Thread Benedikt Ritter
Tibor Digana  schrieb am Di., 11. Okt. 2016 um
10:01 Uhr:

> Both old Jenkins builds [1] already use JDK 8.
> So this should not be a problem.
>

Perfect! I have something put together in a local branch. I think you're
going to like it :-)


>
> [1]
> https://builds.apache.org/job/maven-surefire/
> https://builds.apache.org/job/maven-surefire-windows/
>
>
> On Mon, Oct 10, 2016 at 7:03 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5882750...@n5.nabble.com> wrote:
>
> > Hello again,
> >
> > Tibor Digana <[hidden email]
> > <http:///user/SendEmail.jtp?type=node&node=5882750&i=0>> schrieb am Mi.,
> > 5. Okt. 2016 um
> > 00:05 Uhr:
> >
> > > >>Or do we want to even share the test projects and work with profiles
> > in
> > > the test project pom?
> > >
> > > I mean this.
> > >
> > > It pretty depends on what we are going to test, either:
> > > + features of surefire-junit5 provider, or
> > > + features of junit5 itself.
> > >
> > > I would say the provider in the first phase, and in the second phase we
> > > should identify junit5 features which do not exist in junit4 but may
> > > influence, e.g. Surefire report.
> > >
> > > The main purpose of integration testing is the interoperability between
> > the
> > > main process of Maven and surefire (forked jvm or in-plugin process).
> > >
> > > This would lower the development time because you can already reuse
> > > existing tests.
> > > It would be nice to have profiles for vintage and jupiter. If we find
> > out
> > > difference between reports, this can be a subject to a discovered bug.
> > > I think we can keep all IT projects and IT classes where they are and
> we
> > > can also keep sources using JUnit4 annotations together with JUnit5
> > > annotations. The best was that you split the JUni5 project into
> > > junit-jupiter-api and the core modules.
> > > If we just add junit-jupiter-api to the main  section in
> > > every POM, we do not break old tests because JUnit5 annotations do not
> > > break JUnit4 runners. If we want to run JUnit5 tests, then the profiles
> > > come into the role (one having junit-jupiter-engine
> > > <https://github.com/junit-team/junit5/tree/master/junit-jupiter-engine
> >
> > > and
> > > another profile with junit-vintage-engine
> > > <https://github.com/junit-team/junit5/tree/master/junit-vintage-engine
> >,
> >
> > > and finally profiles for old junit4 or 47). Unfortunately JUnit4 does
> > not
> > > have separate module with annotations only. Therefore we may use
> > >  to exclude it if really necessary, see
> > >
> > > http://maven.apache.org/surefire/maven-surefire-
> > plugin/examples/configuring-classpath.html
> > > Do you think this would work?
> > >
> >
> > I found a way to this. I've modified the Junit4VersionIT again to run
> > against JUnit 4 and JUnit 5.
> >
> > Drawbacks:
> > - all tests have to be run on Java 8. Otherwise we can't have the JUnit 5
> > annotations
> > - the profile for the jupiter engine also needs a dependency to junit
> 4.x.
> > Otherwise we get a compilation error because the old annotations are not
> > available.
> >
> > I think this could be a starting point. After we have merged #126 from
> > GitHub, I can build this on top the parameterized test.
> >
> > Regards,
> > Benedikt
> >
> >
> > >
> > > Cheers
> > > Tibor
> > >
> > >
> > > On Tue, Oct 4, 2016 at 7:48 PM, Benedikt Ritter [via Maven] <
> > > [hidden email]  /user/SendEmail.jtp?type=node&node=5882750&i=1>>
> > wrote:
> > >
> > > > Hello Tibor,
> > > >
> > > > Tibor Digana <[hidden email]
> > > > <http:///user/SendEmail.jtp?type=node&node=5882181&i=0>> schrieb am
> > Di.,
> > > > 4. Okt. 2016 um
> > > > 02:29 Uhr:
> > > >
> > > > > Can you simplify and speed up writing integration tests in the way
> > that
> > > > you
> > > > > would parameterize the existing JUnit 4 testing by adding Maven
> > > profiles
> > > > > (one default profile and junit5 profile) having another
> dependencies
> > > and
> > > > > @RunWith(Parameterized.class)?
> > > > > This would be cool because we can h

Re: [SUREFIRE] Hangout with Marc Philipp

2016-10-12 Thread Benedikt Ritter
Hello Tibor,

okay, I will prepare something, after we have integrated the parameterized
JUnit4VersionsIT into junit5 branch.

Tibor Digana  schrieb am Di., 11. Okt. 2016 um
19:50 Uhr:

> Sorry for my typos, again:
> *Maybe we should setup second trigger in Jenkins build for junir5 branch
> and run ASF build if any change is pushed to ASF or JUnit Git repository.*
>
> On Tue, Oct 11, 2016 at 7:48 PM, Tibor Digana 
> wrote:
>
> > We can follow already existing principle and declare JUnit5 version in
> [1].
> > There is already such good example:
> >
> > 
> > ${project.version}
> > ${testng.version}
> >
> > [1] https://github.com/apache/maven-surefire/blob/master/
> > surefire-integration-tests/pom.xml
> >
> > Regarding Surefire JUnit5 Provider, you can integrate it right now since
> > the branch *junit5 *is apart of master.
> > I think this would be quite good signal sent outside as well.
> > The drawback is a little work in *AbstractSurefireMojo.java* at the
> > starting point when the provider is being moved.
> > I don't know how easy it would be to frequently migrate all changes but
> we
> > can have some intermediate step and that is surefire-junit5 provider as a
> > new module dependent on SNAPSHOT of JUnit5 provider. This means our
> provide
> > can be a kind of wrapper around real provider in JUnit5 project. This way
> > we have ITs and immediate reflection of changes in JUnit5 project. Maybe
> we
> > should setup second trigger in Jenkins build and run ASF build if any
> > change in pushed to ASF of JUnit Git repository.
> >
> >
> >
> >
> > On Tue, Oct 11, 2016 at 1:25 PM, Tibor Digana 
> > wrote:
> >
> >> Hi Benedikt,
> >>
> >> I do not have any problem with SNAPSHOT version in the branch junit5.
> >> Even this can be very easily accomplished. Let's put property, e.g.
> >> version.junit5,  into [1] and reference to this POM via parent
> declaration
> >> in your root POM of particular IT having JUnit5 tests.
> >> [1]
> >> https://github.com/apache/maven-surefire/blob/master/surefir
> >> e-integration-tests/src/test/resources/pom.xml
> >>
> >>
> >> On Mon, Oct 10, 2016 at 6:43 PM, Benedikt Ritter [via Maven] <
> >> ml-node+s40175n5882740...@n5.nabble.com> wrote:
> >>
> >> > Hi,
> >> >
> >> > I've had the chance to talk to Marc Philipp from the JUnit team again,
> >> and
> >> > I'd like to share with you some of our discussions.
> >> >
> >> > First we talked about the progress with the surefire provider:
> >> > Currently I've done not much. We have a junit5 branch with a single PR
> >> > merged (the JUnit5VersionsIT). Further more we have some pending PRs
> >> which
> >> > need to be reviewed/merged. We haven't done any "real" development
> yet.
> >> >
> >> > I told Marc that a problem with implementing the ITs at the moment is,
> >> > that
> >> > the JUnit 5.0.0-m2 provider implementation is very limited. So we
> >> thought
> >> > it may be a good thing to depend on the SNAPSHOT of JUnit5 in the
> junit5
> >> > as
> >> > long as we do not have our own implementation. This may be fragile,
> but
> >> we
> >> > can pick up the latest changes to the provider implementation.
> >> >
> >> > Then we talked about moving the provider code from junit to
> >> > maven-surefire.
> >> > This is clearly the preferred way to go for the junit team. From a
> user
> >> > perspective it would also be better to not have a junit provider and a
> >> > maven-surefire provider for junit5.
> >> >
> >> > The problem we identified may be when this code could be released at
> >> > maven-surefire. Currently JUnit 5 GA is expected sometime in early
> 2017.
> >> > If
> >> > the provider code is moved to maven-surefire there should be a release
> >> > very
> >> > soon after the JUnit 5 GA release. Otherwise nobody can use JUnit 5
> with
> >> > maven.
> >> > The alternative would be, that JUnit releases the provider
> >> implementation
> >> > when they go GA and after that the code is moved to surefire. The
> >> problem
> >> > with this approach is, that it may lead to confusion for the users.
> >> >
> >> > My personal view is, that it would be good t

Re: [SUREFIRE] Parameterized Tests for Junit 4 and Junit 5? (Was: [SUREFIRE] JUnit 5 support - how to move things forward?)

2016-10-10 Thread Benedikt Ritter
Hello again,

Tibor Digana  schrieb am Mi., 5. Okt. 2016 um
00:05 Uhr:

> >>Or do we want to even share the test projects and work with profiles in
> the test project pom?
>
> I mean this.
>
> It pretty depends on what we are going to test, either:
> + features of surefire-junit5 provider, or
> + features of junit5 itself.
>
> I would say the provider in the first phase, and in the second phase we
> should identify junit5 features which do not exist in junit4 but may
> influence, e.g. Surefire report.
>
> The main purpose of integration testing is the interoperability between the
> main process of Maven and surefire (forked jvm or in-plugin process).
>
> This would lower the development time because you can already reuse
> existing tests.
> It would be nice to have profiles for vintage and jupiter. If we find out
> difference between reports, this can be a subject to a discovered bug.
> I think we can keep all IT projects and IT classes where they are and we
> can also keep sources using JUnit4 annotations together with JUnit5
> annotations. The best was that you split the JUni5 project into
> junit-jupiter-api and the core modules.
> If we just add junit-jupiter-api to the main  section in
> every POM, we do not break old tests because JUnit5 annotations do not
> break JUnit4 runners. If we want to run JUnit5 tests, then the profiles
> come into the role (one having junit-jupiter-engine
> <https://github.com/junit-team/junit5/tree/master/junit-jupiter-engine>
> and
> another profile with junit-vintage-engine
> <https://github.com/junit-team/junit5/tree/master/junit-vintage-engine>,
> and finally profiles for old junit4 or 47). Unfortunately JUnit4 does not
> have separate module with annotations only. Therefore we may use
>  to exclude it if really necessary, see
>
> http://maven.apache.org/surefire/maven-surefire-plugin/examples/configuring-classpath.html
> Do you think this would work?
>

I found a way to this. I've modified the Junit4VersionIT again to run
against JUnit 4 and JUnit 5.

Drawbacks:
- all tests have to be run on Java 8. Otherwise we can't have the JUnit 5
annotations
- the profile for the jupiter engine also needs a dependency to junit 4.x.
Otherwise we get a compilation error because the old annotations are not
available.

I think this could be a starting point. After we have merged #126 from
GitHub, I can build this on top the parameterized test.

Regards,
Benedikt


>
> Cheers
> Tibor
>
>
> On Tue, Oct 4, 2016 at 7:48 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5882181...@n5.nabble.com> wrote:
>
> > Hello Tibor,
> >
> > Tibor Digana <[hidden email]
> > <http:///user/SendEmail.jtp?type=node&node=5882181&i=0>> schrieb am Di.,
> > 4. Okt. 2016 um
> > 02:29 Uhr:
> >
> > > Can you simplify and speed up writing integration tests in the way that
> > you
> > > would parameterize the existing JUnit 4 testing by adding Maven
> profiles
> > > (one default profile and junit5 profile) having another dependencies
> and
> > > @RunWith(Parameterized.class)?
> > > This would be cool because we can have identical assertion statements,
> > > means behavior, for multiple providers.
> > >
> >
> > I was already thinking about this, because right now I'm duplicating a
> lot
> > of the code from the ITs. This is definitely a good idea. But right know
> I
> > don't have a clear view of how we could implement that. Do we just share
> > the test class and work with separate test projects? Or do we want to
> even
> > share the test projects and work with profiles in the test project pom?
> >
> > JUnit 5 also has support for running legacy tests (they call it
> > "vintage").
> > To make a complete IT suite, we would have to run all the JUnit 4 tests
> > against the JUnit 5 vintage engine as well.
> >
> > Lot a work ahead :-)
> >
> > Regards,
> > Benedikt
> >
> >
> > >
> > > On Mon, Oct 3, 2016 at 7:38 PM, Benedikt Ritter <[hidden email]
> > <http:///user/SendEmail.jtp?type=node&node=5882181&i=1>>
> > > wrote:
> > >
> > > > Hi,
> > > >
> > > > now that we have a separate branch for the JUnit 5 support in the
> > > surefire
> > > > repo, I'm asking myself how to much things forward. I've added some
> > > > additional IT implementations in my GitHub fork, but they all fail
> > > because
> > > > the 5.0.0-M2 release of junit-surefire-provider does not implement
> the
> > > > desired features.
&g

[SUREFIRE] Hangout with Marc Philipp

2016-10-10 Thread Benedikt Ritter
Hi,

I've had the chance to talk to Marc Philipp from the JUnit team again, and
I'd like to share with you some of our discussions.

First we talked about the progress with the surefire provider:
Currently I've done not much. We have a junit5 branch with a single PR
merged (the JUnit5VersionsIT). Further more we have some pending PRs which
need to be reviewed/merged. We haven't done any "real" development yet.

I told Marc that a problem with implementing the ITs at the moment is, that
the JUnit 5.0.0-m2 provider implementation is very limited. So we thought
it may be a good thing to depend on the SNAPSHOT of JUnit5 in the junit5 as
long as we do not have our own implementation. This may be fragile, but we
can pick up the latest changes to the provider implementation.

Then we talked about moving the provider code from junit to maven-surefire.
This is clearly the preferred way to go for the junit team. From a user
perspective it would also be better to not have a junit provider and a
maven-surefire provider for junit5.

The problem we identified may be when this code could be released at
maven-surefire. Currently JUnit 5 GA is expected sometime in early 2017. If
the provider code is moved to maven-surefire there should be a release very
soon after the JUnit 5 GA release. Otherwise nobody can use JUnit 5 with
maven.
The alternative would be, that JUnit releases the provider implementation
when they go GA and after that the code is moved to surefire. The problem
with this approach is, that it may lead to confusion for the users.

My personal view is, that it would be good to integrate the provider code
into maven-surefire as soon as possible, but I not yet firm enough with the
code base the be really productive. I hope to get a better understanding of
the codebase during the next few weeks.

Regards,
Benedikt


Re: [SUREFIRE] Jenkins job for junit5 branch

2016-10-10 Thread Benedikt Ritter
Hi Christopher,

Christofer Dutz  schrieb am Mo., 10. Okt. 2016
um 10:49 Uhr:

> Hi Guys ... I just setup such a build with the Pipeline Plugin for the
> Flex project ... is working nicely. I could help you with this, if you like?
>

I leave this to the maven guys :-)


>
>
> Chris
>
> 
> Von: Tibor Digana 
> Gesendet: Montag, 10. Oktober 2016 08:15:25
> An: dev@maven.apache.org
> Betreff: Re: [SUREFIRE] Jenkins job for junit5 branch
>
> Excellent!
>
> On Sun, Oct 9, 2016 at 2:16 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5882475...@n5.nabble.com> wrote:
>
> > Hello,
> >
> > I've created a build job for the junit5 branch [1]. It's a copy of the
> > maven-surefire job with the only difference that it does not publish
> build
> > artifacts to the snapshots repository. It maybe possible to configure
> this
> > more elegantly by using the Jenkins Pipeline Plugin, but I don't know
> how.
> > I think this is good enough for a start.
> >
> > Regards,
> > Benedikt
> >
> > [1] https://builds.apache.org/job/maven-surefire-junit5/
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/SUREFIRE-Jenkins-job-for-
> > junit5-branch-tp5882475.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Re-SUREFIRE-Jenkins-job-for-junit5-branch-tp5882626.html
> Sent from the Maven Developers mailing list archive at Nabble.com.
>


Re: [SUREFIRE] Parameterized Tests for Junit 4 and Junit 5? (Was: [SUREFIRE] JUnit 5 support - how to move things forward?)

2016-10-09 Thread Benedikt Ritter
Hello Tibor,

I've played around a bit with the idea and I think there is a problem:

If you want to compile and run test using JUnit 4 annotations using JUnit
5, you need the vintage enigne in your dependencies. Otherwise you get
compile errors, because neither jupiter engine nor jupiter api ship the old
annotations. So I end up with a dependency declaration like this:


  
org.junit.jupiter
junit-jupiter-engine
${junit.jupiter.version}
test
  
  
  
org.junit.vintage
junit-vintage-engine
${junit.vintage.version}
test
  



This will cause JUnit 5 to execute everything on the vintage engine. If we
want to share test code across JUnit versions, I need to also add the
jupiter API annotations to the test code. But I still need the vintage
engine in order to get it to compile.
Now I have a test like:

public class MixedTest
{

  @org.junit.Test
  @org.junit.api.jupiter.Test
  public test()
  {
// do something
  }

}

Which engine will execute the test? No one knows... So I don't think it's a
good idea to share test classes.

Regards,
Benedikt

Benedikt Ritter  schrieb am So., 9. Okt. 2016 um
14:25 Uhr:

> Hello Tibor,
>
> I think I understand what you're thinking of. I'll give it a try and see
> if I can come up with something useful :-)
>
> Regards,
> Benedikt
>
>
> Tibor Digana  schrieb am Mi., 5. Okt. 2016
> um 00:18 Uhr:
>
> Instead of using  in Surefire (may affect the
> ITs), there is a better trick.
> See the POM surefire-junit47.
> You will see the section of compiler endorsed classpath but Surefire has
> different one:
>
>
>
> 
>   maven-dependency-plugin
>   
> 
>   main
>   process-sources
>   
> copy
>   
>   
>
> ${project.build.directory}/endorsed
> false
> true
> 
>   
> junit
> junit
> 4.7
> jar
>   
> 
>   
> 
> 
>   test
>   process-sources
>   
> copy
>   
>   
>
> ${project.build.directory}/endorsed-test
> false
> true
> 
>   
> junit
> junit
> 4.12
> jar
>   
> 
>   
> 
>   
> 
> 
>   maven-compiler-plugin
>   
> 
>   ${project.build.directory}/endorsed
> 
> 
>   ${project.build.directory}/endorsed-test
> 
>   
> 
>
>
>
>
>
> On Tue, Oct 4, 2016 at 7:47 PM, Benedikt Ritter 
> wrote:
>
> > Hello Tibor,
> >
> > Tibor Digana  schrieb am Di., 4. Okt. 2016
> um
> > 02:29 Uhr:
> >
> > > Can you simplify and speed up writing integration tests in the way that
> > you
> > > would parameterize the existing JUnit 4 testing by adding Maven
> profiles
> > > (one default profile and junit5 profile) having another dependencies
> and
> > > @RunWith(Parameterized.class)?
> > > This would be cool because we can have identical assertion statements,
> > > means behavior, for multiple providers.
> > >
> >
> > I was already thinking about this, because right now I'm duplicating a
> lot
> > of the code from the ITs. This is definitely a good idea. But right know
> I
> > don't have a clear view of how we could implement that. Do we just share
> > the test class and work with separate test projects? Or do we want to
> even
> > share the test projects and work with profiles in the test project pom?
> >
> > JUnit 5 also has support for running legacy tests (they call it
> "vintage").
> > To make a complete IT suite, we would have to run all the JUnit 4 tests
> > against the JUnit 5 vintage engine as well.
> >
> > Lot a work ahead :-)
> >
> > Regards,
> > Benedikt
> >
> >
> > >
> > > On Mon, Oct 3, 2016 at 7:38 PM, Benedikt Ritter 
> > > wrote:
> > >
> > > > Hi,
> > > >
> > > > now that we have a separate branch for the JUnit 5 support in the
> > > surefire
> > > > repo, I'm asking myself how to much things forward. I've added some
> > > > additional IT implementations in my GitHub fork, but they all fail
> > > because
> > > > the 5.0.0-M2 release of junit-surefire-provider does not implement
> the
> > > > desired features.
> > > >
> > > > At this point I'm pretty much blocked: I can not pick up the latest
> > > changes
> > > > to the JUnit 5 provider, because the JUnit team has not released it.
> > The
> > > > JUnit team does not push the development of the provider further,
> since
> > > > they don't have integration tests...
> > > > Right now I think it would be best to start implementing a JUnit 5
> > > provider
> > > > ourself in the junit5 branch, so we can add the missing features and
> > have
> > > > it ready when JUnit 5 reaches GA.
> > > >
> > > > Thoughts?
> > > >
> > > > Benedikt
> > > >
> > >
> > >
> > >
> > > --
> > > Cheers
> > > Tibor
> > >
> >
>
>
>
> --
> Cheers
> Tibor
>
>


Re: [SUREFIRE] Parameterized Tests for Junit 4 and Junit 5? (Was: [SUREFIRE] JUnit 5 support - how to move things forward?)

2016-10-09 Thread Benedikt Ritter
Hello Tibor,

I think I understand what you're thinking of. I'll give it a try and see if
I can come up with something useful :-)

Regards,
Benedikt

Tibor Digana  schrieb am Mi., 5. Okt. 2016 um
00:18 Uhr:

> Instead of using  in Surefire (may affect the
> ITs), there is a better trick.
> See the POM surefire-junit47.
> You will see the section of compiler endorsed classpath but Surefire has
> different one:
>
>
>
> 
>   maven-dependency-plugin
>   
> 
>   main
>   process-sources
>   
> copy
>   
>   
>
> ${project.build.directory}/endorsed
> false
> true
> 
>   
> junit
> junit
> 4.7
> jar
>   
> 
>   
> 
> 
>   test
>   process-sources
>   
> copy
>   
>   
>
> ${project.build.directory}/endorsed-test
> false
> true
> 
>   
> junit
> junit
> 4.12
> jar
>   
> 
>   
> 
>   
> 
> 
>   maven-compiler-plugin
>   
> 
>   ${project.build.directory}/endorsed
> 
> 
>   ${project.build.directory}/endorsed-test
> 
>   
> 
>
>
>
>
>
> On Tue, Oct 4, 2016 at 7:47 PM, Benedikt Ritter 
> wrote:
>
> > Hello Tibor,
> >
> > Tibor Digana  schrieb am Di., 4. Okt. 2016
> um
> > 02:29 Uhr:
> >
> > > Can you simplify and speed up writing integration tests in the way that
> > you
> > > would parameterize the existing JUnit 4 testing by adding Maven
> profiles
> > > (one default profile and junit5 profile) having another dependencies
> and
> > > @RunWith(Parameterized.class)?
> > > This would be cool because we can have identical assertion statements,
> > > means behavior, for multiple providers.
> > >
> >
> > I was already thinking about this, because right now I'm duplicating a
> lot
> > of the code from the ITs. This is definitely a good idea. But right know
> I
> > don't have a clear view of how we could implement that. Do we just share
> > the test class and work with separate test projects? Or do we want to
> even
> > share the test projects and work with profiles in the test project pom?
> >
> > JUnit 5 also has support for running legacy tests (they call it
> "vintage").
> > To make a complete IT suite, we would have to run all the JUnit 4 tests
> > against the JUnit 5 vintage engine as well.
> >
> > Lot a work ahead :-)
> >
> > Regards,
> > Benedikt
> >
> >
> > >
> > > On Mon, Oct 3, 2016 at 7:38 PM, Benedikt Ritter 
> > > wrote:
> > >
> > > > Hi,
> > > >
> > > > now that we have a separate branch for the JUnit 5 support in the
> > > surefire
> > > > repo, I'm asking myself how to much things forward. I've added some
> > > > additional IT implementations in my GitHub fork, but they all fail
> > > because
> > > > the 5.0.0-M2 release of junit-surefire-provider does not implement
> the
> > > > desired features.
> > > >
> > > > At this point I'm pretty much blocked: I can not pick up the latest
> > > changes
> > > > to the JUnit 5 provider, because the JUnit team has not released it.
> > The
> > > > JUnit team does not push the development of the provider further,
> since
> > > > they don't have integration tests...
> > > > Right now I think it would be best to start implementing a JUnit 5
> > > provider
> > > > ourself in the junit5 branch, so we can add the missing features and
> > have
> > > > it ready when JUnit 5 reaches GA.
> > > >
> > > > Thoughts?
> > > >
> > > > Benedikt
> > > >
> > >
> > >
> > >
> > > --
> > > Cheers
> > > Tibor
> > >
> >
>
>
>
> --
> Cheers
> Tibor
>


[SUREFIRE] Jenkins job for junit5 branch

2016-10-09 Thread Benedikt Ritter
Hello,

I've created a build job for the junit5 branch [1]. It's a copy of the
maven-surefire job with the only difference that it does not publish build
artifacts to the snapshots repository. It maybe possible to configure this
more elegantly by using the Jenkins Pipeline Plugin, but I don't know how.
I think this is good enough for a start.

Regards,
Benedikt

[1] https://builds.apache.org/job/maven-surefire-junit5/


[SUREFIRE] Parameterized Tests for Junit 4 and Junit 5? (Was: [SUREFIRE] JUnit 5 support - how to move things forward?)

2016-10-04 Thread Benedikt Ritter
Hello Tibor,

Tibor Digana  schrieb am Di., 4. Okt. 2016 um
02:29 Uhr:

> Can you simplify and speed up writing integration tests in the way that you
> would parameterize the existing JUnit 4 testing by adding Maven profiles
> (one default profile and junit5 profile) having another dependencies and
> @RunWith(Parameterized.class)?
> This would be cool because we can have identical assertion statements,
> means behavior, for multiple providers.
>

I was already thinking about this, because right now I'm duplicating a lot
of the code from the ITs. This is definitely a good idea. But right know I
don't have a clear view of how we could implement that. Do we just share
the test class and work with separate test projects? Or do we want to even
share the test projects and work with profiles in the test project pom?

JUnit 5 also has support for running legacy tests (they call it "vintage").
To make a complete IT suite, we would have to run all the JUnit 4 tests
against the JUnit 5 vintage engine as well.

Lot a work ahead :-)

Regards,
Benedikt


>
> On Mon, Oct 3, 2016 at 7:38 PM, Benedikt Ritter 
> wrote:
>
> > Hi,
> >
> > now that we have a separate branch for the JUnit 5 support in the
> surefire
> > repo, I'm asking myself how to much things forward. I've added some
> > additional IT implementations in my GitHub fork, but they all fail
> because
> > the 5.0.0-M2 release of junit-surefire-provider does not implement the
> > desired features.
> >
> > At this point I'm pretty much blocked: I can not pick up the latest
> changes
> > to the JUnit 5 provider, because the JUnit team has not released it. The
> > JUnit team does not push the development of the provider further, since
> > they don't have integration tests...
> > Right now I think it would be best to start implementing a JUnit 5
> provider
> > ourself in the junit5 branch, so we can add the missing features and have
> > it ready when JUnit 5 reaches GA.
> >
> > Thoughts?
> >
> > Benedikt
> >
>
>
>
> --
> Cheers
> Tibor
>


Re: [SUREFIRE] JUnit 5 support - how to move things forward?

2016-10-04 Thread Benedikt Ritter
Hello Tibor,

Tibor Digana  schrieb am Di., 4. Okt. 2016 um
02:14 Uhr:

> +1 to adopt JUnit 5 provider to Maven Surefire git branch.
> You would have to extend AbstractSurefireMojo.java with JUnit 5 provider in
> classpath builder.
>

I see, we would have to implement a ProviderInfo for JUnit 5. I think I can
work on thins.


> Do we have to keep using other license?
>

JUnit is EPL. I talked to Marc and they asked all contributors to
explicitly state that they are donating the code under the terms of EPL.
I'm not sure how we would handle it. If they explicitly contribute the
code, it should be no problem. Or do we have to talk to legal?


> Can we ask the JUnit team to continuously migrate stable changes to
> Surefire?
>

I think there should be only one home for the provider. Either it is
maven-surefire or it's junit but not both. So if we decide to take over the
provider they will probably stop maintaining their own implementation
because it is not their scope. I've volunteered to help maintaining the
code since I'd like to get involved in the maven project and in junit. If
that's not enough, we have to find a different solution.

Regards,
Benedikt


>
>
> On Mon, Oct 3, 2016 at 7:39 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5882104...@n5.nabble.com> wrote:
>
> > Hi,
> >
> > now that we have a separate branch for the JUnit 5 support in the
> surefire
> > repo, I'm asking myself how to much things forward. I've added some
> > additional IT implementations in my GitHub fork, but they all fail
> because
> > the 5.0.0-M2 release of junit-surefire-provider does not implement the
> > desired features.
> >
> > At this point I'm pretty much blocked: I can not pick up the latest
> > changes
> > to the JUnit 5 provider, because the JUnit team has not released it. The
> > JUnit team does not push the development of the provider further, since
> > they don't have integration tests...
> > Right now I think it would be best to start implementing a JUnit 5
> > provider
> > ourself in the junit5 branch, so we can add the missing features and have
> > it ready when JUnit 5 reaches GA.
> >
> > Thoughts?
> >
> > Benedikt
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/SUREFIRE-JUnit-5-support-
> > how-to-move-things-forward-tp5882104.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Re-SUREFIRE-JUnit-5-support-how-to-move-things-forward-tp5882138.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


[SUREFIRE] JUnit 5 support - how to move things forward?

2016-10-03 Thread Benedikt Ritter
Hi,

now that we have a separate branch for the JUnit 5 support in the surefire
repo, I'm asking myself how to much things forward. I've added some
additional IT implementations in my GitHub fork, but they all fail because
the 5.0.0-M2 release of junit-surefire-provider does not implement the
desired features.

At this point I'm pretty much blocked: I can not pick up the latest changes
to the JUnit 5 provider, because the JUnit team has not released it. The
JUnit team does not push the development of the provider further, since
they don't have integration tests...
Right now I think it would be best to start implementing a JUnit 5 provider
ourself in the junit5 branch, so we can add the missing features and have
it ready when JUnit 5 reaches GA.

Thoughts?

Benedikt


Re: Would Commons Lang on Java 8 be a problem for the Apache Maven project?

2016-10-03 Thread Benedikt Ritter
Hi,

sorry for the late reply... :-)

Stephen Connolly  schrieb am Mo., 26. Sep.
2016 um 12:03 Uhr:

> Well another question is how much longer will we support Java 7 anyway...
> what we have said in the past is the next release line after JDK9 is
> released will only support Java 8+... now we could change that, but that is
> *currently* what we have currently stated. I suspect that a bump to Java 8
> by commons-lang would have a corresponding major version bump... would that
> also include a package name relocation (ala commons-lang3) to indicate
> breaking API changes?
>

Not necessarily. If we just add new API for example for dealing with
java.util.function.Function, we would consider this a compatible change and
not bump the major version number.

If we decide to do a major version number bump, we would change maven and
package coordinates.

But TBH, I don't see Commons Lang 4.0 any time soon, simply because we
don't have enough man power.


>
> Would the commons PMC object to us looking for maintenance releases on the
> old line (perhaps even with us committing the fixes to the maintenance line
> if necessary... IIUI commons is "open commit" so any Maven committer
> *should* have a commit bit on commons... we'd just need the commons PMC to
> assist getting releases out)?
>

Traditionally we just maintain two release lines. For Commons Lang we still
provide the 2.x release line but the main development line is 3.x. TBH
there has been no development/fixes in the 2.x line for ages. But
theoretically we could push out a new 2.x release if anything critical
shows up.
So we would usually not maintain, say 3.4.x and 3.4+, because we don't have
enough resources.

Yes Commons is open for all Apache Committers and we invite every Apache
Committer to come and collaborate at Apache Commons. So if any Maven
Committer wants to push out a Maintenance release of Component XY, we're
happy to help with that.
When it comes to releases, at the moment only the Commons PMC LDAP group
can deploy to our group ID. But if anybody shows serious interest in
pushing a release we simply grant the necessary karma.


>
> I suspect the answers to the above are all favourable... in which case I
> say "don't let us hold you back"
>

Okay, I still think it will take some time until we seriously consider
moving to Java 8 (we're currently Java 6), but it's good to get some
feedback early.

Thank you!
Benedikt


>
> On 25 September 2016 at 15:20, Robert Scholte 
> wrote:
>
> > On Sun, 25 Sep 2016 16:11:22 +0200, Benedikt Ritter 
> > wrote:
> >
> > Hello Robert,
> >>
> >> just watched your JavaOne presentation. Very interesting :-)
> >>
> >
> > thanks!
> >
> >
> >> Robert Scholte  schrieb am So., 25. Sep. 2016 um
> >> 13:48 Uhr:
> >>
> >> It depends. If you are changing existing methods to only work with
> Java8,
> >>> that would be a problem (read: we cannot upgrade). If you have both
> Java8
> >>> and pre-Java8 implementations, either by reflection or proper
> >>> encapsulated
> >>> code it'll work for us.
> >>> We do it ourselves too[1]
> >>>
> >>> for us it would be nice if the target is still 1.7
> >>>
> >>> if ( isJava8() )
> >>> { // do java8 stuff }
> >>> else
> >>> { do classic stuff } )
> >>>
> >>> if the java8 stuff uses reflection, you can build it with JDK7,
> otherwise
> >>> you must use JDK8
> >>>
> >>>
> >> We're thinking about adding APIs for dealing with e.g. Functions. So
> >> maven.compiler.source and maven.compiler.target would be 1.8. This would
> >> require downstream user to also compile with Java 8. If I understand
> >> correctly, this would be a problem for Maven, right?
> >>
> >
> > As long as we say that users can run Maven with Java7, then yes it would
> > block us from upgrading. Is that a problem? Maybe, as long as we don't
> hit
> > a bug commons-lang.
> >
> > Robert
> >
> >
> >
> >> Regards,
> >> Benedikt
> >>
> >>
> >>
> >>> Robert
> >>>
> >>> [1]
> >>>
> >>> https://maven.apache.org/shared/maven-shared-utils/xref/org/
> >>> apache/maven/shared/utils/io/FileUtils.html#L831
> >>>
> >>> On Sun, 25 Sep 2016 09:48:56 +0200, Benedikt Ritter <
> brit...@apache.org>
> >>> wrote:
> >>>
> >>> > Hi,
> >>> >
> >>> > at the Apache Commons Pr

Re: Would Commons Lang on Java 8 be a problem for the Apache Maven project?

2016-09-25 Thread Benedikt Ritter
Hello Robert,

just watched your JavaOne presentation. Very interesting :-)

Robert Scholte  schrieb am So., 25. Sep. 2016 um
13:48 Uhr:

> It depends. If you are changing existing methods to only work with Java8,
> that would be a problem (read: we cannot upgrade). If you have both Java8
> and pre-Java8 implementations, either by reflection or proper encapsulated
> code it'll work for us.
> We do it ourselves too[1]
>
> for us it would be nice if the target is still 1.7
>
> if ( isJava8() )
> { // do java8 stuff }
> else
> { do classic stuff } )
>
> if the java8 stuff uses reflection, you can build it with JDK7, otherwise
> you must use JDK8
>

We're thinking about adding APIs for dealing with e.g. Functions. So
maven.compiler.source and maven.compiler.target would be 1.8. This would
require downstream user to also compile with Java 8. If I understand
correctly, this would be a problem for Maven, right?

Regards,
Benedikt


>
> Robert
>
> [1]
>
> https://maven.apache.org/shared/maven-shared-utils/xref/org/apache/maven/shared/utils/io/FileUtils.html#L831
>
> On Sun, 25 Sep 2016 09:48:56 +0200, Benedikt Ritter 
> wrote:
>
> > Hi,
> >
> > at the Apache Commons Project we're currently discussing where we can
> > host
> > utility classes for working with the features introduced in Java 8. One
> > proposal add this to Commons Lang [1]. Since Apache Maven makes use of
> > Commons Lang, I would like to know whether it would be a problem for you
> > if
> > Commons Lang would require Java 8.
> >
> > Thank you,
> > Benedikt
> >
> > [1] http://markmail.org/message/ecxc4brpxufamuzu
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: Would Commons Lang on Java 8 be a problem for the Apache Maven project?

2016-09-25 Thread Benedikt Ritter
Hello Tibor,

Tibor Digana  schrieb am So., 25. Sep. 2016 um
10:27 Uhr:

> What a new API is needed from commons-lang:3.7 in Maven?
> Do we need to have something from the latest commons-lang and latest JDK ?
> Benedikt, can you provide us with API changes on the web so that we would
> better understand.
> I understand that commons might be needed in the integration test classes
> in Surefire project but Maven is compiles with Java 1.7. If we compiled
> Maven with Java 1.8 then we forced the users to change their CI jobs and
> infrastructure.
>

We don't have any concrete plans yet. We're just thinking about it. So I
can't give you any examples yet.

Regards,
Benedikt


>
> Cheers
> Tibor
>
>
>
> On Sun, Sep 25, 2016 at 9:49 AM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5881450...@n5.nabble.com> wrote:
>
> > Hi,
> >
> > at the Apache Commons Project we're currently discussing where we can
> host
> > utility classes for working with the features introduced in Java 8. One
> > proposal add this to Commons Lang [1]. Since Apache Maven makes use of
> > Commons Lang, I would like to know whether it would be a problem for you
> > if
> > Commons Lang would require Java 8.
> >
> > Thank you,
> > Benedikt
> >
> > [1] http://markmail.org/message/ecxc4brpxufamuzu
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/Would-Commons-Lang-on-
> > Java-8-be-a-problem-for-the-Apache-Maven-project-tp5881450.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Would-Commons-Lang-on-Java-8-be-a-problem-for-the-Apache-Maven-project-tp5881450p5881451.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


Would Commons Lang on Java 8 be a problem for the Apache Maven project?

2016-09-25 Thread Benedikt Ritter
Hi,

at the Apache Commons Project we're currently discussing where we can host
utility classes for working with the features introduced in Java 8. One
proposal add this to Commons Lang [1]. Since Apache Maven makes use of
Commons Lang, I would like to know whether it would be a problem for you if
Commons Lang would require Java 8.

Thank you,
Benedikt

[1] http://markmail.org/message/ecxc4brpxufamuzu


Re: [SUREFIRE] Branch for JUnit 5 integration?

2016-09-18 Thread Benedikt Ritter
Hello,

the branch with the first JUnit 5 IT is ready for reintegration. My
proposal is to create a separate branch for working on JUnit 5 so that we
can merge it to master only when all work on JUnit 5 support is done.

Regards,
Benedikt

Tibor Digana  schrieb am Mo., 12. Sep. 2016 um
01:13 Uhr:

> ok, go on with the branch but then please create CI job in
> https://builds.apache.org/job/maven-surefire/.
> We will nicely see how the branch is still in green.
> I want to be brief. I understand that this work takes private time, I was
> idle for cca 5 months in Surefire and recovered recently. Many colleagues
> in Maven are idle for years and only send some emails but these emails move
> us ahead which is positive as well.
>
> On Sat, Sep 10, 2016 at 2:49 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5880358...@n5.nabble.com> wrote:
>
> > Hi,
> >
> > I'm currently a bit stuck with the JUnit 5 integration, because I don't
> > really know where it should be going. At first I thought we should just
> > merge the Integration tests I'm about to write directly to master. But
> > maybe that's not the best idea. But I don't want to implement the whole
> > thing in my fork as well since this will lead to a unmanagable huge
> change
> > set when it will be reintegrated. So my proposal is to create a separate
> > branch in the maven-surefire repo I can open PRs against. This way I can
> > iteratively add more stuff in small chucks but is does not land directly
> > on
> > master.
> >
> > WDYT?
> >
> > Regards,
> > Benedikt
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/SUREFIRE-Branch-for-JUnit-
> > 5-integration-tp5880358.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Re-SUREFIRE-Branch-for-JUnit-5-integration-tp5880473.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


[SUREFIRE] Branch for JUnit 5 integration?

2016-09-10 Thread Benedikt Ritter
Hi,

I'm currently a bit stuck with the JUnit 5 integration, because I don't
really know where it should be going. At first I thought we should just
merge the Integration tests I'm about to write directly to master. But
maybe that's not the best idea. But I don't want to implement the whole
thing in my fork as well since this will lead to a unmanagable huge change
set when it will be reintegrated. So my proposal is to create a separate
branch in the maven-surefire repo I can open PRs against. This way I can
iteratively add more stuff in small chucks but is does not land directly on
master.

WDYT?

Regards,
Benedikt


Re: Code Style for IntelliJ 2016.2

2016-09-10 Thread Benedikt Ritter
Hello Tibor,

that file does not seem to work with IntelliJ 2016.2 :-( I think I will
have to configure the code style myself.

Benedikt

Tibor Digana  schrieb am Fr., 9. Sep. 2016 um
15:00 Uhr:

> Sorry I did not get notice this email. You need to configure your IDE for
> you pending PR in GitHub, right? :)
> I use IntelliJ IDEA 14.1.7. The code style for IDEA:
> https://maven.apache.org/developers/maven-idea-codestyle.xml
> https://maven.apache.org/developers/conventions/code.html
>
> On Wed, Sep 7, 2016 at 9:04 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5880158...@n5.nabble.com> wrote:
>
> > Hi maven devs,
> >
> > which code style file do you use for IntelliJ 2016.2? I've imported the
> > file from http://maven.apache.org/developers/committer-environment.html
> but
> >
> > is does not work properly (for example is does not place curly braces on
> > new lines)
> >
> > Thank you!
> > Benedikt
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> > http://maven.40175.n5.nabble.com/Code-Style-for-IntelliJ-
> > 2016-2-tp5880158.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Code-Style-for-IntelliJ-2016-2-tp5880158p5880283.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


Re: [SUREFIRE] How does Junit4VersionsIT work?

2016-09-08 Thread Benedikt Ritter
Hello Tibor,

thank you for solving the mistery :-) I will put together a PR to fix the
IT.

Regards,
Benedikt

Tibor Digana  schrieb am Do., 8. Sep. 2016 um 22:08:

> This is obviously a bug and junit.version should be used.
> If you type "unpack().debugLogging()." in the IT you will see that the
> classpath contains junit-4.4 instead of the providing one.
> There are several such cases but not all ITs alter the version.
>
> On Wed, Sep 7, 2016 at 9:10 PM, Benedikt Ritter [via Maven] <
> ml-node+s40175n5880159...@n5.nabble.com> wrote:
>
> > Hi surefire devs,
> >
> > I don't understand how the Junit4VersionsIT sets the JUnit version to use
> > during testing. The version is set as sysProp "junit.version" on
> > mavenLauncher, but the test pom.xml in the resources/junit4 folder
> > uses ${junitVersion} to set the JUnit version. How does this work? Does
> > maven automatically convert between the two property keys?
> >
> > Regards,
> > Benedikt
> >
> >
> > --
> > If you reply to this email, your message will be added to the discussion
> > below:
> >
> http://maven.40175.n5.nabble.com/SUREFIRE-How-does-Junit4VersionsIT-work-
> > tp5880159.html
> > To start a new topic under Maven Developers, email
> > ml-node+s40175n142166...@n5.nabble.com
> > To unsubscribe from Maven Developers, click here
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==
> >
> > .
> > NAML
> > <
> http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml
> >
> >
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/Re-SUREFIRE-How-does-Junit4VersionsIT-work-tp5880204.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


[SUREFIRE] How does Junit4VersionsIT work?

2016-09-07 Thread Benedikt Ritter
Hi surefire devs,

I don't understand how the Junit4VersionsIT sets the JUnit version to use
during testing. The version is set as sysProp "junit.version" on
mavenLauncher, but the test pom.xml in the resources/junit4 folder
uses ${junitVersion} to set the JUnit version. How does this work? Does
maven automatically convert between the two property keys?

Regards,
Benedikt


Code Style for IntelliJ 2016.2

2016-09-07 Thread Benedikt Ritter
Hi maven devs,

which code style file do you use for IntelliJ 2016.2? I've imported the
file from http://maven.apache.org/developers/committer-environment.html but
is does not work properly (for example is does not place curly braces on
new lines)

Thank you!
Benedikt


Re: MNG-5889 fix available

2016-09-07 Thread Benedikt Ritter
Hello Robert,

you can either create a git diff and attach it as txt file to the jira
issue or you submit a pull request using the GitHub mirror at
https://github.com/apache/maven.

Regards,
Benedikt

Robert Patrick  schrieb am Mi., 7. Sep. 2016 um
00:04 Uhr:

> I have fixed for Bug HYPERLINK "
> https://issues.apache.org/jira/browse/MNG-5889"MNG-5889.  How should I
> submit the fix?
>
>
>
>
>
> --
>
> Robert Patrick mailto:robert.patr...@oracle.com";
> robert.patr...@oracle.com>
>
> VP, OPC Development, Oracle Corporation
>
> 7460 Warren Pkwy, Ste. 300 Office: +1.972.963.2872
>
> Frisco, TX 75034, USA Mobile: +1.469.556.9450
>
>
>
> HYPERLINK "
> http://www.amazon.com/Professional-Oracle-WebLogic-Server-Patrick/dp/0470484306/"Professional
> Oracle WebLogic Server
>
> by Robert Patrick, Gregory Nyberg, and Philip Aston
>
> with Josh Bregman and Paul Done
>
> Book Home Page: HYPERLINK "
> http://www.wrox.com/WileyCDA/WroxTitle/Professional-Oracle-WebLogic-Server.productCd-0470484306.html
> "http://www.wrox.com/
>
> Kindle Version: HYPERLINK "
> http://www.amazon.com/Professional-Oracle-WebLogic-Server-ebook/dp/B004HD69J2/
> "http://www.amazon.com/
>
>
>
>
>


Re: [GitHub] maven-plugins pull request #91: MCHANGES-373

2016-09-07 Thread Benedikt Ritter
Hello Mikael,

Michael Osipov has asked two times why the change set shows the whole class
as changed. So there has been feedback.

Regards,
Benedikt

Mikael Petterson  schrieb am Mi., 7. Sep.
2016 um 08:08 Uhr:

> Hi,
>
> Is there no one that can help out with my pull request or is the
> maven-changes plugin "dead"?
> We really need this fix to be able to use the plugin. And also we want to
> be able to help out.
> It is a bit frustrating when nothing happens
>
> //mikael
>
>
> -Original Message-
> From: Mikael Petterson [mailto:mikael.petter...@ericsson.com]
> Sent: den 5 september 2016 14:48
> To: Maven Developers List 
> Subject: RE: [GitHub] maven-plugins pull request #91: MCHANGES-373
>
> Hi,
>
> Just a friendly reminder to review code and if ok commit.
>
> Br,
>
> //mikael
>
> -Original Message-
> From: mpet [mailto:g...@git.apache.org]
> Sent: den 29 augusti 2016 13:51
> To: dev@maven.apache.org
> Subject: [GitHub] maven-plugins pull request #91: MCHANGES-373
>
> GitHub user mpet opened a pull request:
>
> https://github.com/apache/maven-plugins/pull/91
>
> MCHANGES-373
>
>
>
> You can merge this pull request into a Git repository by running:
>
> $ git pull https://github.com/mpet/maven-plugins MCHANGES-373
>
> Alternatively you can review and apply these changes as the patch at:
>
> https://github.com/apache/maven-plugins/pull/91.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 #91
>
> 
> commit 699dd3bf3649c150913a0dd1eda25966bc6f6f1c
> Author: mike 
> Date:   2016-08-29T11:39:24Z
>
> MCHANGES-373
>
> 
>
>
> ---
> 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 infrastruct...@apache.org or
> file a JIRA ticket with INFRA.
> ---
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional
> commands, e-mail: dev-h...@maven.apache.org
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional
> commands, e-mail: dev-h...@maven.apache.org
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: [SUREFIRE] Surefire provider for JUnit 5?

2016-09-03 Thread Benedikt Ritter
Hello Tibor,

good idea! I've talked to Marc Philipp from the JUnit team and he want's to
join us, so we can make a plan how to move things forward. When would you
be available for a chat on freenode?

Best Regards,
Benedikt

Tibor Digana  schrieb am Di., 30. Aug. 2016 um
22:46 Uhr:

> Benedikt, let's go to IRC freenode.net for developers.
>
> On Tue, Aug 30, 2016 at 10:44 PM, Tibor Digana 
> wrote:
>
> > I would like to help you but currently I can't because I am in hurry with
> > release and few issues and then I have to fix one IT in Surefire 3.0
> which
> > fails because of unresolved transitive dependencies. So this may take
> some
> > time for me.
> >
> > On Tue, Aug 30, 2016 at 10:06 PM, Benedikt Ritter [via Maven] <
> > ml-node+s40175n587945...@n5.nabble.com> wrote:
> >
> > > Hello Tibor,
> > >
> > > Tibor Digana <[hidden email]
> > > <http:///user/SendEmail.jtp?type=node&node=5879457&i=0>> schrieb am
> Mo.,
> > > 29. Aug. 2016 um
> > > 12:56 Uhr:
> > >
> > > > Hi Benedikt,
> > > >
> > > > I found out that JUNit 5 was release with ALPHA version in Maven
> > > Central.
> > > > I guess there is no need to rush in Surefire yet.
> > > > The JUnit team should contribute in JUnit code line in the artifact
> > > > project org.junit.surefire-junit5 and test that provider. AFter the
> it
> > > > is stable with non-alpha and non-beta version we can take over, but
> > > > the next question would be license of JUnit 5.
> > > >
> > >
> > > I think the team has enough to do with getting JUnit 5 out of the door.
> > > That's why they would be happy if the provider would be moved to the
> > maven
> > > project.
> > >
> > >
> > > > JUnit 5 is developed with license Eclipse Public License v1.0.
> > > > We can accept ASF 2.0 license.
> > > > See http://repo1.maven.org/maven2/org/junit/surefire-junit5/
> > > >
> > > >
> > > EPL falls under the category of weak-copy left licenses. So this may be
> > an
> > > issue [1]. Would probably be best to check with legal before importing
> > any
> > > code.
> > >
> > > Regards,
> > > Benedikt
> > >
> > > [1] http://www.apache.org/legal/resolved.html#category-b
> > >
> > >
> > > > --
> > > > Cheers
> > > > Tibor
> > > >
> > > >
> > > >
> > > >
> > > > --
> > > > View this message in context:
> > > > http://maven.40175.n5.nabble.com/SUREFIRE-Surefire-
> > provider-for-JUnit-5-
> > > tp5879054p5879245.html
> > > > Sent from the Maven Developers mailing list archive at Nabble.com.
> > >
> > >
> > > --
> > > If you reply to this email, your message will be added to the
> discussion
> > > below:
> > >
> http://maven.40175.n5.nabble.com/SUREFIRE-Surefire-provider-for-JUnit-5-
> > > tp5879054p5879457.html
> > > To start a new topic under Maven Developers, email
> > > ml-node+s40175n142166...@n5.nabble.com
> > > To unsubscribe from Maven Developers, click here
> > > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?
> > macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3
> > wxNDIxNjZ8LTI4OTQ5MjEwMg==>
> > > .
> > > NAML
> > > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?
> > macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&
> > base=nabble.naml.namespaces.BasicNamespace-nabble.view.
> > web.template.NabbleNamespace-nabble.view.web.template.
> > NodeNamespace&breadcrumbs=notify_subscribers%21nabble%
> > 3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_
> > instant_email%21nabble%3Aemail.naml>
> > >
> >
> >
> >
> >
> > --
> > View this message in context: http://maven.40175.n5.nabble.
> > com/SUREFIRE-Surefire-provider-for-JUnit-5-tp5879054p5879471.html
> > Sent from the Maven Developers mailing list archive at Nabble.com.
> >
>


Re: [SUREFIRE] Surefire provider for JUnit 5?

2016-08-30 Thread Benedikt Ritter
Hello Tibor,

Tibor Digana  schrieb am Mo., 29. Aug. 2016 um
12:56 Uhr:

> Hi Benedikt,
>
> I found out that JUNit 5 was release with ALPHA version in Maven Central.
> I guess there is no need to rush in Surefire yet.
> The JUnit team should contribute in JUnit code line in the artifact
> project org.junit.surefire-junit5 and test that provider. AFter the it
> is stable with non-alpha and non-beta version we can take over, but
> the next question would be license of JUnit 5.
>

I think the team has enough to do with getting JUnit 5 out of the door.
That's why they would be happy if the provider would be moved to the maven
project.


> JUnit 5 is developed with license Eclipse Public License v1.0.
> We can accept ASF 2.0 license.
> See http://repo1.maven.org/maven2/org/junit/surefire-junit5/
>
>
EPL falls under the category of weak-copy left licenses. So this may be an
issue [1]. Would probably be best to check with legal before importing any
code.

Regards,
Benedikt

[1] http://www.apache.org/legal/resolved.html#category-b


> --
> Cheers
> Tibor
>
>
>
>
> --
> View this message in context:
> http://maven.40175.n5.nabble.com/SUREFIRE-Surefire-provider-for-JUnit-5-tp5879054p5879245.html
> Sent from the Maven Developers mailing list archive at Nabble.com.


Re: [SUREFIRE] Surefire provider for JUnit 5?

2016-08-28 Thread Benedikt Ritter
Hello Tibor,

Tibor Digana  schrieb am Sa., 27. Aug. 2016 um
22:48 Uhr:

> The JUnit5 team create junit5-provider which might be nice to have a
> look inside. I don't know if this provider has all features Surefire
> provided through surefire-junit47-provider, I know for certain that
> junit5-provider converts JUnit 5 Executor to JUnit 4 Runners and
> that's the way to support all features again.
>

Their idea was to contribute the code of the junit5-provider to the maven
project. So there would be a new provider sub module for JUnit 5.
Given the fact that they changed the whole architecture of JUnit in the
Jupiter release, I think it makes sense to have a new provider for this.
Who do you feel about that?

Regards,
Benedikt


> On 8/27/16, Tibor Digana  wrote:
> > hm, you know, I don't like new provider because it's a lot of work to
> > support all the features we had in surefire-junit47. This may always
> > go with bugs and finally more work for you, integration tests, ...
> >
> > On 8/27/16, Benedikt Ritter  wrote:
> >> Tibor Digana  schrieb am Sa., 27. Aug.
> 2016
> >> um
> >> 15:01 Uhr:
> >>
> >>> I won't ha ve much time for JUnit 5 provider because I am preparing
> >>> Version 2.19.2 to release and next release with blocker and critical
> >>> fix, then 3.0-RC1.
> >>>
> >>> Feel free to open pull request in GitHub for JUnit 5 provider.
> >>>
> >>
> >> Great, I'll review what has already be implemented at the JUnit project
> >> and
> >> then create a PR for integrating it into the surefire code base.
> >>
> >> Best regards,
> >> Benedikt
> >>
> >>
> >>>
> >>>
> >>> On 8/27/16, Kristian Rosenvold  wrote:
> >>> > Hi, Benedikt :)
> >>> >
> >>> > JUnit 5 provider is cool, yes please :)
> >>> >
> >>> > Providers have different language levels (the different modules have
> >>> > different language levels), and I am sure we can build with jdk8.
> jdk8
> >>> > still supports the target 1.6, right ?
> >>> >
> >>> > Kristian
> >>> >
> >>> >
> >>> >
> >>> >
> >>> > 2016-08-27 11:23 GMT+02:00 Benedikt Ritter :
> >>> >> Hi,
> >>> >>
> >>> >> I'm currently at SoCraTes 2016 [1] and I got a chance to talk to
> Marc
> >>> >> Philipp [2], who is one of the maintainers of the JUnit project. As
> >>> >> you
> >>> >> might know, the JUnit team is currently working on the next major
> >>> release
> >>> >> of JUnit (JUnit 5 a.k.a Jupiter) [3]. Quite a lot will change in
> >>> >> JUnit
> >>> >> 5
> >>> >> and Marc gave me a kick start about the new architecture. The JUnit
> >>> >> team
> >>> >> already implemented a rudimentary surefire provider for JUnit 5 [4]
> >>> >> and
> >>> >> they eventually like to donate it to the maven project. This is
> where
> >>> >> I
> >>> >> come into play, since I know Marc know and I'm a member of the ASF
> >>> >> and
> >>> >> I
> >>> >> though it would be good to help with this.
> >>> >>
> >>> >> What I'd like to find out is how we can move this forward. So here
> >>> >> are
> >>> >> two
> >>> >> questions to get us started:
> >>> >> - is the maven community interested in a donation of a surefire
> >>> >> provider
> >>> >> for JUnit 5? (I volunteer to drive that and maintain the code
> >>> afterwards)
> >>> >> - JUnit 5 is Java 8. Will this be a blocker?
> >>> >>
> >>> >> Looking forward to hearing from you,
> >>> >> Benedikt
> >>> >>
> >>> >> [1] http://socrates-conference.de/
> >>> >> [2] https://twitter.com/marcphilipp
> >>> >> [3] http://junit.org/junit5/
> >>> >> [4]
> >>> >>
> >>>
> https://github.com/junit-team/junit5/tree/master/junit-platform-surefire-provider
> >>> >
> >>> > -
> >>> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> >>> > For additional commands, e-mail: dev-h...@maven.apache.org
> >>> >
> >>> >
> >>>
> >>>
> >>> --
> >>> Cheers
> >>> Tibor
> >>>
> >>
> >
> >
> > --
> > Cheers
> > Tibor
> >
>
>
> --
> Cheers
> Tibor
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: [SUREFIRE] Surefire provider for JUnit 5?

2016-08-27 Thread Benedikt Ritter
Tibor Digana  schrieb am Sa., 27. Aug. 2016 um
15:01 Uhr:

> I won't ha ve much time for JUnit 5 provider because I am preparing
> Version 2.19.2 to release and next release with blocker and critical
> fix, then 3.0-RC1.
>
> Feel free to open pull request in GitHub for JUnit 5 provider.
>

Great, I'll review what has already be implemented at the JUnit project and
then create a PR for integrating it into the surefire code base.

Best regards,
Benedikt


>
>
> On 8/27/16, Kristian Rosenvold  wrote:
> > Hi, Benedikt :)
> >
> > JUnit 5 provider is cool, yes please :)
> >
> > Providers have different language levels (the different modules have
> > different language levels), and I am sure we can build with jdk8. jdk8
> > still supports the target 1.6, right ?
> >
> > Kristian
> >
> >
> >
> >
> > 2016-08-27 11:23 GMT+02:00 Benedikt Ritter :
> >> Hi,
> >>
> >> I'm currently at SoCraTes 2016 [1] and I got a chance to talk to Marc
> >> Philipp [2], who is one of the maintainers of the JUnit project. As you
> >> might know, the JUnit team is currently working on the next major
> release
> >> of JUnit (JUnit 5 a.k.a Jupiter) [3]. Quite a lot will change in JUnit 5
> >> and Marc gave me a kick start about the new architecture. The JUnit team
> >> already implemented a rudimentary surefire provider for JUnit 5 [4] and
> >> they eventually like to donate it to the maven project. This is where I
> >> come into play, since I know Marc know and I'm a member of the ASF and I
> >> though it would be good to help with this.
> >>
> >> What I'd like to find out is how we can move this forward. So here are
> >> two
> >> questions to get us started:
> >> - is the maven community interested in a donation of a surefire provider
> >> for JUnit 5? (I volunteer to drive that and maintain the code
> afterwards)
> >> - JUnit 5 is Java 8. Will this be a blocker?
> >>
> >> Looking forward to hearing from you,
> >> Benedikt
> >>
> >> [1] http://socrates-conference.de/
> >> [2] https://twitter.com/marcphilipp
> >> [3] http://junit.org/junit5/
> >> [4]
> >>
> https://github.com/junit-team/junit5/tree/master/junit-platform-surefire-provider
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > For additional commands, e-mail: dev-h...@maven.apache.org
> >
> >
>
>
> --
> Cheers
> Tibor
>


[SUREFIRE] Surefire provider for JUnit 5?

2016-08-27 Thread Benedikt Ritter
Hi,

I'm currently at SoCraTes 2016 [1] and I got a chance to talk to Marc
Philipp [2], who is one of the maintainers of the JUnit project. As you
might know, the JUnit team is currently working on the next major release
of JUnit (JUnit 5 a.k.a Jupiter) [3]. Quite a lot will change in JUnit 5
and Marc gave me a kick start about the new architecture. The JUnit team
already implemented a rudimentary surefire provider for JUnit 5 [4] and
they eventually like to donate it to the maven project. This is where I
come into play, since I know Marc know and I'm a member of the ASF and I
though it would be good to help with this.

What I'd like to find out is how we can move this forward. So here are two
questions to get us started:
- is the maven community interested in a donation of a surefire provider
for JUnit 5? (I volunteer to drive that and maintain the code afterwards)
- JUnit 5 is Java 8. Will this be a blocker?

Looking forward to hearing from you,
Benedikt

[1] http://socrates-conference.de/
[2] https://twitter.com/marcphilipp
[3] http://junit.org/junit5/
[4]
https://github.com/junit-team/junit5/tree/master/junit-platform-surefire-provider


Introducing myself

2016-08-27 Thread Benedikt Ritter
Hello maven community,

I'm Benedikt Ritter, 30 years old from Neuss (Germany). I'm committer and
member of the project management committee of the Apache Commons project. I
just wanted to introduce myself, since I'd like to contribute to maven
(more on that in a separate mail).
Looking forward to working with you!

Best regards,
Benedikt