Show all type of messages( sdtOut, stdErr etc) for passed tests (#809)

* Show all type of messages likesdtOut, stdErr etc for passed tests

* Address PR comment

Author: Faizan2304

src/vstest.console/Internal/ConsoleLogger.cs

@@ -263,6 +263,19 @@ private static void DisplayFullInformation(TestResult result)
                 }
             }
 
+            var DbgTrcMessagesCollection = GetTestMessages(result.Messages, TestResultMessage.DebugTraceCategory);
+            if (DbgTrcMessagesCollection.Count > 0)
+            {
+                addAdditionalNewLine = false;
+                var dbgTrcMessages = GetFormattedOutput(DbgTrcMessagesCollection);
+
+                if (!string.IsNullOrEmpty(dbgTrcMessages))
+                {
+                    Output.Information(CommandLineResources.DbgTrcMessagesBanner);
+                    Output.Information(dbgTrcMessages);
+                }
+            }
+
             var addnlInfoMessagesCollection = GetTestMessages(result.Messages, TestResultMessage.AdditionalInfoCategory);
             if (addnlInfoMessagesCollection.Count > 0)
             {
@@ -275,6 +288,7 @@ private static void DisplayFullInformation(TestResult result)
                     Output.Information(addnlInfoMessages);
                 }
             }
+
             if (addAdditionalNewLine)
             {
                 Output.WriteLine(String.Empty, OutputLevel.Information);
@@ -354,6 +368,7 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
                 {
                     var output = string.Format(CultureInfo.CurrentCulture, CommandLineResources.PassedTestIndicator, name);
                     Output.Information(output);
+                    DisplayFullInformation(e.Result);
                 }
                 this.testsPassed++;
             }

src/vstest.console/Resources/Resources.Designer.cs

@@ -652,4 +652,7 @@
   <data name="DataCollectorFriendlyNameInvalid" xml:space="preserve">
     <value>The Data Collector friendly name '{0}' is not valid.  The Data Collector will be ignored.</value>
   </data>
+  <data name="DbgTrcMessagesBanner" xml:space="preserve">
+    <value>Debug Traces Messages:</value>
+  </data>
 </root>

src/vstest.console/Resources/Resources.resx

@@ -314,6 +314,59 @@ public void TestResultHandlerShouldWriteToConsoleShouldShowPassedTestsForNormalV
             this.mockOutput.Verify(o => o.WriteLine(string.Format(CultureInfo.CurrentCulture, CommandLineResources.NotRunTestIndicator, "TestName"), OutputLevel.Information), Times.Exactly(2));
         }
 
+        [TestMethod]
+        public void TestResultHandlerShouldShowStdOutMsgOfPassedTestIfVerbosityIsNormal()
+        {
+            var parameters = new Dictionary<string, string>();
+            parameters.Add("verbosity", "normal");
+            this.consoleLogger.Initialize(this.events.Object, parameters);
+
+            var testcase = new TestCase("TestName", new Uri("some://uri"), "TestSource");
+
+            string message = "Dummy message";
+            TestResultMessage testResultMessage = new TestResultMessage(TestResultMessage.StandardOutCategory, message);
+
+            var testresult = new ObjectModel.TestResult(testcase);
+            testresult.Outcome = TestOutcome.Passed;
+            testresult.Messages.Add(testResultMessage);
+
+            var eventArgs = new TestRunChangedEventArgs(null, new List<ObjectModel.TestResult> { testresult }, null);
+
+            // Raise an event on mock object
+            this.testRunRequest.Raise(m => m.OnRunStatsChange += null, eventArgs);
+            this.FlushLoggerMessages();
+
+            this.mockOutput.Verify(o => o.WriteLine(CommandLineResources.StdOutMessagesBanner, OutputLevel.Information), Times.Once());
+            this.mockOutput.Verify(o => o.WriteLine(" " + message, OutputLevel.Information), Times.Once());
+        }
+
+        [TestMethod]
+        public void TestResultHandlerShouldShowDbgTrcMsg()
+        {
+            var parameters = new Dictionary<string, string>();
+            parameters.Add("verbosity", "normal");
+            this.consoleLogger.Initialize(this.events.Object, parameters);
+
+            var testcase = new TestCase("TestName", new Uri("some://uri"), "TestSource");
+
+            string message = "Dummy message";
+            TestResultMessage testResultMessage = new TestResultMessage(TestResultMessage.DebugTraceCategory, message);
+
+            var testresult = new ObjectModel.TestResult(testcase);
+            testresult.Outcome = TestOutcome.Passed;
+            testresult.Messages.Add(testResultMessage);
+
+            var eventArgs = new TestRunChangedEventArgs(null, new List<ObjectModel.TestResult> { testresult }, null);
+
+            // Raise an event on mock object
+            this.testRunRequest.Raise(m => m.OnRunStatsChange += null, eventArgs);
+            this.FlushLoggerMessages();
+
+            this.mockOutput.Verify(o => o.WriteLine(CommandLineResources.DbgTrcMessagesBanner, OutputLevel.Information), Times.Once());
+            this.mockOutput.Verify(o => o.WriteLine(" " + message, OutputLevel.Information), Times.Once());
+        }
+
+
         [TestMethod]
         public void TestResultHandlerShouldWriteToConsoleButSkipPassedTestsForMinimalVerbosity()
         {