Skip to content

Fix tests involving wp-cli-test/updateable-package - #246

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

Fix tests involving wp-cli-test/updateable-package#246
swissspidy merged 1 commit into
mainfrom
fix/tests

Conversation

@swissspidy

@swissspidy swissspidy commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Tests
    • Updated package update scenarios to use version 3.0.0.
    • Updated expected Composer metadata, help output, and version constraints for single- and multi-package updates.

@swissspidy swissspidy added this to the 3.0.1 milestone Aug 5, 2026
@swissspidy
swissspidy requested a review from a team as a code owner August 5, 2026 09:18
Copilot AI lite review requested due to automatic review settings August 5, 2026 09:18
@swissspidy swissspidy added the scope:testing Related to testing label Aug 5, 2026
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: aa2d7340-4044-4a01-9f97-ebd595d7fc13

📥 Commits

Reviewing files that changed from the base of the PR and between b53f71c and 813becd.

📒 Files selected for processing (1)
  • features/package-update.feature

📝 Walkthrough

Walkthrough

The package-update feature scenarios now use wp-cli-test/updateable-package version v3.0.0. Composer metadata, help output, and version constraints now match the new package version across general, specific-package, and multiple-package scenarios.

Changes

Package update scenarios

Layer / File(s) Summary
Update scenario expectations
features/package-update.feature
General, specific-package, and multiple-package scenarios now install version v3.0.0, expect matching Composer metadata and help output, and update constraints to >=3.0.0.

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

Suggested reviewers: copilot, brianhenryie

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the test fixes for the package updated in the pull request.
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.
✨ 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.

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

Updates the Behat package update acceptance tests to use a newer wp-cli-test/updateable-package version, aligning the test fixture versions with what’s currently expected/available in the WP-CLI package index.

Changes:

  • Bump wp-cli-test/updateable-package install version from v1.0.0 to v3.0.0 in update-related scenarios.
  • Update corresponding assertions (install output, composer.json contents, and wp help updateable-package v3).
  • Adjust the wp eval ... str_replace(...) steps to update constraints from v3.0.0 to >=3.0.0.

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

@swissspidy
swissspidy merged commit d46a067 into main Aug 5, 2026
95 of 145 checks passed
@swissspidy
swissspidy deleted the fix/tests branch August 5, 2026 10:11
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