Changeset 50951 for trunk/tests/phpunit/tests/image/intermediateSize.php

Timestamp:

05/22/2021 12:58:46 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Media: Some documentation and test improvements for the image_editor_output_format filter:

• Update the filter DocBlock per the documentation standards.
• Use a shorter variable name for consistency with the surrounding code.
• Delete the test file before performing assertions to avoid leftovers in case the test fails.

Follow-up to [50943].

See #52867.

File:

• trunk/tests/phpunit/tests/image/intermediateSize.php (modified) (4 diffs)

trunk/tests/phpunit/tests/image/intermediateSize.php

@@ -53 +53 @@
         $image = image_make_intermediate_size( DIR_TESTDATA . '/images/a2-small.jpg', 100, 75, true );
 
+        unlink( DIR_TESTDATA . '/images/a2-small-100x75.jpg' );
+
         $this->assertInternalType( 'array', $image );
         $this->assertSame( 100, $image['width'] );
@@ -59 +61 @@
 
         $this->assertFalse( isset( $image['path'] ) );
-
-        unlink( DIR_TESTDATA . '/images/a2-small-100x75.jpg' );
-    }
-
-    /**
-     * @requires function imagejpeg
+    }
+
+    /**
      * @ticket 52867
+     * @requires function imagejpeg
      */
     function test_image_editor_output_format_filter() {
@@ -74 +74 @@
             }
         );
+
         $file   = DIR_TESTDATA . '/images/waffles.jpg';
         $image  = image_make_intermediate_size( $file, 100, 75, true );
         $editor = wp_get_image_editor( $file );
+
+        unlink( DIR_TESTDATA . '/images/' . $image['file'] );
+        remove_all_filters( 'image_editor_output_format' );
+
         if ( is_wp_error( $editor ) || ! $editor->supports_mime_type( 'image/webp' ) ) {
             $this->assertSame( 'image/jpeg', $image['mime-type'] );
@@ -82 +87 @@
             $this->assertSame( 'image/webp', $image['mime-type'] );
         }
-        unlink( DIR_TESTDATA . '/images/' . $image['file'] );
-        remove_all_filters( 'image_editor_output_format' );
     }