Changeset 39065

Timestamp:

11/01/2016 03:13:27 AM (9 years ago)

Author:

dd32

Message:

Themes: Update the unit tests to handle [39064] and #31550.

test_default_theme_in_default_theme_list() was always being skipped after #31550, this causes it to once again check that the unit tests are up to date and include the latest default theme.
test_default_themes_have_textdomain() didn't play happy when a default theme wasn't installed on a site.

See #31550, #29925, #38551.

File:

• trunk/tests/phpunit/tests/theme.php (modified) (2 diffs)

trunk/tests/phpunit/tests/theme.php

@@ -11 +11 @@
     protected $default_themes = array(
         'twentyten', 'twentyeleven', 'twentytwelve', 'twentythirteen',
-        'twentyfourteen', 'twentyfifteen',
+        'twentyfourteen', 'twentyfifteen', 'twentysixteen', 'twentyseventeen',
     );
 
@@ -188 +188 @@
      */
     function test_default_theme_in_default_theme_list() {
-        if ( 'twenty' === substr( WP_DEFAULT_THEME, 0, 6 ) ) {
-            $this->assertContains( WP_DEFAULT_THEME, $this->default_themes );
-        }
+        $latest_default_theme = WP_Theme::get_core_default_theme();
+        if ( ! $latest_default_theme->exists() || 'twenty' !== substr( $latest_default_theme->get_stylesheet(), 0, 6 ) ) {
+            $this->markTestSkipped( 'No Twenty* series default themes are installed' ); 
+        }
+        $this->assertContains( $latest_default_theme->get_stylesheet(), $this->default_themes );
     }
 
     function test_default_themes_have_textdomain() {
         foreach ( $this->default_themes as $theme ) {
-            $this->assertEquals( $theme, wp_get_theme( $theme )->get( 'TextDomain' ) );
+            if ( wp_get_theme( $theme )->exists() ) {
+                $this->assertEquals( $theme, wp_get_theme( $theme )->get( 'TextDomain' ) );
+            }
         }
     }