Changeset 25354 for trunk/tests/phpunit/tests/theme.php

Timestamp:

09/11/2013 06:30:06 PM (11 years ago)

Author:

wonderboymusic

Message:

Add action/filter to Tests_Theme::setUp() to suppress errors while running back-compat tests on get_theme(), get_themes(), get_theme_data(), get_current_theme().

See #25282.

File:

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

trunk/tests/phpunit/tests/theme.php

@@ -16 +16 @@
         wp_clean_themes_cache();
         unset( $GLOBALS['wp_themes'] );
+
+        add_action( 'deprecated_function_run', array( $this, 'deprecated_function_run_check' ) );
+    }
+
+    function deprecated_function_run_check( $function ) {
+        if ( in_array( $function, array( 'get_theme', 'get_themes', 'get_theme_data', 'get_current_theme' ) ) )
+            add_filter( 'deprecated_function_trigger_error', array( $this, 'filter_deprecated_function_trigger_error' ) );
+    }
+
+    function filter_deprecated_function_trigger_error() {
+        remove_filter( 'filter_deprecated_function_trigger_error', array( $this, 'filter_deprecated_function_trigger_error' ) );
+        return false;
     }
 
@@ -70 +82 @@
     function test_get_themes_contents() {
         $themes = get_themes();
-
         // Generic tests that should hold true for any theme
-        foreach ($themes as $k=>$theme) {
-            $this->assertEquals($theme['Name'], $k);
-            $this->assertTrue(!empty($theme['Title']));
+        foreach ( $themes as $k => $theme ) {
+            $this->assertEquals( $theme['Name'], $k );
+            $this->assertNotEmpty( $theme['Title'] );
 
             // important attributes should all be set
@@ -101 +112 @@
 
             // important attributes should all not be empty as well
-            $this->assertTrue(!empty($theme['Description']));
-            $this->assertTrue(!empty($theme['Author']));
+            $this->assertNotEmpty( $theme['Description'] );
+            $this->assertNotEmpty( $theme['Author'] );
             $this->assertTrue(version_compare($theme['Version'], 0) > 0);
-            $this->assertTrue(!empty($theme['Template']));
-            $this->assertTrue(!empty($theme['Stylesheet']));
+            $this->assertNotEmpty( $theme['Template'] );
+            $this->assertNotEmpty( $theme['Stylesheet'] );
 
             // template files should all exist
@@ -180 +191 @@
 
         for ( $i = 0; $i < 3; $i++ ) {
-            foreach ($themes as $name=>$theme) {
+            foreach ( $themes as $name => $theme ) {
                 // switch to this theme
                 if ( $i === 2 )