Changeset 53531 for trunk/tests/phpunit/tests/image/functions.php

Timestamp:

06/19/2022 05:10:48 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Refactor Tests_Image_Functions::test_inferred_mime_types() to use a data provider.

Using a data provider has a number of advantages:

1. If the first test case fails, it won't prevent the other test cases from being tested.
2. The output from PHPUnit will be more descriptive in case of failure when using a data provider.
3. Using named test cases in the data provider will also make the --testdox output much more descriptive and informative.

The actual cases being tested, or the test itself have not been changed.

Includes:

• Adding a @covers annotation.
• Adding a failure message to each assertion.
• Adding a skip annotation for unsupported mime types.
• Making the test method name more specific.

Follow-up to [1061/tests], [53495], [53497], [53521], [53523], [53524], [53525], [53526], [53529], [53530].

Props jrf.
See #55652.

File:

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

trunk/tests/phpunit/tests/image/functions.php

@@ -268 +268 @@
                 }
 
+                // Save the file.
                 $file = wp_tempnam();
                 $ret  = wp_save_image_file( $file, $img, $mime_type, 1 );
 
+                // Make assertions.
                 $this->assertNotEmpty( $ret, 'Image failed to save - "empty" response returned' );
                 $this->assertNotWPError( $ret, 'Image failed to save - WP_Error returned' );
@@ -318 +320 @@
          * Test that a passed mime type overrides the extension in the filename when saving an image.
          *
-         * @dataProvider data_mime_overrides_filename
+         * @dataProvider data_mime_overrides_filename_when_saving_an_image
          *
          * @ticket 6821
@@ -352 +354 @@
          * @return array
          */
-        public function data_mime_overrides_filename() {
+        public function data_mime_overrides_filename_when_saving_an_image() {
                 return $this->text_array_to_dataprovider( $this->get_image_editor_engine_classes() );
         }
 
         /**
-         * Test that mime types are correctly inferred from file extensions
+         * Test that mime types are correctly inferred from file extensions when saving an image.
+         *
+         * @dataProvider data_inferred_mime_types_when_saving_an_image
          *
          * @ticket 6821
+         * @covers WP_Image_Editor::get_mime_type
+         * @covers WP_Image_Editor::get_output_format
          * @requires extension fileinfo
-         */
-        public function test_inferred_mime_types() {
+         *
+         * @param string $class_name Name of the image editor engine class to be tested.
+         * @param string $extension  File extension.
+         * @param string $mime_type  The mime type to test.
+         */
+        public function test_inferred_mime_types_when_saving_an_image( $class_name, $extension, $mime_type ) {
+                $img    = new $class_name( DIR_TESTDATA . '/images/canola.jpg' );
+                $loaded = $img->load();
+
+                $img = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' );
+                $this->assertNotWPError( $img );
+
+                if ( ! $img->supports_mime_type( $mime_type ) ) {
+                        $this->markTestSkipped(
+                                sprintf(
+                                        'The %s mime type is not supported by the %s engine',
+                                        $mime_type,
+                                        str_replace( 'WP_Image_Editor_', '', $class_name )
+                                )
+                        );
+                }
+
+                // Save the file.
+                $temp = get_temp_dir();
+                $file = wp_unique_filename( $temp, uniqid() . ".$extension" );
+                $ret  = $img->save( trailingslashit( $temp ) . $file );
+
+                // Make assertions.
+                $this->assertNotEmpty( $ret, 'Image failed to save - "empty" response returned' );
+                $this->assertNotWPError( $ret, 'Image failed to save - WP Error returned' );
+                $this->assertSame( $mime_type, $this->get_mime_type( $ret['path'] ), 'Mime type of the saved image was not inferred correctly' );
+
+                // Clean up.
+                unlink( $ret['path'] );
+                unset( $img );
+        }
+
+        /**
+         * Data provider.
+         *
+         * @return array
+         */
+        public function data_inferred_mime_types_when_saving_an_image() {
                 $classes = $this->get_image_editor_engine_classes();
 
@@ -373 +420 @@
                         'png'  => 'image/png',
                         'webp' => 'image/webp',
-                        'unk'  => 'image/jpeg',   // Default, unknown.
-                );
-
-                // Test each image editor engine.
+                        'unk'  => 'image/jpeg', // Default, unknown.
+                );
+
+                $data = array();
+
                 foreach ( $classes as $class ) {
-                        $img    = new $class( DIR_TESTDATA . '/images/canola.jpg' );
-                        $loaded = $img->load();
-
-                        // Save the image as each file extension, check the mime type.
-                        $img = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' );
-                        $this->assertNotWPError( $img );
-
-                        $temp = get_temp_dir();
                         foreach ( $mime_types as $ext => $mime_type ) {
-                                if ( ! $img->supports_mime_type( $mime_type ) ) {
-                                        continue;
-                                }
-
-                                $file = wp_unique_filename( $temp, uniqid() . ".$ext" );
-                                $ret  = $img->save( trailingslashit( $temp ) . $file );
-                                $this->assertNotEmpty( $ret );
-                                $this->assertNotWPError( $ret );
-                                $this->assertSame( $mime_type, $this->get_mime_type( $ret['path'] ) );
-                                unlink( $ret['path'] );
+                                $data[ $class . '; Extension: ' . $ext . '; Mime type: ' . $mime_type ] = array(
+                                        'class_name' => $class,
+                                        'extension'  => $ext,
+                                        'mime_type'  => $mime_type,
+                                );
                         }
-
-                        // Clean up.
-                        unset( $img );
-                }
+                }
+
+                return $data;
         }