(GH-1309) Support this.Log.Trace()

Allow calling trace directly and have it do the appropriate thing. If trace is called through a logger, redirect to the trace method.
chocolatey · May 29, 2017 · 95f2d0e · 95f2d0e
1 parent 5469ccd
commit 95f2d0e
Showing 1 changed file with 45 additions and 4 deletions.
diff --git a/src/chocolatey/ILogExtensions.cs b/src/chocolatey/ILogExtensions.cs
@@ -26,6 +26,21 @@ namespace chocolatey
     /// </summary>
     public static class ILogExtensions
     {
+        /// <summary>
+        /// This is changed for testing only
+        /// </summary>
+        public static bool LogTraceMessages = true;
+
+        public static void Trace(this ILog logger, string message, params object[] formatting)
+        {
+           if (LogTraceMessages) ChocolateyLoggers.Trace.to_string().Log().Debug(message, formatting);
+        }
+
+        public static void Trace(this ILog logger, Func<string> message)
+        {
+            if (LogTraceMessages) ChocolateyLoggers.Trace.to_string().Log().Debug(message);
+        }
+
         public static void Debug(this ILog logger, ChocolateyLoggers logType, string message, params object[] formatting)
         {
             switch (logType)
@@ -34,6 +49,9 @@ public static void Debug(this ILog logger, ChocolateyLoggers logType, string mes
 
                     logger.Debug(message, formatting);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger,message,formatting);
+                    break;
                 default:
                     logType.to_string().Log().Debug(message, formatting);
                     break;
@@ -47,6 +65,9 @@ public static void Debug(this ILog logger, ChocolateyLoggers logType, Func<strin
                 case ChocolateyLoggers.Normal:
                     logger.Debug(message);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message);
+                    break;
                 default:
                     logType.to_string().Log().Debug(message);
                     break;
@@ -58,9 +79,11 @@ public static void Info(this ILog logger, ChocolateyLoggers logType, string mess
             switch (logType)
             {
                 case ChocolateyLoggers.Normal:
-
                     logger.Info(message, formatting);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message, formatting);
+                    break;
                 default:
                     logType.to_string().Log().Info(message, formatting);
                     break;
@@ -74,6 +97,9 @@ public static void Info(this ILog logger, ChocolateyLoggers logType, Func<string
                 case ChocolateyLoggers.Normal:
                     logger.Info(message);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message);
+                    break;
                 default:
                     logType.to_string().Log().Info(message);
                     break;
@@ -85,9 +111,11 @@ public static void Warn(this ILog logger, ChocolateyLoggers logType, string mess
             switch (logType)
             {
                 case ChocolateyLoggers.Normal:
-
                     logger.Warn(message, formatting);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message, formatting);
+                    break;
                 default:
                     logType.to_string().Log().Warn(message, formatting);
                     break;
@@ -101,6 +129,9 @@ public static void Warn(this ILog logger, ChocolateyLoggers logType, Func<string
                 case ChocolateyLoggers.Normal:
                     logger.Warn(message);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message);
+                    break;
                 default:
                     logType.to_string().Log().Warn(message);
                     break;
@@ -112,9 +143,11 @@ public static void Error(this ILog logger, ChocolateyLoggers logType, string mes
             switch (logType)
             {
                 case ChocolateyLoggers.Normal:
-
                     logger.Error(message, formatting);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message, formatting);
+                    break;
                 default:
                     logType.to_string().Log().Error(message, formatting);
                     break;
@@ -128,6 +161,9 @@ public static void Error(this ILog logger, ChocolateyLoggers logType, Func<strin
                 case ChocolateyLoggers.Normal:
                     logger.Error(message);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message);
+                    break;
                 default:
                     logType.to_string().Log().Error(message);
                     break;
@@ -139,9 +175,11 @@ public static void Fatal(this ILog logger, ChocolateyLoggers logType, string mes
             switch (logType)
             {
                 case ChocolateyLoggers.Normal:
-
                     logger.Fatal(message, formatting);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message, formatting);
+                    break;
                 default:
                     logType.to_string().Log().Fatal(message, formatting);
                     break;
@@ -155,6 +193,9 @@ public static void Fatal(this ILog logger, ChocolateyLoggers logType, Func<strin
                 case ChocolateyLoggers.Normal:
                     logger.Fatal(message);
                     break;
+                case ChocolateyLoggers.Trace:
+                    Trace(logger, message);
+                    break;
                 default:
                     logType.to_string().Log().Fatal(message);
                     break;