Improve tests coverage 2#35

Merged

frigini merged 204 commits intomasterfrom

improve-tests-coverage-2

Dec 10, 2025

Commits on Dec 5, 2025

fix: replace Dictionary with ConcurrentDictionary in PermissionMetricsServiceTests
Filipe Frigini
committed
test: add DbContext transaction tests with PostgreSQL
Filipe Frigini
committed
test: add ExternalServicesHealthCheck tests with 9 passing tests
Filipe Frigini
committed
refactor: remove reflection from health check tests
Filipe Frigini
committed
test: add HelpProcessingHealthCheck and DatabasePerformanceHealthCheck tests
Filipe Frigini
committed
chore(coverage): exclude compiler-generated code from coverage analysis
Filipe Frigini
committed
fix: resource leaks, username validation, coverage thresholds, formatting
Filipe Frigini
committed
fix: suppress CA2000 in test mocks, improve test documentation
Filipe Frigini
committed
chore: trigger CI/CD pipeline
Filipe Frigini
committed
fix: YAML syntax error in pr-validation workflow line 331
Filipe Frigini
committed
fix: improve monitor script robustness and test documentation
Filipe Frigini
committed
fix: YAML parser error with XML heredoc - use echo instead
Filipe Frigini
committed
test: add Quick Win tests for Options and NoOpClaimsTransformation
Filipe Frigini
committed
test: add Quick Win 3 - Swagger filter tests
Filipe Frigini
committed
test: add Sprint 2 Task 2.1 - Environment & Configuration tests
Filipe Frigini
committed
test: add Sprint 2 Task 2.2 - StaticFilesMiddleware logic tests
Filipe Frigini
committed
test: add Sprint 3 Task 3.1 - SecurityExtensions null validation tests
Filipe Frigini
committed
refactor: apply code review fixes
Filipe Frigini
committed
refactor: apply additional code review suggestions
Filipe Frigini
committed
refactor: apply code review fixes for test naming and timing
Filipe Frigini
committed
feat: add coverage.runsettings for accurate local coverage testing
Filipe Frigini
committed
refactor: apply code review fixes
Filipe Frigini
committed
fix: resolve build errors and YAML validation issues
Filipe Frigini
committed
test: remove problematic SecurityExtensions tests
Filipe Frigini
committed
test: add Email value object tests
Filipe Frigini
committed
test: add Documents module unit tests
Filipe Frigini
committed
test: add Providers module value object tests
Filipe Frigini
committed
refactor: improve test assertions and YAML readability
Filipe Frigini
committed
test: add value object and enum tests for multiple modules
Filipe Frigini
committed
test: add domain event tests for Users and Providers modules
Filipe Frigini
committed
fix: correct coverage filters in pr-validation workflow
Filipe Frigini
committed
test: apply code review improvements
Filipe Frigini
committed
test: apply additional code review improvements
Filipe Frigini
committed
fix: add missing CORS configuration to security tests
Filipe Frigini
committed
ci: block EF Core 10.0.0 stable updates until Npgsql compatibility
Filipe Frigini
committed
docs: add EF Core 10 migration status and PR #41 rejection explanation
Filipe Frigini
committed
docs: remove pr-41-rejection-comment.md
Filipe Frigini
committed
feat: configure automated Npgsql 10.0.0 stable monitoring
Filipe Frigini
committed
refactor: apply code review suggestions to test files
Filipe Frigini
committed
docs: apply code review improvements to Npgsql monitoring documentation
Filipe Frigini
committed
ci: fix coverage collection - exclude Integration and E2E tests
Filipe Frigini
committed
ci: separate test steps for better visibility in GitHub Actions UI
Filipe Frigini
committed
test: remove duplicate integration tests (Phase 1 - Quick Wins)
Filipe Frigini
committed
test: remove low-value smoke tests and DbContext duplicates (Phase 2)
Filipe Frigini
committed
test: Phase 3 - Remove E2E test duplicates and low-value tests
Filipe Frigini
committed
test: Phase 4 - Remove Integration test duplicates and trivial tests
Filipe Frigini
committed
refactor(tests): Phase 5 - Remove remaining duplicate tests and ImplementedFeaturesTests
Filipe Frigini
committed
refactor(tests): apply code review improvements and fix test quality issues
Filipe Frigini
committed
fix(ci): improve test separation visibility and fix coverage calculation
Filipe Frigini
committed
test: remove 11 tests with Skip attribute
Filipe Frigini
committed
test: remove all remaining Skip tests - no value in keeping disabled tests
Filipe Frigini
committed
fix(ci): resolve YAML validation errors and warnings
Filipe Frigini
committed
feat(users): implement optimistic concurrency control using PostgreSQL xmin
Filipe Frigini
committed
fix(tests): address code review feedback - improve test precision and fix concurrency tests
Filipe Frigini
committed
fix(ci): remove Hangfire compatibility test step - tests no longer exist
Filipe Frigini
committed
fix(ci): use explicit coverage paths in PR Validation workflow
Filipe Frigini
committed
fix(ci): add visual test separation headers to PR Validation workflow
Filipe Frigini
committed
fix: address final code review feedback
Filipe Frigini
committed
fix: improve PR validation coverage accuracy with aggregated reports
Filipe Frigini
committed
fix: add dotnet tools to PATH for reportgenerator command
Filipe Frigini
committed
fix: correct ReportGenerator classfilters syntax with - prefix
Filipe Frigini
committed
fix: skip Integration/E2E tests in PR validation (require database)
Filipe Frigini
committed
fix: use comma-separated file globs for CodeCoverageSummary action
Filipe Frigini
committed
fix: restore Integration/E2E tests and fix SearchProviders connection string validation
Filipe Frigini
committed
refactor: apply code review feedback for SearchProviders and coverage workflow
Filipe Frigini
committed
fix: use Cobertura output for ReportGenerator (OpenCover requires Pro)
Filipe Frigini
committed
fix: accept both 401 and 403 in DocumentsApiTests mismatched provider test
Filipe Frigini
committed
fix: aggregate coverage after all tests to include Integration/E2E
Filipe Frigini
committed
fix: apply code review feedback - compilation error and XML docs
Filipe Frigini
committed
fix: honor INTEGRATION_TESTS environment variable and remove duplicate assertion
Filipe Frigini
committed
fix: correct assembly filters in ReportGenerator to match actual assembly names
Filipe Frigini
committed
fix: apply code review feedback - ExcludeByFile for Integration/E2E tests
Filipe Frigini
committed
fix: exclude .Tests assemblies from coverage and truncate percentage
Filipe Frigini
committed
fix: correct step ID references and extract reusable runsettings generator
Filipe Frigini
committed
test: add unit tests for Locations External API clients
Filipe Frigini
committed
feat(tests): add unit tests for Locations External API clients
Filipe Frigini
committed
feat(tests): add unit tests for PagedSearchResultDto
Filipe Frigini
committed
feat(tests): add comprehensive unit tests for UserRoles
Filipe Frigini
committed
feat(tests): add unit tests for Providers Value Objects
Filipe Frigini
committed
feat(tests): add unit tests for ServiceCatalogs Domain Events
Filipe Frigini
committed
fix(ci): exclude Migrations, Database, and Contracts from coverage
Filipe Frigini
committed
feat(tests): add unit tests for Users Event Handlers
Filipe Frigini
committed
feat(tests): add unit tests for ServiceCatalogsModuleApi
Filipe Frigini
committed
fix(ci): escape heredoc variables in pr-validation workflow
Filipe Frigini
committed
fix: address CodeRabbit review findings
Filipe Frigini
committed
fix(ci): replace heredoc with echo blocks in all workflow steps
Filipe Frigini
committed
refactor: apply code review feedback
Filipe Frigini
committed
refactor(ci): extract runsettings generation to shared script
Filipe Frigini
committed
fix(ci): use sed instead of bash string substitution in escape_xml
Filipe Frigini
committed
docs(ci-cd): add comprehensive workflows documentation
Filipe Frigini
committed
docs(roadmap): add MkDocs Material decision for Sprint 2
Filipe Frigini
committed
docs(roadmap): add documentation curation process for MkDocs migration
Filipe Frigini
committed
feat(tests): add SearchProvidersModuleApi tests
Filipe Frigini
committed
feat(tests): add 35 Shared authorization tests (PermissionExtensions + PermissionRequirement)
Filipe Frigini
committed
feat(tests): add 26 Shared UserId value object tests
Filipe Frigini
committed
refactor(tests): improve test assertions per code review
Filipe Frigini
committed
feat(tests): add Batch 2B/2C infrastructure tests (44 new tests total)
Filipe Frigini
committed
feat(tests): add 38 exception tests (NotFoundException, ForbiddenAccessException, BusinessRuleException)
Filipe Frigini
committed
feat(tests): add 21 EndpointExtensions tests + document HandlePaged bug
Filipe Frigini
committed
refactor(tests): apply code review improvements
Filipe Frigini
committed
fix(tests): correct ESubscriptionTier enum reference
Filipe Frigini
committed
fix(shared): correct PagedResponse constructor parameter order
Filipe Frigini
committed
refactor(tests): apply code review improvements
Filipe Frigini
committed
test(users): add UsersPermissions policy tests
Filipe Frigini
committed
test(documents): add 14 infrastructure integration tests
Filipe Frigini
committed
test(shared): add 17 infrastructure integration tests
Filipe Frigini
committed
refactor: apply code review improvements
Filipe Frigini
committed
test: add 57 integration tests across Users, SearchProviders, and ServiceCatalogs modules
Filipe Frigini
committed
fix: properly exclude Migrations from coverage and fix failing integration tests
Filipe Frigini
committed
test: fix remaining ServiceCatalogs integration test failures
Filipe Frigini
committed
refactor: apply code review improvements
Filipe Frigini
committed
test: add missing IsSuccess check in ServiceCatalogs workflow test
Filipe Frigini
committed
fix: correct Migrations exclusion pattern in coverage
Filipe Frigini
committed
refactor: rename cancellation test to reflect actual behavior
Filipe Frigini
committed
fix: add INCLUDE_FILTER parameter to E2E tests coverage generation
Filipe Frigini
committed
fix: use broader pattern to exclude Migrations from coverage
Filipe Frigini
committed
refactor: use awk-only coverage extraction (remove bc dependency)
Filipe Frigini
committed
fix: use .runsettings in ReportGenerator to exclude Migrations
Filipe Frigini
committed
test: add Shared.Messaging and BusinessMetrics tests (63 passing tests)
Filipe Frigini
committed
test: add event handler and messaging tests (partial)
Filipe Frigini
committed
test: add event handler and GlobalExceptionHandler tests
Filipe Frigini
committed
test: add PostgreSqlExceptionProcessor tests (12 tests)
Filipe Frigini
committed
test: add Shared.Logging and API Extensions tests
Filipe Frigini
committed
refactor: improve test quality and maintainability
Filipe Frigini
committed
refactor: implement code review improvements and fix .NET 10 compatibility
Filipe Frigini
committed
refactor: strengthen test assertions and align coverage thresholds
Filipe Frigini
committed
test: add RabbitMqMessageBus unit tests and fix parse-coverage script
Filipe Frigini
committed
style: Fix code formatting issues from dotnet format
Filipe Frigini
committed
docs: Add XML documentation and improve test quality
Filipe Frigini
committed
fix: Resolve compilation errors and documentation issues
Filipe Frigini
committed

