fix same file access exception

microsoft · Feb 15, 2022 · 1a700a4 · 1a700a4
1 parent 17cf142
commit 1a700a4
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/src/Microsoft.TestPlatform.Extensions.HtmlLogger/HtmlLogger.cs b/src/Microsoft.TestPlatform.Extensions.HtmlLogger/HtmlLogger.cs
@@ -344,14 +344,30 @@ private void PopulateHtmlFile()
     private string GetFilePath(string fileExtension, string fileName)
     {
         var fullFileFormat = $".{fileExtension}";
-        return Path.Combine(TestResultsDirPath, string.Concat("TestResult_", fileName, fullFileFormat));
+        string fullFilePath;
+        for (short i = 0; i < short.MaxValue; i++)
+        {
+            fileName = i == 0 ? fileName : GetNextIterationFile(fileName, i);
+            fullFilePath = Path.Combine(TestResultsDirPath, string.Concat("TestResult_", fileName, fullFileFormat));
+            if (!File.Exists(fullFilePath))
+            {
+                return fullFilePath;
+            }
+        }
+
+        throw new Exception($"Cannot generate unique filename for: {fileName}");
     }
 
     private string FormatDateTimeForRunName(DateTime timeStamp)
     {
         return timeStamp.ToString("yyyyMMdd_HHmmss", DateTimeFormatInfo.InvariantInfo);
     }
 
+    private string GetNextIterationFile(string baseName, short iteration)
+    {
+        return $"{Path.GetFileNameWithoutExtension(baseName)}[{iteration}]{(Path.HasExtension(baseName) ? Path.GetExtension(baseName) : string.Empty)}";
+    }
+
     /// <summary>
     /// Gives the parent execution id of a TestResult.
     /// </summary>