Skip to content

Update behat-data URLs in tests #250

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
swissspidy merged 2 commits into from
Apr 30, 2026
Merged

Update behat-data URLs in tests #250

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
71257b1
Update behat-data URLs in tests
swissspidy Apr 27, 2026
226b525
Update another test
swissspidy Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 24 additions & 24 deletions features/media-import.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ Feature: Manage WordPress attachments
Given a WP install

Scenario: Import media from remote URL
When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' --post_id=1`
When I run `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' --post_id=1`
Then STDOUT should contain:
"""
Imported file 'http://wp-cli.org/behat-data/codeispoetry.png'
Imported file 'http://wp-cli.github.io/behat-data/codeispoetry.png'
Comment thread
swissspidy marked this conversation as resolved.
"""
Comment thread
swissspidy marked this conversation as resolved.
And STDOUT should contain:
"""
Expand All @@ -26,7 +26,7 @@ Feature: Manage WordPress attachments
"""

Scenario: Import media from remote URL and use input file as attachment name
When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' --file_name=abc`
When I run `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' --file_name=abc`
Then STDOUT should contain:
"""
file name abc.png
Expand Down Expand Up @@ -57,7 +57,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file as attachment from a local image
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |

When I run `wp media import {CACHE_DIR}/large-image.jpg --post_id=1 --featured_image`
Then STDOUT should contain:
Expand All @@ -74,7 +74,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file as attachment from a local image and preserve the file modified time.
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
And I run `TZ=UTC touch -t 8001031305 {CACHE_DIR}/large-image.jpg`
And I run `wp option update gmt_offset -5`

Expand All @@ -96,7 +96,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file as an attachment but porcelain style
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |

When I run `wp media import {CACHE_DIR}/large-image.jpg --title="My imported attachment" --caption="My fabulous caption" --post_name="My post name" --porcelain`
Then save STDOUT as {ATTACHMENT_ID}
Expand All @@ -122,7 +122,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file as attachment from a local image and leave it in it's current location
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
And I run `wp option update uploads_use_yearmonth_folders 0`

When I run `wp media import {CACHE_DIR}/large-image.jpg --skip-copy`
Expand All @@ -141,7 +141,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file and use its filename as the title
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |

When I run `wp media import {CACHE_DIR}/large-image.jpg --porcelain`
Then save STDOUT as {ATTACHMENT_ID}
Expand All @@ -155,7 +155,7 @@ Feature: Manage WordPress attachments
Scenario: Import a file and persist its original metadata
Given download:
| path | url |
| {CACHE_DIR}/canola.jpg | http://wp-cli.org/behat-data/canola.jpg |
| {CACHE_DIR}/canola.jpg | http://wp-cli.github.io/behat-data/canola.jpg |

When I run `wp media import {CACHE_DIR}/canola.jpg --porcelain`
Then save STDOUT as {ATTACHMENT_ID}
Expand All @@ -173,7 +173,7 @@ Feature: Manage WordPress attachments
"""

Scenario: Make sure WordPress receives the slashed data it expects
When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' --post_id=1 --title='My\Title' --caption='Caption\Here' --alt='Alt\Here' --desc='Desc\Here' --porcelain`
When I run `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' --post_id=1 --title='My\Title' --caption='Caption\Here' --alt='Alt\Here' --desc='Desc\Here' --porcelain`
Then save STDOUT as {ATTACHMENT_ID}

When I run `wp post get {ATTACHMENT_ID} --format=csv --fields=post_title,post_excerpt,post_content`
Expand All @@ -199,27 +199,27 @@ Feature: Manage WordPress attachments
Scenario: Import multiple images
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |

When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' {CACHE_DIR}/large-image.jpg`
When I run `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' {CACHE_DIR}/large-image.jpg`
Then STDOUT should contain:
"""
Success: Imported 2 of 2 items.
"""

Scenario: Fail to import one image but continue trying the next
When I try `wp media import gobbledygook.png 'http://wp-cli.org/behat-data/codeispoetry.png'`
When I try `wp media import gobbledygook.png 'http://wp-cli.github.io/behat-data/codeispoetry.png'`
Then STDERR should contain:
"""
Error: Only imported 1 of 2 items.
"""
And the return code should be 1

Scenario: Fail when download_url() fails
When I try `wp media import 'http://wp-cli.org/404'`
When I try `wp media import 'http://example.com/404'`
Then STDERR should be:
"""
Warning: Unable to import file 'http://wp-cli.org/404'. Reason: Not Found
Warning: Unable to import file 'http://example.com/404'. Reason: Not Found
Error: No items imported.
"""
And the return code should be 1
Expand All @@ -235,7 +235,7 @@ Feature: Manage WordPress attachments
Scenario: Return upload URL after importing a single valid file
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |

