Changeset 57737 for trunk/tests/phpunit/tests/functions/wpParseList.php

Timestamp:

02/28/2024 06:09:38 PM (13 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.

File:

• trunk/tests/phpunit/tests/functions/wpParseList.php (modified) (2 diffs)

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

@@ -12 +12 @@
     /**
      * @ticket 43977
+     *
      * @dataProvider data_wp_parse_list
      */
-    public function test_wp_parse_list( $expected, $actual ) {
-        $this->assertSame( $expected, array_values( wp_parse_list( $actual ) ) );
+    public function test_wp_parse_list( $input_list, $expected ) {
+        $this->assertSameSets( $expected, wp_parse_list( $input_list ) );
     }
 
@@ -25 +26 @@
     public function data_wp_parse_list() {
         return array(
-            array( array( '1', '2', '3', '4' ), '1,2,3,4' ),
-            array( array( 'apple', 'banana', 'carrot', 'dog' ), 'apple,banana,carrot,dog' ),
-            array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana' ),
-            array( array( '1', '2', 'apple', 'banana' ), '1, 2,apple,banana' ),
-            array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,,banana' ),
-            array( array( '1', '2', 'apple', 'banana' ), ',1,2,apple,banana' ),
-            array( array( '1', '2', 'apple', 'banana' ), '1,2,apple,banana,' ),
-            array( array( '1', '2', 'apple', 'banana' ), '1,2 ,apple,banana' ),
-            array( array(), '' ),
-            array( array(), ',' ),
-            array( array(), ',,' ),
+            'ids only'           => array(
+                'input_list' => '1,2,3,4',
+                'expected'   => array( '1', '2', '3', '4' ),
+            ),
+            'slugs only'         => array(
+                'input_list' => 'apple,banana,carrot,dog',
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'ids and slugs'      => array(
+                'input_list' => '1,2,apple,banana',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'space after comma'  => array(
+                'input_list' => '1, 2,apple,banana',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'double comma'       => array(
+                'input_list' => '1,2,apple,,banana',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'leading comma'      => array(
+                'input_list' => ',1,2,apple,banana',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'trailing comma'     => array(
+                'input_list' => '1,2,apple,banana,',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'space before comma' => array(
+                'input_list' => '1,2 ,apple,banana',
+                'expected'   => array( '1', '2', 'apple', 'banana' ),
+            ),
+            'empty string'       => array(
+                'input_list' => '',
+                'expected'   => array(),
+            ),
+            'comma only'         => array(
+                'input_list' => ',',
+                'expected'   => array(),
+            ),
+            'double comma only'  => array(
+                'input_list' => ',,',
+                'expected'   => array(),
+            ),
         );
     }