Skip to content

Tests: Do not use deprecated plugin status command - #391

Merged
swissspidy merged 1 commit into
mainfrom
fix/tests
Aug 4, 2026
Merged

Tests: Do not use deprecated plugin status command#391
swissspidy merged 1 commit into
mainfrom
fix/tests

Conversation

@swissspidy

@swissspidy swissspidy commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Tests
    • Updated plugin scaffolding verification to use detailed plugin information.
    • Adjusted expected results to match the revised command output format.

@swissspidy swissspidy added this to the 3.0.1 milestone Aug 4, 2026
@swissspidy
swissspidy requested a review from a team as a code owner August 4, 2026 15:43
Copilot AI lite review requested due to automatic review settings August 4, 2026 15:43
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The scaffold test now uses wp plugin get with selected fields and validates the resulting status, title, and description output.

Changes

Scaffold verification

Layer / File(s) Summary
Plugin field verification
features/scaffold.feature
The test uses wp plugin get hello-world --fields=title,status,description and updates the expected status, title, and description output.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: copilot

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the test update that removes use of the deprecated plugin status command.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/tests

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

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Behat acceptance test for scaffolding a plugin to avoid using the deprecated wp plugin status command, keeping the scaffold feature tests aligned with current WP-CLI commands.

Changes:

  • Replaced wp plugin status hello-world with wp plugin get ... in the scaffold plugin prompt scenario.
  • Updated expected output assertions to match the new command output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread features/scaffold.feature
@swissspidy swissspidy added the scope:testing Related to testing label Aug 4, 2026
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@swissspidy swissspidy changed the title Test: Do not use deprecated plugin status command Tests: Do not use deprecated plugin status command Aug 4, 2026
@swissspidy
swissspidy merged commit 8767148 into main Aug 4, 2026
66 of 68 checks passed
@swissspidy
swissspidy deleted the fix/tests branch August 4, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants