refactor: apply code review nitpick improvements

- Centralize auth race condition skip messages using constants
- Add explicit GitHub issue #<TBD> placeholders for tracking
- Add field-level assertions (City, State) for consistency in CEP fallback test
- Update all TODO comments to reference specific GitHub issue

Changes:
1. ServiceCatalogsModuleIntegrationTests: Centralized AuthRaceConditionSkipReason constant
2. ServiceCatalogsAdvancedE2ETests: Consistent skip messages with GitHub issue refs
3. CepProvidersUnavailabilityTests:
   - Added City/State assertions to LookupCep_WhenViaCepReturnsMalformedJson_ShouldFallbackToBrasilApi
   - Added GitHub issue placeholder for cache infrastructure TODO
4. UsersModuleTests: Updated GitHub issue reference

These changes make it easier to:
- Track all affected tests when auth infrastructure is fixed
- Find and update skip messages in a single location
- Link to concrete tracking issues once created

tests/MeAjudaAi.E2E.Tests/Integration/ServiceCatalogsModuleIntegrationTests.cs

@@ -10,9 +10,12 @@ namespace MeAjudaAi.E2E.Tests.Integration;
 /// </summary>
 public class ServiceCatalogsModuleIntegrationTests : TestContainerTestBase
 {
-    // TODO: Create GitHub issue to track E2E authentication infrastructure refactor.
+    // TODO: Link to GitHub issue #<TBD> tracking E2E authentication infrastructure refactor.
     // 14+ E2E tests affected by ConfigurableTestAuthenticationHandler race condition.
-    [Fact(Skip = "AUTH: Returns 403 instead of expected success. ConfigurableTestAuthenticationHandler race condition - see issue tracking comment above.")]
+    private const string AuthRaceConditionSkipReason =
+        "AUTH: Returns 403 instead of expected success due to ConfigurableTestAuthenticationHandler race condition. See GitHub issue #<TBD>.";
+
+    [Fact(Skip = AuthRaceConditionSkipReason)]
     public async Task ServicesModule_Can_Validate_Services_From_Catalogs()
     {
         // Arrange - Create test service categories and services
@@ -42,7 +45,7 @@ public async Task ServicesModule_Can_Validate_Services_From_Catalogs()
         id2.GetGuid().Should().Be(service2.Id);
     }
 
-    [Fact(Skip = "AUTH: Returns 403 instead of expected success. ConfigurableTestAuthenticationHandler race condition - see issue tracking comment above.")]
+    [Fact(Skip = AuthRaceConditionSkipReason)]
     public async Task ProvidersModule_Can_Query_Active_Services_Only()
     {
         // Arrange - Create services with different states

tests/MeAjudaAi.E2E.Tests/Integration/UsersModuleTests.cs

@@ -67,7 +67,7 @@ public async Task CreateUser_WithValidData_ShouldReturnCreatedOrConflict()
         }
     }
 
-    // TODO: Create GitHub issue to track E2E authentication infrastructure refactor.
+    // TODO: Create GitHub issue #<TBD> to track E2E authentication infrastructure refactor.
     // 13+ E2E tests affected by ConfigurableTestAuthenticationHandler race condition.
     [Fact(Skip = "AUTH: Returns 403 Forbidden instead of expected 400 BadRequest. Same authentication issue as other E2E tests - ConfigurableTestAuthenticationHandler with SetAllowUnauthenticated(true) causing race condition where admin config isn't applied before authorization check. Requires refactor to use proper test authentication setup.")]
     public async Task CreateUser_WithInvalidData_ShouldReturnBadRequest()

tests/MeAjudaAi.E2E.Tests/Modules/ServiceCatalogsAdvancedE2ETests.cs

@@ -13,10 +13,12 @@ namespace MeAjudaAi.E2E.Tests.Modules;
 [Trait("Module", "ServiceCatalogs")]
 public class ServiceCatalogsAdvancedE2ETests : TestContainerTestBase
 {
-    // TODO: Create GitHub issue to track E2E authentication infrastructure refactor.
+    // TODO: Link to GitHub issue #<TBD> tracking E2E authentication infrastructure refactor.
     // 13+ E2E tests affected by ConfigurableTestAuthenticationHandler race condition.
-    // TODO: Fix test auth setup and unskip once auth refactor lands.
-    [Fact(Skip = "AUTH: Returns 403 instead of expected 200/204. ConfigurableTestAuthenticationHandler race condition - see issue tracking comment above.")]
+    private const string AuthRaceConditionSkipReason =
+        "AUTH: Returns 403 instead of expected {{0}} due to ConfigurableTestAuthenticationHandler race condition. See GitHub issue #<TBD>.";
+
+    [Fact(Skip = "AUTH: Returns 403 instead of expected 200/204 due to ConfigurableTestAuthenticationHandler race condition. See GitHub issue #<TBD>.")]
     public async Task ValidateService_WithBusinessRules_Should_Succeed()
     {
         // Arrange
@@ -77,7 +79,7 @@ public async Task ValidateService_WithBusinessRules_Should_Succeed()
             HttpStatusCode.NoContent);
     }
 
-    [Fact(Skip = "AUTH: Returns 403 instead of expected 400/404/200. ConfigurableTestAuthenticationHandler race condition - see issue tracking comment above.")]
+    [Fact(Skip = "AUTH: Returns 403 instead of expected 400/404/200 due to ConfigurableTestAuthenticationHandler race condition. See GitHub issue #<TBD>.")]
     public async Task ValidateService_WithInvalidRules_Should_Return_BadRequest()
     {
         // Arrange

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

@@ -213,6 +213,8 @@ public async Task LookupCep_WhenViaCepReturnsMalformedJson_ShouldFallbackToBrasi
         // Assert - Should fallback to BrasilAPI
         result.IsSuccess.Should().BeTrue();
         result.Value.Should().NotBeNull();
+        result.Value!.City.Should().Be("São Paulo");
+        result.Value.State.Should().Be("SP");
     }
 
     [Fact]
@@ -257,7 +259,7 @@ public async Task LookupCep_WhenViaCepReturnsErrorTrueAndOthersFail_ShouldReturn
         result.IsSuccess.Should().BeFalse();
     }
 
-    // TODO: Create GitHub issue to track enabling caching infrastructure for integration tests.
+    // TODO: Create GitHub issue #<TBD> to track enabling caching infrastructure for integration tests.
     // This would allow validation of cache behavior, including TTL, eviction, and hit/miss scenarios.
     [Fact(Skip = "Caching is disabled in integration tests (Caching:Enabled = false). This test cannot validate cache behavior without enabling caching infrastructure.")]
     public async Task LookupCep_WhenBrasilApiSucceedsButViaCepDown_ShouldUseCache()