Commits on Dec 6, 2025

docs: Add XML documentation and use UUID v7 for test IDs
Filipe Frigini
committed
test: Improve test code quality with nameof() and UuidGenerator
Filipe Frigini
committed
refactor: Use collection expression instead of Array.Empty
Filipe Frigini
committed
docs: Improve XML documentation and test assertions
Filipe Frigini
committed
ci: Optimize artifact uploads to prevent timeouts
Filipe Frigini
committed
feat: Measure actual response time in ExternalServicesHealthCheck
Filipe Frigini
committed

Commits on Dec 8, 2025

refactor: optimize coverage filters to exclude infrastructure classes
Filipe Frigini
committed
wip: add initial tests for Provider event handlers (needs signature fixes)
Filipe Frigini
committed
fix: correct event handler test signatures and improve coverage tooling
Filipe Frigini
committed
fix: remove non-existent workflow outputs and add logger verification to tests
Filipe Frigini
committed
feat: update threshold to 90%, improve test quality and add ApiVersionOperationFilter tests
Filipe Frigini
committed
refactor: move warning suppressions from pragmas to .editorconfig
Filipe Frigini
committed
fix: correct workflow script sourcing, ReportGenerator filters, and OpenAPI types
Filipe Frigini
committed
fix: correct OpenApi namespace usage in ApiVersionOperationFilterTests
Filipe Frigini
committed
refactor: consolidate CA2000 suppression and optimize workflow script sourcing
Filipe Frigini
committed
test: add GetProviderByDocumentQueryHandler tests (9 comprehensive tests)
Filipe Frigini
committed
test: add Documents module integration tests (16 tests, 10 passing)
Filipe Frigini
committed
fix: use admin auth for Documents upload tests
Filipe Frigini
committed
test: add comprehensive DocumentExtensions tests (25 tests)
Filipe Frigini
committed
refactor: apply code review improvements to tests
Filipe Frigini
committed

