diff --git a/server/NeonSprawl.Server.Tests/Game/Contracts/ContractOutcomePersistenceIntegrationTests.cs b/server/NeonSprawl.Server.Tests/Game/Contracts/ContractOutcomePersistenceIntegrationTests.cs index 3bd6c0a..d4f14f5 100644 --- a/server/NeonSprawl.Server.Tests/Game/Contracts/ContractOutcomePersistenceIntegrationTests.cs +++ b/server/NeonSprawl.Server.Tests/Game/Contracts/ContractOutcomePersistenceIntegrationTests.cs @@ -39,19 +39,17 @@ public sealed class ContractOutcomePersistenceIntegrationTests(PostgresIntegrati RecordedAt); // Act — write through first host - using (var firstScope = Factory.Services.CreateScope()) - { - var instanceStore = firstScope.ServiceProvider.GetRequiredService(); - var outcomeStore = firstScope.ServiceProvider.GetRequiredService(); - Assert.True(instanceStore.TryCreateActive( - PlayerId, - InstanceId, - TemplateId, - SeedBucket, - IssuedAt, - out _)); - Assert.True(outcomeStore.TryAppend(outcomeRow)); - } + using var firstScope = Factory.Services.CreateScope(); + var instanceStore = firstScope.ServiceProvider.GetRequiredService(); + var outcomeStore = firstScope.ServiceProvider.GetRequiredService(); + Assert.True(instanceStore.TryCreateActive( + PlayerId, + InstanceId, + TemplateId, + SeedBucket, + IssuedAt, + out _)); + Assert.True(outcomeStore.TryAppend(outcomeRow)); ContractOutcomeRow readBack; await using var secondFactory = new PostgresWebApplicationFactory(); @@ -145,24 +143,22 @@ public sealed class ContractOutcomePersistenceIntegrationTests(PostgresIntegrati [], [], RecordedAt); - using (var scope = Factory.Services.CreateScope()) - { - var instanceStore = scope.ServiceProvider.GetRequiredService(); - var outcomeStore = scope.ServiceProvider.GetRequiredService(); - Assert.True(instanceStore.TryCreateActive( - PlayerId, - InstanceId, - TemplateId, - SeedBucket, - IssuedAt, - out _)); - await SeedPostgresPlayerAsync(otherPlayerId); - // Act - var appended = outcomeStore.TryAppend(outcomeRow); - // Assert - Assert.False(appended); - Assert.Empty(outcomeStore.GetOutcomesForInstance(InstanceId)); - } + using var scope = Factory.Services.CreateScope(); + var instanceStore = scope.ServiceProvider.GetRequiredService(); + var outcomeStore = scope.ServiceProvider.GetRequiredService(); + Assert.True(instanceStore.TryCreateActive( + PlayerId, + InstanceId, + TemplateId, + SeedBucket, + IssuedAt, + out _)); + await SeedPostgresPlayerAsync(otherPlayerId); + // Act + var appended = outcomeStore.TryAppend(outcomeRow); + // Assert + Assert.False(appended); + Assert.Empty(outcomeStore.GetOutcomesForInstance(InstanceId)); } [RequirePostgresFact] @@ -180,17 +176,15 @@ public sealed class ContractOutcomePersistenceIntegrationTests(PostgresIntegrati [], [], RecordedAt); - using (var scope = Factory.Services.CreateScope()) - { - var instanceStore = scope.ServiceProvider.GetRequiredService(); - Assert.True(instanceStore.TryCreateActive( - PlayerId, - InstanceId, - TemplateId, - SeedBucket, - IssuedAt, - out _)); - } + using var seedScope = Factory.Services.CreateScope(); + var instanceStore = seedScope.ServiceProvider.GetRequiredService(); + Assert.True(instanceStore.TryCreateActive( + PlayerId, + InstanceId, + TemplateId, + SeedBucket, + IssuedAt, + out _)); // Act var results = await Task.WhenAll(