fix unresponsive Test Explorer problem (RetryHelper.Do AggregateException)

- Changed task in `publish-coverage-results.yml` from
  `PublishCodeCoverageResults@1` to `PublishCodeCoverageResults@2`
  and added `reportDirectory` parameter for coverage report location.

- Added overloaded `BackupOriginalModule` methods in
  `InstrumentationHelper.cs` to improve backup flexibility,
  including a new `withBackupList` parameter.

- Updated `InstrumentationHelperTests.cs` to align tests with
  the new method signature for backing up original modules.

Author: Bertk

eng/publish-coverage-results.yml

@@ -26,10 +26,11 @@ steps:
   artifact: CoverageResults_$(Agent.Os)_$(BuildConfiguration)
   condition: ${{parameters.condition}}
 
-- task: PublishCodeCoverageResults@2
+- task: PublishCodeCoverageResults@1
   displayName: 'Publish code coverage'
   condition: ${{parameters.condition}}
   inputs:
     codeCoverageTool: Cobertura
     summaryFileLocation: '$(Build.SourcesDirectory)/artifacts/CoverageReport/Cobertura.xml'
+    reportDirectory: '$(Build.SourcesDirectory)/artifacts/CoverageReport'
     failIfCoverageEmpty: ${{parameters.breakBuild}}

src/coverlet.core/Helpers/InstrumentationHelper.cs

@@ -236,12 +236,28 @@ private bool MatchDocumentsWithSourcesMissingAll(MetadataReader metadataReader)
       return (true, string.Empty);
     }
 
+    /// <summary>
+    /// Backs up the original module to a specified location.
+    /// </summary>
+    /// <param name="module">The path to the module to be backed up.</param>
+    /// <param name="identifier">A unique identifier to distinguish the backup file.</param>
     public void BackupOriginalModule(string module, string identifier)
+    {
+      BackupOriginalModule(module, identifier, true);
+    }
+
+    /// <summary>
+    /// Backs up the original module to a specified location.
+    /// </summary>
+    /// <param name="module">The path to the module to be backed up.</param>
+    /// <param name="identifier">A unique identifier to distinguish the backup file.</param>
+    /// <param name="withBackupList">Indicates whether to add the backup to the backup list. Required for test TestBackupOriginalModule</param>
+    public void BackupOriginalModule(string module, string identifier, bool withBackupList)
     {
       string backupPath = GetBackupPath(module, identifier);
       string backupSymbolPath = Path.ChangeExtension(backupPath, ".pdb");
       _fileSystem.Copy(module, backupPath, true);
-      if (!_backupList.TryAdd(module, backupPath))
+      if (withBackupList && !_backupList.TryAdd(module, backupPath))
       {
         throw new ArgumentException($"Key already added '{module}'");
       }
@@ -250,13 +266,18 @@ public void BackupOriginalModule(string module, string identifier)
       if (_fileSystem.Exists(symbolFile))
       {
         _fileSystem.Copy(symbolFile, backupSymbolPath, true);
-        if (!_backupList.TryAdd(symbolFile, backupSymbolPath))
+        if (withBackupList && !_backupList.TryAdd(symbolFile, backupSymbolPath))
         {
           throw new ArgumentException($"Key already added '{module}'");
         }
       }
     }
 
+    /// <summary>
+    /// Restores the original module from a backup.
+    /// </summary>
+    /// <param name="module">The path to the module to be restored.</param>
+    /// <param name="identifier">A unique identifier to distinguish the backup file.</param>
     public virtual void RestoreOriginalModule(string module, string identifier)
     {
       string backupPath = GetBackupPath(module, identifier);

test/coverlet.core.tests/Helpers/InstrumentationHelperTests.cs

@@ -102,7 +102,7 @@ public void TestBackupOriginalModule()
       string module = typeof(InstrumentationHelperTests).Assembly.Location;
       string identifier = Guid.NewGuid().ToString();
 
-      _instrumentationHelper.BackupOriginalModule(module, identifier);
+      _instrumentationHelper.BackupOriginalModule(module, identifier, false);
 
       string backupPath = Path.Combine(
           Path.GetTempPath(),