Thanks for the reminder, I upgraded the to junit5 for the PR https://github.com/apache/incubator-hudi/pull/1536 and will take an eye on when reviewing PRs.
Bhavani Sudha <[email protected]> 于2020年4月22日周三 下午3:31写道： > +1. Thanks for the update Raymond and great work on the migration. > > -Sudha > > On Tue, Apr 21, 2020 at 10:39 PM Vinoth Chandar <[email protected]> wrote: > > > +1 Appreciate the efforts, Raymond! > > > > [Wondering if there is a way to stick a checkstyle rule to this effect. > > guess it won't check for new changes alone, rather complain about > existing > > junit 4 tests?] > > > > On Tue, Apr 21, 2020 at 5:10 PM Shiyan Xu <[email protected]> > > wrote: > > > > > Hi all, > > > > > > We're in progress with JUnit 5 migration for all test classes. So far > the > > > JUnit 5 dependencies (including Mockito) have been added to all > modules. > > > The APIs/modules migration status is shown here > > > https://github.com/apache/incubator-hudi/pull/1530#issue-405575235 > > > > > > I would like to kindly ask for support from the community in these 2 > > > aspects > > > > > > - To PR submitters: for newly added test classes, please start using > > JUnit > > > 5 APIs (org.junit.jupiter.*) > > > - To PR reviewers: please help look out for the JUnit adopt in the new > > test > > > classes > > > > > > Really appreciate the coordination efforts on this matter. > > > > > > Thank you. > > > > > > Regards, > > > Raymond > > > > > >
