WordPress.org

Make WordPress Core


Ignore:
Timestamp:
09/02/2020 12:35:36 AM (17 months 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.