Fix console logger spacing

microsoft · May 31, 2022 · a317579 · a317579
1 parent ca7d537
commit a317579
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/src/vstest.console/Internal/ConsoleLogger.cs b/src/vstest.console/Internal/ConsoleLogger.cs
@@ -39,6 +39,11 @@ internal class ConsoleLogger : ITestLoggerWithParameters
     /// </summary>
     private const string TestResultPrefix = "  ";
 
+    /// <summary>
+    /// Suffix used for formatting the result output
+    /// </summary>
+    private const string TestResultSuffix = " ";
+
     /// <summary>
     /// Bool to decide whether Verbose level should be added as prefix or not in log messages.
     /// </summary>
@@ -538,7 +543,7 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                     // Pause the progress indicator before displaying test result information
                     _progressIndicator?.Pause();
 
-                    Output.Write(TestResultPrefix + CommandLineResources.SkippedTestIndicator, OutputLevel.Information, ConsoleColor.Yellow);
+                    Output.Write(GetFormattedTestIndicator(CommandLineResources.SkippedTestIndicator), OutputLevel.Information, ConsoleColor.Yellow);
                     Output.WriteLine(testDisplayName, OutputLevel.Information);
                     if (VerbosityLevel == Verbosity.Detailed)
                     {
@@ -561,7 +566,7 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                     // Pause the progress indicator before displaying test result information
                     _progressIndicator?.Pause();
 
-                    Output.Write(TestResultPrefix + CommandLineResources.FailedTestIndicator, OutputLevel.Information, ConsoleColor.Red);
+                    Output.Write(GetFormattedTestIndicator(CommandLineResources.FailedTestIndicator), OutputLevel.Information, ConsoleColor.Red);
                     Output.WriteLine(testDisplayName, OutputLevel.Information);
                     DisplayFullInformation(e.Result);
 
@@ -578,7 +583,7 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                         // Pause the progress indicator before displaying test result information
                         _progressIndicator?.Pause();
 
-                        Output.Write(TestResultPrefix + CommandLineResources.PassedTestIndicator, OutputLevel.Information, ConsoleColor.Green);
+                        Output.Write(GetFormattedTestIndicator(CommandLineResources.PassedTestIndicator), OutputLevel.Information, ConsoleColor.Green);
                         Output.WriteLine(testDisplayName, OutputLevel.Information);
                         if (VerbosityLevel == Verbosity.Detailed)
                         {
@@ -602,7 +607,7 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                     // Pause the progress indicator before displaying test result information
                     _progressIndicator?.Pause();
 
-                    Output.Write(TestResultPrefix + CommandLineResources.SkippedTestIndicator, OutputLevel.Information, ConsoleColor.Yellow);
+                    Output.Write(GetFormattedTestIndicator(CommandLineResources.SkippedTestIndicator), OutputLevel.Information, ConsoleColor.Yellow);
                     Output.WriteLine(testDisplayName, OutputLevel.Information);
                     if (VerbosityLevel == Verbosity.Detailed)
                     {
@@ -615,6 +620,9 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                     break;
                 }
         }
+
+        // Local functions
+        static string GetFormattedTestIndicator(string indicator) => TestResultPrefix + indicator + TestResultSuffix;
     }
 
     private string GetFormattedDurationString(TimeSpan duration)