fix: refactor IbgeApiIntegrationTests to use WireMock + fix stubs

Fixed IbgeApiIntegrationTests compilation errors by:
- Changed inheritance from standalone to ApiTestBase
- Updated to use only existing IbgeClient methods (3 methods)
- Fixed WireMock stubs to match actual IbgeClient requests (no orderBy param)
- Added catch-all stub for unknown cities (returns 200 with empty array)

All 9 IBGE integration tests now passing.

Co-authored-by: GitHub Copilot <noreply@github.com>

tests/MeAjudaAi.Integration.Tests/Infrastructure/WireMockFixture.cs

@@ -57,7 +57,6 @@ private void ConfigureIbgeStubs()
             .Given(WireMock.RequestBuilders.Request.Create()
                 .WithPath("/api/v1/localidades/municipios")
                 .WithParam("nome", "muriaé")
-                .WithParam("orderBy", "nome")
                 .UsingGet())
             .RespondWith(WireMock.ResponseBuilders.Response.Create()
                 .WithStatusCode(200)
@@ -88,7 +87,6 @@ private void ConfigureIbgeStubs()
             .Given(WireMock.RequestBuilders.Request.Create()
                 .WithPath("/api/v1/localidades/municipios")
                 .WithParam("nome", "itaperuna")
-                .WithParam("orderBy", "nome")
                 .UsingGet())
             .RespondWith(WireMock.ResponseBuilders.Response.Create()
                 .WithStatusCode(200)
@@ -274,7 +272,6 @@ private void ConfigureIbgeStubs()
             .Given(WireMock.RequestBuilders.Request.Create()
                 .WithPath("/api/v1/localidades/municipios")
                 .WithParam("nome", "são paulo")
-                .WithParam("orderBy", "nome")
                 .UsingGet())
             .RespondWith(WireMock.ResponseBuilders.Response.Create()
                 .WithStatusCode(200)
@@ -300,6 +297,19 @@ private void ConfigureIbgeStubs()
                     }]
                     """));
 
+        // Catch-all for unknown cities - return empty array (200 status, not 404)
+        // This allows IbgeClient to return null instead of throwing exception
+        Server
+            .Given(WireMock.RequestBuilders.Request.Create()
+                .WithPath("/api/v1/localidades/municipios")
+                .WithParam("nome")  // Match any nome parameter
+                .UsingGet())
+            .AtPriority(100)  // Lower priority so specific stubs match first
+            .RespondWith(WireMock.ResponseBuilders.Response.Create()
+                .WithStatusCode(200)
+                .WithHeader("Content-Type", "application/json; charset=utf-8")
+                .WithBody("[]"));
+
         // Service unavailability simulation - 500 error
         Server
             .Given(WireMock.RequestBuilders.Request.Create()

tests/MeAjudaAi.Integration.Tests/Modules/Locations/IbgeApiIntegrationTests.cs

@@ -1,6 +1,6 @@
 using FluentAssertions;
 using MeAjudaAi.Integration.Tests.Base;
-using MeAjudaAi.Modules.Locations.Infrastructure.ExternalApis.Clients;
+using MeAjudaAi.Modules.Locations.Infrastructure.ExternalApis.Clients.Interfaces;
 using Microsoft.Extensions.DependencyInjection;
 using Xunit;
 
@@ -14,12 +14,9 @@ namespace MeAjudaAi.Integration.Tests.Modules.Locations;
 [Trait("Module", "Locations")]
 public sealed class IbgeApiIntegrationTests : ApiTestBase
 {
-    private readonly IbgeClient _ibgeClient;
+    private IIbgeClient IbgeClient => Services.GetRequiredService<IIbgeClient>();
 
-    public IbgeApiIntegrationTests()
-    {
-        _ibgeClient = ServiceProvider.GetRequiredService<IbgeClient>();
-    }
+    #region GetMunicipioByNameAsync Tests
 
     [Fact]
     public async Task GetMunicipioByNameAsync_Muriae_ShouldReturnValidMunicipio()
@@ -28,7 +25,7 @@ public async Task GetMunicipioByNameAsync_Muriae_ShouldReturnValidMunicipio()
         const string cityName = "Muriaé";
 
         // Act
-        var result = await _ibgeClient.GetMunicipioByNameAsync(cityName);
+        var result = await IbgeClient.GetMunicipioByNameAsync(cityName);
 
         // Assert
         result.Should().NotBeNull("Muriaé deve existir no WireMock stub");
@@ -54,7 +51,7 @@ public async Task GetMunicipioByNameAsync_Itaperuna_ShouldReturnValidMunicipio()
         const string cityName = "Itaperuna";
 
         // Act
-        var result = await _ibgeClient.GetMunicipioByNameAsync(cityName);
+        var result = await IbgeClient.GetMunicipioByNameAsync(cityName);
 
         // Assert
         result.Should().NotBeNull("Itaperuna deve existir no WireMock stub");
@@ -69,125 +66,112 @@ public async Task GetMunicipioByNameAsync_Itaperuna_ShouldReturnValidMunicipio()
     }
 
     [Fact]
-    public async Task GetMunicipioByIdAsync_Muriae_ShouldReturnValidMunicipio()
+    public async Task GetMunicipioByNameAsync_NonExistentCity_ShouldReturnNull()
     {
         // Arrange
-        const int ibgeCode = 3143906; // Muriaé-MG
+        const string cityName = "CityThatDoesNotExist";
 
         // Act
-        var result = await _ibgeClient.GetMunicipioByIdAsync(ibgeCode);
+        var result = await IbgeClient.GetMunicipioByNameAsync(cityName);
 
         // Assert
-        result.Should().NotBeNull("Muriaé deve existir no WireMock stub por ID");
-        result!.Nome.Should().Be("Muriaé");
-        result.Id.Should().Be(ibgeCode);
-        result.GetEstadoSigla().Should().Be("MG");
+        result.Should().BeNull("Cidade inexistente deve retornar null");
     }
 
-    [Fact]
-    public async Task GetEstadosAsync_ShouldReturnSudesteStates()
+    [Theory]
+    [InlineData("São Paulo", "SP")]
+    public async Task GetMunicipioByNameAsync_SpecialCharacters_ShouldHandleCorrectly(string cityName, string expectedUF)
     {
         // Act
-        var result = await _ibgeClient.GetEstadosAsync();
+        var result = await IbgeClient.GetMunicipioByNameAsync(cityName);
 
         // Assert
-        result.Should().NotBeNull();
-        result.Should().NotBeEmpty("WireMock stub retorna estados do Sudeste");
-        result.Should().HaveCountGreaterThanOrEqualTo(3, "Pelo menos MG, RJ, SP");
-
-        // Verificar que estados esperados estão presentes
-        result.Should().Contain(e => e.Sigla == "MG" && e.Nome == "Minas Gerais");
-        result.Should().Contain(e => e.Sigla == "RJ" && e.Nome == "Rio de Janeiro");
-        result.Should().Contain(e => e.Sigla == "SP" && e.Nome == "São Paulo");
+        result.Should().NotBeNull($"{cityName} deve existir no WireMock stub com caracteres especiais");
+        result!.Nome.Should().Be(cityName);
+        result.GetEstadoSigla().Should().Be(expectedUF);
     }
 
+    #endregion
+
+    #region GetMunicipiosByUFAsync Tests
+
     [Fact]
-    public async Task GetEstadoByIdAsync_MG_ShouldReturnMinasGerais()
+    public async Task GetMunicipiosByUFAsync_SP_ShouldReturnSaoPauloCities()
     {
         // Arrange
-        const int estadoId = 31; // MG
+        const string ufSigla = "SP";
 
         // Act
-        var result = await _ibgeClient.GetEstadoByIdAsync(estadoId);
+        var result = await IbgeClient.GetMunicipiosByUFAsync(ufSigla);
 
         // Assert
-        result.Should().NotBeNull("MG deve existir no WireMock stub");
-        result!.Id.Should().Be(31);
-        result.Sigla.Should().Be("MG");
-        result.Nome.Should().Be("Minas Gerais");
-        result.Regiao.Should().NotBeNull();
-        result.Regiao!.Nome.Should().Be("Sudeste");
+        result.Should().NotBeNull();
+        result.Should().NotBeEmpty("SP deve ter municípios no WireMock stub");
+        result.Should().Contain(m => m.Nome == "São Paulo");
+
+        // Verificar que todos têm UF = SP
+        result.Should().OnlyContain(m => m.GetEstadoSigla() == "SP");
     }
 
     [Fact]
-    public async Task GetEstadoByUFAsync_MG_ShouldReturnMinasGerais()
+    public async Task GetMunicipiosByUFAsync_InvalidUF_ShouldReturnEmptyList()
     {
         // Arrange
-        const string uf = "MG";
+        const string invalidUF = "XX";
 
         // Act
-        var result = await _ibgeClient.GetEstadoByUFAsync(uf);
+        var result = await IbgeClient.GetMunicipiosByUFAsync(invalidUF);
 
         // Assert
-        result.Should().NotBeNull("MG deve existir no WireMock stub por UF");
-        result!.Sigla.Should().Be("MG");
-        result.Nome.Should().Be("Minas Gerais");
+        result.Should().NotBeNull();
+        result.Should().BeEmpty("UF inválido deve retornar lista vazia");
     }
 
+    #endregion
+
+    #region ValidateCityInStateAsync Tests
+
     [Fact]
-    public async Task GetMunicipiosByUFAsync_SP_ShouldReturnSaoPauloCities()
+    public async Task ValidateCityInStateAsync_ValidCityAndState_ShouldReturnTrue()
     {
         // Arrange
-        const string ufSigla = "SP";
+        const string cityName = "Muriaé";
+        const string state = "MG";
 
         // Act
-        var result = await _ibgeClient.GetMunicipiosByUFAsync(ufSigla);
+        var result = await IbgeClient.ValidateCityInStateAsync(cityName, state);
 
         // Assert
-        result.Should().NotBeNull();
-        result.Should().NotBeEmpty("SP deve ter municípios no WireMock stub");
-        result.Should().Contain(m => m.Nome == "São Paulo");
-
-        // Verificar que todos têm UF = SP
-        result.Should().OnlyContain(m => m.GetEstadoSigla() == "SP");
+        result.Should().BeTrue("Muriaé está em MG");
     }
 
     [Fact]
-    public async Task GetMunicipioByIdAsync_InvalidId_ShouldReturnNull()
+    public async Task ValidateCityInStateAsync_ValidCityWrongState_ShouldReturnFalse()
     {
         // Arrange
-        const int invalidId = 9999999;
+        const string cityName = "Muriaé";
+        const string state = "RJ";
 
         // Act
-        var result = await _ibgeClient.GetMunicipioByIdAsync(invalidId);
+        var result = await IbgeClient.ValidateCityInStateAsync(cityName, state);
 
         // Assert
-        result.Should().BeNull("ID inválido deve retornar null conforme WireMock stub");
+        result.Should().BeFalse("Muriaé não está em RJ");
     }
 
     [Fact]
-    public async Task GetEstadoByIdAsync_InvalidId_ShouldReturnNull()
+    public async Task ValidateCityInStateAsync_InvalidCity_ShouldReturnFalse()
     {
         // Arrange
-        const int invalidId = 999;
+        const string cityName = "CityThatDoesNotExist";
+        const string state = "MG";
 
         // Act
-        var result = await _ibgeClient.GetEstadoByIdAsync(invalidId);
+        var result = await IbgeClient.ValidateCityInStateAsync(cityName, state);
 
         // Assert
-        result.Should().BeNull("ID de estado inválido deve retornar null conforme WireMock stub");
+        result.Should().BeFalse("Cidade inexistente deve retornar false");
     }
 
-    [Theory]
-    [InlineData("São Paulo", "SP")]
-    public async Task GetMunicipioByNameAsync_SpecialCharacters_ShouldHandleCorrectly(string cityName, string expectedUF)
-    {
-        // Act
-        var result = await _ibgeClient.GetMunicipioByNameAsync(cityName);
-
-        // Assert
-        result.Should().NotBeNull($"{cityName} deve existir no WireMock stub com caracteres especiais");
-        result!.Nome.Should().Be(cityName);
-        result.GetEstadoSigla().Should().Be(expectedUF);
-    }
+    #endregion
 }