Skip to content

fix(ci): eliminate PHPUnit 12 false-positive coverage warnings#4

Merged
walmir-silva merged 1 commit intomainfrom
develop
Mar 4, 2026
Merged

fix(ci): eliminate PHPUnit 12 false-positive coverage warnings#4
walmir-silva merged 1 commit intomainfrom
develop

Conversation

@walmir-silva
Copy link
Copy Markdown
Contributor

@walmir-silva walmir-silva commented Mar 4, 2026

  • DiscoveryExceptionTest: replace #[CoversClass] with #[UsesClass] — PHPUnit 12 + PCOV emits 'not a valid target' for classes inheriting native RuntimeException methods; UsesClass avoids the coverage check
  • Workflow (both jobs): revert XML patch from script back to sed, also strip restrictWarnings/Deprecations/Notices from tag to prevent internal PHPUnit warnings from triggering failOnWarning

@walmir-silva
fix(ci): eliminate PHPUnit 12 false-positive coverage warnings
8536688 
- DiscoveryExceptionTest: replace #[CoversClass] with #[UsesClass] —
  PHPUnit 12 + PCOV emits 'not a valid target' for classes inheriting
  native RuntimeException methods; UsesClass avoids the coverage check
- Workflow (both jobs): revert XML patch from script back to sed,
  also strip restrictWarnings/Deprecations/Notices from <source> tag
  to prevent internal PHPUnit warnings from triggering failOnWarning
@walmir-silva walmir-silva merged commit 2ecc569 into main Mar 4, 2026
8 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@walmir-silva