fix(#8): fix flaky integration tests caused by wrong dispose order

https-richardy · https-richardy · commit a8c40c6fc48e · 2026-03-05T00:21:51.000-03:00
diff --git a/Boundaries/Comanda.Subscriptions/Tests/Fixtures/WebApplicationFixture.cs b/Boundaries/Comanda.Subscriptions/Tests/Fixtures/WebApplicationFixture.cs
@@ -21,6 +21,7 @@ public WebApplicationFixture()
     public async ValueTask InitializeAsync()
     {
         await _databaseFixture.InitializeAsync();
+        await _databaseFixture.CleanDatabaseAsync();
 
         Environment.SetEnvironmentVariable("Settings__Database__ConnectionString", _databaseFixture.ConnectionString);
         Environment.SetEnvironmentVariable("Settings__Database__DatabaseName", _databaseFixture.DatabaseName);
@@ -73,7 +74,6 @@ public async ValueTask DisposeAsync()
         HttpClient.Dispose();
 
         await _factory.DisposeAsync();
-        await _databaseFixture.CleanDatabaseAsync();
         await _databaseFixture.DisposeAsync();
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@ public WebApplicationFixture()`
`21`	`21`	`public async ValueTask InitializeAsync()`
`22`	`22`	`{`
`23`	`23`	`await _databaseFixture.InitializeAsync();`
	`24`	`+ await _databaseFixture.CleanDatabaseAsync();`
`24`	`25`
`25`	`26`	`Environment.SetEnvironmentVariable("Settings__Database__ConnectionString", _databaseFixture.ConnectionString);`
`26`	`27`	`Environment.SetEnvironmentVariable("Settings__Database__DatabaseName", _databaseFixture.DatabaseName);`
`@@ -73,7 +74,6 @@ public async ValueTask DisposeAsync()`
`73`	`74`	`HttpClient.Dispose();`
`74`	`75`
`75`	`76`	`await _factory.DisposeAsync();`
`76`		`- await _databaseFixture.CleanDatabaseAsync();`
`77`	`77`	`await _databaseFixture.DisposeAsync();`
`78`	`78`	`}`
`79`	`79`	`}`