* Avoid notice by making WP_Image_Editor_Mock::test()
compatible with WP_Image_Editor::test()
.
* Suppress deprecated function notice for `wp_load_image()`. * Add assertion for `wp_get_image_editor()`. See #25282. git-svn-id: https://develop.svn.wordpress.org/trunk@25380 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
dc35c8d11e
commit
a76e6cef7d
@ -11,7 +11,7 @@ if (class_exists( 'WP_Image_Editor' ) ) :
|
||||
public function load() {
|
||||
return self::$load_return;
|
||||
}
|
||||
public static function test() {
|
||||
public static function test( $args = array() ) {
|
||||
return self::$test_return;
|
||||
}
|
||||
public static function supports_mime_type( $mime_type ) {
|
||||
|
@ -9,12 +9,30 @@ class Tests_Image_Functions extends WP_UnitTestCase {
|
||||
/**
|
||||
* Setup test fixture
|
||||
*/
|
||||
public function setup() {
|
||||
public function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
require_once( ABSPATH . WPINC . '/class-wp-image-editor.php' );
|
||||
require_once( ABSPATH . WPINC . '/class-wp-image-editor-gd.php' );
|
||||
require_once( ABSPATH . WPINC . '/class-wp-image-editor-imagick.php' );
|
||||
|
||||
include_once( DIR_TESTDATA . '/../includes/mock-image-editor.php' );
|
||||
add_action( 'deprecated_function_run', array( $this, 'deprecated_function_run' ) );
|
||||
}
|
||||
|
||||
function tearDown() {
|
||||
parent::tearDown();
|
||||
remove_action( 'deprecated_function_run', array( $this, 'deprecated_function_run' ) );
|
||||
}
|
||||
|
||||
function deprecated_function_run( $function ) {
|
||||
if ( in_array( $function, array( 'wp_load_image' ) ) )
|
||||
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;
|
||||
}
|
||||
|
||||
/**
|
||||
@ -236,8 +254,11 @@ class Tests_Image_Functions extends WP_UnitTestCase {
|
||||
public function test_load_directory() {
|
||||
|
||||
// First, test with deprecated wp_load_image function
|
||||
$editor = wp_load_image( DIR_TESTDATA );
|
||||
$this->assertNotInternalType( 'resource', $editor );
|
||||
$editor1 = wp_load_image( DIR_TESTDATA );
|
||||
$this->assertNotInternalType( 'resource', $editor1 );
|
||||
|
||||
$editor2 = wp_get_image_editor( DIR_TESTDATA );
|
||||
$this->assertNotInternalType( 'resource', $editor2 );
|
||||
|
||||
// Then, test with editors.
|
||||
$classes = array('WP_Image_Editor_GD', 'WP_Image_Editor_Imagick');
|
||||
|
Loading…
Reference in New Issue
Block a user