Commits on Dec 9, 2025

refactor: remove redundant E2E tests and add unit tests for Documents module
Filipe Frigini
committed
fix: apply code review improvements to Documents tests
Filipe Frigini
committed
refactor: apply DRY improvements to Documents tests
Filipe Frigini
committed
fix: update enum converter test for InMemory compatibility
Filipe Frigini
committed
fix: ensure pipeline fails immediately when tests fail
Filipe Frigini
committed
fix: call Times.Once() instead of passing method group
Filipe Frigini
committed
fix: resolve all compiler warnings and code review feedback
Filipe Frigini
committed
fix: resolve all compilation errors and code review feedback
Filipe Frigini
committed
refactor: apply comprehensive code review improvements
Filipe Frigini
committed
feat: Add 27 error handling tests for DocumentsModuleApi
Filipe Frigini
committed
feat: Add 27 error handling tests to DocumentsModuleApiTests
Filipe Frigini
committed
chore: reorganize coverage scripts for better discoverability
Filipe Frigini
committed
test: add unit tests for RateLimitingMiddleware and module extensions
Filipe Frigini
committed
refactor: reorganize test structure and remove redundant tests
Filipe Frigini
committed
fix: remove deprecated Aspire workload installation from CI/CD
Filipe Frigini
committed
test: expand AzureDocumentIntelligenceService tests and fix issues
Filipe Frigini
committed
fix: correct namespaces, cross-platform script, and CEP validation tests
Filipe Frigini
committed
test: add comprehensive extension tests for Documents and SearchProviders modules
Filipe Frigini
committed
refactor: improve test quality and code cleanliness
Filipe Frigini
committed
test: improve test assertions and diagnostics
Filipe Frigini
committed
test: replace NSubstitute with Moq in SecurityExtensionsTests
Filipe Frigini
committed
test: add comprehensive tests for RequestLoggingMiddleware (100% line coverage)
Filipe Frigini
committed
test: add comprehensive tests for DocumentVerificationJob (15 tests)
Filipe Frigini
committed
test: add tests for CepLookupService caching behavior (4 tests)
Filipe Frigini
committed
test: add tests for GeocodingService caching behavior (6 tests)
Filipe Frigini
committed
chore: suppress CA2000 warnings in test projects via local .editorconfig files
Filipe Frigini
committed
refactor: improve test quality and coverage configuration
Filipe Frigini
committed
test: enhance ModuleApi tests to achieve 100% DTO coverage
Filipe Frigini
committed
Merge branch 'master' into improve-tests-coverage-2
Filipe Frigini
committed
chore: suppress CS8604 warnings in test projects
Filipe Frigini
committed

