Changeset 12841 for branches/7.0/tests/phpunit/testcases/core/class-bp-attachment.php

Timestamp:

02/06/2021 08:39:28 PM (4 years ago)

Author:

imath

Message:

PHPUnit: fix failing tests about media

Since WP ​50170 revision, errors about getimagesize() and exif_read_data() are not silenced anymore. We must now make sure to write media files where they should be for these two functions.

Tests involving exif_read_data() will be skipped from now on when testing against PHP 5.6 as no matter the image you use, you'll always get an Illegal IDF size error. It appears to be a bug with this version of PHP.

Fixes #8432 (7.0 branch)

File:

• branches/7.0/tests/phpunit/testcases/core/class-bp-attachment.php (modified) (3 diffs)

branches/7.0/tests/phpunit/testcases/core/class-bp-attachment.php

@@ -378 +378 @@
      */
     public function test_bp_attachment_avatar_shrink() {
-        if ( false === _wp_image_editor_choose() ) {
-            $this->markTestSkipped( 'This test requires PHP to have a valid image editor that is compatible with WordPress.' );
+        if ( false === _wp_image_editor_choose() || version_compare( phpversion(), '7.0' , '<' ) ) {
+            $this->markTestSkipped( 'This test requires PHP >= 7.0 and to have a valid image editor that is compatible with WordPress.' );
         }
 
@@ -430 +430 @@
         }
 
-        $image = BP_TESTS_DIR . 'assets/upside-down.jpg';
+        $image = BP_TESTS_DIR . 'assets/test-image-large.jpg';
 
         $cover_image_class = new BP_Attachment_Cover_Image();
 
-        $abs_path_copy = $cover_image_class->upload_path . '/upside-down.jpg';
+        $abs_path_copy = $cover_image_class->upload_path . '/test-image-large.jpg';
 
         copy( $image, $abs_path_copy );
@@ -465 +465 @@
      */
     public function test_bp_attachment_get_image_data() {
-        if ( ! is_callable( 'exif_read_data' ) ) {
-            $this->markTestSkipped( 'This test requires PHP to be compiled with EXIF support.' );
+        if ( ! is_callable( 'exif_read_data' ) || version_compare( phpversion(), '7.0' , '<' ) ) {
+            $this->markTestSkipped( 'This test requires PHP >= 7.0 and to be compiled with EXIF support.' );
         }