Skip to content

Commit

Permalink
use Async naming convention
Browse files Browse the repository at this point in the history
  • Loading branch information
MarcoRossignoli committed Feb 10, 2022
1 parent 6ee53b4 commit 5c5467f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@ namespace Microsoft.VisualStudio.TestPlatform.ObjectModel.Engine;
internal interface IArtifactProcessingManager
{
void CollectArtifacts(TestRunCompleteEventArgs testRunCompleteEventArgs, string runSettingsXml);
Task PostProcessArtifacts();
Task PostProcessArtifactsAsync();
}
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ public void CollectArtifacts(TestRunCompleteEventArgs testRunCompleteEventArgs,
}
}

public async Task PostProcessArtifacts()
public async Task PostProcessArtifactsAsync()
{
if (!_featureFlag.IsEnabled(FeatureFlag.ARTIFACTS_POSTPROCESSING))
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ public ArgumentProcessorResult Execute()
// We don't have async execution at the moment for the argument processors.
// Anyway post processing could involve a lot of I/O and so we make some space
// for some possible parallelization async/await and fair I/O for the callee.
_artifactProcessingManage.PostProcessArtifacts().Wait();
_artifactProcessingManage.PostProcessArtifactsAsync().Wait();
return ArgumentProcessorResult.Success;
}
catch (Exception e)
Expand Down

0 comments on commit 5c5467f

Please sign in to comment.