feat: update package versions and refactor tests for improved stability and clarity

aalmada · aalmada · commit 3cffa44461e3 · 2026-04-17T23:52:01.000+01:00
diff --git a/Directory.Packages.props b/Directory.Packages.props
@@ -14,9 +14,7 @@
     <PackageVersion Include="Azure.Storage.Blobs" Version="12.27.0" />
     <PackageVersion Include="Blazored.LocalStorage" Version="4.3.0" />
     <PackageVersion Include="Bogus" Version="35.6.5" />
-    <PackageVersion Include="bunit" Version="1.40.0" />
-    <PackageVersion Include="bunit.web" Version="1.40.0" />
-    <PackageVersion Include="coverlet.collector" Version="6.0.4" />
+    <PackageVersion Include="bunit" Version="2.7.2" />
     <PackageVersion Include="JasperFx.Core" Version="1.12.0" />
     <PackageVersion Include="MailKit" Version="4.15.1" />
     <PackageVersion Include="Marten" Version="8.18.0" />
@@ -53,11 +51,11 @@
     <PackageVersion Include="SkiaSharp.NativeAssets.Linux.NoDependencies" Version="3.119.1" />
     <PackageVersion Include="SkiaSharp.NativeAssets.macOS" Version="3.119.1" />
     <PackageVersion Include="System.IdentityModel.Tokens.Jwt" Version="8.15.0" />
-    <PackageVersion Include="TUnit" Version="1.12.15" />
+    <PackageVersion Include="TUnit" Version="1.36.0" />
     <PackageVersion Include="Weasel.Core" Version="8.4.3" />
     <PackageVersion Include="WolverineFx.Http" Version="5.11.0" />
     <PackageVersion Include="WolverineFx.Marten" Version="5.11.0" />
-    <PackageVersion Include="Microsoft.SourceLink.GitHub" Version="10.0.102" />
+    <PackageVersion Include="Microsoft.SourceLink.GitHub" Version="10.0.202" />
     <PackageVersion Include="Roslynator.Analyzers" Version="4.15.0" />
   </ItemGroup>
 </Project>
diff --git a/tests/BookStore.ApiService.UnitTests/BookStore.ApiService.UnitTests.csproj b/tests/BookStore.ApiService.UnitTests/BookStore.ApiService.UnitTests.csproj
@@ -8,10 +8,6 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="coverlet.collector">
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-      <PrivateAssets>all</PrivateAssets>
-    </PackageReference>
     <PackageReference Include="NSubstitute" />
     <PackageReference Include="TUnit" />
   </ItemGroup>
diff --git a/tests/BookStore.AppHost.Tests/AuthTests.cs b/tests/BookStore.AppHost.Tests/AuthTests.cs
@@ -9,9 +9,10 @@ namespace BookStore.AppHost.Tests;
 
 public class AuthTests
 {
-    readonly IIdentityClient _client;
+    IIdentityClient _client = null!;
 
-    public AuthTests()
+    [Before(Test)]
+    public void Setup()
     {
         var httpClient = HttpClientHelpers.GetUnauthenticatedClient();
         _client = RestService.For<IIdentityClient>(httpClient);
diff --git a/tests/BookStore.AppHost.Tests/BookStore.AppHost.Tests.csproj b/tests/BookStore.AppHost.Tests/BookStore.AppHost.Tests.csproj
@@ -12,10 +12,6 @@
     <PackageReference Include="Azure.Storage.Blobs" />
     <PackageReference Include="Bogus" />
     <PackageReference Include="Microsoft.Playwright" />
-    <PackageReference Include="coverlet.collector">
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-      <PrivateAssets>all</PrivateAssets>
-    </PackageReference>
     <PackageReference Include="Microsoft.Extensions.Logging.Console" />
     <PackageReference Include="Marten" />
     <PackageReference Include="Npgsql" />
diff --git a/tests/BookStore.AppHost.Tests/MultiLanguageTranslationTests.cs b/tests/BookStore.AppHost.Tests/MultiLanguageTranslationTests.cs
@@ -148,7 +148,7 @@ public async Task Book_Update_ShouldPreserveAllDescriptions()
             }
         };
 
-        var updatedBook = await BookHelpers.UpdateBookAsync(client, book.Id, updateRequest, book.ETag);
+        var updatedBook = await BookHelpers.UpdateBookAsync(client, book.Id, updateRequest, etag!);
         _ = await Assert.That(updatedBook).IsNotNull();
 
         // 4. Verify using Accept-Language
