From b85b5b8725e62deb2161a11beb2898c50d02cafe Mon Sep 17 00:00:00 2001 From: peter Date: Wed, 1 Mar 2017 19:42:21 +0100 Subject: [PATCH] no debug logging in stress/performance tests by default --- .../intellij/testFramework/TestLogger.java | 22 ++++++++++++++----- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/platform/testFramework/src/com/intellij/testFramework/TestLogger.java b/platform/testFramework/src/com/intellij/testFramework/TestLogger.java index c330e014a603..3e0daaa092d9 100644 --- a/platform/testFramework/src/com/intellij/testFramework/TestLogger.java +++ b/platform/testFramework/src/com/intellij/testFramework/TestLogger.java @@ -15,6 +15,7 @@ */ package com.intellij.testFramework; +import com.intellij.openapi.application.impl.ApplicationInfoImpl; import com.intellij.openapi.diagnostic.Log4jBasedLogger; import org.apache.log4j.Level; import org.apache.log4j.Logger; @@ -41,20 +42,26 @@ public class TestLogger extends Log4jBasedLogger { @Override public void debug(@NonNls String message) { - super.debug(message); - TestLoggerFactory.log(myLogger, Level.DEBUG, message, null); + if (isDebugEnabled()) { + super.debug(message); + TestLoggerFactory.log(myLogger, Level.DEBUG, message, null); + } } @Override public void debug(@Nullable Throwable t) { - super.debug(t); - TestLoggerFactory.log(myLogger, Level.DEBUG, null, t); + if (isDebugEnabled()) { + super.debug(t); + TestLoggerFactory.log(myLogger, Level.DEBUG, null, t); + } } @Override public void debug(@NonNls String message, @Nullable Throwable t) { - super.debug(message, t); - TestLoggerFactory.log(myLogger, Level.DEBUG, message, t); + if (isDebugEnabled()) { + super.debug(message, t); + TestLoggerFactory.log(myLogger, Level.DEBUG, message, t); + } } @Override @@ -71,6 +78,9 @@ public class TestLogger extends Log4jBasedLogger { @Override public boolean isDebugEnabled() { + if (ApplicationInfoImpl.isInStressTest()) { + return super.isDebugEnabled(); + } return true; } }