Repository navigation
Use cached plugin and theme installs in tests - #192
Conversation
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update installed-asset scenario setup composer.json, features/language-core.feature, features/language-plugin.feature, features/language-theme.feature |
The wp-cli/wp-cli-tests constraint changes to ^5.4.0. Theme and plugin scenarios declare assets as installed, with version-specific setup retained around translation checks. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Other
Suggested reviewers: swissspidy, ernilambar
Merge Risk: 
Minimal · up to 1f863
-
The change uses the supported cached-install test steps and updates their dependency constraint without introducing a merge-blocking risk.
Pre-merge checks: 
5
-
Full details: Title check
-
Explanation
The title "Use cached plugin and theme installs in tests" matches the main change. The PR replaces direct
wp plugin installandwp theme installcalls in the feature files with thethese installed plugins:andthese installed themes:steps, and raises the wp-cli-tests dependency to ^5.4.0. The title is specific and short, and it names the primary change from the developer's view. It does not cover the version-specific translation checks or the timing gain, which is acceptable for a title.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)
-
Finishing Touches
-
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
3185a11 to
1f86336
Compare
Switches the feature files to the
these installed plugins:/these installed themes:steps from wp-cli/wp-cli-tests#404, and requireswp-cli/wp-cli-tests^5.4.0.Measured on this test suite: Behat time -15.8% (187.3 → 157.8 job minutes, summed over the matrix). Details in wp-cli/wp-cli-tests#404.
Summary by CodeRabbit