Skip to content

Commit bcd0673

Browse files
committed
Fixing multi test finalization manager tests
1 parent fc7a1c0 commit bcd0673

3 files changed

Lines changed: 147 additions & 102 deletions

File tree

src/Microsoft.TestPlatform.CrossPlatEngine/MultiTestRunFinalization/MultiTestRunFinalizationManager.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ private async Task<Collection<AttachmentSet>> InternalFinalizeMultiTestRunAsync(
9999
finally
100100
{
101101
stopwatch.Stop();
102-
requestData.MetricsCollection.Metrics.Add(TelemetryDataConstants.TimeTakenInSecForFinalization, stopwatch.Elapsed.TotalSeconds);
102+
requestData.MetricsCollection.Add(TelemetryDataConstants.TimeTakenInSecForFinalization, stopwatch.Elapsed.TotalSeconds);
103103
}
104104
}
105105

@@ -137,7 +137,7 @@ private Collection<AttachmentSet> FinalizeOperation(IRequestData requestData, Co
137137
eventHandler?.HandleMultiTestRunFinalizationComplete(attachments);
138138
testPlatformEventSource.MultiTestRunFinalizationStop(attachments.Count);
139139
requestData.MetricsCollection.Add(TelemetryDataConstants.NumberOfAttachmentsAfterFinalization, attachments.Count);
140-
requestData.MetricsCollection.Add(TelemetryDataConstants.FinalizationState, attachments.Count);
140+
requestData.MetricsCollection.Add(TelemetryDataConstants.FinalizationState, finalizationState);
141141

142142
return attachments;
143143
}

0 commit comments

Comments
 (0)