Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
60de2bd
refactor: apply code review nitpick improvements
Nov 25, 2025
516a2a0
fix: remove ConfigurableTestAuthenticationHandler race condition
Nov 25, 2025
0e48824
docs: remove obsolete documentation files
Nov 25, 2025
e50f728
feat: add IDocumentsModuleApi for cross-module communication
Nov 25, 2025
e046895
wip: add comprehensive IBGE API stubs to WireMock + refactor IbgeApiI…
Nov 25, 2025
efdd1ef
feat: add IServiceCatalogsModuleApi for cross-module service validation
Nov 25, 2025
6302d16
fix: refactor IbgeApiIntegrationTests to use WireMock + fix stubs
Nov 25, 2025
916e4dd
test: remove Skip from 2 ServiceCatalogs integration tests after AUTH…
Nov 25, 2025
4940bc2
fix: enable IBGE fail-open with fallback to simple validation
Nov 25, 2025
97dc7ac
feat(providers): integrate IDocumentsModuleApi in provider activation
Nov 25, 2025
62e46fa
feat(providers): prepare ISearchModuleApi integration for provider in…
Nov 25, 2025
a6ad166
feat(search): implement IndexProviderAsync and RemoveProviderAsync in…
Nov 25, 2025
c63ad65
test: remove duplicate GeographicRestrictionFeatureFlagTests
Nov 25, 2025
65897e9
docs: update Sprint 1 progress and skipped tests analysis
Nov 25, 2025
8ca60ea
docs: comprehensive Module APIs documentation in architecture.md
Nov 25, 2025
234a20f
docs: Sprint 1 final summary and completion report
Nov 25, 2025
b77eb08
feat(providers): implement full provider data sync for search indexing
Nov 25, 2025
2cf5816
feat(providers): add ProviderServices many-to-many table
Nov 25, 2025
b355776
test(providers): add comprehensive unit tests for ProviderServices
Nov 25, 2025
bff2381
refactor: rename infrastructure/database/modules folders and update S…
Nov 25, 2025
3d8b469
fix: make SearchProviders schema migration conditional for TestContai…
Nov 25, 2025
4aa726a
fix: corrigir schema hardcoded 'search' para 'search_providers' em qu…
Nov 25, 2025
938c55d
fix: atualizar schema 'search' para 'search_providers' em SearchProvi…
Nov 25, 2025
b317f38
feat: implement cross-module event handlers and fix SearchProviders s…
Nov 26, 2025
ca4b2f9
test: skip CreateAndUpdateUser_ShouldMaintainConsistency in CI/CD
Nov 26, 2025
17b6e55
refactor: code review fixes - remove duplicates, fix schemas, regener…
Nov 26, 2025
5dad19a
chore: final code review fixes - documentation, security, domain events
Nov 26, 2025
ac601cb
fix: correct schema naming to meajudaai_* pattern across all modules
Nov 26, 2025
b98c132
refactor: code quality improvements from review feedback
Nov 26, 2025
59e6265
refactor: minor code quality improvements
Nov 26, 2025
025aee0
test: improve test quality with explicit assertions and Theory pattern
Nov 26, 2025
ee4d1b2
refactor: use IReadOnlyList in ModuleServiceValidationResultDto
Nov 26, 2025
3d7b31b
fix: resolve race condition in ConfigurableTestAuthenticationHandler
Nov 26, 2025
78e47cb
fix: consolidate schema naming to meajudaai_* convention across all m…
Nov 26, 2025
dfa7f10
perf: reduce WireMock timeout simulation from 30s to 5s
Nov 26, 2025
5221f44
refactor: implement code review feedback - Result Match pattern, repo…
Nov 26, 2025
51f7e0d
refactor: rename Mock* to LocalDevelopment* services and move to dedi…
Nov 26, 2025
512cd66
fix: allowUnauthenticated should return anonymous principal, not admin
Nov 26, 2025
780cae9
fix: address CodeRabbit review feedback
Nov 26, 2025
9bfff7c
refactor: implement CodeRabbit Round 2 feedback - null guards, XML do…
Nov 26, 2025
0be5389
refactor: apply minor CodeRabbit polish - constants and comments
Nov 26, 2025
c69c404
feat: implement remaining CodeRabbit suggestions - UUID v7, migration…
Nov 27, 2025
93592d7
refactor: implement CodeRabbit final round - schema standardization, …
Nov 27, 2025
e0b4db3
refactor: complete CodeRabbit final polish - ProductVersion stable, h…
Nov 27, 2025
63a4a2d
fix: resolve E2E test failures - authentication and category deactiva…
Nov 27, 2025
274b378
refactor: apply CodeRabbit feedback - schema naming, async cleanup, t…
Nov 27, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: minor code quality improvements
Changes:
- Fixed misleading SQL comment in search_providers permissions
  (Document schema -> Search Providers schema)
- Removed unused AuthRaceConditionSkipReason constant from ServiceCatalogsModuleIntegrationTests
- Cleaned up comment structure in ApiVersioningTests for better readability

All changes improve code clarity and remove dead code.

Refs: Code review feedback on minor quality issues
  • Loading branch information
Filipe Frigini committed Nov 26, 2025
commit 59e62652397d14855821c3e75beb2d4d8802df39
Original file line number Diff line number Diff line change
Expand Up @@ -43,5 +43,5 @@ GRANT SELECT ON ALL TABLES IN SCHEMA providers TO search_role;
GRANT SELECT ON TABLE public.spatial_ref_sys TO search_role;
GRANT SELECT ON TABLE public.spatial_ref_sys TO meajudaai_app_role;

-- Document schema purpose
-- Search Providers schema purpose
COMMENT ON SCHEMA search_providers IS 'Search & Discovery module - Geospatial provider search with PostGIS';
6 changes: 4 additions & 2 deletions tests/MeAjudaAi.E2E.Tests/Integration/ApiVersioningTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,10 @@ await ApiClient.GetAsync("/api/v2/users") // Versão não suportada - deve ser 4
[Fact]
public async Task ApiVersioning_ShouldWork_ForDifferentModules()
{
// Arrange
AuthenticateAsAdmin(); // Ensure authenticated for API access & Act - Testa se o versionamento funciona para qualquer padrão de módulo
// Arrange - Configure authentication for API access
AuthenticateAsAdmin();

// Act - Test that versioning works for different module patterns
var responses = new[]
{
await ApiClient.GetAsync("/api/v1/users"),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,6 @@ namespace MeAjudaAi.E2E.Tests.Integration;
/// </summary>
public class ServiceCatalogsModuleIntegrationTests : TestContainerTestBase
{
// TODO: Link to GitHub issue #<TBD> tracking E2E authentication infrastructure refactor.
// 14+ E2E tests affected by ConfigurableTestAuthenticationHandler race condition.
private const string AuthRaceConditionSkipReason =
"AUTH: Returns 403 instead of expected success due to ConfigurableTestAuthenticationHandler race condition. See GitHub issue #<TBD>.";

[Fact]
public async Task ServicesModule_Can_Validate_Services_From_Catalogs()
{
Expand Down