Update behat-data URLs in tests#250
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the test asset URLs across several Behat feature files, migrating from http://wp-cli.org/behat-data/ to http://wp-cli.github.io/behat-data/. The feedback recommends using https:// for the new GitHub Pages URLs to enhance security and prevent potential redirects.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Pull request overview
Updates Behat fixture URLs in the media-command feature tests in response to the wp-cli.org redirect work referenced in the PR description.
Changes:
- Replaced
http://wp-cli.org/behat-data/...URLs withhttp://wp-cli.github.io/behat-data/...across media-related Behat features. - Updated expected output strings in tests that assert on the imported remote URL.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| features/media-replace.feature | Updates behat-data fixture URLs used by media replace scenarios. |
| features/media-regenerate.feature | Updates behat-data fixture URLs used by media regenerate scenarios (including PDFs/videos). |
| features/media-prune.feature | Updates behat-data fixture URLs used by media prune scenarios. |
| features/media-import.feature | Updates behat-data fixture URLs used by media import scenarios and related assertions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
See https://make.wordpress.org/systems/2026/04/23/redirect-request-for-wp-cli-org/