Fix a Race in Installer Test Infrastructure

src/installer/tests/Microsoft.NET.HostModel.Tests/AppHost.Bundle.Tests/BundleRename.cs

@@ -23,7 +23,6 @@ public BundleRename(SharedTestState fixture)
         }
 
         [Theory]
-        [ActiveIssue("https://github.com/dotnet/runtime/issues/38013")]
         [InlineData(true)]  // Test renaming the single-exe when contents are extracted
         [InlineData(false)] // Test renaming the single-exe when contents are not extracted 
         private void Bundle_can_be_renamed_while_running(bool testExtraction)

src/installer/tests/Microsoft.NET.HostModel.Tests/AppHost.Bundle.Tests/StaticHost.cs

@@ -82,7 +82,7 @@ public class SharedTestState : IDisposable
             public SharedTestState()
             {
                 RepoDirectories = new RepoDirectoriesProvider();
-                TestFixture = new TestProjectFixture("StaticHostApp", RepoDirectories);
+                TestFixture = new TestProjectFixture("StandaloneApp", RepoDirectories);
                 TestFixture
                     .EnsureRestoredForRid(TestFixture.CurrentRid, RepoDirectories.CorehostPackages)
                     .PublishProject(runtime: TestFixture.CurrentRid, 

src/installer/tests/TestUtils/TestApp.cs

@@ -2,7 +2,6 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
-using Microsoft.DotNet.CoreSetup.Test.HostActivation;
 using System.IO;
 
 namespace Microsoft.DotNet.CoreSetup.Test
@@ -37,8 +36,6 @@ public TestApp(TestApp source)
         public static TestApp CreateEmpty(string name)
         {
             string location = GetNewTestArtifactPath(name);
-            FileUtils.EnsureDirectoryExists(location);
-
             return new TestApp(location);
         }
 

src/installer/tests/TestUtils/TestArtifact.cs

@@ -2,9 +2,11 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+using Microsoft.DotNet.CoreSetup.Test.HostActivation;
 using System;
 using System.Collections.Generic;
 using System.IO;
+using System.Threading;
 
 namespace Microsoft.DotNet.CoreSetup.Test
 {
@@ -67,25 +69,26 @@ public virtual void Dispose()
             _copies.Clear();
         }
 
+        private static readonly object _pathCountLock = new object();
         protected static string GetNewTestArtifactPath(string artifactName)
         {
             int projectCount = 0;
-            string projectDirectory = Path.Combine(TestArtifactsPath, projectCount.ToString(), artifactName);
+            string projectCountDir() => Path.Combine(TestArtifactsPath, projectCount.ToString(), artifactName);
 
-            while (Directory.Exists(projectDirectory))
+            for (; Directory.Exists(projectCountDir()); projectCount++);
+
+            lock (_pathCountLock)
             {
-                projectDirectory = Path.Combine(TestArtifactsPath, (++projectCount).ToString(), artifactName);
+                string projectDirectory;
+                for (; Directory.Exists(projectDirectory = projectCountDir()); projectCount++);
+                FileUtils.EnsureDirectoryExists(projectDirectory);
+                return projectDirectory;
             }
-
-            return projectDirectory;
         }
 
         protected static void CopyRecursive(string sourceDirectory, string destinationDirectory, bool overwrite = false)
         {
-            if (!Directory.Exists(destinationDirectory))
-            {
-                Directory.CreateDirectory(destinationDirectory);
-            }
+            FileUtils.EnsureDirectoryExists(destinationDirectory);
 
             foreach (var dir in Directory.EnumerateDirectories(sourceDirectory))
             {