Changeset 48488 for branches/4.8

Timestamp:

07/14/2020 09:47:56 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Tests: Temporarily skip PDF tests if they fail due to ImageMagick permission errors.

Merges [48338], [48341] to the 4.8 branch.

See #50573.

Location:

branches/4.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/image/functions.php (modified) (3 diffs)

branches/4.8/tests/phpunit/tests/image/functions.php

@@ -371 +371 @@
         $test_file = '/tmp/wordpress-gsoc-flyer.pdf';
         copy( $orig_file, $test_file );
+
+        $editor = wp_get_image_editor( $test_file );
+        if ( is_wp_error( $editor ) ) {
+            $this->markTestSkipped( $editor->get_error_message() );
+        }
 
         $attachment_id = $this->factory->attachment->create_object( $test_file, 0, array(
@@ -428 +433 @@
         copy( $orig_file, $test_file );
 
+        $editor = wp_get_image_editor( $test_file );
+        if ( is_wp_error( $editor ) ) {
+            $this->markTestSkipped( $editor->get_error_message() );
+        }
+
         $attachment_id = $this->factory->attachment->create_object( $test_file, 0, array(
             'post_mime_type' => 'application/pdf',
@@ -482 +492 @@
         $pdf_path = '/tmp/test.pdf';
         copy( DIR_TESTDATA . '/images/wordpress-gsoc-flyer.pdf', $pdf_path );
+
+        $editor = wp_get_image_editor( $test_file );
+        if ( is_wp_error( $editor ) ) {
+            $this->markTestSkipped( $editor->get_error_message() );
+        }
 
         $attachment_id = $this->factory->attachment->create_object( $pdf_path, 0, array(