You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by pk...@apache.org on 2022/07/24 18:54:38 UTC
[logging-log4j2] 02/04: Do not reset the `StatusLogger` in tests
This is an automated email from the ASF dual-hosted git repository.
pkarwasz pushed a commit to branch parallel-tests
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git
commit c041f1372491e9e18b6bff7fafc1366c8bc8b3d1
Author: Piotr P. Karwasz <pi...@karwasz.org>
AuthorDate: Sun Jul 17 12:03:18 2022 +0200
Do not reset the `StatusLogger` in tests
Instead of resetting the `StatusLogger`, we use the global
`StatusLoggerExtension`.
---
.../test/java/org/apache/logging/log4j/core/config/AdvertiserTest.java | 1 -
.../log4j/core/config/plugins/util/PluginManagerPackagesTest.java | 1 -
.../apache/logging/log4j/core/config/xml/XmlConfigurationPropsTest.java | 1 -
3 files changed, 3 deletions(-)
diff --git a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/AdvertiserTest.java b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/AdvertiserTest.java
index 61f7892911..114b3fb06a 100644
--- a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/AdvertiserTest.java
+++ b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/AdvertiserTest.java
@@ -56,7 +56,6 @@ public class AdvertiserTest {
System.clearProperty(ConfigurationFactory.CONFIGURATION_FILE_PROPERTY);
final LoggerContext ctx = LoggerContext.getContext();
ctx.reconfigure();
- StatusLogger.getLogger().reset();
}
private void verifyExpectedEntriesAdvertised(final Map<Object, Map<String, String>> entries) {
diff --git a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/plugins/util/PluginManagerPackagesTest.java b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/plugins/util/PluginManagerPackagesTest.java
index 1cd15e2e10..5872a81cc2 100644
--- a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/plugins/util/PluginManagerPackagesTest.java
+++ b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/plugins/util/PluginManagerPackagesTest.java
@@ -53,7 +53,6 @@ public class PluginManagerPackagesTest {
if (ctx != null) {
ctx.reconfigure();
}
- StatusLogger.getLogger().reset();
}
@AfterAll
diff --git a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/xml/XmlConfigurationPropsTest.java b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/xml/XmlConfigurationPropsTest.java
index 94fa1528f1..90f8c5f5f0 100644
--- a/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/xml/XmlConfigurationPropsTest.java
+++ b/log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/xml/XmlConfigurationPropsTest.java
@@ -37,7 +37,6 @@ public class XmlConfigurationPropsTest {
System.clearProperty(ConfigurationFactory.CONFIGURATION_FILE_PROPERTY);
final LoggerContext ctx = LoggerContext.getContext();
ctx.reconfigure();
- StatusLogger.getLogger().reset();
}
@Test