diff --git a/tests/BookStore.Web.Tests/BookStore.Web.Tests.csproj b/tests/BookStore.Web.Tests/BookStore.Web.Tests.csproj
@@ -11,7 +11,6 @@
     <PackageReference Include="Aspire.Hosting.Testing" />
     <PackageReference Include="Bogus" />
     <PackageReference Include="bunit" />
-    <PackageReference Include="bunit.web" />
     <PackageReference Include="Microsoft.JSInterop" />
     <PackageReference Include="Microsoft.AspNetCore.Components.Web" />
     <PackageReference Include="MudBlazor" />
diff --git a/tests/BookStore.Web.Tests/Components/AddBookDialogTests.cs b/tests/BookStore.Web.Tests/Components/AddBookDialogTests.cs
@@ -102,7 +102,7 @@ await dialogProvider.InvokeAsync(async () =>
         dialogProvider.WaitForState(() => dialogProvider.Markup.Contains("Edit Book", StringComparison.Ordinal));
 
         // Assert pre-population
-        var expectedPriceValue = existingGbpPrice.ToString(System.Globalization.CultureInfo.CurrentCulture);
+        var expectedPriceValue = existingGbpPrice.ToString(System.Globalization.CultureInfo.InvariantCulture);
         var gbpInput = dialogProvider.Find($"input[value='{expectedPriceValue}']");
         _ = await Assert.That(gbpInput.GetAttribute("value")).IsEqualTo(expectedPriceValue);
 
diff --git a/tests/BookStore.Web.Tests/Components/CurrencySelectorTests.cs b/tests/BookStore.Web.Tests/Components/CurrencySelectorTests.cs
@@ -38,8 +38,8 @@ public async Task CurrencySelector_ShouldRenderCurrentCurrency()
         var cut = RenderComponent<CurrencySelector>();
 
         // Assert
-        var input = cut.Find("input");
-        _ = await Assert.That(input.GetAttribute("value")).IsEqualTo("GBP");
+        var button = cut.Find("button[aria-label='Change Currency']");
+        _ = await Assert.That(button.TextContent).Contains("GBP");
     }
 
     [Test]
@@ -52,7 +52,7 @@ public async Task CurrencySelector_ShouldChangeCurrency_OnClick()
         await cut.InvokeAsync(async () => await _currencyService.SetCurrencyAsync("EUR"));
 
         // Assert
-        var input = cut.Find("input");
-        _ = await Assert.That(input.GetAttribute("value")).IsEqualTo("EUR");
+        var button = cut.Find("button[aria-label='Change Currency']");
+        _ = await Assert.That(button.TextContent).Contains("EUR");
     }
 }
