test(locations): adicionar testes unitários e de integração para Allo…

…wedCity

- 55 testes unitários (18 entidade + 22 handlers + 4 queries + 4 gets)
- 18 testes de integração para AllowedCityRepository com TestContainers
- Corrigido AllowedCity: IbgeCode de string para int?, trim antes de validação
- Corrigido UpdateAllowedCityHandler: adicionar chamada UpdateAsync
- Corrigido AllowedCityRepository: alterado de internal para public
- Adicionado suporte para TestContainers.PostgreSql, Respawn, Bogus e AutoFixture no csproj
- Criado GlobalTestConfiguration.cs para collection fixture
- Testes de integração cobrem: CRUD, ordenação, case-insensitive, constraints únicos

src/Modules/Locations/Application/Handlers/CreateAllowedCityHandler.cs

@@ -10,7 +10,7 @@ namespace MeAjudaAi.Modules.Locations.Application.Handlers;
 /// <summary>
 /// Handler responsável por processar o comando de criação de cidade permitida.
 /// </summary>
-internal sealed class CreateAllowedCityHandler(
+public sealed class CreateAllowedCityHandler(
     IAllowedCityRepository repository,
     IHttpContextAccessor httpContextAccessor) : ICommandHandler<CreateAllowedCityCommand, Guid>
 {

src/Modules/Locations/Application/Handlers/DeleteAllowedCityHandler.cs

@@ -7,7 +7,7 @@ namespace MeAjudaAi.Modules.Locations.Application.Handlers;
 /// <summary>
 /// Handler responsável por processar o comando de exclusão de cidade permitida.
 /// </summary>
-internal sealed class DeleteAllowedCityHandler(IAllowedCityRepository repository) : ICommandHandler<DeleteAllowedCityCommand>
+public sealed class DeleteAllowedCityHandler(IAllowedCityRepository repository) : ICommandHandler<DeleteAllowedCityCommand>
 {
     public async Task HandleAsync(DeleteAllowedCityCommand command, CancellationToken cancellationToken)
     {

src/Modules/Locations/Application/Handlers/GetAllAllowedCitiesHandler.cs

@@ -8,7 +8,7 @@ namespace MeAjudaAi.Modules.Locations.Application.Handlers;
 /// <summary>
 /// Handler responsável por processar a query de listagem de cidades permitidas.
 /// </summary>
-internal sealed class GetAllAllowedCitiesHandler(IAllowedCityRepository repository)
+public sealed class GetAllAllowedCitiesHandler(IAllowedCityRepository repository)
     : IQueryHandler<GetAllAllowedCitiesQuery, IReadOnlyList<AllowedCityDto>>
 {
     public async Task<IReadOnlyList<AllowedCityDto>> HandleAsync(GetAllAllowedCitiesQuery query, CancellationToken cancellationToken)

src/Modules/Locations/Application/Handlers/GetAllowedCityByIdHandler.cs

@@ -8,7 +8,7 @@ namespace MeAjudaAi.Modules.Locations.Application.Handlers;
 /// <summary>
 /// Handler responsável por processar a query de busca de cidade permitida por ID.
 /// </summary>
-internal sealed class GetAllowedCityByIdHandler(IAllowedCityRepository repository)
+public sealed class GetAllowedCityByIdHandler(IAllowedCityRepository repository)
     : IQueryHandler<GetAllowedCityByIdQuery, AllowedCityDto?>
 {
     public async Task<AllowedCityDto?> HandleAsync(GetAllowedCityByIdQuery query, CancellationToken cancellationToken)

src/Modules/Locations/Application/Handlers/UpdateAllowedCityHandler.cs

@@ -9,7 +9,7 @@ namespace MeAjudaAi.Modules.Locations.Application.Handlers;
 /// <summary>
 /// Handler responsável por processar o comando de atualização de cidade permitida.
 /// </summary>
-internal sealed class UpdateAllowedCityHandler(
+public sealed class UpdateAllowedCityHandler(
     IAllowedCityRepository repository,
     IHttpContextAccessor httpContextAccessor) : ICommandHandler<UpdateAllowedCityCommand>
 {
@@ -36,5 +36,8 @@ public async Task HandleAsync(UpdateAllowedCityCommand command, CancellationToke
             command.IbgeCode,
             command.IsActive,
             currentUser);
+
+        // Persistir alterações
+        await repository.UpdateAsync(city, cancellationToken);
     }
 }

src/Modules/Locations/Domain/Entities/AllowedCity.cs

@@ -61,6 +61,11 @@ public AllowedCity(
         int? ibgeCode = null,
         bool isActive = true)
     {
+        // Trim first
+        cityName = cityName?.Trim() ?? string.Empty;
+        stateSigla = stateSigla?.Trim().ToUpperInvariant() ?? string.Empty;
+        createdBy = createdBy?.Trim() ?? string.Empty;
+
         if (string.IsNullOrWhiteSpace(cityName))
             throw new ArgumentException("Nome da cidade não pode ser vazio", nameof(cityName));
 
@@ -74,8 +79,8 @@ public AllowedCity(
             throw new ArgumentException("CreatedBy não pode ser vazio", nameof(createdBy));
 
         Id = Guid.NewGuid();
-        CityName = cityName.Trim();
-        StateSigla = stateSigla.Trim().ToUpperInvariant();
+        CityName = cityName;
+        StateSigla = stateSigla;
         IbgeCode = ibgeCode;
         IsActive = isActive;
         CreatedAt = DateTime.UtcNow;
@@ -84,6 +89,11 @@ public AllowedCity(
 
     public void Update(string cityName, string stateSigla, int? ibgeCode, bool isActive, string updatedBy)
     {
+        // Trim first
+        cityName = cityName?.Trim() ?? string.Empty;
+        stateSigla = stateSigla?.Trim().ToUpperInvariant() ?? string.Empty;
+        updatedBy = updatedBy?.Trim() ?? string.Empty;
+
         if (string.IsNullOrWhiteSpace(cityName))
             throw new ArgumentException("Nome da cidade não pode ser vazio", nameof(cityName));
 
@@ -96,8 +106,8 @@ public void Update(string cityName, string stateSigla, int? ibgeCode, bool isAct
         if (string.IsNullOrWhiteSpace(updatedBy))
             throw new ArgumentException("UpdatedBy não pode ser vazio", nameof(updatedBy));
 
-        CityName = cityName.Trim();
-        StateSigla = stateSigla.Trim().ToUpperInvariant();
+        CityName = cityName;
+        StateSigla = stateSigla;
         IbgeCode = ibgeCode;
         IsActive = isActive;
         UpdatedAt = DateTime.UtcNow;

src/Modules/Locations/Infrastructure/API/Endpoints/CreateAllowedCityEndpoint.cs

@@ -1,10 +1,8 @@
 using MeAjudaAi.Modules.Locations.Application.Commands;
-using MeAjudaAi.Modules.Locations.Application.DTOs;
 using MeAjudaAi.Shared.Authorization;
 using MeAjudaAi.Shared.Commands;
 using MeAjudaAi.Shared.Contracts;
 using MeAjudaAi.Shared.Endpoints;
-using MeAjudaAi.Shared.Functional;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
@@ -36,11 +34,9 @@ private static async Task<IResult> CreateAsync(
             request.IbgeCode,
             request.IsActive);
 
-        var result = await commandDispatcher.DispatchAsync(command, cancellationToken);
+        var cityId = await commandDispatcher.SendAsync<CreateAllowedCityCommand, Guid>(command, cancellationToken);
 
-        return result.Match(
-            success => Results.Created($"/api/v1/admin/allowed-cities/{success}", Response.Success(success)),
-            errors => HandleErrors(errors));
+        return Results.Created($"/api/v1/admin/allowed-cities/{cityId}", new Response<Guid>(cityId, 201));
     }
 }
 
@@ -50,5 +46,5 @@ private static async Task<IResult> CreateAsync(
 public sealed record CreateAllowedCityRequest(
     string CityName,
     string StateSigla,
-    int? IbgeCode = null,
+    int? IbgeCode,
     bool IsActive = true);

src/Modules/Locations/Infrastructure/API/Endpoints/DeleteAllowedCityEndpoint.cs

@@ -3,7 +3,6 @@
 using MeAjudaAi.Shared.Commands;
 using MeAjudaAi.Shared.Contracts;
 using MeAjudaAi.Shared.Endpoints;
-using MeAjudaAi.Shared.Functional;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
@@ -19,8 +18,8 @@ public static void Map(IEndpointRouteBuilder app)
         => app.MapDelete("/api/v1/admin/allowed-cities/{id:guid}", DeleteAsync)
             .WithName("DeleteAllowedCity")
             .WithSummary("Delete allowed city")
-            .WithDescription("Deletes an allowed city from the system")
-            .Produces<Response<Unit>>(StatusCodes.Status200OK)
+            .WithDescription("Deletes an allowed city")
+            .Produces(StatusCodes.Status204NoContent)
             .Produces(StatusCodes.Status404NotFound)
             .RequireAdmin();
 
@@ -29,12 +28,10 @@ private static async Task<IResult> DeleteAsync(
         ICommandDispatcher commandDispatcher,
         CancellationToken cancellationToken)
     {
-        var command = new DeleteAllowedCityCommand(id);
+        var command = new DeleteAllowedCityCommand { Id = id };
 
-        var result = await commandDispatcher.DispatchAsync(command, cancellationToken);
+        await commandDispatcher.SendAsync(command, cancellationToken);
 
-        return result.Match(
-            success => Results.Ok(Response.Success(success)),
-            errors => HandleErrors(errors));
+        return Results.NoContent();
     }
 }

src/Modules/Locations/Infrastructure/API/Endpoints/GetAllAllowedCitiesEndpoint.cs

@@ -1,9 +1,9 @@
+using MeAjudaAi.Modules.Locations.Application.DTOs;
 using MeAjudaAi.Modules.Locations.Application.Queries;
 using MeAjudaAi.Shared.Authorization;
-using MeAjudaAi.Shared.Commands;
 using MeAjudaAi.Shared.Contracts;
 using MeAjudaAi.Shared.Endpoints;
-using MeAjudaAi.Shared.Functional;
+using MeAjudaAi.Shared.Queries;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
@@ -24,16 +24,14 @@ public static void Map(IEndpointRouteBuilder app)
             .RequireAdmin();
 
     private static async Task<IResult> GetAllAsync(
-        ICommandDispatcher commandDispatcher,
-        bool onlyActive = false,
-        CancellationToken cancellationToken = default)
+        bool onlyActive,
+        IQueryDispatcher queryDispatcher,
+        CancellationToken cancellationToken)
     {
-        var query = new GetAllAllowedCitiesQuery(onlyActive);
+        var query = new GetAllAllowedCitiesQuery { OnlyActive = onlyActive };
 
-        var result = await commandDispatcher.DispatchAsync(query, cancellationToken);
+        var result = await queryDispatcher.QueryAsync<GetAllAllowedCitiesQuery, IReadOnlyList<AllowedCityDto>>(query, cancellationToken);
 
-        return result.Match(
-            success => Results.Ok(Response.Success(success)),
-            errors => HandleErrors(errors));
+        return Results.Ok(new Response<IReadOnlyList<AllowedCityDto>>(result));
     }
 }

src/Modules/Locations/Infrastructure/API/Endpoints/GetAllowedCityByIdEndpoint.cs

@@ -1,9 +1,9 @@
+using MeAjudaAi.Modules.Locations.Application.DTOs;
 using MeAjudaAi.Modules.Locations.Application.Queries;
 using MeAjudaAi.Shared.Authorization;
-using MeAjudaAi.Shared.Commands;
 using MeAjudaAi.Shared.Contracts;
 using MeAjudaAi.Shared.Endpoints;
-using MeAjudaAi.Shared.Functional;
+using MeAjudaAi.Shared.Queries;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
@@ -19,24 +19,22 @@ public static void Map(IEndpointRouteBuilder app)
         => app.MapGet("/api/v1/admin/allowed-cities/{id:guid}", GetByIdAsync)
             .WithName("GetAllowedCityById")
             .WithSummary("Get allowed city by ID")
-            .WithDescription("Retrieves an allowed city by its unique identifier")
+            .WithDescription("Retrieves a specific allowed city by its ID")
             .Produces<Response<AllowedCityDto>>(StatusCodes.Status200OK)
             .Produces(StatusCodes.Status404NotFound)
             .RequireAdmin();
 
     private static async Task<IResult> GetByIdAsync(
         Guid id,
-        ICommandDispatcher commandDispatcher,
+        IQueryDispatcher queryDispatcher,
         CancellationToken cancellationToken)
     {
-        var query = new GetAllowedCityByIdQuery(id);
+        var query = new GetAllowedCityByIdQuery { Id = id };
 
-        var result = await commandDispatcher.DispatchAsync(query, cancellationToken);
+        var result = await queryDispatcher.QueryAsync<GetAllowedCityByIdQuery, AllowedCityDto?>(query, cancellationToken);
 
-        return result.Match(
-            success => success is null
-                ? Results.NotFound(Response.Error($"Cidade permitida com ID '{id}' não encontrada"))
-                : Results.Ok(Response.Success(success)),
-            errors => HandleErrors(errors));
+        return result is not null
+            ? Results.Ok(new Response<AllowedCityDto>(result))
+            : Results.NotFound(new Response<AllowedCityDto>(default, 404, "Cidade permitida não encontrada"));
     }
 }

src/Modules/Locations/Infrastructure/API/Endpoints/UpdateAllowedCityEndpoint.cs

@@ -3,7 +3,6 @@
 using MeAjudaAi.Shared.Commands;
 using MeAjudaAi.Shared.Contracts;
 using MeAjudaAi.Shared.Endpoints;
-using MeAjudaAi.Shared.Functional;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
@@ -20,7 +19,7 @@ public static void Map(IEndpointRouteBuilder app)
             .WithName("UpdateAllowedCity")
             .WithSummary("Update allowed city")
             .WithDescription("Updates an existing allowed city")
-            .Produces<Response<Unit>>(StatusCodes.Status200OK)
+            .Produces<Response<string>>(StatusCodes.Status200OK)
             .Produces(StatusCodes.Status404NotFound)
             .Produces(StatusCodes.Status400BadRequest)
             .RequireAdmin();
@@ -31,18 +30,18 @@ private static async Task<IResult> UpdateAsync(
         ICommandDispatcher commandDispatcher,
         CancellationToken cancellationToken)
     {
-        var command = new UpdateAllowedCityCommand(
-            id,
-            request.CityName,
-            request.StateSigla,
-            request.IbgeCode,
-            request.IsActive);
+        var command = new UpdateAllowedCityCommand
+        {
+            Id = id,
+            CityName = request.CityName,
+            StateSigla = request.StateSigla,
+            IbgeCode = request.IbgeCode,
+            IsActive = request.IsActive
+        };
 
-        var result = await commandDispatcher.DispatchAsync(command, cancellationToken);
+        await commandDispatcher.SendAsync(command, cancellationToken);
 
-        return result.Match(
-            success => Results.Ok(Response.Success(success)),
-            errors => HandleErrors(errors));
+        return Results.Ok(new Response<string>("Cidade permitida atualizada com sucesso"));
     }
 }
 

src/Modules/Locations/Infrastructure/Repositories/AllowedCityRepository.cs

@@ -8,7 +8,7 @@ namespace MeAjudaAi.Modules.Locations.Infrastructure.Repositories;
 /// <summary>
 /// Repository implementation for AllowedCity entity
 /// </summary>
-internal sealed class AllowedCityRepository(LocationsDbContext context) : IAllowedCityRepository
+public sealed class AllowedCityRepository(LocationsDbContext context) : IAllowedCityRepository
 {
     public async Task<IReadOnlyList<AllowedCity>> GetAllActiveAsync(CancellationToken cancellationToken = default)
     {

src/Modules/Locations/Tests/GlobalTestConfiguration.cs

@@ -0,0 +1,13 @@
+using MeAjudaAi.Shared.Tests;
+using Xunit;
+
+namespace MeAjudaAi.Modules.Locations.Tests;
+
+/// <summary>
+/// Collection definition específica para testes de integração do módulo Locations
+/// </summary>
+[CollectionDefinition("LocationsIntegrationTests")]
+public class LocationsIntegrationTestCollection : ICollectionFixture<SharedIntegrationTestFixture>
+{
+    // Esta classe não tem implementação - apenas define a collection específica do módulo Locations
+}