Fixing multi test finalization manager tests

microsoft · Jun 19, 2020 · bcd0673 · bcd0673
1 parent fc7a1c0
commit bcd0673
Show file tree

Hide file tree

Showing 3 changed files with 147 additions and 102 deletions.
diff --git a/....TestPlatform.CrossPlatEngine/MultiTestRunFinalization/MultiTestRunFinalizationManager.cs b/....TestPlatform.CrossPlatEngine/MultiTestRunFinalization/MultiTestRunFinalizationManager.cs
@@ -99,7 +99,7 @@ private async Task<Collection<AttachmentSet>> InternalFinalizeMultiTestRunAsync(
             finally
             {
                 stopwatch.Stop();
-                requestData.MetricsCollection.Metrics.Add(TelemetryDataConstants.TimeTakenInSecForFinalization, stopwatch.Elapsed.TotalSeconds);
+                requestData.MetricsCollection.Add(TelemetryDataConstants.TimeTakenInSecForFinalization, stopwatch.Elapsed.TotalSeconds);
             }
         }
 
@@ -137,7 +137,7 @@ private Collection<AttachmentSet> FinalizeOperation(IRequestData requestData, Co
             eventHandler?.HandleMultiTestRunFinalizationComplete(attachments);
             testPlatformEventSource.MultiTestRunFinalizationStop(attachments.Count);
             requestData.MetricsCollection.Add(TelemetryDataConstants.NumberOfAttachmentsAfterFinalization, attachments.Count);
-            requestData.MetricsCollection.Add(TelemetryDataConstants.FinalizationState, attachments.Count);
+            requestData.MetricsCollection.Add(TelemetryDataConstants.FinalizationState, finalizationState);
 
             return attachments;
         }