Better names

Author: daveaglick

src/AzurePipelines.TestLogger/AzurePipelinesTestLogger.cs

@@ -41,7 +41,7 @@ public void Initialize(TestLoggerEvents events, string testRunDirectory)
             ApiClient apiClient = new ApiClient(accessToken, collectionUri, teamProject);
             _queue = new LoggerQueue(apiClient, buildId, agentName, jobName);
 
-            // Register for the events.
+            // Register for the events
             events.TestRunMessage += TestMessageHandler;
             events.TestResult += TestResultHandler;
             events.TestRunComplete += TestRunCompleteHandler;
@@ -65,21 +65,25 @@ private void TestMessageHandler(object sender, TestRunMessageEventArgs e)
 
         private void TestResultHandler(object sender, TestResultEventArgs e)
         {
-            string filename = string.IsNullOrEmpty(e.Result.TestCase.Source) ? string.Empty : Path.GetFileName(e.Result.TestCase.Source);
-
             Dictionary<string, object> testResult = new Dictionary<string, object>()
             {
-                { "testCaseTitle", e.Result.TestCase.DisplayName },
+                { "testCaseTitle", e.Result.TestCase.FullyQualifiedName },
                 { "automatedTestName", e.Result.TestCase.FullyQualifiedName },
                 { "outcome", e.Result.Outcome.ToString() },
                 { "state", "Completed" },
                 { "automatedTestType", "UnitTest" },
                 { "automatedTestTypeId", "13cdc9d9-ddb5-4fa4-a97d-d965ccfc6d4b" }, // This is used in the sample response and also appears in web searches
             };
 
+            string filename = string.IsNullOrEmpty(e.Result.TestCase.Source) ? string.Empty : Path.GetFileName(e.Result.TestCase.Source);
             if (!string.IsNullOrEmpty(filename))
             {
                 testResult.Add("automatedTestStorage", filename);
+
+                if(string.IsNullOrEmpty(_queue.Filename))
+                {
+                    _queue.Filename = filename;
+                }
             }
 
             if (e.Result.Outcome == TestOutcome.Passed || e.Result.Outcome == TestOutcome.Failed)
@@ -121,9 +125,6 @@ private void TestResultHandler(object sender, TestResultEventArgs e)
             _queue.Enqueue(testResult.ToJson());
         }
 
-        private void TestRunCompleteHandler(object sender, TestRunCompleteEventArgs e)
-        {
-            _queue.Flush();
-        }
+        private void TestRunCompleteHandler(object sender, TestRunCompleteEventArgs e) => _queue.Flush();
     }
 }

src/AzurePipelines.TestLogger/LoggerQueue.cs

@@ -9,8 +9,9 @@ namespace AzurePipelines.TestLogger
     internal class LoggerQueue
     {
         private readonly ApiClient _apiClient;
-        private readonly string _runName;
         private readonly string _buildId;
+        private readonly string _agentName;
+        private readonly string _jobName;
 
         private readonly AsyncProducerConsumerCollection<string> _queue = new AsyncProducerConsumerCollection<string>();
         private readonly Task _consumeTask;
@@ -24,10 +25,15 @@ public LoggerQueue(ApiClient apiClient, string buildId, string agentName, string
         {
             _apiClient = apiClient;
             _buildId = buildId;
-            _runName = $"{ jobName } on { agentName } at {DateTime.UtcNow.ToString("o")}";
+            _agentName = agentName;
+            _jobName = jobName;
+            
             _consumeTask = ConsumeItemsAsync(_consumeTaskCancellationSource.Token);
         }
 
+        // This gets set once the first test result comes in
+        public string Filename { get; set; }
+
         public void Enqueue(string json)
         {
             _queue.Add(json);
@@ -102,9 +108,10 @@ private async Task PostResultsAsync(ICollection<string> jsonEntities, Cancellati
 
         private async Task<int> CreateTestRun(CancellationToken cancellationToken)
         {
+            string runName = $"{( string.IsNullOrEmpty(Filename) ? "Unknown Test File" : Filename)} (OS: { System.Runtime.InteropServices.RuntimeInformation.OSDescription }, Job: { _jobName }, Agent: { _agentName }) at {DateTime.UtcNow.ToString("o")}";
             Dictionary<string, object> request = new Dictionary<string, object>
             {
-                { "name", _runName },
+                { "name", runName },
                 { "build", new Dictionary<string, object> { { "id", _buildId } } },
                 { "isAutomated", true }
             };