Enable post processing (#3384)

microsoft · Feb 18, 2022 · 326bc26 · 326bc26
1 parent 885bd8d
commit 326bc26
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/Microsoft.TestPlatform.CoreUtilities/FeatureFlag/FeatureFlag.cs b/src/Microsoft.TestPlatform.CoreUtilities/FeatureFlag/FeatureFlag.cs
@@ -3,8 +3,6 @@
 
 #if !NETSTANDARD1_0
 
-#nullable disable
-
 namespace Microsoft.VisualStudio.TestPlatform.Utilities;
 
 using System;
@@ -20,7 +18,7 @@ internal partial class FeatureFlag : IFeatureFlag
 
  static FeatureFlag()
  {
- FeatureFlags.Add(ARTIFACTS_POSTPROCESSING, false);
+ FeatureFlags.Add(ARTIFACTS_POSTPROCESSING, true);
  FeatureFlags.Add(ARTIFACTS_POSTPROCESSING_SDK_KEEP_OLD_UX, false);
  }
 

diff --git a/src/vstest.console/Internal/ConsoleLogger.cs b/src/vstest.console/Internal/ConsoleLogger.cs
@@ -663,7 +663,12 @@ private void TestRunCompleteHandler(object sender, TestRunCompleteEventArgs e)
  var runLevelAttachementCount = (e.AttachmentSets == null) ? 0 : e.AttachmentSets.Sum(attachmentSet => attachmentSet.Attachments.Count);
  if (runLevelAttachementCount > 0)
  {
- if (!_featureFlag.IsEnabled(FeatureFlag.ARTIFACTS_POSTPROCESSING) || _featureFlag.IsEnabled(FeatureFlag.ARTIFACTS_POSTPROCESSING_SDK_KEEP_OLD_UX))
+ // If ARTIFACTS_POSTPROCESSING is disabled
+ if (!_featureFlag.IsEnabled(FeatureFlag.ARTIFACTS_POSTPROCESSING) ||
+ // ARTIFACTS_POSTPROCESSING_SDK_KEEP_OLD_UX(old UX) is enabled
+ _featureFlag.IsEnabled(FeatureFlag.ARTIFACTS_POSTPROCESSING_SDK_KEEP_OLD_UX) ||
+ // TestSessionCorrelationId is null(we're not running through the dotnet SDK).
+ CommandLineOptions.Instance.TestSessionCorrelationId is null)
  {
  Output.Information(false, CommandLineResources.AttachmentsBanner);
  foreach (var attachmentSet in e.AttachmentSets)