From a067d588c3bf412eca9cf47efba3e63537371f4e Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Tue, 14 Jun 2022 11:44:58 +0000 Subject: [PATCH] Tests: Remove redundant skip call in `Tests_Image_Functions::get_image_editor_engine_classes()`. The test bootstrap requires GD to be available, so this test skip condition will never be matched. Also, test skipping from within a helper method, which may be used in a data provider, can lead to test runtime errors. Follow-up to [49009], [49014], [49535], [49571], [51415]. Props jrf. See #55652. git-svn-id: https://develop.svn.wordpress.org/trunk@53497 602fd350-edb4-49c9-b593-d223f7449a82 --- tests/phpunit/tests/image/functions.php | 4 ---- 1 file changed, 4 deletions(-) diff --git a/tests/phpunit/tests/image/functions.php b/tests/phpunit/tests/image/functions.php index 9105f244a1..c2c5ffe39d 100644 --- a/tests/phpunit/tests/image/functions.php +++ b/tests/phpunit/tests/image/functions.php @@ -42,10 +42,6 @@ class Tests_Image_Functions extends WP_UnitTestCase { } } - if ( empty( $classes ) ) { - $this->markTestSkipped( 'Image editor engines WP_Image_Editor_GD and WP_Image_Editor_Imagick are not supported on this system.' ); - } - return $classes; }