Skip to content

Fix test warnings, achieve 100% coverage, add CI coverage job#12

Merged
alganet merged 1 commit intomasterfrom
ci-cov
Mar 15, 2026
Merged

Fix test warnings, achieve 100% coverage, add CI coverage job#12
alganet merged 1 commit intomasterfrom
ci-cov

Conversation

@alganet
Copy link
Member

@alganet alganet commented Mar 15, 2026

Remove invalid CoversClass attributes from integration tests targeting test-only InMemoryMapper. Add 29 tests covering AbstractMapper state management, Collection edge cases, style no-match paths, and sub-collection factory methods to reach 100% line and method coverage. Add composer coverage script. Add code-coverage CI job with Codecov upload and PHPCS to the static-analysis CI job.

Remove invalid CoversClass attributes from integration tests
targeting test-only InMemoryMapper. Add 29 tests covering
AbstractMapper state management, Collection edge cases, style
no-match paths, and sub-collection factory methods to reach 100%
line and method coverage. Add composer coverage script. Add
code-coverage CI job with Codecov upload and PHPCS to the
static-analysis CI job.
@alganet alganet marked this pull request as ready for review March 15, 2026 07:24
@alganet alganet merged commit a201365 into master Mar 15, 2026
6 checks passed
@alganet alganet deleted the ci-cov branch March 15, 2026 07:24
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants