More verbose info in datacollector log (#2430)

microsoft · May 11, 2020 · d8bc445 · d8bc445
1 parent bcad654
commit d8bc445
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/Microsoft.TestPlatform.CommunicationUtilities/DataCollectionTestCaseEventHandler.cs b/src/Microsoft.TestPlatform.CommunicationUtilities/DataCollectionTestCaseEventHandler.cs
@@ -84,7 +84,7 @@ public void ProcessRequests()
 
                         if (EqtTrace.IsInfoEnabled)
                         {
-                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Test case started.");
+                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Test case '{0} - {1}' started.", testCaseStartEventArgs.TestCaseName, testCaseStartEventArgs.TestCaseId);
                         }
 
                         break;
@@ -101,7 +101,7 @@ public void ProcessRequests()
 
                         if (EqtTrace.IsInfoEnabled)
                         {
-                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Test case completed");
+                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Test case '{0} - {1}' completed", testCaseEndEventArgs.TestCaseName, testCaseEndEventArgs.TestCaseId);
                         }
 
                         break;
@@ -121,7 +121,7 @@ public void ProcessRequests()
                     default:
                         if (EqtTrace.IsInfoEnabled)
                         {
-                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Invalid Message types");
+                            EqtTrace.Info("DataCollectionTestCaseEventHandler: Invalid Message type '{0}'", message.MessageType);
                         }
 
                         break;

diff --git a/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/TestRunCache.cs b/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/TestRunCache.cs
@@ -259,7 +259,7 @@ public bool OnTestCompletion(TestCase completedTest)
 
                 if (this.inProgressTests == null || this.inProgressTests.Count == 0)
                 {
-                    EqtTrace.Warning("InProgressTests is null");
+                    EqtTrace.Warning("TestRunCache: InProgressTests is null");
                     return false;
                 }
 
@@ -394,7 +394,7 @@ private void RemoveInProgress(TestResult result)
             var removed = this.OnTestCompletion(result.TestCase);
             if (!removed)
             {
-                EqtTrace.Warning("TestRunCache: No test found corresponding to testResult '{0}' in inProgress list.", result);
+                EqtTrace.Warning("TestRunCache: No test found corresponding to testResult '{0}' in inProgress list.", result.DisplayName);
             }
         }