-
Notifications
You must be signed in to change notification settings - Fork 0
Improve tests coverage #31
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 153 commits
Commits
Show all changes
155 commits
Select commit
Hold shift + click to select a range
323fe96
fix: remove mandatory secrets validation from PR workflow
c0d798e
docs: update roadmap and create Sprint 2 detailed tasks
e021447
feat: create Bruno API collections for Documents and SearchProviders …
abf366f
feat: complete Bruno API collections for ServiceCatalogs and Location…
fb2e63d
chore: update MigrationTool to use Central Package Management
683dcc0
feat: add all module DbContexts to MigrationTool
5ea45f0
docs: organize Sprint 1 documentation and update API collections README
9f55b63
fix: resolve ServiceCatalogs integration test failures (duplicate key…
5aea927
fix: enable PostGIS support in test containers
2ad85e9
chore: standardize PostgreSQL image to postgis/postgis:16-3.4 across …
e49a682
feat: add Azurite TestContainer for Documents E2E tests
2f7310e
fix(tests): resolve authentication race condition by clearing context…
7c81915
fix(tests): resolve IBGE geographic restriction test by fixing Allowe…
bd368b3
docs: archive skipped tests documentation and create resolution summary
a19ae06
fix: address code review feedback across multiple components
0cdd6b5
docs: apply code review feedback for documentation quality
225cb43
refactor(tests): improve ServiceCatalogs test data generation with sh…
ff9bce9
fix(ci): resolve workflow and formatting issues
332f3f4
docs: update README.md and configure test coverage
8b1186e
fix: code review corrections
32de9a5
test(providers): add unit tests for command handlers
3a0bab3
test(providers): add comprehensive validator tests - increase coverag…
45c5608
chore: remove duplicate migrations
5928a0e
test(providers): add handler tests for CompleteBasicInfo and SetPrima…
17d8164
fix: resolve CI/CD validation and test issues
144f145
refactor: apply code review suggestions
7cc9eb6
refactor: improve code quality and test coverage
a203d35
test: add comprehensive GetProviderByDocumentQueryHandler tests
1d4b1c2
test: add query handler tests for GetProvidersByCity, GetProvidersByS…
c52073c
fix: correct migration file encoding for dotnet format compliance
fc964b9
test: add [Trait("Category", "Unit")] to all Domain tests and create …
25d799a
fix: correct OccurredAt property name in Document events tests
3b7d549
fix: corrige mocks ProviderId e assertions DocumentNumber
5f83493
refactor: aplica melhorias sugeridas nos testes
b91729c
fix: corrige configuração GeographicRestriction para formato correto
493e244
fix: corrige workflow shell script e documenta\u00e7\u00e3o de testes
2aa55c5
refactor: aplica melhorias da code review
2bf6eae
fix: corrige erros críticos de CI/CD
79b1b66
refactor: aplica recomenda\u00e7\u00f5es de code review
79d8d5e
feat: adiciona testes Domain para Locations IBGE models
9040e8d
feat: adiciona 11 testes Provider Domain Events restantes
867c4dc
fix: adiciona delays para eventual consistency em testes E2E + testes…
c069f06
fix: corrige testes unitários Providers
a8d8642
refactor: apply code review improvements
789afd3
fix: improve error handling and fix PostgreSQL integration tests
5570398
fix: resolve DocumentsVerificationE2ETests failures
1e03e05
fix: update unit tests to match improved error handling
fba3cb0
style: fix code formatting issues
fe94b28
fix: address code review feedback on error handling
e98fab5
refactor: implement code review improvements
f4e5f56
feat(tests): Implement E2E TestContainerFixture and improve CI/CD cov…
1ef5322
docs: Consolidate testing documentation and remove duplicates
7862c5c
fix: Address code review feedback
26b11e7
fix: Address CI/CD issues from code review
e68f1be
fix: Sanitize connection strings and fix FluentAssertions usage
9b0656c
fix: Suppress PendingModelChangesWarning in E2E tests
480cbf2
test: Add GetProvidersQueryHandler and GetUsersByIdsQueryHandler tests
d6e8794
fix: Address code review feedback
3b0e247
fix: Correct PagedResult parameter order in GetProvidersQueryHandler
a202d49
test: Add AddressTests for Providers module
53d7a29
test: Add 3 unit repository tests (68 tests total)
e7cc56f
fix: address code review feedback
9c189ec
refactor(tests): apply code review improvements
799160f
refactor(tests): apply code review improvements - polling strategy
3825ca1
fix(tests): resolve compilation error and remove magic number
b36dabf
refactor(tests): use enum directly and improve assertions
b074635
refactor(tests): apply final code review improvements
7e5c0f2
fix(coverage): add filters to exclude generated code (OpenApi, RegexG…
9521a5a
feat(tests): add SearchProviders API integration tests (10 tests)
d293a5d
feat(tests): add 5 Documents API integration tests
e81eb97
feat(tests): add 10 API integration tests for Providers and ServiceCa…
d6bcb5e
feat(tests): add 22 repository unit tests for Providers and Documents
e02bb4d
feat(tests): add 28 Shared component unit tests
6bd6912
feat(tests): add 46 extension method unit tests
65e50dd
feat(tests): add 13 Shared exception unit tests
83d5bce
feat(tests): add 33 Shared infrastructure unit tests
87bd27c
feat(tests): add 18 SearchProviders domain unit tests
4e6485a
feat(tests): add 44 Shared Contracts unit tests
ad0de2e
refactor(tests): fix API inconsistencies and improve test quality
861d448
fix(formatting): resolve pipeline formatting errors and add XML docs
ef4c933
refactor(tests): implement code review improvements
0f15b5d
feat(tests): add repository integration tests with real database
8cebc98
refactor(tests): apply code review improvements
75131e9
feat(tests): add repository integration tests for Users and ServiceCa…
6dcb391
docs(tests): add XML documentation to GetByProviderIdAsync test
57273ee
refactor(tests): improve integration test robustness and documentation
2e3ac98
docs: fix broken markdown links and standardize kebab-case naming
d5fa00f
docs: clean up redundant and obsolete documentation
b56f39a
fix(tests): add retry logic for PostgreSQL startup timing issue
34a4643
test: add 44 comprehensive edge case tests for Provider Entity
5c72cb3
test: add comprehensive tests for SecurityExtensions (0% → 95%+ cover…
c858856
test: add comprehensive tests for PermissionMetricsService (0% → 95%+…
6c0d6ae
test: add comprehensive tests for ProviderConfiguration (0% → 95%+ co…
40881fc
test: add comprehensive tests for KeycloakExtensions (0% → 95%+ cover…
ea6d12f
test: add comprehensive tests for DocumentationExtensions (0% → 95%+ …
d049a15
test: add comprehensive tests for ServiceDefaults Extensions (0% → ~7…
65189d3
fix: code review corrections - formatting, links, and documentation
d1127e5
test: add comprehensive PerformanceExtensions tests (64 tests, ~95% c…
f041264
test: add comprehensive PerformanceExtensions tests (64 tests, ~95% c…
ce09759
test: add comprehensive ModuleTagsDocumentFilter tests (35 tests, ~95…
12f6762
refactor: code review fixes - improve test quality and documentation
5242b2d
fix: add missing ILogger registration in PermissionMetricsServiceTests
5d2b8db
test: add MessagingExtensions tests (Options classes)
5ca791c
test: add comprehensive tests for PermissionOptimizationMiddleware (4…
06be765
test: add comprehensive SearchableProvider entity tests (51 tests)
238893b
test: add RateLimitingMiddleware tests (26 tests, 13 passing) - WIP
85b12e7
style: fix code formatting (whitespace)
316a557
chore: remove RateLimitingMiddlewareTests (cache isolation issues)
1708094
refactor(tests): apply code review fixes
89faa54
ci(coverage): include Shared and ApiService tests in coverage collection
dfaa3fd
fix(tests): resolve ApiService.Tests failures (IWebHostEnvironment, v…
c73d186
refactor(tests): apply code review feedback
302ee90
test(shared): add 15 tests for ModuleApiRegistry and PerformanceHealt…
6a9ce4b
chore: implement code review feedback
cdecc23
test(shared): add 11 tests for PermissionSystemHealthCheck
ee0ace9
fix(workflow): correct XPlat Code Coverage filter syntax and update T…
4ec68a2
test: add comprehensive AuthorizationExtensions unit tests (22 tests)
73f4391
test: add PermissionClaimsTransformation and PermissionRequirementHan…
51757ea
style: fix code formatting (imports ordering and whitespace)
4949a6a
test: unskip ClockSkew test, add Keycloak null guard tests, fix workf…
df04368
refactor: remove permanently skipped CORS test and update STRICT_COVE…
4ecba35
refactor: simplify tests by removing duplicate ServiceProvider builds
91b327f
fix: correct find syntax for recursive coverage file search
60dc95c
fix: register IConfiguration in simplified tests
6483a8f
fix: correct find syntax for recursive coverage file search
bd4dda2
fix(ci): add retry logic and fallback for OSV-Scanner installation
06d281f
docs: link STRICT_COVERAGE TODO to tracking issue #33
66890e5
Merge branch 'improve-tests-coverage' of https://github.com/frigini/M…
3422224
style(ci): remove trailing whitespace from OSV-Scanner installation b…
ec078f7
test: add 24 comprehensive GlobalExceptionHandler tests (+8-10% cover…
33ee244
test: add 13 comprehensive CommandDispatcher tests (+4-5% coverage)
0c24736
test: add 11 comprehensive DomainEventProcessor tests (+3-4% coverage)
4530a99
test: add 9 comprehensive QueryDispatcher tests (+3% coverage)
422cdb5
test: add 14 comprehensive ServiceBusMessageBus tests (+4-5% coverage)
b14fc50
test: add 8 comprehensive EventTypeRegistry tests (+2-3% coverage)
ba440d1
test: add 11 comprehensive TopicStrategySelector tests (+3% coverage)
febe334
test: add 9 comprehensive ValidationBehavior tests (+2-3% coverage)
9789a32
ci: enable all domain module tests in coverage pipeline
4e66bcd
docs(api): add comprehensive XML documentation to QueryDispatcher
0ef104e
fix(ci): correct coverage Include/Exclude filters to use wildcard syntax
c86ff44
docs(api): add singleton lifetime recommendation to QueryDispatcher
2fe4895
fix(ci): remove --no-build from module tests and add coverage debug l…
7d73642
refactor(ci): extract coverage validation to reusable action
1319fff
fix(ci): handle edge case for coverage decimal values without leading…
c116e4b
refactor(ci): apply code review improvements to coverage validation
bb9f162
chore: merge master to get workflow updates for coverage collection
3945b4f
test: add 30 unit tests for Shared coverage gaps
726318f
revert: remove incompilable test files
6de4fc8
chore: trigger pipeline with updated workflow
6e3705d
test: add 39 unit tests for Shared core classes
5acb4d8
fix: resolve ValidationException namespace ambiguity
b0d6849
style: fix code formatting issues
e66152a
docs: add comprehensive code coverage roadmap and analysis
dcf08f7
docs: fix corrupted text and reorganize coverage projections
342067e
docs: fix formatting inconsistencies and add Phase 5 roadmap
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| <?xml version="1.0" encoding="utf-8" ?> | ||
| <RunSettings> | ||
| <DataCollectionRunSettings> | ||
| <DataCollectors> | ||
| <DataCollector friendlyName="XPlat code coverage"> | ||
| <Configuration> | ||
| <Format>cobertura,json,opencover</Format> | ||
| <Exclude>[*.Tests]*,[*.Testing]*,[testhost]*,[MeAjudaAi.AppHost]*,[MeAjudaAi.ServiceDefaults]*</Exclude> | ||
| <ExcludeByAttribute>Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute,ExcludeFromCodeCoverageAttribute</ExcludeByAttribute> | ||
| <ExcludeByFile>**/Migrations/**/*.cs,**/bin/**,**/obj/**,**/Logging/**/*.cs,**/Jobs/Hangfire*.cs,**/Messaging/RabbitMq/**/*.cs,**/Messaging/ServiceBus/**/*.cs,**/Monitoring/**/*.cs,**/HealthChecks/**/*.cs</ExcludeByFile> | ||
| <IncludeDirectory>./src/</IncludeDirectory> | ||
| <SingleHit>false</SingleHit> | ||
| <UseSourceLink>true</UseSourceLink> | ||
| <IncludeTestAssembly>false</IncludeTestAssembly> | ||
| <SkipAutoProps>true</SkipAutoProps> | ||
| <DeterministicReport>false</DeterministicReport> | ||
| <DoesNotReturnAttribute>DoesNotReturnAttribute</DoesNotReturnAttribute> | ||
| </Configuration> | ||
| </DataCollector> | ||
| </DataCollectors> | ||
| </DataCollectionRunSettings> | ||
| <RunConfiguration> | ||
| <MaxCpuCount>0</MaxCpuCount> | ||
| <ResultsDirectory>.\TestResults\Coverage</ResultsDirectory> | ||
| <TargetFramework>net10.0</TargetFramework> | ||
| </RunConfiguration> | ||
| </RunSettings> |
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
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
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.