diff --git a/log4j-api-test/src/test/java/org/apache/logging/log4j/status/StatusConsoleListenerTest.java b/log4j-api-test/src/test/java/org/apache/logging/log4j/status/StatusConsoleListenerTest.java index 5fb84eb3173..6ca21afefad 100644 --- a/log4j-api-test/src/test/java/org/apache/logging/log4j/status/StatusConsoleListenerTest.java +++ b/log4j-api-test/src/test/java/org/apache/logging/log4j/status/StatusConsoleListenerTest.java @@ -50,7 +50,7 @@ void StatusData_getFormattedStatus_should_be_used() { } @Test - void level_and_stream_should_be_honored() throws Exception { + void stream_should_be_honored() throws Exception { // Create the listener. final ByteArrayOutputStream outputStream = new ByteArrayOutputStream(); @@ -58,7 +58,7 @@ void level_and_stream_should_be_honored() throws Exception { final PrintStream printStream = new PrintStream(outputStream, false, encoding); final StatusConsoleListener listener = new StatusConsoleListener(Level.WARN, printStream); - // First, log a message that is expected to be logged. + // log a message that is expected to be logged. final RuntimeException expectedThrowable = new RuntimeException("expectedThrowable"); expectedThrowable.setStackTrace(new StackTraceElement[] { new StackTraceElement("expectedThrowableClass", "expectedThrowableMethod", "expectedThrowableFile", 1) @@ -71,29 +71,12 @@ void level_and_stream_should_be_honored() throws Exception { expectedThrowable, null)); // as set by `StatusLogger` itself - // Second, log a message that is expected to be discarded due to its insufficient level. - final RuntimeException discardedThrowable = new RuntimeException("discardedThrowable"); - discardedThrowable.setStackTrace(new StackTraceElement[] { - new StackTraceElement("discardedThrowableClass", "discardedThrowableMethod", "discardedThrowableFile", 2) - }); - final Message discardedMessage = MESSAGE_FACTORY.newMessage("discardedMessage"); - listener.log(new StatusData( - null, // since ignored by `SimpleLogger` - Level.INFO, - discardedMessage, - discardedThrowable, - null)); // as set by `StatusLogger` itself - // Collect the output. printStream.flush(); final String output = outputStream.toString(encoding); // Verify the output. - assertThat(output) - .contains(expectedThrowable.getMessage()) - .contains(expectedMessage.getFormattedMessage()) - .doesNotContain(discardedThrowable.getMessage()) - .doesNotContain(discardedMessage.getFormattedMessage()); + assertThat(output).contains(expectedThrowable.getMessage()).contains(expectedMessage.getFormattedMessage()); } @Test diff --git a/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java b/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java index 99bcd8d1a74..6261c172aa9 100644 --- a/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java +++ b/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java @@ -134,10 +134,8 @@ public Level getStatusLevel() { @Override public void log(final StatusData data) { requireNonNull(data, "data"); - if (level.isLessSpecificThan(data.getLevel())) { - final String formattedStatus = data.getFormattedStatus(); - stream.println(formattedStatus); - } + final String formattedStatus = data.getFormattedStatus(); + stream.println(formattedStatus); } /**