WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
09/02/2020 12:35:36 AM (4 years ago)
Author:
SergeyBiryukov
Message:

Tests: First pass at using assertSame() instead of assertEquals() in most of the unit tests.

This ensures that not only the return values match the expected results, but also that their type is the same.

Going forward, stricter type checking by using assertSame() should generally be preferred to assertEquals() where appropriate, to make the tests more reliable.

Props johnbillion, jrf, SergeyBiryukov.
See #38266.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/image/dimensions.php

    r47122 r48937  
    1111        $out = image_resize_dimensions( 640, 480, 400, 400, false );
    1212        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    13         $this->assertEquals( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
     13        $this->assertSame( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
    1414
    1515        // Portrait: resize 480x640 to fit 400x400: 300x400.
    1616        $out = image_resize_dimensions( 480, 640, 400, 400, false );
    1717        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    18         $this->assertEquals( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
     18        $this->assertSame( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
    1919    }
    2020
    … …  
    2323        $out = image_resize_dimensions( 640, 480, 400, 0, false );
    2424        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    25         $this->assertEquals( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
     25        $this->assertSame( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
    2626
    2727        // Portrait: resize 480x640 to fit 400w: 400x533.
    2828        $out = image_resize_dimensions( 480, 640, 400, 0, false );
    2929        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    30         $this->assertEquals( array( 0, 0, 0, 0, 400, 533, 480, 640 ), $out );
     30        $this->assertSame( array( 0, 0, 0, 0, 400, 533, 480, 640 ), $out );
    3131    }
    3232
    … …  
    3535        $out = image_resize_dimensions( 640, 480, 0, 400, false );
    3636        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    37         $this->assertEquals( array( 0, 0, 0, 0, 533, 400, 640, 480 ), $out );
     37        $this->assertSame( array( 0, 0, 0, 0, 533, 400, 640, 480 ), $out );
    3838
    3939        // Portrait: resize 480x640 to fit 400h: 300x400.
    4040        $out = image_resize_dimensions( 480, 640, 0, 400, false );
    4141        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    42         $this->assertEquals( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
     42        $this->assertSame( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
    4343    }
    4444
    … …  
    4747        $out = image_resize_dimensions( 640, 480, 800, 800, false );
    4848        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    49         $this->assertEquals( false, $out );
     49        $this->assertFalse( $out );
    5050
    5151        // Portrait: resize 480x640 to fit 800x800.
    5252        $out = image_resize_dimensions( 480, 640, 800, 800, false );
    5353        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    54         $this->assertEquals( false, $out );
     54        $this->assertFalse( $out );
    5555    }
    5656
    … …  
    5959        $out = image_resize_dimensions( 640, 480, 800, 0, false );
    6060        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    61         $this->assertEquals( false, $out );
     61        $this->assertFalse( $out );
    6262
    6363        // Portrait: resize 480x640 to fit 800w.
    6464        $out = image_resize_dimensions( 480, 640, 800, 0, false );
    6565        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    66         $this->assertEquals( false, $out );
     66        $this->assertFalse( $out );
    6767    }
    6868
    … …  
    7171        $out = image_resize_dimensions( 640, 480, 0, 800, false );
    7272        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    73         $this->assertEquals( false, $out );
     73        $this->assertFalse( $out );
    7474
    7575        // Portrait: resize 480x640 to fit 800h.
    7676        $out = image_resize_dimensions( 480, 640, 0, 800, false );
    7777        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    78         $this->assertEquals( false, $out );
     78        $this->assertFalse( $out );
    7979    }
    8080
    … …  
    8585        $out = image_resize_dimensions( 640, 480, 400, 400, true );
    8686        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    87         $this->assertEquals( array( 0, 0, 80, 0, 400, 400, 480, 480 ), $out );
     87        $this->assertSame( array( 0, 0, 80, 0, 400, 400, 480, 480 ), $out );
    8888
    8989        // Portrait: resize 480x640 to fit 400x400: 400x400 taken from a 480x480 crop at (0. 80).
    9090        $out = image_resize_dimensions( 480, 640, 400, 400, true );
    9191        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    92         $this->assertEquals( array( 0, 0, 0, 80, 400, 400, 480, 480 ), $out );
     92        $this->assertSame( array( 0, 0, 0, 80, 400, 400, 480, 480 ), $out );
    9393    }
    9494
    … …  
    9797        $out = image_resize_dimensions( 640, 480, 400, 0, true );
    9898        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    99         $this->assertEquals( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
     99        $this->assertSame( array( 0, 0, 0, 0, 400, 300, 640, 480 ), $out );
    100100
    101101        // Portrait: resize 480x640 to fit 400w: 400x533.
    102102        $out = image_resize_dimensions( 480, 640, 400, 0, true );
    103103        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    104         $this->assertEquals( array( 0, 0, 0, 0, 400, 533, 480, 640 ), $out );
     104        $this->assertSame( array( 0, 0, 0, 0, 400, 533, 480, 640 ), $out );
    105105    }
    106106
    … …  
    109109        $out = image_resize_dimensions( 640, 480, 0, 400, true );
    110110        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    111         $this->assertEquals( array( 0, 0, 0, 0, 533, 400, 640, 480 ), $out );
     111        $this->assertSame( array( 0, 0, 0, 0, 533, 400, 640, 480 ), $out );
    112112
    113113        // Portrait: resize 480x640 to fit 400h: 300x400.
    114114        $out = image_resize_dimensions( 480, 640, 0, 400, true );
    115115        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    116         $this->assertEquals( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
     116        $this->assertSame( array( 0, 0, 0, 0, 300, 400, 480, 640 ), $out );
    117117    }
    118118
    … …  
    121121        $out = image_resize_dimensions( 640, 480, 400, 500, true );
    122122        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    123         $this->assertEquals( array( 0, 0, 120, 0, 400, 480, 400, 480 ), $out );
     123        $this->assertSame( array( 0, 0, 120, 0, 400, 480, 400, 480 ), $out );
    124124
    125125        // Portrait: resize 480x640 to fit 400x400: 400x400 taken from a 480x480 crop at (0. 80).
    126126        $out = image_resize_dimensions( 480, 640, 400, 500, true );
    127127        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    128         $this->assertEquals( array( 0, 0, 0, 20, 400, 500, 480, 600 ), $out );
     128        $this->assertSame( array( 0, 0, 0, 20, 400, 500, 480, 600 ), $out );
    129129    }
    130130
    … …  
    144144        $out = image_resize_dimensions( 640, 480, 640, 480, true );
    145145        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    146         $this->assertEquals( array( 0, 0, 0, 0, 640, 480, 640, 480 ), $out );
     146        $this->assertSame( array( 0, 0, 0, 0, 640, 480, 640, 480 ), $out );
    147147
    148148        // Resize 640x480 to fit 640x480 (no change).
    149149        $out = image_resize_dimensions( 640, 480, 640, 480, false );
    150150        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    151         $this->assertEquals( array( 0, 0, 0, 0, 640, 480, 640, 480 ), $out );
     151        $this->assertSame( array( 0, 0, 0, 0, 640, 480, 640, 480 ), $out );
    152152
    153153        remove_filter( 'wp_image_resize_identical_dimensions', '__return_true' );
    … …  
    162162        $out = image_resize_dimensions( 640, 480, 400, 500, array( 'left', 'top' ) );
    163163        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    164         $this->assertEquals( array( 0, 0, 0, 0, 400, 480, 400, 480 ), $out );
     164        $this->assertSame( array( 0, 0, 0, 0, 400, 480, 400, 480 ), $out );
    165165
    166166        // Portrait: resize 480x640 to fit 400x400: 400x400 taken from a 480x480 crop.
    … …  
    168168        $out = image_resize_dimensions( 480, 640, 400, 500, array( 'left', 'top' ) );
    169169        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    170         $this->assertEquals( array( 0, 0, 0, 0, 400, 500, 480, 600 ), $out );
     170        $this->assertSame( array( 0, 0, 0, 0, 400, 500, 480, 600 ), $out );
    171171
    172172        // Landscape: crop 640x480 to fit 400x500: 400x400 taken from a 480x480 crop.
    … …  
    174174        $out = image_resize_dimensions( 640, 480, 400, 500, array( 'right', 'bottom' ) );
    175175        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    176         $this->assertEquals( array( 0, 0, 240, 0, 400, 480, 400, 480 ), $out );
     176        $this->assertSame( array( 0, 0, 240, 0, 400, 480, 400, 480 ), $out );
    177177
    178178        // Portrait: resize 480x640 to fit 400x400: 400x400 taken from a 480x480 crop.
    … …  
    180180        $out = image_resize_dimensions( 480, 640, 400, 500, array( 'right', 'bottom' ) );
    181181        // dst_x, dst_y, src_x, src_y, dst_w, dst_h, src_w, src_h.
    182         $this->assertEquals( array( 0, 0, 0, 40, 400, 500, 480, 600 ), $out );
     182        $this->assertSame( array( 0, 0, 0, 40, 400, 500, 480, 600 ), $out );
    183183    }
    184184
Note: See TracChangeset for help on using the changeset viewer.