Hi Ivan,
While I have your attention, I would like to say huge thanks for your
contribution to migration efforts.
I am specifically talking about trick you proposed when it started, with
using TestRule in GridAbstractTest to enable switching between JUnit 3 and 4
execution modes.
Looking back, I think that approach I originally planned (before you
proposed this way) would cause us much more hassle in migration.
REGARDS, Oleg
Павлухин Иван wrote
> Cheers!
>
> вт, 22 янв. 2019 г. в 20:19, oignatenko <
> oignatenko@
> >:
>>
>> ...sort of a follow-up. I gave folks a bit more leeway than originally
>> planned week or two.
>>
>> It's been about a month since master was changed in a way that wouldn't
>> allow JUnit 3 tests execute anymore (per IGNITE-10177) and I just started
>> progress on IGNITE-10629 and made a promised inspection for tests that
>> could
>> slip through without required annotation while we were migrating the
>> project
>> to JUnit 4.
>>
>> Results of this check look pretty good, IDEA reported only 6 (six)
>> troublesome test cases (4 of which were by the way annotated @Ignore,
>> meaning that even with added @Test these won't run anyway). Compared to
>> over
>> 11,600 properly annotated test cases this looks really minor. We probably
>> can say that tests in master migrated to JUnit 4 quite smoothly.
>>
>> regards, Oleg
>>
>>
>> Vladimir Ozerov wrote
>> > Hi Oleg,
>> >
>> > Yes, makes perfect sense. Thank you.
>> >
>> > On Mon, Dec 10, 2018 at 10:14 PM oignatenko <
>>
>> > oignatenko@
>>
>> > > wrote:
>> >
>> >> Hi Vovan,
>> >>
>> >> I just created JIRA ticket to address your concerns:
>> >> -
https://issues.apache.org/jira/browse/IGNITE-10629>> >>
>> >> In brief, the plan is that a week or two after migration is over we
>> will
>> >> run
>> >> code inspection that detects JUnit 3 style tests that lack @Test
>> >> annotation
>> >> and fix these tests if there are any.
>> >>
>> >> Does that answer your question?
>> >>
>> >> regards, Oleg
>> >> Vladimir Ozerov wrote
>> >> > Ed,
>> >> >
>> >> > Several questions from my side:
>> >> > 1) When the change is expected to be merged?
>> >> > 2) What contributors with opened PRs and new or updated JUnit3 tests
>> >> are
>> >> > supposed to do? Rewrite to JUnit4?
>> >> >
>> >> > If yes, then we should give them time to have a chance to get used
>> to
>> >> new
>> >> > approach and resolve possible conflicts.
>> >> >
>> >> > Vladimir.
>> >> >
>> >> [cut]
>> >>
>> >> --
>> >> Sent from:
http://apache-ignite-developers.2346864.n4.nabble.com/>> >>
>>
>>
>>
>>
>>
>> --
>> Sent from:
http://apache-ignite-developers.2346864.n4.nabble.com/>
>
>
> --
> Best regards,
> Ivan Pavlukhin
--
Sent from:
http://apache-ignite-developers.2346864.n4.nabble.com/
Locked
