You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "Matt Sicker (Jira)" <ji...@apache.org> on 2022/01/17 21:20:00 UTC

[jira] [Commented] (LOG4J2-2653) Migrate to JUnit 5

    [ https://issues.apache.org/jira/browse/LOG4J2-2653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17477422#comment-17477422 ] 

Matt Sicker commented on LOG4J2-2653:
-------------------------------------

Added CassandraFixture in master to replace old CassandraRule.

> Migrate to JUnit 5
> ------------------
>
>                 Key: LOG4J2-2653
>                 URL: https://issues.apache.org/jira/browse/LOG4J2-2653
>             Project: Log4j 2
>          Issue Type: Improvement
>          Components: Tests
>    Affects Versions: 3.0.0
>            Reporter: Matt Sicker
>            Assignee: Matt Sicker
>            Priority: Major
>             Fix For: 3.0.0
>
>
> JUnit 5 has a ton of neat features that make writing tests easier and more expressive. It provides a compatibility layer for JUnit 4 and 3, so we should be able to migrate pretty easily there.
> In order to migrate v4 tests to v5, we'll need to support the following:
>  * LoggerContextRule should have an equivalent ParameterResolver [extension|https://junit.org/junit5/docs/current/user-guide/#extensions] which injects a LoggerContext parameter among other types. (This class should remain for JUnit 4 users, and an abstraction might help in making something equivalent available for JMH tests as well).
>  ** This is currently implemented via the LoggerContextSource annotation.
>  * Test categories should be converted to [tags|https://junit.org/junit5/docs/current/user-guide/#writing-tests-tagging-and-filtering].
>  * {{@Ignore}} should be replaced with {{@Disabled}} or appropriate conditional disabled annotations (also useful for replacing assumeThat() tests).
>  ** IntelliJ's convert to JUnit 5 refactoring action does this automatically.
>  * Hamcrest must be added as an explicit dependency and/or replaced by something better like AssertJ.
>  ** IntelliJ's convert to JUnit 5 refactoring action does this automatically.
>  * The other test annotations need to be updated to the new ones.
>  ** IntelliJ's convert to JUnit 5 refactoring action does this automatically.
>  * Parameterized tests can use the new parameters annotations to simplify things a lot.
>  * TemporaryFolder rule can be replaced with {{@TempDir}} parameter injection (experimental feature).
>  * Other rule usage needs to be converted to use appropriate v5 features or turned into extensions.
> Note that this upgrade should be backported to the release-2.x branch after Java 8 is made the baseline there.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)