WordPress.org
Get WordPress

Make WordPress Core

Ticket #22356: 22356.9.tests.diff

File 22356.9.tests.diff, 6.9 KB (added by kirasong, 12 years 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;