Commits on Dec 10, 2025

merge: sync with master after Aspire investigation resolution
Filipe Frigini
committed
fix: address CodeRabbit review feedback
Filipe Frigini
committed
fix: upgrade to stable packages and address CodeRabbit feedback
Filipe Frigini
committed
fix: regenerate lockfiles for linux-x64 and update Hangfire compatibility warnings
Filipe Frigini
committed
fix: exclude Program.cs and DbContextFactory from coverage metrics
Filipe Frigini
committed
feat: add DTO unit tests to strengthen coverage above 90%
Filipe Frigini
committed
fix: enforce locked-mode restore and correct documentation references
Filipe Frigini
committed
fix: Corrigir UsersModuleApiTests e regenerar AppHost lockfile
Filipe Frigini
committed
fix: Corrigir issues do CodeRabbit e regenerar lockfiles
Filipe Frigini
committed
fix: correct Hangfire.PostgreSql version and cleanup duplicate test files
Filipe Frigini
committed
fix: remove RuntimeIdentifier from AppHost to support cross-platform lockfiles
Filipe Frigini
committed
feat: upgrade to EF Core 10.0.1 GA and Npgsql 10.0.0 GA
Filipe Frigini
committed
feat: add Linux lockfile regeneration workflow and GetUserByUsernameAsync tests
Filipe Frigini
committed
docs: update Hangfire.PostgreSql version references and add EFCore.NamingConventions monitoring
Filipe Frigini
committed
fix: remove Aspire lockfiles to resolve NU1004 errors in CI
Filipe Frigini
committed
refactor: address CodeRabbit feedback and improve code quality
Filipe Frigini
committed
fix: remove invalid tests for non-existent GetUserByUsernameAsync method
Filipe Frigini
committed
refactor: address CodeRabbit feedback and improve code quality
Filipe Frigini
committed
refactor: address CodeRabbit final review feedback (partial)
Filipe Frigini
committed
fix: correct failing tests and apply code formatting
Filipe Frigini
committed
fix: resolve CS8625 warning - use default(HealthCheckService) instead of null
Filipe Frigini
committed
refactor: improve IsAvailableAsync test coverage and fix markdown linting
Filipe Frigini
committed
fix: disable RestoreLockedMode for AppHost and dependent test projects
Filipe Frigini
committed
fix: remove --locked-mode from CI workflows and cleanup test code
Filipe Frigini
committed