When I run `wp media import {CACHE_DIR}/large-image.jpg --porcelain=url`
Then STDOUT should contain:
Expand All @@ -251,10 +251,10 @@ Feature: Manage WordPress attachments
Scenario: Return upload URL after importing a multiple valid files
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/audio-with-no-cover.mp3 | http://wp-cli.org/behat-data/audio-with-no-cover.mp3 |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
| {CACHE_DIR}/audio-with-no-cover.mp3 | http://wp-cli.github.io/behat-data/audio-with-no-cover.mp3 |

When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' {CACHE_DIR}/large-image.jpg {CACHE_DIR}/audio-with-no-cover.mp3 --porcelain=url`
When I run `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' {CACHE_DIR}/large-image.jpg {CACHE_DIR}/audio-with-no-cover.mp3 --porcelain=url`
Then STDOUT should contain:
"""
https://example.com/wp-content/uploads/
Expand All @@ -281,7 +281,7 @@ Feature: Manage WordPress attachments
"""

Scenario: Errors when invalid --porcelain flag is applied.
When I try `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' --porcelain=invalid`
When I try `wp media import 'http://wp-cli.github.io/behat-data/codeispoetry.png' --porcelain=invalid`
Then STDERR should be:
"""
Error: Invalid value for <porcelain>: invalid. Expected flag or 'url'.
Expand All @@ -290,7 +290,7 @@ Feature: Manage WordPress attachments
Scenario: Import media from STDIN
Given download:
| path | url |
| {CACHE_DIR}/codeispoetry.png | http://wp-cli.org/behat-data/codeispoetry.png |
| {CACHE_DIR}/codeispoetry.png | http://wp-cli.github.io/behat-data/codeispoetry.png |

When I run `cat {CACHE_DIR}/codeispoetry.png | wp media import - --title="From STDIN" --porcelain`
Then save STDOUT as {ATTACHMENT_ID}
Expand All @@ -304,7 +304,7 @@ Feature: Manage WordPress attachments
Scenario: Import media from STDIN with file_name
Given download:
| path | url |
| {CACHE_DIR}/codeispoetry.png | http://wp-cli.org/behat-data/codeispoetry.png |
| {CACHE_DIR}/codeispoetry.png | http://wp-cli.github.io/behat-data/codeispoetry.png |

When I run `cat {CACHE_DIR}/codeispoetry.png | wp media import - --file_name=my-image.png --porcelain`
Then save STDOUT as {ATTACHMENT_ID}
Expand Down Expand Up @@ -336,7 +336,7 @@ Feature: Manage WordPress attachments
Scenario: Upload files into a custom directory, relative to ABSPATH, when --destination-dir flag is applied.
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
When I run `wp media import --destination-dir="foo" {CACHE_DIR}/large-image.jpg --porcelain=url`

Then STDOUT should not contain:
Expand All @@ -352,7 +352,7 @@ Feature: Manage WordPress attachments
Scenario: Upload files into a custom directory, not relative to ABSPATH, when --destination-dir flag is applied.
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
When I run `wp media import --destination-dir="{RUN_DIR}/foo" {CACHE_DIR}/large-image.jpg --porcelain=url`

Then STDOUT should not contain:
Expand Down
10 changes: 5 additions & 5 deletions features/media-prune.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ Feature: Prune WordPress attachment thumbnails
Scenario: Prune all thumbnails for all images
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/canola.jpg | http://wp-cli.org/behat-data/canola.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
| {CACHE_DIR}/canola.jpg | http://wp-cli.github.io/behat-data/canola.jpg |
Comment thread
swissspidy marked this conversation as resolved.
And I run `wp option update uploads_use_yearmonth_folders 0`

When I run `wp media import {CACHE_DIR}/large-image.jpg --title="My large attachment" --porcelain`
Expand Down Expand Up @@ -72,7 +72,7 @@ Feature: Prune WordPress attachment thumbnails
Scenario: Prune a specific image size
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
And I run `wp option update uploads_use_yearmonth_folders 0`

When I run `wp media import {CACHE_DIR}/large-image.jpg --title="My large attachment" --porcelain`
Expand All @@ -96,7 +96,7 @@ Feature: Prune WordPress attachment thumbnails
Scenario: Prune does not remove abandoned (unregistered) thumbnails by default
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
And a wp-content/mu-plugins/media-settings.php file:
"""
<?php
Expand Down Expand Up @@ -133,7 +133,7 @@ Feature: Prune WordPress attachment thumbnails
Scenario: Prune removes abandoned thumbnails with --remove-abandoned
Given download:
| path | url |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.org/behat-data/large-image.jpg |
| {CACHE_DIR}/large-image.jpg | http://wp-cli.github.io/behat-data/large-image.jpg |
And a wp-content/mu-plugins/media-settings.php file:
"""
<?php
Expand Down
Loading