MsTest: Only use TestContext for output and not Console.WriteLine (#368)

reqnroll · Jan 10, 2025 · 3713c14 · 3713c14
1 parent 1ab7ee0
commit 3713c14
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@
 * Reduced MsBuild log output and consistent use of [Reqnroll] prefix (#381)
 
 ## Bug fixes:
+* MsTest: Only use TestContext for output and not Console.WriteLine (#368) 
 
 * Fix: Replace deprecated dependency `Specflow.Internal.Json` with `System.Text.Json`. The dependency was used for laoding `reqnroll.json`, for Visual Studio integration and for telemetry. (#373)
 

diff --git a/Plugins/Reqnroll.MSTest.ReqnrollPlugin/MSTestTraceListener.cs b/Plugins/Reqnroll.MSTest.ReqnrollPlugin/MSTestTraceListener.cs
@@ -15,13 +15,11 @@ public MSTestTraceListener(ITraceListenerQueue traceListenerQueue, IObjectContai
         public override void WriteTestOutput(string message)
         {
             _testContextProvider.GetTestContext().WriteLine(message);
-            base.WriteTestOutput(message);
         }
 
         public override void WriteToolOutput(string message)
         {
             _testContextProvider.GetTestContext().WriteLine("-> " + message);
-            base.WriteToolOutput(message);
         }
     }
 }
diff --git a/Tests/TestProjectGenerator/Reqnroll.TestProjectGenerator/TRXParser.cs b/Tests/TestProjectGenerator/Reqnroll.TestProjectGenerator/TRXParser.cs
@@ -184,7 +184,7 @@ private static IEnumerable<TestStepResult> ParseTestOutput(XElement stdOutElemen
 
             var logLines = stdOutText.Split(new string[] { "\r\n", "\r", "\n" }, StringSplitOptions.RemoveEmptyEntries);
 
-            var primaryOutput = logLines.TakeWhile(line => line != "TestContext Messages:");
+            var primaryOutput = logLines.Where(line => line != "TestContext Messages:");
 
             TestStepResult step = null;