Fixes #799 by testing logged messages against "null or whitespace" in…

…stead of "null or empty" (#892) Co-authored-by: Olivier DALET <olivier.dalet@addupsolutions.com>
microsoft · Jun 26, 2021 · 50c322c · 50c322c
1 parent 2d0f311
commit 50c322c
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs
@@ -469,17 +469,17 @@ private void LogCleanupResult(ITestExecutionRecorder testExecutionRecorder, RunC
         {
             Debug.Assert(testExecutionRecorder != null, "Logger should not be null");
 
-            if (!string.IsNullOrEmpty(result.StandardOut))
+            if (!string.IsNullOrWhiteSpace(result.StandardOut))
             {
                 testExecutionRecorder.SendMessage(TestMessageLevel.Informational, result.StandardOut);
             }
 
-            if (!string.IsNullOrEmpty(result.DebugTrace))
+            if (!string.IsNullOrWhiteSpace(result.DebugTrace))
             {
                 testExecutionRecorder.SendMessage(TestMessageLevel.Informational, result.DebugTrace);
             }
 
-            if (!string.IsNullOrEmpty(result.StandardError))
+            if (!string.IsNullOrWhiteSpace(result.StandardError))
             {
                 testExecutionRecorder.SendMessage(
                     MSTestSettings.CurrentSettings.TreatClassAndAssemblyCleanupWarningsAsErrors ? TestMessageLevel.Error : TestMessageLevel.Warning,
@@ -490,9 +490,12 @@ private void LogCleanupResult(ITestExecutionRecorder testExecutionRecorder, RunC
             {
                 foreach (string warning in result.Warnings)
                 {
-                    testExecutionRecorder.SendMessage(
-                        MSTestSettings.CurrentSettings.TreatClassAndAssemblyCleanupWarningsAsErrors ? TestMessageLevel.Error : TestMessageLevel.Warning,
-                        warning);
+                    if (!string.IsNullOrWhiteSpace(warning))
+                    {
+                        testExecutionRecorder.SendMessage(
+                            MSTestSettings.CurrentSettings.TreatClassAndAssemblyCleanupWarningsAsErrors ? TestMessageLevel.Error : TestMessageLevel.Warning,
+                            warning);
+                    }
                 }
             }
         }