4 ci

Author: KSemenenko

tests/PrompterOne.Testing/EnvironmentAwareParallelLimitBase.cs

@@ -7,7 +7,7 @@ namespace PrompterOne.Testing;
 /// </summary>
 public abstract class EnvironmentAwareParallelLimitBase : IParallelLimit
 {
-    protected virtual int CiLimit { get; } = 8;
+    protected virtual int CiLimit { get; } = 4;
     protected virtual int LocalLimit { get; } = 15;
 
     public int Limit => ResolveLimit();

tests/PrompterOne.Web.UITests/Infrastructure/StandaloneAppFixture.cs

@@ -2,6 +2,7 @@
 using System.Net;
 using System.Runtime.CompilerServices;
 using Microsoft.Playwright;
+using PrompterOne.Testing;
 
 namespace PrompterOne.Web.UITests;
 
@@ -79,11 +80,8 @@ private async Task<IPage> CreateAdditionalPageAsync()
 
             try
             {
-                var page = await context.NewPageAsync();
-                PreparePage(page);
-                await PrimeIsolatedBrowserStorageAsync(page);
-                await WarmUpContextPageIfNeededAsync(page, BaseAddress);
-                return page;
+                await WarmUpContextWithSacrificialPageAsync(context);
+                return await CreatePrimedPageAsync(context);
             }
             catch (PlaywrightException exception) when (attempt < ContextBootstrapAttemptCount && IsBrowserClosedException(exception))
             {
@@ -104,20 +102,13 @@ private async Task<IPage> CreateSharedPageAsync(string contextKey)
 
             try
             {
-                var page = await context.NewPageAsync();
-                PreparePage(page);
-
                 if (isNewSharedContext)
                 {
-                    await PrimeIsolatedBrowserStorageAsync(page);
-                    await WarmUpContextPageIfNeededAsync(page, BaseAddress, warmAllRuntimeRoutes: true);
-                }
-                else
-                {
-                    await page.GotoAsync($"{BaseAddress}{UiTestHostConstants.BlankPagePath}");
+                    await WarmUpContextWithSacrificialPageAsync(context, warmAllRuntimeRoutes: true);
+                    return await CreatePrimedPageAsync(context);
                 }
 
-                return page;
+                return await CreateBlankPageAsync(context);
             }
             catch (PlaywrightException exception) when (attempt < ContextBootstrapAttemptCount && IsBrowserClosedException(exception))
             {
@@ -241,9 +232,61 @@ private static bool IsBrowserClosedException(PlaywrightException exception) =>
     private static bool IsContextWarmupFailure(InvalidOperationException exception) =>
         exception.Message.StartsWith("Browser context warmup failed.", StringComparison.Ordinal);
 
+    private async Task<IPage> CreatePrimedPageAsync(IBrowserContext context)
+    {
+        var page = await context.NewPageAsync();
+        PreparePage(page);
+        await PrimeIsolatedBrowserStorageAsync(page);
+        return page;
+    }
+
+    private async Task<IPage> CreateBlankPageAsync(IBrowserContext context)
+    {
+        var page = await context.NewPageAsync();
+        PreparePage(page);
+        await page.GotoAsync($"{BaseAddress}{UiTestHostConstants.BlankPagePath}");
+        return page;
+    }
+
+    private async Task WarmUpContextWithSacrificialPageAsync(IBrowserContext context, bool warmAllRuntimeRoutes = false)
+    {
+        if (!TestEnvironment.IsCiEnvironment)
+        {
+            return;
+        }
+
+        var warmupPage = await context.NewPageAsync();
+
+        try
+        {
+            // Warm a disposable page so the returned test page stays clean and isolated.
+            PreparePage(warmupPage);
+            await PrimeIsolatedBrowserStorageAsync(warmupPage);
+            await WarmUpContextPageIfNeededAsync(warmupPage, BaseAddress, warmAllRuntimeRoutes);
+        }
+        finally
+        {
+            await ClosePageAsync(warmupPage);
+        }
+    }
+
     private Task PrimeIsolatedBrowserStorageAsync(IPage page) =>
         PrimeIsolatedBrowserStorageAsync(page, BaseAddress);
 
+    private static async Task ClosePageAsync(IPage page)
+    {
+        try
+        {
+            if (!page.IsClosed)
+            {
+                await page.CloseAsync();
+            }
+        }
+        catch
+        {
+        }
+    }
+
     private static void PreparePage(IPage page)
     {
         page.SetDefaultNavigationTimeout(BrowserTestConstants.Timing.DefaultNavigationTimeoutMs);