Fix flaky MVC functional test by warming up the OrchardCore tenant pipeline before tests run

test/OrchardCore.Tests.Functional/Helpers/OrchardTestServer.cs

@@ -104,7 +104,13 @@ public static async Task<OrchardTestServer> StartMvcAsync(string contentRoot, st
 
         await app.StartAsync();
 
-        return new OrchardTestServer(app, GetListeningAddress(app), loggerProvider.Collector);
+        var address = GetListeningAddress(app);
+
+        // The OrchardCore tenant pipeline initializes lazily on the first request.
+        // Warm up the app now so tests don't race against pipeline initialization.
+        await WarmUpAsync(address);
+
+        return new OrchardTestServer(app, address, loggerProvider.Collector);
     }
 
     public void AssertNoLoggedIssues()
@@ -341,6 +347,42 @@ private static string ReplaceDatabaseName(string connectionString, string newDbN
             System.Text.RegularExpressions.RegexOptions.IgnoreCase);
     }
 
+    /// <summary>
+    /// Polls the root URL until the app serves a non-empty 200 response, ensuring the
+    /// OrchardCore tenant pipeline has finished its lazy first-request initialization
+    /// before the Playwright tests begin.
+    /// </summary>
+    private static async Task WarmUpAsync(string baseAddress, int timeoutSeconds = 30)
+    {
+        using var client = new HttpClient();
+        var deadline = DateTime.UtcNow.AddSeconds(timeoutSeconds);
+
+        while (DateTime.UtcNow < deadline)
+        {
+            try
+            {
+                var response = await client.GetAsync($"{baseAddress}/");
+                if (response.IsSuccessStatusCode)
+                {
+                    var content = await response.Content.ReadAsStringAsync();
+                    if (!string.IsNullOrWhiteSpace(content))
+                    {
+                        return;
+                    }
+                }
+            }
+            catch
+            {
+                // Server not yet accepting connections — keep waiting.
+            }
+
+            await Task.Delay(500);
+        }
+
+        throw new TimeoutException(
+            $"The MVC application at '{baseAddress}' did not serve content within {timeoutSeconds} seconds.");
+    }
+
     private static string GetListeningAddress(WebApplication app)
     {
         var server = app.Services.GetRequiredService<IServer>();

test/OrchardCore.Tests.Functional/Tests/Cms/AgencyTests.cs

@@ -11,7 +11,9 @@ public AgencyTests(AgencyFixture fixture) : base(fixture) { }
     public async Task DisplaysTheHomePageOfTheAgencyTheme()
     {
         var page = await Fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator("#services")).ToContainTextAsync("Lorem ipsum dolor sit amet consectetur");
         await page.CloseAsync();
     }
@@ -21,7 +23,9 @@ public async Task AgencyAdminLoginShouldWork()
     {
         var page = await Fixture.CreatePageAsync();
         await page.LoginAsync();
-        await page.GotoAsync("/Admin");
+        var response = await page.GotoAsync("/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Cms/BlogTests.cs

@@ -11,7 +11,9 @@ public BlogTests(BlogFixture fixture) : base(fixture) { }
     public async Task DisplaysTheHomePageOfTheBlogRecipe()
     {
         var page = await Fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".subheading")).ToContainTextAsync("This is the description of your blog");
         await page.CloseAsync();
     }
@@ -21,7 +23,9 @@ public async Task BlogAdminLoginShouldWork()
     {
         var page = await Fixture.CreatePageAsync();
         await page.LoginAsync();
-        await page.GotoAsync("/Admin");
+        var response = await page.GotoAsync("/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Cms/ComingSoonTests.cs

@@ -11,7 +11,9 @@ public ComingSoonTests(ComingSoonFixture fixture) : base(fixture) { }
     public async Task DisplaysTheHomePageOfTheComingSoonTheme()
     {
         var page = await Fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator("h1")).ToContainTextAsync("Coming Soon");
         await Assertions.Expect(page.Locator("p")).ToContainTextAsync("We're working hard to finish the development of this site.");
         await page.CloseAsync();
@@ -22,7 +24,9 @@ public async Task ComingSoonAdminLoginShouldWork()
     {
         var page = await Fixture.CreatePageAsync();
         await page.LoginAsync();
-        await page.GotoAsync("/Admin");
+        var response = await page.GotoAsync("/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Cms/HeadlessTests.cs

@@ -11,7 +11,9 @@ public HeadlessTests(HeadlessFixture fixture) : base(fixture) { }
     public async Task DisplaysTheLoginScreenForTheHeadlessTheme()
     {
         var page = await Fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator("h1")).ToContainTextAsync("Log in");
         await page.CloseAsync();
     }
@@ -21,7 +23,9 @@ public async Task HeadlessAdminLoginShouldWork()
     {
         var page = await Fixture.CreatePageAsync();
         await page.LoginAsync();
-        await page.GotoAsync("/Admin");
+        var response = await page.GotoAsync("/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Cms/MigrationsTests.cs

@@ -11,7 +11,9 @@ public MigrationsTests(MigrationsFixture fixture) : base(fixture) { }
     public async Task DisplaysTheHomePageOfTheMigrationsRecipe()
     {
         var page = await Fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.GetByText("Testing features having database migrations")).ToBeVisibleAsync();
         await page.CloseAsync();
     }
@@ -21,7 +23,9 @@ public async Task MigrationsAdminLoginShouldWork()
     {
         var page = await Fixture.CreatePageAsync();
         await page.LoginAsync();
-        await page.GotoAsync("/Admin");
+        var response = await page.GotoAsync("/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Cms/SaasTests.cs

@@ -24,7 +24,9 @@ public ValueTask DisposeAsync()
     public async Task DisplaysTheHomePageOfTheSaasTheme()
     {
         var page = await _fixture.CreatePageAsync();
-        await page.GotoAsync($"/{_fixture.Tenant.Prefix}");
+        var response = await page.GotoAsync($"/{_fixture.Tenant.Prefix}");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator("h4")).ToContainTextAsync("Welcome to Orchard Core, your site has been successfully set up.");
         await page.CloseAsync();
     }
@@ -34,7 +36,9 @@ public async Task SaasAdminLoginShouldWork()
     {
         var page = await _fixture.CreatePageAsync();
         await page.LoginAsync($"/{_fixture.Tenant.Prefix}");
-        await page.GotoAsync($"/{_fixture.Tenant.Prefix}/Admin");
+        var response = await page.GotoAsync($"/{_fixture.Tenant.Prefix}/Admin");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator(".menu-admin")).ToHaveAttributeAsync("id", "adminMenu");
         await page.CloseAsync();
     }

test/OrchardCore.Tests.Functional/Tests/Mvc/MvcTests.cs

@@ -23,7 +23,9 @@ public ValueTask DisposeAsync()
     public async Task ShouldDisplayHelloWorld()
     {
         var page = await _fixture.CreatePageAsync();
-        await page.GotoAsync("/");
+        var response = await page.GotoAsync("/");
+        Assert.NotNull(response);
+        Assert.True(response.Ok, $"Expected HTTP 200 but got {response.Status} for {response.Url}");
         await Assertions.Expect(page.Locator("body")).ToContainTextAsync("Hello World");
         await page.CloseAsync();
     }