Changeset 25402 for trunk/tests/phpunit/includes/testcase.php

Timestamp:

09/12/2013 02:47:58 PM (13 years ago)

Author:

wonderboymusic

Message:

There was way too much duplicated code in my notice cleanup, it built up over time, and there's definitely a need to standardize.

• Remove duplicated code for deprecated function notice suppression
• Add support in WP_UnitTestCase setUp/tearDown methods for $deprecated_functions fixture if the extending class has added it
• Add a $deprecated_functions fixture to each extending class that needs it

To use this fixture, add something to your Test Case class like so:
protected $deprecated_functions = array( 'get_theme', 'get_themes', 'get_theme_data', 'get_current_theme' );

See #25282.

File:

• trunk/tests/phpunit/includes/testcase.php (modified) (3 diffs)

trunk/tests/phpunit/includes/testcase.php

@@ -7 +7 @@
 
     protected static $forced_tickets = array();
+    protected $deprecated_functions = array();
 
     /**
@@ -25 +26 @@
         $this->start_transaction();
         add_filter( 'wp_die_handler', array( $this, 'get_wp_die_handler' ) );
+
+        if ( ! empty( $this->deprecated_functions ) )
+            add_action( 'deprecated_function_run', array( $this, 'deprecated_function_run' ) );
     }
 
@@ -33 +37 @@
         remove_filter( 'query', array( $this, '_drop_temporary_tables' ) );
         remove_filter( 'wp_die_handler', array( $this, 'get_wp_die_handler' ) );
+        if ( ! empty( $this->deprecated_functions ) )
+            remove_action( 'deprecated_function_run', array( $this, 'deprecated_function_run' ) );
+    }
+
+    function deprecated_function_run( $function ) {
+        if ( in_array( $function, $this->deprecated_functions ) )
+            add_filter( 'deprecated_function_trigger_error', array( $this, 'deprecated_function_trigger_error' ) );
+    }
+
+    function deprecated_function_trigger_error() {
+        remove_filter( 'deprecated_function_trigger_error', array( $this, 'deprecated_function_trigger_error' ) );
+        return false;
     }