diff --git a/tests/BookStore.Web.Tests/Infrastructure/BunitTestContext.cs b/tests/BookStore.Web.Tests/Infrastructure/BunitTestContext.cs
@@ -7,7 +7,7 @@ namespace BookStore.Web.Tests.Infrastructure;
 
 public abstract class BunitTestContext : IDisposable
 {
-    protected Bunit.TestContext Context { get; } = new();
+    protected BunitContext Context { get; } = new();
 
     protected BunitTestContext()
     {
@@ -36,7 +36,7 @@ protected IRenderedComponent<TComponent> RenderComponent<TComponent>(
         Action<ComponentParameterCollectionBuilder<TComponent>>? parameterBuilder = null)
         where TComponent : Microsoft.AspNetCore.Components.IComponent
     {
-        _ = Context.RenderComponent<MudPopoverProvider>();
-        return Context.RenderComponent(parameterBuilder);
+        _ = Context.Render<MudPopoverProvider>();
+        return Context.Render(parameterBuilder);
     }
 }
diff --git a/tests/BookStore.Web.Tests/Services/TenantServiceTests.cs b/tests/BookStore.Web.Tests/Services/TenantServiceTests.cs
@@ -26,7 +26,7 @@ public void Setup()
         _localStorage = Substitute.For<ILocalStorageService>();
         _js = Substitute.For<IJSRuntime>();
 
-        // NavigationManager is already registered in bUnit Context.Services as FakeNavigationManager
+        // NavigationManager is already registered in bUnit Context.Services as BunitNavigationManager
         var navigation = Context.Services.GetRequiredService<NavigationManager>();
 
         _sut = new TenantService(_tenantClient, navigation, _localStorage, _js);
@@ -39,7 +39,7 @@ public void Setup()
     public async Task InitializeAsync_ShouldSetTenantFromUrl_WhenPresent()
     {
         // Arrange
-        var navigation = Context.Services.GetRequiredService<FakeNavigationManager>();
+        var navigation = Context.Services.GetRequiredService<BunitNavigationManager>();
         navigation.NavigateTo("http://localhost/?tenant=apple");
 
         var tenantInfo = new TenantInfoDto("apple", "Apple Store", "Tagline", "#FFFFFF", true);
@@ -58,7 +58,7 @@ public async Task InitializeAsync_ShouldSetTenantFromUrl_WhenPresent()
     public async Task InitializeAsync_ShouldFallbackToLocalStorage_WhenUrlEmpty()
     {
         // Arrange
-        var navigation = Context.Services.GetRequiredService<FakeNavigationManager>();
+        var navigation = Context.Services.GetRequiredService<BunitNavigationManager>();
         navigation.NavigateTo("http://localhost/");
 
         _ = _localStorage.GetItemAsStringAsync("selected-tenant").Returns("banana");

Original file line number	Diff line number	Diff line change
`@@ -9,9 +9,10 @@ namespace BookStore.AppHost.Tests;`
`9`	`9`
`10`	`10`	`public class AuthTests`
`11`	`11`	`{`
`12`		`- readonly IIdentityClient _client;`
	`12`	`+ IIdentityClient _client = null!;`
`13`	`13`
`14`		`- public AuthTests()`
	`14`	`+ [Before(Test)]`
	`15`	`+ public void Setup()`
`15`	`16`	`{`
`16`	`17`	`var httpClient = HttpClientHelpers.GetUnauthenticatedClient();`
`17`	`18`	`_client = RestService.For<IIdentityClient>(httpClient);`
Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ public async Task Book_Update_ShouldPreserveAllDescriptions()`
`148`	`148`	`}`
`149`	`149`	`};`
`150`	`150`
`151`		`- var updatedBook = await BookHelpers.UpdateBookAsync(client, book.Id, updateRequest, book.ETag);`
	`151`	`+ var updatedBook = await BookHelpers.UpdateBookAsync(client, book.Id, updateRequest, etag!);`
`152`	`152`	`_ = await Assert.That(updatedBook).IsNotNull();`
`153`	`153`
`154`	`154`	`// 4. Verify using Accept-Language`
Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,8 @@ public async Task CurrencySelector_ShouldRenderCurrentCurrency()`
`38`	`38`	`var cut = RenderComponent<CurrencySelector>();`
`39`	`39`
`40`	`40`	`// Assert`
`41`		`- var input = cut.Find("input");`
`42`		`- _ = await Assert.That(input.GetAttribute("value")).IsEqualTo("GBP");`
	`41`	`+ var button = cut.Find("button[aria-label='Change Currency']");`
	`42`	`+ _ = await Assert.That(button.TextContent).Contains("GBP");`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`[Test]`
`@@ -52,7 +52,7 @@ public async Task CurrencySelector_ShouldChangeCurrency_OnClick()`
`52`	`52`	`await cut.InvokeAsync(async () => await _currencyService.SetCurrencyAsync("EUR"));`
`53`	`53`
`54`	`54`	`// Assert`
`55`		`- var input = cut.Find("input");`
`56`		`- _ = await Assert.That(input.GetAttribute("value")).IsEqualTo("EUR");`
	`55`	`+ var button = cut.Find("button[aria-label='Change Currency']");`
	`56`	`+ _ = await Assert.That(button.TextContent).Contains("EUR");`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ namespace BookStore.Web.Tests.Infrastructure;`
`7`	`7`
`8`	`8`	`public abstract class BunitTestContext : IDisposable`
`9`	`9`	`{`
`10`		`- protected Bunit.TestContext Context { get; } = new();`
	`10`	`+ protected BunitContext Context { get; } = new();`
`11`	`11`
`12`	`12`	`protected BunitTestContext()`
`13`	`13`	`{`
`@@ -36,7 +36,7 @@ protected IRenderedComponent<TComponent> RenderComponent<TComponent>(`
`36`	`36`	`Action<ComponentParameterCollectionBuilder<TComponent>>? parameterBuilder = null)`
`37`	`37`	`where TComponent : Microsoft.AspNetCore.Components.IComponent`
`38`	`38`	`{`
`39`		`- _ = Context.RenderComponent<MudPopoverProvider>();`
`40`		`- return Context.RenderComponent(parameterBuilder);`
	`39`	`+ _ = Context.Render<MudPopoverProvider>();`
	`40`	`+ return Context.Render(parameterBuilder);`
`41`	`41`	`}`
`42`	`42`	`}`