WordPress.org

Make WordPress Core

Ticket #22356: 22356.9.tests.diff

File 22356.9.tests.diff, 6.9 KB (added by DH-Shredder, 17 months ago)

Make tests pass with new factory functions

  • tests/image/editor.php

     
    1717         * Setup test fixture 
    1818         */ 
    1919        public function setup() { 
    20                 if ( !class_exists( 'WP_Image_Editor' ) ) 
    21                         $this->markTestSkipped(); 
     20                require_once( ABSPATH . WPINC . '/class-wp-image-editor.php' ); 
    2221 
    23                 // Include our custom mock 
    2422                include_once( DIR_TESTDATA . '/../includes/mock-image-editor.php' ); 
    2523 
    2624                // Mock up an abstract image editor based on WP_Image_Editor 
     
    6159        } 
    6260 
    6361        /** 
    64          * Test get_instance where load returns true 
     62         * Test wp_get_image_editor() where load returns true 
    6563         * @ticket 6821 
    6664         */ 
    67         public function test_get_instance_load_returns_true() { 
     65        public function test_get_editor_load_returns_true() { 
    6866 
    6967                // Swap out the PHPUnit mock with our custom mock 
    7068                $func = create_function( '', 'return "WP_Image_Editor_Mock";'); 
     
    7573                WP_Image_Editor_Mock::$load_return = true; 
    7674 
    7775                // Load an image 
    78                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     76                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    7977 
    8078                // Everything should work 
    8179                $this->assertInstanceOf( 'WP_Image_Editor_Mock', $editor ); 
     
    8583        } 
    8684 
    8785        /** 
    88          * Test get_instance where load returns false 
     86         * Test wp_get_image_editor() where load returns false 
    8987         * @ticket 6821 
    9088         */ 
    91         public function test_get_instance_load_returns_false() { 
     89        public function test_get_editor_load_returns_false() { 
    9290 
    9391                // Swap out the PHPUnit mock with our custom mock 
    9492                $func = create_function( '', 'return "WP_Image_Editor_Mock";'); 
     
    9997                WP_Image_Editor_Mock::$load_return = new WP_Error(); 
    10098 
    10199                // Load an image 
    102                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     100                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    103101 
    104102                // Everything should work 
    105103                $this->assertInstanceOf( 'WP_Error', $editor ); 
     
    115113        public function test_set_quality() { 
    116114 
    117115                // Get an editor 
    118                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     116                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    119117 
    120118                // Make quality readable 
    121119                $property = new ReflectionProperty( $editor, 'quality' ); 
     
    142140        public function test_generate_filename() { 
    143141 
    144142                // Get an editor 
    145                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     143                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
     144 
    146145                $property = new ReflectionProperty( $editor, 'size' ); 
    147146                $property->setAccessible( true ); 
    148147                $property->setValue( $editor, array( 
     
    172171         */ 
    173172        public function test_get_size() { 
    174173 
    175                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     174                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    176175 
    177176                // Size should be false by default 
    178177                $this->assertNull( $editor->get_size() ); 
     
    194193         * @ticket 6821 
    195194         */ 
    196195        public function test_get_suffix() { 
    197                 $editor = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     196                $editor = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    198197 
    199198                // Size should be false by default 
    200199                $this->assertFalse( $editor->get_suffix() ); 
  • tests/image/functions.php

     
    88class Tests_Image_Functions extends WP_UnitTestCase { 
    99 
    1010        /** 
     11         * Setup test fixture 
     12         */ 
     13        public function setup() { 
     14                require_once ABSPATH . WPINC . '/class-wp-image-editor.php'; 
     15                require_once ABSPATH . WPINC . '/class-wp-image-editor-gd.php'; 
     16                require_once ABSPATH . WPINC . '/class-wp-image-editor-imagick.php'; 
     17        } 
     18 
     19        /** 
    1120         * Get the MIME type of a file 
    1221         * @param string $filename 
    1322         * @return string 
     
    120129                        add_filter( 'image_editor_class', $filter ); 
    121130 
    122131                        // Call wp_save_image_file 
    123                         $img = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     132                        $img = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    124133 
    125134                        // Save a file as each mime type, assert it works 
    126135                        foreach ( $mime_types as $mime_type ) { 
     
    158167                        add_filter( 'image_editor_class', $filter ); 
    159168 
    160169                        // Save the file 
    161                         $img = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     170                        $img = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    162171                        $mime_type = 'image/gif'; 
    163172                        $file = wp_tempnam( 'tmp.jpg' ); 
    164173                        $ret = $img->save( $file, $mime_type ); 
     
    203212                        add_filter( 'image_editor_class', $filter ); 
    204213 
    205214                        // Save the image as each file extension, check the mime type 
    206                         $img = WP_Image_Editor::get_instance( DIR_TESTDATA . '/images/canola.jpg' ); 
     215                        $img = wp_get_image_editor( DIR_TESTDATA . '/images/canola.jpg' ); 
    207216                        $temp = get_temp_dir(); 
    208217                        foreach ( $mime_types as $ext => $mime_type ) { 
    209218                                $file = wp_unique_filename( $temp, uniqid() . ".$ext" ); 
     
    225234         * @ticket 17814 
    226235         */ 
    227236        public function test_load_directory() { 
     237 
    228238                // First, test with deprecated wp_load_image function 
    229239                $editor = wp_load_image( DIR_TESTDATA ); 
    230240                $this->assertNotInternalType( 'resource', $editor ); 
     
    237247                        } 
    238248                        $filter = create_function( '', "return '$class';" ); 
    239249                        add_filter( 'image_editor_class', $filter ); 
    240                         $editor = WP_Image_Editor::get_instance( DIR_TESTDATA ); 
     250                        $editor = wp_get_image_editor( DIR_TESTDATA ); 
    241251                        $this->assertInstanceOf( 'WP_Error', $editor ); 
    242252                        $this->assertEquals( 'error_loading_image', $editor->get_error_code() ); 
    243253                } 
     
    248258                                                          0, 0, 100, 100, 100, 100 ); 
    249259                $this->assertNotInstanceOf( 'WP_Error', $file ); 
    250260                $this->assertFileExists( $file ); 
    251                 $image = WP_Image_Editor::get_instance( $file ); 
     261                $image = wp_get_image_editor( $file ); 
    252262                $size = $image->get_size(); 
    253263                $this->assertEquals( 100, $size['height'] ); 
    254264                $this->assertEquals( 100, $size['width'] ); 
     
    260270                                                          DIR_TESTDATA . '/images/' . rand_str() . '.jpg' ); 
    261271                $this->assertNotInstanceOf( 'WP_Error', $file ); 
    262272                $this->assertFileExists( $file ); 
    263                 $image = WP_Image_Editor::get_instance( $file ); 
     273                $image = wp_get_image_editor( $file ); 
    264274                $size = $image->get_size(); 
    265275                $this->assertEquals( 100, $size['height'] ); 
    266276                $this->assertEquals( 100, $size['width'] ); 
  • includes/mock-image-editor.php

     
    77                public static $load_return = true; 
    88                public static $test_return = true; 
    99 
    10                 protected function load() { 
     10                public function load() { 
    1111                        return self::$load_return; 
    1212                } 
    1313                public static function test() { 
     
    3939                } 
    4040        } 
    4141 
    42 endif; 
    43  No newline at end of file 
     42endif;