WordPress.org
Get WordPress

Make WordPress Core

Changeset 57737


Ignore:
Timestamp:
02/28/2024 06:09:38 PM (12 months ago)
Author:
SergeyBiryukov
Message:

Tests: Expand wp_parse_id_list() unit tests.

Includes:

  • Moving pre-existing wp_parse_id_list() tests to their own file.
  • Merging new and pre-existing wp_parse_slug_list() tests.
  • Using named data provider in wp_parse_list() tests.

Follow-up to [25170], [40044], [44546], [57284], [57725].

Props pbearne, mukesh27, SergeyBiryukov.
Fixes #60218. See #60217, #59647.

Location:
trunk/tests/phpunit/tests
Files:
1 added
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/functions.php

    r57524 r57737  
    725725
    726726        update_option( 'blog_charset', $orig_blog_charset );
    727     }
    728 
    729     /**
    730      * @dataProvider data_wp_parse_id_list
    731      */
    732     public function test_wp_parse_id_list( $expected, $actual ) {
    733         $this->assertSame( $expected, array_values( wp_parse_id_list( $actual ) ) );
    734     }
    735 
    736     public function data_wp_parse_id_list() {
    737         return array(
    738             array( array( 1, 2, 3, 4 ), '1,2,3,4' ),
    739             array( array( 1, 2, 3, 4 ), '1, 2,,3,4' ),
    740             array( array( 1, 2, 3, 4 ), '1,2,2,3,4' ),
    741             array( array( 1, 2, 3, 4 ), array( '1', '2', '3', '4', '3' ) ),
    742             array( array( 1, 2, 3, 4 ), array( 1, '2', 3, '4' ) ),
    743             array( array( 1, 2, 3, 4 ), '-1,2,-3,4' ),
    744             array( array( 1, 2, 3, 4 ), array( -1, 2, '-3', '4' ) ),
    745         );
    746     }
    747 
    748     /**
    749      * @dataProvider data_wp_parse_slug_list
    750      */
    751     public function test_wp_parse_slug_list( $expected, $actual ) {
    752         $this->assertSame( $expected, array_values( wp_parse_slug_list( $actual ) ) );
    753     }
    754 
    755     public function data_wp_parse_slug_list() {
    756         return array(
    757             array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple,banana,carrot,dog' ),
    758             array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple, banana,,carrot,dog' ),
    759             array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple banana carrot dog' ),
    760             array( array( 'apple', 'banana-carrot', 'd-o-g' ), array( 'apple ', 'banana carrot', 'd o g' ) ),
    761         );
    762727    }
    763728

  • trunk/tests/phpunit/tests/functions/wpParseList.php

    r57735 r57737  
    1212    /**
    1313     * @ticket 43977
     14     *
    1415     * @dataProvider data_wp_parse_list
    1516     */
    16     public function test_wp_parse_list( $expected, $actual ) {
    17         $this->assertSame( $expected, array_values( wp_parse_list( $actual ) ) );
     17    public function test_wp_parse_list( $input_list, $expected ) {
     18        $this->assertSameSets( $expected, wp_parse_list( $input_list ) );
    1819    }
    1920
    … …  
    2526    public function data_wp_parse_list() {
    2627        return array(
    27             array( array( '1', '2', '3', '4' ), '1,2,3,4' ),
    28             array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple,banana,carrot,dog' ),
    29             array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana' ),
    30             array( array( '1', '2', 'apple', 'banana' ), '1, 2,apple,banana' ),
    31             array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,,banana' ),
    32             array( array( '1', '2', 'apple', 'banana' ), ',1,2,apple,banana' ),
    33             array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana,' ),
    34             array( array( '1', '2', 'apple', 'banana' ), '1,2 ,apple,banana' ),
    35             array( array(), '' ),
    36             array( array(), ',' ),
    37             array( array(), ',,' ),
     28            'ids only'           => array(
     29                'input_list' => '1,2,3,4',
     30                'expected'   => array( '1', '2', '3', '4' ),
     31            ),
     32            'slugs only'         => array(
     33                'input_list' => 'apple,banana,carrot,dog',
     34                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     35            ),
     36            'ids and slugs'      => array(
     37                'input_list' => '1,2,apple,banana',
     38                'expected'   => array( '1', '2', 'apple', 'banana' ),
     39            ),
     40            'space after comma'  => array(
     41                'input_list' => '1, 2,apple,banana',
     42                'expected'   => array( '1', '2', 'apple', 'banana' ),
     43            ),
     44            'double comma'       => array(
     45                'input_list' => '1,2,apple,,banana',
     46                'expected'   => array( '1', '2', 'apple', 'banana' ),
     47            ),
     48            'leading comma'      => array(
     49                'input_list' => ',1,2,apple,banana',
     50                'expected'   => array( '1', '2', 'apple', 'banana' ),
     51            ),
     52            'trailing comma'     => array(
     53                'input_list' => '1,2,apple,banana,',
     54                'expected'   => array( '1', '2', 'apple', 'banana' ),
     55            ),
     56            'space before comma' => array(
     57                'input_list' => '1,2 ,apple,banana',
     58                'expected'   => array( '1', '2', 'apple', 'banana' ),
     59            ),
     60            'empty string'       => array(
     61                'input_list' => '',
     62                'expected'   => array(),
     63            ),
     64            'comma only'         => array(
     65                'input_list' => ',',
     66                'expected'   => array(),
     67            ),
     68            'double comma only'  => array(
     69                'input_list' => ',,',
     70                'expected'   => array(),
     71            ),
    3872        );
    3973    }

  • trunk/tests/phpunit/tests/functions/wpParseSlugList.php

    r57735 r57737  
    1111
    1212    /**
     13     * @ticket 35582
    1314     * @ticket 60217
    1415     *
    1516     * @dataProvider data_wp_parse_slug_list
     17     * @dataProvider data_unexpected_input
    1618     */
    1719    public function test_wp_parse_slug_list( $input_list, $expected ) {
    18 
    1920        $this->assertSameSets( $expected, wp_parse_slug_list( $input_list ) );
    2021    }
    2122
    2223    /**
    23      * Data provider for test_wp_parse_slug_list().
     24     * Data provider.
    2425     *
    2526     * @return array[]
    … …  
    2728    public function data_wp_parse_slug_list() {
    2829        return array(
    29             'simple'             => array(
     30            'regular'                    => array(
     31                'input_list' => 'apple,banana,carrot,dog',
     32                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     33            ),
     34            'double comma'               => array(
     35                'input_list' => 'apple, banana,,carrot,dog',
     36                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     37            ),
     38            'duplicate slug in a string' => array(
     39                'input_list' => 'apple,banana,carrot,carrot,dog',
     40                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     41            ),
     42            'duplicate slug in an array' => array(
     43                'input_list' => array( 'apple', 'banana', 'carrot', 'carrot', 'dog' ),
     44                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     45            ),
     46            'string with spaces'         => array(
     47                'input_list' => 'apple banana carrot dog',
     48                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
     49            ),
     50            'array with spaces'          => array(
     51                'input_list' => array( 'apple ', 'banana carrot', 'd o g' ),
     52                'expected'   => array( 'apple', 'banana-carrot', 'd-o-g' ),
     53            ),
     54        );
     55    }
     56
     57    /**
     58     * Data provider.
     59     *
     60     * @return array[]
     61     */
     62    public function data_unexpected_input() {
     63        return array(
     64            'string with commas' => array(
     65                'input_list' => '1,2,string with spaces',
     66                'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
     67            ),
     68            'array'              => array(
    3069                'input_list' => array( '1', 2, 'string with spaces' ),
    3170                'expected'   => array( '1', '2', 'string-with-spaces' ),
    3271            ),
    33             'simple_with_comma' => array(
    34                 'input_list' => '1,2,string with spaces',
     72            'string with spaces' => array(
     73                'input_list' => '1 2 string with spaces',
    3574                'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
    3675            ),
    37             'array_with_spaces'  => array(
     76            'array with spaces'  => array(
    3877                'input_list' => array( '1 2 string with spaces' ),
    3978                'expected'   => array( '1-2-string-with-spaces' ),
    4079            ),
    41             'simple_with_spaces' => array(
    42                 'input_list' => '1 2 string with spaces',
    43                 'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
     80            'string with html'  => array(
     81                'input_list' => '1 2 string <strong>with</strong> <h1>HEADING</h1>',
     82                'expected'   => array( '1', '2', 'string', 'with', 'heading' ),
    4483            ),
    45             'array_html'         => array(
     84            'array with html'    => array(
    4685                'input_list' => array( '1', 2, 'string <strong>with</strong> <h1>HEADING</h1>' ),
    4786                'expected'   => array( '1', '2', 'string-with-heading' ),
    4887            ),
    49             'simple_html_spaces' => array(
    50                 'input_list' => '1 2 string <strong>with</strong> <h1>HEADING</h1>',
    51                 'expected'   => array( '1', '2', 'string', 'with', 'heading' ),
     88            'array with null'    => array(
     89                'input_list' => array( 1, 2, null ),
     90                'expected'   => array( '1', '2' ),
    5291            ),
    53             'dup_id'             => array(
    54                 'input_list' => '1 1 string string',
    55                 'expected'   => array( '1', 'string' ),
     92            'array with false'   => array(
     93                'input_list' => array( 1, 2, false ),
     94                'expected'   => array( '1', '2', '' ),
    5695            ),
    5796        );
Note: See TracChangeset for help on using the changeset viewer.