Fix PHPCS errors in Behat test files - #259
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe pull request reformats embedded PHP fixtures in three Behat feature files. It changes indentation, callback layout, and one regex quote delimiter. Test logic and behavior remain unchanged. ChangesMedia fixture formatting
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change only cleans up formatting in Behat test fixtures without altering product behavior, so no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.) ✨ 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 |
There was a problem hiding this comment.
Pull request overview
This PR standardizes the formatting of inline PHP snippets embedded in Behat .feature files, aligning them with the project’s PHPCS expectations while preserving test behavior.
Changes:
- Reformats inline
add_action()/add_filter()blocks into multi-line, consistently indented calls across media-related scenarios. - Normalizes minor PHP style details (e.g.,
preg_replace()quoting) and indentation within embedded helper scripts.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| features/media-regenerate.feature | Re-indents and expands inline PHP used in regenerate scenarios (actions/filters, helper script snippet). |
| features/media-prune.feature | Reformats inline PHP that registers the temporary custom image size used by prune scenarios. |
| features/media-image-size.feature | Re-indents the inline plugin example used to register a custom image size for listing tests. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary by CodeRabbit
Style
Tests