-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 4: Health Checks + Data Seeding #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
162 commits
Select commit
Hold shift + click to select a range
68b79ef
docs: remove all staging environment references (project has only dev…
0ba72ee
feat(health-checks): add business-specific health checks and UI dashb…
4027f8a
docs(roadmap): update Sprint 4 progress - health checks completed
a6f2db8
feat(health-checks): add business-specific health checks and UI dashb…
db02a20
docs(roadmap): update Sprint 4 progress - health checks completed
531feb0
feat(data-seeding): add ServiceCatalogs seed script
bb41691
refactor(seeding): separate essential domain data from test data
8555c4f
refactor(structure): reorganize seeds and automation into infrastructure
2b390cc
docs(roadmap): update Sprint 4 with final structure changes
7ef4523
docs(sprint4): add comprehensive TODO document for pending tasks
f88b0e5
chore: remove TODO document - implement instead of documenting
aece8eb
feat(health-checks): complete external services health checks and met…
2dd6cf5
docs(roadmap): mark Sprint 4 as COMPLETE - all health checks implemented
91bd063
Merge branch 'master' into feature/health-checks-seeding
fd9b7fa
refactor: address CodeRabbit review comments
0d8bda6
fix: correct NuGet package IDs and configuration keys
f6bac82
fix: revert to correct NuGet package ID AspNetCore.HealthChecks.Npgsql
9fadf0b
refactor: optimize health check package dependencies
6b733cf
fix: update IBGE API tests for /estados endpoint
6760abd
fix: regenerate package lock files for consistency
de99cc4
test: add missing HTTP mock for IBGE API in health check test
0d58baa
test: improve health check test assertions
54f094b
ci: add database environment variables for integration tests
32f47a8
refactor: apply DRY principle and fix mock isolation
3eea4ef
refactor: separate test execution in aspire-ci-cd workflow and improv…
83e5bbe
refactor: remove dynamic and reflection from health check tests
28a5b03
ci: fix CI issues - install Aspire workload and configure health chec…
6a1f662
fix: apply CodeRabbit suggestions and improve Aspire workload install…
1daa0b4
fix: external services health check should never return Unhealthy
a917b71
perf(ci): conditionally install Aspire workload only for integration …
a4e7442
fix: improve Aspire workload installation and health check endpoint f…
76b74fb
fix(tests): change HealthCheck_ShouldIncludeProvidersDatabase to vali…
adeaca1
ci: add detailed DCP binary verification with explicit path checks
ec371ad
fix(ci): copy improved Aspire workload installation to pr-validation …
c023061
fix: correct FluentAssertions syntax and nullability warning
424f6c8
refactor: remove trailing whitespace and improve health check test ro…
f093a44
fix(ci): remove deprecated Aspire workload installation - use NuGet p…
12d2573
fix(ci): correct YAML indentation and improve code quality
5a4e146
test: skip Aspire DCP tests and fix health check structure
9fd970c
refactor: apply CodeRabbit review suggestions
0659413
fix(e2e): disable external services health checks in E2E tests
31d116e
refactor: apply final CodeRabbit improvements
3622581
fix(e2e): accept 503 as valid response for readiness check in E2E tests
8d27a7a
fix: dispose HttpResponseMessage to prevent connection leaks in healt…
faacbc9
fix: compilation error and apply final CodeRabbit improvements
ebd5010
fix(ci): pin ReportGenerator to v5.3.11 for .NET 8 compatibility
284a329
fix(ci): use latest ReportGenerator with .NET 10 verification
3adfe20
fix(ci): set DOTNET_ROOT and add job timeouts to prevent hangs
8052cac
feat: add configurable health endpoint paths and CI improvements
6ce4667
fix: resolve CS1744 compilation error and apply remaining CodeRabbit …
7c289ea
fix: address CodeRabbit review and fix ReportGenerator DOTNET_ROOT issue
2c26c7b
fix: resolve DOTNET_ROOT detection and strengthen validation
e87dd42
fix: update lychee-action to v2.8.2 to resolve download error
de96402
fix: correct lychee-action version to v2.8.0 and document fixture usage
05e6b7f
fix: use lychee-action@v2 major version tag
c76c416
fix: correct ReportGenerator validation check
afe68ef
chore: update Aspire packages and suppress CS9113 warning
987cfeb
chore: apply CodeRabbit review suggestions
02e0d2f
feat: add Hangfire health check with tests
0503eaa
feat: enable STRICT_COVERAGE enforcement
83d5d7d
Merge branch 'master' into feature/health-checks-seeding
2be1482
fix: critical CI fixes and package casing corrections
9b3da6c
fix: remove Skip from DataSeeding tests and implement TODOs
999e1d3
fix: use EXECUTE...USING for parameterized DO block and fail fast in CI
c3fbba0
fix: address CodeRabbit review feedback on DataSeeding tests
8605b7a
fix: improve CI detection to use truthy value whitelist
dea2f28
fix: remove AspireIntegrationFixture dependency from DataSeeding tests
c15f7ec
fix: remove fail-fast CI logic and hardcode last schema interpolation
c12967d
fix: use NpgsqlConnectionStringBuilder to properly handle special cha…
a190dca
refactor(aspire): reorganize AppHost/ServiceDefaults structure and fi…
85abe72
security: remove hardcoded credentials and extract shared test helper
142221c
fix: enforce credential validation and snake_case naming consistency
629b660
fix: correct TestConnectionHelper method calls
be44c63
refactor(ApiService): reorganizar estrutura e melhorar organização de…
311d783
refactor: consolidar HostedServices e remover código de teste da prod…
db08289
refactor: implementar melhorias de code review
4d2c937
fix: corrigir teste de rate limiting para mensagem em português
58b45a4
fix: padronizar idiomas - logs em inglês, comentários em português
f5a4189
fix: corrigir compression providers e configuração Keycloak
f327c31
fix: atualizar teste GeographicRestrictionMiddleware para mensagem em…
5dfc46b
fix: corrigir problemas de integração de testes
3c644a6
fix: corrigir todos os problemas dos testes
630e066
fix: apply code review corrections and migrate integration tests to T…
805ed5d
refactor: improve HangfireHealthCheck verification and test quality
465104f
perf: optimize RateLimitingMiddleware and improve monitoring
6f62bc6
fix(ci): remove broken ReportGenerator version check
38efede
refactor(documents): reorganize module structure and improve code qua…
d671112
feat(Documents): Implement configurable MinimumConfidence and integra…
490777c
feat(Documents): Add migration control and handler documentation
ce7d4e9
refactor(Documents): Language standardization and IsTransientExceptio…
f28d7dd
refactor(Documents): Apply code review improvements
9a9b533
refactor(Documents): Apply additional code review improvements
ebdfd7d
refactor: Apply security and quality improvements from code review
253cf4b
refactor: Translate all Portuguese log messages to English
b1d40f7
refactor: Complete log message translation for remaining modules
f01b4da
refactor: Complete English log translation and code review improvements
4d127b6
refactor(locations): Implement comprehensive architectural improvements
1beed9e
fix: corrigir feedback do code review e traduzir comentários
32462c0
fix(ci): usar find case-insensitive para localizar pacote Aspire DCP
fcf365e
refactor(locations): criar camada API separada para consistência
f9ef646
fix: implementar correções completas do code review
17c25e4
fix(ci): usar build ao invés de restore para baixar pacotes Aspire DCP
2f0ef8c
fix(ci): forçar download explícito do pacote Aspire DCP via projeto t…
9f32366
fix(ci): remover trailing whitespace do workflow YAML
b596573
fix: padronizar mensagens de API para inglês e melhorar YAML lint
3371fbb
fix: atualizar assertions de testes para mensagens em inglês
cdbd77c
fix: reverter mensagens de API para português (user-facing)
b807b07
fix: corrigir seeding de allowed cities em testes de integracao
b70ce6b
refactor: padronizar estrutura do modulo Locations com outros modulos
27a1b86
docs: traduzir comentarios em ingles para portugues em Providers Valu…
bd82d17
refactor: mover Migrations para dentro de Persistence no modulo Provi…
855e31e
docs: remover CONFIGURATION.md redundante do modulo Providers
dc6d0d1
fix: corrigir testes de GeographicRestriction falhando em CI
5d845f7
refactor: reorganizar testes do modulo Providers
84858b5
refactor: eliminar duplicacoes e organizar testes de Queries vs Query…
8203938
fix: corrigir testes e endpoints conforme code review
5877834
refactor: implementar melhorias sugeridas no code review
ae451d1
fix: corrigir testes com mensagens de excecao em portugues
9a065f1
refactor: aplicar sugestoes do code review
af45c7c
fix: corrigir rotas e tratamento de excecoes do modulo Locations
3c615e8
refactor: aplicar melhorias de code review e corrigir mensagens de ex…
a840a8a
fix: corrigir configuracao da URL base do IBGE API no WireMock
40cce91
fix: adicionar diagnostico para ReportGenerator e escapar wildcards LIKE
5ae7e48
fix: usar GitHub Action oficial do ReportGenerator ao invés de instal…
e050f99
fix: migrar ci-cd.yml para GitHub Action oficial do ReportGenerator
fc76126
refactor: remover configuração obsoleta do DOTNET_ROOT
c0ab7ae
fix: corrigir issues apontados pelo code review
70a7a20
fix: configurar DOTNET_ROOT para ReportGenerator encontrar runtime .N…
ea129ac
fix: aplicar correções finais do code review
2d0ee0e
fix: validate-coverage deve usar relatório agregado do ReportGenerator
048b8a3
test: add GetCacheTags test for GetProvidersByIdsQuery
db96d4c
style: translate inline comment to Portuguese in GetProvidersByIdsQue…
f7ddc8b
test: add IsExpiredAt tests for Qualification value object
b55c379
test: add business logic tests and remove redundant tests
fd00f4a
refactor: improve test quality and add business logic tests
1bdd79c
test: add comprehensive business logic tests across modules
d01267e
refactor: improve test quality based on code review
16a5d8e
test: add tests for AddService and RemoveService command handlers
56156e7
refactor: finalize SearchProviders cleanup - translate comments and r…
23cac1c
feat: implement provider validation in DeleteServiceCommandHandler
9bc7b18
refactor: translate ServiceCatalogs Application comments to Portuguese
1feae97
docs: add comprehensive WithDescription to all ServiceCatalogs API en…
cd943fe
refactor: move Migrations folder inside Persistence
ee7513d
refactor: translate Infrastructure comments to Portuguese
e36f43c
fix: correct test IDs and translate error messages to Portuguese
d0f17ce
refactor: consolidate ServiceCatalogs integration tests into single file
f770cce
test: add ServiceCatalogsMappingExtensions tests and rename class
1686624
refactor(ServiceCatalogs): rename DtoMappingExtensions to ServiceCata…
7397c52
fix(Providers): translate user-facing error messages to Portuguese
8487805
refactor(Users): improve code organization and translations
c5c89e5
refactor(Users): remove unnecessary tests and code
7a50144
refactor(Users): resolve CA2000 warnings via .editorconfig
9c6d7d7
refactor(Users): resolve CA2000 pragmas via .editorconfig and transla…
6bd248b
chore: Remove orphaned .bru files and fix test assertions
0875c17
refactor(Authorization): Reorganize structure and improve organizatio…
ab58d5e
refactor(Authorization): Organize Authorization structure and fix com…
4815628
refactor(Shared): Massive cleanup and reorganization
16f0982
refactor(auth): migrate from Permission wrapper to EPermission enum
a861903
docs: add BusinessMetrics middleware alignment to technical debt and …
ca2b73a
docs: translate DapperConnection English comments to Portuguese
941f1fa
refactor: apply code review feedback and fix architecture test
413232c
docs: update health checks strategy - use Aspire native dashboard
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.