Merge pull request #417 from tfirdaus/fix/new-theme-feature-tests

Author: swissspidy

features/theme-delete.feature

@@ -2,32 +2,34 @@ Feature: Delete WordPress themes
 
   Background:
     Given a WP install
-    And I run `wp theme install p2`
+    And I run `wp theme delete --all --force`
+    And I run `wp theme install twentytwelve`
+    And I run `wp theme install twentyeleven --activate`
 
   Scenario: Delete an installed theme
-    When I run `wp theme delete p2`
+    When I run `wp theme delete twentytwelve`
     Then STDOUT should be:
       """
-      Deleted 'p2' theme.
+      Deleted 'twentytwelve' theme.
       Success: Deleted 1 of 1 themes.
       """
     And the return code should be 0
 
   Scenario: Delete an active theme
-    When I run `wp theme activate p2`
+    When I run `wp theme activate twentytwelve`
     Then STDOUT should not be empty
 
-    When I try `wp theme delete p2`
+    When I try `wp theme delete twentytwelve`
     Then STDERR should be:
       """
-      Warning: Can't delete the currently active theme: p2
+      Warning: Can't delete the currently active theme: twentytwelve
       Error: No themes deleted.
       """
 
-    When I try `wp theme delete p2 --force`
+    When I try `wp theme delete twentytwelve --force`
     Then STDOUT should contain:
       """
-      Deleted 'p2' theme.
+      Deleted 'twentytwelve' theme.
       """
 
   Scenario: Delete all installed themes
@@ -108,7 +110,7 @@ Feature: Delete WordPress themes
     Then STDOUT should be empty
 
   Scenario: Attempting to delete a theme that doesn't exist
-    When I run `wp theme delete p2`
+    When I run `wp theme delete twentytwelve`
     Then STDOUT should not be empty
 
     When I try the previous command again
@@ -118,6 +120,6 @@ Feature: Delete WordPress themes
       """
     And STDERR should be:
       """
-      Warning: The 'p2' theme could not be found.
+      Warning: The 'twentytwelve' theme could not be found.
       """
     And the return code should be 0

features/theme-install.feature

@@ -1,50 +1,52 @@
 Feature: Install WordPress themes
 
-  Scenario: Return code is 1 when one or more theme installations fail
+  Background:
     Given a WP install
+    And I run `wp theme delete --all --force`
 
-    When I try `wp theme install p2 p2-not-a-theme`
+  Scenario: Return code is 1 when one or more theme installations fail
+    When I try `wp theme install twentytwelve twentytwelve-not-a-theme`
     Then STDERR should contain:
       """
       Warning:
       """
     And STDERR should contain:
       """
-      p2-not-a-theme
+      twentytwelve-not-a-theme
       """
     And STDERR should contain:
       """
       Error: Only installed 1 of 2 themes.
       """
     And STDOUT should contain:
       """
-      Installing P2
+      Installing Twenty Twelve
       """
     And STDOUT should contain:
       """
       Theme installed successfully.
       """
     And the return code should be 1
 
-    When I try `wp theme install p2`
+    When I try `wp theme install twentytwelve`
     Then STDOUT should be:
       """
       Success: Theme already installed.
       """
     And STDERR should be:
       """
-      Warning: p2: Theme already installed.
+      Warning: twentytwelve: Theme already installed.
       """
     And the return code should be 0
 
-    When I try `wp theme install p2-not-a-theme`
+    When I try `wp theme install twentytwelve-not-a-theme`
     Then STDERR should contain:
       """
       Warning:
       """
     And STDERR should contain:
       """
-      p2-not-a-theme
+      twentytwelve-not-a-theme
       """
     And STDERR should contain:
       """
@@ -88,20 +90,18 @@ Feature: Install WordPress themes
       """
 
   Scenario: Verify installed theme activation
-    Given a WP install
-
-    When I run `wp theme install p2`
+    When I run `wp theme install twentytwelve`
     Then STDOUT should not be empty
 
-    When I try `wp theme install p2 --activate`
+    When I try `wp theme install twentytwelve --activate`
     Then STDERR should contain:
     """
-    Warning: p2: Theme already installed.
+    Warning: twentytwelve: Theme already installed.
     """
 
     And STDOUT should contain:
     """
-    Activating 'p2'...
-    Success: Switched to 'P2' theme.
+    Activating 'twentytwelve'...
+    Success: Switched to 'Twenty Twelve' theme.
     Success: Theme already installed.
     """

features/theme-update.feature

@@ -19,17 +19,18 @@ 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`
 
-    When I run `wp theme install p2 --version=1.4.1`
+    When I run `wp theme install twentytwelve --version=3.0`
     Then STDOUT should not be empty
 
-    When I run `wp theme update p2 --version=1.4.2`
+    When I run `wp theme update twentytwelve --version=4.0`
     Then STDOUT should not be empty
 
     When I run `wp theme list --fields=name,version`
     Then STDOUT should be a table containing rows:
-      | name       | version   |
-      | p2         | 1.4.2     |
+      | name         | version   |
+      | twentytwelve | 4.0       |
 
   Scenario: Not giving a slug on update should throw an error unless --all given
     Given a WP install
@@ -51,7 +52,7 @@ Feature: Update WordPress themes
       """
 
     # One theme installed.
-    Given I run `wp theme install p2 --version=1.4.2`
+    Given I run `wp theme install moina --version=1.0.2`
 
     When I try `wp theme update`
     Then the return code should be 1
@@ -74,7 +75,7 @@ Feature: Update WordPress themes
       """
 
     # Note: if given version then re-installs.
-    When I run `wp theme update --version=1.4.2 --all`
+    When I run `wp theme update --version=1.0.2 --all`
     Then STDOUT should contain:
       """
       Success: Installed 1 of 1 themes.
@@ -103,17 +104,17 @@ Feature: Update WordPress themes
       """
 
     # Using version with all rarely makes sense and should probably error and do nothing.
-    When I try `wp theme update --version=1.4.2 --all`
+    When I try `wp theme update --version=1.0.3 --all`
     Then the return code should be 1
     And STDOUT should contain:
       """
       Success: Installed 1 of 1 themes.
       """
     And STDERR should be:
       """
-      Error: Can't find the requested theme's version 1.4.2 in the WordPress.org theme repository (HTTP code 404).
+      Error: Can't find the requested theme's version 1.0.3 in the WordPress.org theme repository (HTTP code 404).
       """
-  
+
   Scenario: Error when both --minor and --patch are provided
     Given a WP install
 
@@ -139,7 +140,7 @@ Feature: Update WordPress themes
       """
       3.9
       """
-  
+
   Scenario: Update a theme to its latest patch release
     Given a WP install
     And I run `wp theme install --force twentytwelve --version=1.1`