fix(tests): serialize SQLite EnsureCreated and disable auto redirect in functional tests

Add a static SemaphoreSlim gate around EnsureCreatedHostedService.StartAsync so parallel host startup cannot race while creating schema in shared in-memory SQLite. Also use CreateClientWithoutAutoRedirect in FunctionalTests so redirect handling stays explicit in GetFollowingGetRedirectsAsync.

Author: BenjaminMichaelis

EssentialCSharp.Web.Tests/FunctionalTests.cs

@@ -13,7 +13,7 @@ public class FunctionalTests : IntegrationTestBase
     [Arguments("/alive")]
     public async Task WhenTheApplicationStarts_ItCanLoadLoadPages(string relativeUrl)
     {
-        using HttpClient client = Factory.CreateClient();
+        using HttpClient client = CreateClientWithoutAutoRedirect();
         using HttpResponseMessage response = await GetFollowingGetRedirectsAsync(client, relativeUrl);
 
         await Assert.That(response.StatusCode).IsEqualTo(HttpStatusCode.OK);
@@ -29,7 +29,7 @@ public async Task WhenTheApplicationStarts_ItCanLoadLoadPages(string relativeUrl
     [Arguments("/about?someOtherParam=value")]
     public async Task WhenPagesAreAccessed_TheyReturnHtml(string relativeUrl)
     {
-        using HttpClient client = Factory.CreateClient();
+        using HttpClient client = CreateClientWithoutAutoRedirect();
         using HttpResponseMessage response = await GetFollowingGetRedirectsAsync(client, relativeUrl);
 
         await Assert.That(response.StatusCode).IsEqualTo(HttpStatusCode.OK);
@@ -45,7 +45,7 @@ public async Task WhenPagesAreAccessed_TheyReturnHtml(string relativeUrl)
     [Test]
     public async Task WhenTheApplicationStarts_NonExistingPage_GivesCorrectStatusCode()
     {
-        using HttpClient client = Factory.CreateClient();
+        using HttpClient client = CreateClientWithoutAutoRedirect();
         using HttpResponseMessage response = await GetFollowingGetRedirectsAsync(client, "/non-existing-page1234");
 
         await Assert.That(response.StatusCode).IsEqualTo(HttpStatusCode.NotFound);

EssentialCSharp.Web.Tests/WebApplicationFactory.cs

@@ -9,11 +9,13 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.Extensions.Hosting;
+using System.Threading;
 
 namespace EssentialCSharp.Web.Tests;
 
 public sealed class WebApplicationFactory : TestWebApplicationFactory<Program>
 {
+    private static readonly SemaphoreSlim SchemaInitializationGate = new(1, 1);
     // One GUID per factory instance → each factory gets its own isolated in-memory database.
     private readonly string _sqlConnectionString =
         $"DataSource=file:{Guid.NewGuid():N}?mode=memory&cache=shared";
@@ -129,10 +131,18 @@ private sealed class EnsureCreatedHostedService(IServiceProvider serviceProvider
     {
         public async Task StartAsync(CancellationToken cancellationToken)
         {
-            using IServiceScope scope = serviceProvider.CreateScope();
-            EssentialCSharpWebContext dbContext =
-                scope.ServiceProvider.GetRequiredService<EssentialCSharpWebContext>();
-            await dbContext.Database.EnsureCreatedAsync(cancellationToken);
+            await SchemaInitializationGate.WaitAsync(cancellationToken);
+            try
+            {
+                using IServiceScope scope = serviceProvider.CreateScope();
+                EssentialCSharpWebContext dbContext =
+                    scope.ServiceProvider.GetRequiredService<EssentialCSharpWebContext>();
+                await dbContext.Database.EnsureCreatedAsync(cancellationToken);
+            }
+            finally
+            {
+                SchemaInitializationGate.Release();
+            }
         }
 
         public Task StopAsync(CancellationToken cancellationToken) => Task.CompletedTask;