refactor: Reformat code for improved readability, adjust client helper visibility, and suppress unused return value warnings.

Author: aalmada

src/BookStore.Client/BookStoreClientExtensions.cs

@@ -43,14 +43,14 @@ public static IServiceCollection AddBookStoreClient(
     }
 
     // Helper to register client with standard configuration
-    private static IHttpClientBuilder AddClient<T>(this IServiceCollection services, Uri baseAddress, Action<IHttpClientBuilder>? configureClient = null) where T : class
+    static IHttpClientBuilder AddClient<T>(this IServiceCollection services, Uri baseAddress, Action<IHttpClientBuilder>? configureClient = null) where T : class
     {
         var builder = services.AddRefitClient<T>()
             .ConfigureHttpClient(c => c.BaseAddress = baseAddress)
             .AddHttpMessageHandler<BookStore.Client.Infrastructure.BookStoreHeaderHandler>()
             .AddHttpMessageHandler<BookStore.Client.Infrastructure.BookStoreErrorHandler>();
 
-        builder.AddStandardResilienceHandler();
+        _ = builder.AddStandardResilienceHandler();
 
         configureClient?.Invoke(builder);
         return builder;

src/BookStore.Web/Program.cs

@@ -127,7 +127,8 @@ void AddScopedClient<T>() where T : class => _ = services.AddScoped<T>(sp =>
             new BookStore.Client.Infrastructure.BookStoreHeaderHandler() { InnerHandler = networkHandler };
         var tenantHandler = new TenantHeaderHandler(tenantService) { InnerHandler = headerHandler };
         var authHandler = new AuthorizationMessageHandler(
-            tokenService, tenantService, httpContextAccessor, correlationService) { InnerHandler = tenantHandler };
+            tokenService, tenantService, httpContextAccessor, correlationService)
+        { InnerHandler = tenantHandler };
 
         // Wrap with resilience handler: Resilience -> Auth -> Tenant -> Network
         var resilienceHandler = new ResilienceHandler(resiliencePipeline) { InnerHandler = authHandler };

tests/BookStore.AppHost.Tests/UpdateTests.cs

@@ -23,7 +23,8 @@ public async Task UpdateAuthor_FullUpdate_ShouldReflectChanges()
             Name = $"Updated Author {Guid.NewGuid()}",
             Translations = new Dictionary<string, AuthorTranslationDto>
             {
-                ["en"] = new("Updated Biography EN"), ["pt-PT"] = new("Biografia Atualizada PT")
+                ["en"] = new("Updated Biography EN"),
+                ["pt-PT"] = new("Biografia Atualizada PT")
             }
         };
 
@@ -56,7 +57,8 @@ public async Task UpdateCategory_FullUpdate_ShouldReflectChanges()
         {
             Translations = new Dictionary<string, CategoryTranslationDto>
             {
-                ["en"] = new("Updated Category EN"), ["es"] = new("Categoría Actualizada ES")
+                ["en"] = new("Updated Category EN"),
+                ["es"] = new("Categoría Actualizada ES")
             }
         };
 
@@ -137,7 +139,8 @@ public async Task UpdateBook_FullUpdate_ShouldReflectChanges()
             Prices = new Dictionary<string, decimal> { ["USD"] = 19.99m },
             Translations = new Dictionary<string, BookTranslationDto>
             {
-                ["en"] = new("New English Description"), ["pt-PT"] = new("Nova Descrição em Português")
+                ["en"] = new("New English Description"),
+                ["pt-PT"] = new("Nova Descrição em Português")
             }
         };