Do not print red success message (#1954)

Resolves Azure/azure-sdk-for-net#22347 and is a port of Azure/azure-sdk-for-net#23210
Azure · Aug 31, 2021 · d5145fc · d5145fc
1 parent 2828d14
commit d5145fc
Showing 1 changed file with 12 additions and 7 deletions.
diff --git a/tools/snippet-generator/Azure.Sdk.Tools.SnippetGenerator/Program.cs b/tools/snippet-generator/Azure.Sdk.Tools.SnippetGenerator/Program.cs
@@ -41,7 +41,6 @@ public async Task OnExecuteAsync()
                 unUsedSnippets = (await new DirectoryProcessor(BasePath).ProcessAsync()).ToList();
             }
             Console.WriteLine();
-            Console.ForegroundColor = ConsoleColor.Red;
             string message = $"Not all snippets were used.\n{string.Join(Environment.NewLine, unUsedSnippets)}";
             unUsedSnippets.Sort();
             if (StrictMode)
@@ -53,27 +52,33 @@ public async Task OnExecuteAsync()
             }
             else
             {
-                Console.WriteLine(message);
+                WriteError(message);
             }
             sw.Stop();
             Console.WriteLine($"SnippetGenerator completed in {sw.Elapsed}");
         }
 
         public static int Main(string[] args)
         {
-            ConsoleColor foreground = Console.ForegroundColor;
-
             try
             {
                 return CommandLineApplication.Execute<Program>(args);
             }
             catch (Exception e)
             {
-                Console.ForegroundColor = ConsoleColor.Red;
-
-                Console.Error.WriteLine(e.ToString());
+                WriteError(e.ToString());
                 return 1;
             }
+        }
+
+        private static void WriteError(string message, params object[] args)
+        {
+            ConsoleColor foreground = Console.ForegroundColor;
+            try
+            {
+                Console.ForegroundColor = ConsoleColor.Red;
+                Console.WriteLine(message, args);
+            }
             finally
             {
                 Console.ForegroundColor = foreground;