3. Should I wait for 1.20 release ? Not commenting on the rest of the proposal but will definitely need to wait until after the 1.19.0 release.
Kevin Risden On Sun, Mar 10, 2019 at 5:43 AM Muhammad Gelbana <m.gelb...@gmail.com> wrote: > The only benefit I can think of out of updating JUnit or our maven plugins > in general, is getting closer to have a faster build if we leverage maven's > or JUnit's parallel execution feature. > > Unfortunately it's still experimental for maven and JUnit5. > > But it's a good idea to stay a couple of steps behind rather than having to > do a lot of changes at once. > > Thanks, > Gelbana > > > On Sun, Mar 10, 2019 at 5:32 AM Andrei Sereda <and...@sereda.cc> wrote: > > > We already require maven 3.5.2 (or newer) > > > > On Sat, Mar 9, 2019, 21:49 YuZhao Chan <yuzhao....@gmail.com> wrote: > > > > > Maven version 3.6.0 seems a too much new version, now most of the > > > developers use 3.2.x and 3.3.x version maven, i think it will be bad to > > run > > > fail test cases just because the maven version is old. > > > > > > Best, > > > YuZhao Chen > > > 在 2019年3月10日 +0800 AM1:28,Andrei Sereda <and...@sereda.cc>,写道: > > > > Greetings, > > > > > > > > > > > > I would like to start a gradual migration of calcite test codebase to > > > > [JUnit5](https://junit.org/junit5/). The plan is to do in several > > steps > > > > outlined below : > > > > > > > > 1. Upgrade maven wrapper to 3.6.0 (surefire plugin needs to work with > > > > JUnit5 >= 2.22.0). Maybe enforce maven 3.6.0 during builds. > > > > 2. Add new dependencies to maven pom (jupiter and vantage). > > > > 3. Migrate all basic tests to new JUnit5 API. Basic in this context > > means > > > > tests without [rules]( > https://github.com/junit-team/junit4/wiki/rules) > > > or > > > > [runners]( > > > > > > > > > > https://github.com/junit-team/junit4/wiki/test-runners#runwith-annotation) > > > > just basic `@Test` / `@Before` / `@Ignore` annotations. Code where I > > can > > > > just apply string/replace and make it work in JUnit5. > > > > 4. Migrate remaining tests (with `@Parameterized` / `@ClassRule` > etc.). > > > For > > > > example, I will have to write custom extensions for existing elastic > / > > > > mongo / cassandra / geode class rules. > > > > > > > > For developers that means you will need to have a reasonably recent > > IDE / > > > > Maven: > > > > 1. For IntelliJ this is >= 2016.2 > > > > 2. For Eclipse this is >= Oxygen.1a (4.7.1a) > > > > 3. For Maven >= 3.6.0 (released on 2018-10-24) > > > > > > > > Questions to fellow calcitians: > > > > > > > > 1. Do you agree with JUnit5 migration ? > > > > 2. Do you agree with the plan ? > > > > 3. Should I wait for 1.20 release ? > > > > 4. Anything I missed ? > > > > > > > > Regards, > > > > Andrei. > > > > > >