[schlessera]

Switches the feature files to the these installed plugins: / these installed themes: steps from wp-cli/wp-cli-tests#404, and requires wp-cli/wp-cli-tests ^5.4.0.

Measured on this test suite: Behat time +0.8% (36.1 → 36.3 job minutes, summed over the matrix). Details in wp-cli/wp-cli-tests#404.

Summary by CodeRabbit

• Tests
  • Updated import-command test scenarios to run with the importer already installed and active. Coverage continues to include file output, missing files, custom importers, standard input, URL imports, and local attachment sources. Expected command behavior remains unchanged.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: b464df38-95a6-4243-8a8a-d99ee1515d49

Commits

Reviewing files that changed from the base of the PR and between be31030 and 91a1982.

Files selected for processing: 2

• composer.json
• features/import.feature

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The development dependency constraint for wp-cli/wp-cli-tests changed from ^5 to ^5.4.0. Eight import scenarios now use an installed, active wordpress-importer fixture instead of running a plugin installation command. Import commands and expectations remain unchanged.

Changes: Import test setup

Layer / File(s)	Summary
Use the importer fixture in import scenarios composer.json, features/import.feature	The development dependency constraint changes from ^5 to ^5.4.0. Eight scenarios set up wordpress-importer as installed and active instead of running a plugin installation command. Their import commands and stated expectations remain unchanged.

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Suggested reviewers: swissspidy

Merge Risk: Minimal · up to 91a19

The cached importer setup introduces no identified merge-blocking risk. The change appears mergeable subject to normal checks.

Pre-merge checks: 4

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title "Use cached plugin and theme installs in tests" matches the main change. The PR switches feature files to the cached these installed plugins: and these installed themes: steps and raises…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title "Use cached plugin and theme installs in tests" matches the main change. The PR switches feature files to the cached these installed plugins: and these installed themes: steps and raises the wp-cli-tests dependency to ^5.4.0, which provides those steps. The title is specific and describes the primary change. It does not cover the composer constraint or the individual scenario edits, which is acceptable.

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.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!