chore(coderabbit): align test review instruction with PHPUnit ^11.2#104
Conversation
The tests path_instruction said "Tests must work across PHPUnit 9.6, 10, and 11", which made CodeRabbit repeatedly flag the project's #[Test]/#[DataProvider] attributes (and the lack of a test* method prefix) as PHPUnit 9.6 incompatibilities — a false positive on three separate PRs. The project pins PHPUnit ^11.2 and uses the attribute API throughout. Point the instruction at ^11.2 and tell CodeRabbit not to flag the attribute API.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the CodeRabbit review configuration for test files to reflect the project’s pinned PHPUnit ^11.2 version and to prevent false-positive warnings about PHPUnit’s attribute-based test API and non-prefixed test method names. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .coderabbit.yaml configuration file to specify that the project pins PHPUnit ^11.2 and uses its attribute API (such as #[Test], #[DataProvider], and #[CoversClass]). It instructs the review tool not to flag these attributes or the absence of a test* method-name prefix as PHPUnit 9.6/10 incompatibilities. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #104 +/- ##
============================================
+ Coverage 18.13% 18.52% +0.38%
- Complexity 7854 7997 +143
============================================
Files 666 670 +4
Lines 43208 43703 +495
============================================
+ Hits 7837 8096 +259
- Misses 35371 35607 +236 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
1 participant
What
Update the
tests/**/*.phppath_instructionin.coderabbit.yamlfrom"Tests must work across PHPUnit 9.6, 10, and 11" to point at PHPUnit
^11.2(the project's pinned version) and explicitly tell CodeRabbit not to flag the
attribute API.
Why
The old instruction made CodeRabbit repeatedly flag the project's PHPUnit
attribute API —
#[Test],#[DataProvider],#[CoversClass]— and theabsence of a
test*method-name prefix as "PHPUnit 9.6 incompatibilities".This is a false positive: the project pins PHPUnit
^11.2(composer /CLAUDE.md) and uses those attributes throughout (CriteriaTest,XoopsSecurityTest, the upgrade suite, etc.). The bogus finding has appeared onthree separate PRs, where it had to be dismissed each time.
Aligning the instruction with the real test baseline stops the recurring noise
without weakening any actual test-quality review.
Summary by Sourcery
CI: