Changeset 49517

Timestamp:

11/06/2020 04:44:34 PM (4 years ago)

Author:

desrosj

Message:

Media: Improve and fix rotate/flip image tests.

Fix off-by-one error in pixel color checks for rotate and flip image tests. Change to using PNG with single pixel to ensure that errors are caught in the future, rather than lost in JPEG noise.

Props Fuegas, mikeschroder.
Merges [45067] to the 5.1 branch.
See #46073, #48301.

Location:

branches/5.1

Files:

• . (modified) (1 prop)
• tests/phpunit/data/images/one-blue-pixel-100x100.png (copied) (copied from trunk/tests/phpunit/data/images/one-blue-pixel-100x100.png)
• tests/phpunit/tests/image/editorImagick.php (modified) (4 diffs)

branches/5.1/tests/phpunit/tests/image/editorImagick.php

@@ -422 +422 @@
      */
     public function test_rotate() {
-        $file = DIR_TESTDATA . '/images/gradient-square.jpg';
+        $file = DIR_TESTDATA . '/images/one-blue-pixel-100x100.png';
 
         $imagick_image_editor = new WP_Image_Editor_Imagick( $file );
@@ -430 +430 @@
         $property->setAccessible( true );
 
-        $color_top_left = $property->getValue( $imagick_image_editor )->getImagePixelColor( 1, 1 )->getColor();
+        $color_top_left = $property->getValue( $imagick_image_editor )->getImagePixelColor( 0, 0 )->getColor();
 
         $imagick_image_editor->rotate( 180 );
@@ -441 +441 @@
      */
     public function test_flip() {
-        $file = DIR_TESTDATA . '/images/gradient-square.jpg';
+        $file = DIR_TESTDATA . '/images/one-blue-pixel-100x100.png';
 
         $imagick_image_editor = new WP_Image_Editor_Imagick( $file );
@@ -449 +449 @@
         $property->setAccessible( true );
 
-        $color_top_left = $property->getValue( $imagick_image_editor )->getImagePixelColor( 1, 1 )->getColor();
+        $color_top_left = $property->getValue( $imagick_image_editor )->getImagePixelColor( 0, 0 )->getColor();
 
         $imagick_image_editor->flip( true, false );