Changeset 38830 for trunk/tests/phpunit/tests/customize/manager.php

Timestamp:

10/19/2016 07:38:27 PM (8 years ago)

Author:

westonruter

Message:

Customize: Fix unit tests when twentyfifteen is WP_DEFAULT_THEME instead of twentysixteen.

Use an inactive core theme for previewing theme switch instead of assuming twentysixteen is installed and active and twentyfifteen is not.

See #30937.

File:

• trunk/tests/phpunit/tests/customize/manager.php (modified) (9 diffs)

trunk/tests/phpunit/tests/customize/manager.php

@@ -69 +69 @@
         $_REQUEST = array();
         parent::tearDown();
+    }
+
+    /**
+     * Get a core theme that is not the same as the current theme.
+     *
+     * @throws Exception If an inactive core Twenty* theme cannot be found.
+     * @return string Theme slug (stylesheet).
+     */
+    function get_inactive_core_theme() {
+        $stylesheet = get_stylesheet();
+        foreach ( wp_get_themes() as $theme ) {
+            if ( $theme->stylesheet !== $stylesheet && 0 === strpos( $theme->stylesheet, 'twenty' ) ) {
+                return $theme->stylesheet;
+            }
+        }
+        throw new Exception( 'Unable to find inactive twenty* theme.' );
     }
 
@@ -348 +364 @@
      */
     function test_add_state_query_params() {
+        $preview_theme = $this->get_inactive_core_theme();
+
         $uuid = wp_generate_uuid4();
         $messenger_channel = 'preview-0';
@@ -367 +385 @@
             'changeset_uuid' => $uuid,
             'messenger_channel' => null,
-            'theme' => 'twentyfifteen',
+            'theme' => $preview_theme,
         ) );
         $url = $wp_customize->add_state_query_params( home_url( '/' ) );
@@ -376 +394 @@
         $this->assertArrayHasKey( 'customize_theme', $query_params );
         $this->assertEquals( $uuid, $query_params['customize_changeset_uuid'] );
-        $this->assertEquals( 'twentyfifteen', $query_params['customize_theme'] );
+        $this->assertEquals( $preview_theme, $query_params['customize_theme'] );
 
         $uuid = wp_generate_uuid4();
@@ -382 +400 @@
             'changeset_uuid' => $uuid,
             'messenger_channel' => null,
-            'theme' => 'twentyfifteen',
+            'theme' => $preview_theme,
         ) );
         $url = $wp_customize->add_state_query_params( 'http://not-allowed.example.com/?q=1' );
@@ -611 +629 @@
         wp_set_current_user( self::$admin_user_id );
 
+        $preview_theme = $this->get_inactive_core_theme();
         $stashed_theme_mods = array(
-            'twentyfifteen' => array(
+            $preview_theme => array(
                 'background_color' => array(
                     'value' => '#123456',
@@ -622 +641 @@
         $manager = new WP_Customize_Manager( array(
             'changeset_uuid' => $uuid,
-            'theme' => 'twentyfifteen',
+            'theme' => $preview_theme,
         ) );
         $manager->register_controls();
         $GLOBALS['wp_customize'] = $manager;
 
-        $manager->set_post_value( 'blogname', 'Hello 2015' );
+        $manager->set_post_value( 'blogname', 'Hello Preview Theme' );
         $post_values = $manager->unsanitized_post_values();
         $manager->save_changeset_post( array( 'status' => 'publish' ) ); // Activate.
 
         $this->assertEquals( '#123456', $post_values['background_color'] );
-        $this->assertEquals( 'twentyfifteen', get_stylesheet() );
-        $this->assertEquals( 'Hello 2015', get_option( 'blogname' ) );
+        $this->assertEquals( $preview_theme, get_stylesheet() );
+        $this->assertEquals( 'Hello Preview Theme', get_option( 'blogname' ) );
     }
 
@@ -750 +769 @@
         wp_set_current_user( self::$admin_user_id );
 
+        $preview_theme = $this->get_inactive_core_theme();
         $stashed_theme_mods = array(
-            'twentyfifteen' => array(
+            $preview_theme => array(
                 'background_color' => array(
                     'value' => '#000000',
@@ -828 +848 @@
         $manager = new WP_Customize_Manager( array(
             'changeset_uuid' => $uuid,
-            'theme' => 'twentyfifteen',
+            'theme' => $preview_theme,
         ) );
         $this->assertFalse( $manager->is_theme_active() );