Changeset 57737 for trunk/tests/phpunit/tests/functions/wpParseSlugList.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/wpParseSlugList.php (modified) (2 diffs)

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

@@ -11 +11 @@
 
     /**
+     * @ticket 35582
      * @ticket 60217
      *
      * @dataProvider data_wp_parse_slug_list
+     * @dataProvider data_unexpected_input
      */
     public function test_wp_parse_slug_list( $input_list, $expected ) {
-
         $this->assertSameSets( $expected, wp_parse_slug_list( $input_list ) );
     }
 
     /**
-     * Data provider for test_wp_parse_slug_list().
+     * Data provider.
      *
      * @return array[]
@@ -27 +28 @@
     public function data_wp_parse_slug_list() {
         return array(
-            'simple'             => array(
+            'regular'                    => array(
+                'input_list' => 'apple,banana,carrot,dog',
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'double comma'               => array(
+                'input_list' => 'apple, banana,,carrot,dog',
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'duplicate slug in a string' => array(
+                'input_list' => 'apple,banana,carrot,carrot,dog',
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'duplicate slug in an array' => array(
+                'input_list' => array( 'apple', 'banana', 'carrot', 'carrot', 'dog' ),
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'string with spaces'         => array(
+                'input_list' => 'apple banana carrot dog',
+                'expected'   => array( 'apple', 'banana', 'carrot', 'dog' ),
+            ),
+            'array with spaces'          => array(
+                'input_list' => array( 'apple ', 'banana carrot', 'd o g' ),
+                'expected'   => array( 'apple', 'banana-carrot', 'd-o-g' ),
+            ),
+        );
+    }
+
+    /**
+     * Data provider.
+     *
+     * @return array[]
+     */
+    public function data_unexpected_input() {
+        return array(
+            'string with commas' => array(
+                'input_list' => '1,2,string with spaces',
+                'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
+            ),
+            'array'              => array(
                 'input_list' => array( '1', 2, 'string with spaces' ),
                 'expected'   => array( '1', '2', 'string-with-spaces' ),
             ),
-            'simple_with_comma'  => array(
-                'input_list' => '1,2,string with spaces',
+            'string with spaces' => array(
+                'input_list' => '1 2 string with spaces',
                 'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
             ),
-            'array_with_spaces'  => array(
+            'array with spaces'  => array(
                 'input_list' => array( '1 2 string with spaces' ),
                 'expected'   => array( '1-2-string-with-spaces' ),
             ),
-            'simple_with_spaces' => array(
-                'input_list' => '1 2 string with spaces',
-                'expected'   => array( '1', '2', 'string', 'with', 'spaces' ),
+            'string with html'   => array(
+                'input_list' => '1 2 string <strong>with</strong> <h1>HEADING</h1>',
+                'expected'   => array( '1', '2', 'string', 'with', 'heading' ),
             ),
-            'array_html'         => array(
+            'array with html'    => array(
                 'input_list' => array( '1', 2, 'string <strong>with</strong> <h1>HEADING</h1>' ),
                 'expected'   => array( '1', '2', 'string-with-heading' ),
             ),
-            'simple_html_spaces' => array(
-                'input_list' => '1 2 string <strong>with</strong> <h1>HEADING</h1>',
-                'expected'   => array( '1', '2', 'string', 'with', 'heading' ),
+            'array with null'    => array(
+                'input_list' => array( 1, 2, null ),
+                'expected'   => array( '1', '2' ),
             ),
-            'dup_id'             => array(
-                'input_list' => '1 1 string string',
-                'expected'   => array( '1', 'string' ),
+            'array with false'   => array(
+                'input_list' => array( 1, 2, false ),
+                'expected'   => array( '1', '2', '' ),
             ),
         );