From a1ca66fe75837d5f7c39aa57d3dc99705576221f Mon Sep 17 00:00:00 2001 From: Thomas Stuart Date: Thu, 28 Apr 2022 11:04:40 -0700 Subject: [PATCH] fix log spewing coming from Telecom#LogUtils LogUtils was causing a ~7k line log spew in the utomation tests. new var created in Telecom Log Frameworks file that can gate ceratin lines causing log spew. bug: 224667124 Test: manual, looked through test logs and log spew lines removed Change-Id: I5aac180832c0c26af201d1286727cc3e9c67d207 --- telecomm/java/android/telecom/Log.java | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/telecomm/java/android/telecom/Log.java b/telecomm/java/android/telecom/Log.java index 922eddb6ac3e2..884dcf2dfbad0 100644 --- a/telecomm/java/android/telecom/Log.java +++ b/telecomm/java/android/telecom/Log.java @@ -75,6 +75,11 @@ public class Log { */ private static boolean sIsUserExtendedLoggingEnabled = false; + /** + * Enabled in telecom testing to help gate log statements causing log spew. + */ + private static boolean sIsUnitTestingEnabled = false; + /** * The time when user-activated extended logging should be ended. Used to determine when * extended logging should automatically be disabled. @@ -330,6 +335,20 @@ public class Log { } } + /** + * Enabled when tests are running to help gate log statements causing log spew. + * + * @param isEnabled {@code true} if running unit tests. false otherwise. + * + */ + public static void setUnitTestingEnabled(boolean isEnabled) { + sIsUnitTestingEnabled = isEnabled; + } + + public static boolean isUnitTestingEnabled() { + return sIsUnitTestingEnabled; + } + private static EventManager getEventManager() { // Checking for null again outside of synchronization because we only need to synchronize // during the lazy loading of the events logger. We don't need to synchronize elsewhere.