Skip to content

Fix docstring indent issue in feature files #81

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 1 commit into from
Apr 10, 2025
Merged

Fix docstring indent issue in feature files #81

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
14 changes: 7 additions & 7 deletions features/fetch.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ Feature: Manage oEmbed fetch.
And STDOUT should contain:
"""
LearningApps.org/
"""
"""
And STDOUT should contain:
"""
<iframe
"""
"""

# How unknown provider checked depends on WP version and post_id so recheck with post_id.
When I run `wp post create --post_title="Foo Bar" --porcelain`
Expand All @@ -46,11 +46,11 @@ Feature: Manage oEmbed fetch.
And STDOUT should contain:
"""
LearningApps.org/
"""
"""
And STDOUT should contain:
"""
<iframe
"""
"""

# Unknown provider requiring discovery but not returning iframe so would be sanitized for WP >= 4.4 without 'skip-sanitization' option.
# Old versions of WP_oEmbed can trigger PHP "Only variables should be passed by reference" notices on discover so use "try" to cater for these.
Expand All @@ -71,11 +71,11 @@ Feature: Manage oEmbed fetch.
And STDOUT should not contain:
"""
<iframe
"""
"""
And STDOUT should not contain:
"""
<script
"""
"""

# WP 4.9 always returns clickable link even for sanitized oEmbed responses.
@require-wp-4.9
Expand Down Expand Up @@ -164,7 +164,7 @@ Feature: Manage oEmbed fetch.
And STDOUT should contain:
"""
LearningApps.org
"""
"""

@require-wp-4.0
Scenario: Bails when no oEmbed provider is found for a raw request
Expand Down