Skip to content

Use only these installed themes in tests #565

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
schlessera merged 1 commit into from
Nov 2, 2026
Merged

Use only these installed themes in tests #565

Show file tree
Hide file tree
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
3 changes: 1 addition & 2 deletions features/theme-auto-update-status.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ Feature: Show the status of auto-updates for WordPress themes

Background:
Given a WP install
And I run `wp theme delete --all --force`
And these installed themes:
And only these installed themes:
"""
twentysixteen
twentyseventeen
Expand Down
3 changes: 1 addition & 2 deletions features/theme-auto-updates-disable.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ Feature: Disable auto-updates for WordPress themes

Background:
Given a WP install
And I run `wp theme delete --all --force`
And these installed themes:
And only these installed themes:
"""
twentysixteen
twentyseventeen
Expand Down
3 changes: 1 addition & 2 deletions features/theme-auto-updates-enable.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ Feature: Enable auto-updates for WordPress themes

Background:
Given a WP install
And I run `wp theme delete --all --force`
And these installed themes:
And only these installed themes:
"""
twentysixteen
twentyseventeen
Expand Down
3 changes: 1 addition & 2 deletions features/theme-delete.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ Feature: Delete WordPress themes

Background:
Given a WP install
And I run `wp theme delete --all --force`
And these installed themes:
And only these installed themes:
"""
twentytwelve
"""
Expand Down
2 changes: 1 addition & 1 deletion features/theme-install.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ Feature: Install WordPress themes

Background:
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

Scenario: Return code is 1 when one or more theme installations fail
When I try `wp theme install twentytwelve twentytwelve-not-a-theme`
Expand Down
4 changes: 2 additions & 2 deletions features/theme-update.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Feature: Update WordPress themes

Scenario: Install a theme, then update to a specific version of that theme
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install twentytwelve --version=3.0`
Then STDOUT should not be empty
Expand Down Expand Up @@ -254,7 +254,7 @@ Feature: Update WordPress themes
@require-wp-4.5
Scenario: Updating all themes should show the name of each theme as it is updated
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install storefront --version=1.0.0`
Then STDOUT should not be empty
Expand Down
38 changes: 16 additions & 22 deletions features/theme.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ Feature: Manage WordPress themes

Scenario: Installing and deleting theme
Given a WP install
And I run `wp theme delete --all --force`
And these installed and active themes:
And only these installed and active themes:
"""
twentyeleven
"""
Expand Down Expand Up @@ -91,8 +90,7 @@ Feature: Manage WordPress themes

Scenario: Install a theme, activate, then force install an older version of the theme
Given a WP install
And I run `wp theme delete --all --force`
And these installed and active themes:
And only these installed and active themes:
"""
twentyeleven
"""
Expand Down Expand Up @@ -151,8 +149,7 @@ Feature: Manage WordPress themes

Scenario: Exclude theme from bulk updates.
Given a WP install
And I run `wp theme delete --all --force`
And these installed and active themes:
And only these installed and active themes:
"""
twentyeleven
"""
Expand Down Expand Up @@ -197,7 +194,7 @@ Feature: Manage WordPress themes

Scenario: Get the path of an installed theme
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install twentytwelve`
Then STDOUT should not be empty
Expand All @@ -210,7 +207,7 @@ Feature: Manage WordPress themes

Scenario: Activate an already active theme
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install twentytwelve`
Then STDOUT should not be empty
Expand Down Expand Up @@ -288,7 +285,7 @@ Feature: Manage WordPress themes

Scenario: Install a theme when the theme directory doesn't yet exist
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `rm -rf wp-content/themes`
And I run `wp eval "if ( is_dir('wp-content/themes') ) echo 'fail';"`
Expand Down Expand Up @@ -514,7 +511,7 @@ Feature: Manage WordPress themes

Scenario: When updating a theme --format should be the same when using --dry-run
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install --force twentytwelve --version=1.0`
Then STDOUT should not be empty
Expand Down Expand Up @@ -554,7 +551,7 @@ Feature: Manage WordPress themes

Scenario: Check json and csv formats when updating a theme
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install --force twentytwelve --version=1.0`
Then STDOUT should not be empty
Expand Down Expand Up @@ -608,7 +605,7 @@ Feature: Manage WordPress themes

Scenario: Get status field in theme detail
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install twentytwelve`
Then STDOUT should not be empty
Expand All @@ -635,7 +632,7 @@ Feature: Manage WordPress themes

Scenario: Theme activation fails when slug does not match exactly
Given a WP install
And I run `wp theme delete --all --force`
And an empty wp-content/themes directory

When I run `wp theme install twentytwelve`
Then the return code should be 0
Expand Down Expand Up @@ -694,8 +691,7 @@ Feature: Manage WordPress themes
@require-wp-5.7
Scenario: Parent theme is active when its child is active
Given a WP install
And I run `wp theme delete --all --force`
And these installed themes:
And only these installed themes:
"""
twentytwelve
"""
Expand All @@ -715,8 +711,7 @@ Feature: Manage WordPress themes

Scenario: Excluding a missing theme should not throw an error
Given a WP install
And I run `wp theme delete --all --force`
And these installed and active themes:
And only these installed and active themes:
"""
twentytwelve --version=1.5
"""
Expand Down Expand Up @@ -921,14 +916,13 @@ Feature: Manage WordPress themes

Scenario: Filter theme list by multiple names
Given a WP install
And I run `wp theme delete --all --force`
And these installed and active themes:
And only these installed themes:
"""
twentyeleven
twentytwelve
"""
And these installed themes:
And these installed and active themes:
"""
twentytwelve
twentyeleven
"""

When I run `wp theme list --name=twentytwelve,twentyeleven --fields=name,status`
Expand Down
Loading