[
https://issues.apache.org/jira/browse/HADOOP-14693?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17402174#comment-17402174
]
Steve Loughran commented on HADOOP-14693:
-----------------------------------------
There's an IntelliJ article on the topic too
https://blog.jetbrains.com/idea/2020/08/migrating-from-junit-4-to-junit-5/
One big PITA is that the Assertion classes have changed. I think it's a real
shame that this has changed as it is generating needless work.
Similarly, @Ignore attributes now need to move to @Disabled. Again, I think
this is "unfortunate"
Even with the automated migration and IDE support this is going to make back
porting really painful. Normally all test suites which avoid Mockito go back to
any of version without problems; now we (I!) are going to have to change them,
recompile, and rerun.
I note that we have alternatives to some of the JUnit methods which will
backport seamlessly
* AssertJ assertions
* {{ContractTestUtils.skip(message)}} over {{@Ignore}}
* LambdaTestUtils.intercept over @Expected (also, it provides better
diagnostics with the toString() of what the l-expression was, etc etc.)
It's makes me think: can we enforce the use of those over the JUnit5 versions?
Or at least have checkstyle recommending them.
AssertJ is a bit verbose for simple asserts, and I would not advocate replacing
all existing assertTrue, assertEquals etc with them, but we could
# warn on new code with the classic JUnit asserts
# warn on @Disabled, @Expected
# reject hamcrest asserts entirely
Given that AssertJ and intercept are, IMO, better, we can justify this on test
Qquality rather than just ease of back porting.
> Upgrade JUnit from 4 to 5
> -------------------------
>
> Key: HADOOP-14693
> URL: https://issues.apache.org/jira/browse/HADOOP-14693
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Akira Ajisaka
> Priority: Major
> Labels: pull-request-available
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> JUnit 4 does not support Java 9. We need to upgrade this.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
