Changeset 45067

Timestamp:

03/29/2019 02:48:08 AM (6 years ago)

Author:

mikeschroder

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.
Fixes #46073.

Location:

trunk/tests/phpunit

Files:

• data/images/one-blue-pixel-100x100.png (added)
• tests/image/editorImagick.php (modified) (4 diffs)

trunk/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 );