#1816 Error message doesn't respect -nocolor option (#1824)

setting a NoColoring bool in TransformFactory if the related CommandLine switch was used. this commit was amended to fix missing 'static' keyword
xunit · Oct 2, 2018 · 5959d74 · 5959d74
1 parent 975ff0c
commit 5959d74
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/xunit.console/CommandLine.cs b/src/xunit.console/CommandLine.cs
@@ -170,6 +170,7 @@ protected XunitProject Parse(Predicate<string> fileExists)
                 {
                     GuardNoOptionValue(option);
                     NoColor = true;
+                    TransformFactory.NoErrorColoring = NoColor;
                 }
                 else if (optionName == "noappdomain")    // Here for historical reasons
                 {

diff --git a/src/xunit.console/Utility/TransformFactory.cs b/src/xunit.console/Utility/TransformFactory.cs
@@ -14,6 +14,8 @@ public class TransformFactory
 
         readonly Dictionary<string, Transform> availableTransforms = new Dictionary<string, Transform>(StringComparer.OrdinalIgnoreCase);
 
+        public static bool NoErrorColoring = false;
+
         protected TransformFactory()
         {
             availableTransforms.Add("xml", new Transform
@@ -65,9 +67,15 @@ static void Handler_DirectWrite(XElement xml, string outputFileName)
         static void Handler_XslTransform(string key, string resourceName, XElement xml, string outputFileName)
         {
 #if NETCOREAPP1_0
-            ConsoleHelper.SetForegroundColor(ConsoleColor.Yellow);
+            if(!NoErrorColoring)
+            {
+                ConsoleHelper.SetForegroundColor(ConsoleColor.Yellow);
+            }
             Console.WriteLine($"Skipping -{key} because XSL-T is not supported on .NET Core 1.x");
-            ConsoleHelper.ResetColor();
+            if(!NoErrorColoring)
+            {
+                ConsoleHelper.ResetColor();
+            }
 #else
             var xmlTransform = new System.Xml.Xsl.XslCompiledTransform();