Changeset 48880 for trunk/tests/phpunit/tests/admin/includesTemplate.php

Timestamp:

08/27/2020 02:46:22 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Taxonomy: Make sure wp_terms_checklist() and Walker_Category_Checklist::start_el() properly handle an array of strings as selected_cats or popular_cats values.

Even with these values documented as an array of integers, they can technically also accept an array of strings, e.g. as form results.

Add a unit test.

Props brianhogg, TimothyBlynJacobs, SergeyBiryukov.
Fixes #51137.

File:

• trunk/tests/phpunit/tests/admin/includesTemplate.php (modified) (2 diffs)

trunk/tests/phpunit/tests/admin/includesTemplate.php

@@ -4 +4 @@
  */
 class Tests_Admin_includesTemplate extends WP_UnitTestCase {
+
     function test_equal() {
         $this->assertEquals( ' selected=\'selected\'', selected( 'foo', 'foo', false ) );
@@ -45 +46 @@
         $this->assertEquals( '', selected( 0, false, false ) );
         $this->assertEquals( '', checked( 0, false, false ) );
+    }
+
+    /**
+     * @ticket 51147
+     * @dataProvider data_wp_terms_checklist_with_selected_cats
+     */
+    public function test_wp_terms_checklist_with_selected_cats( $term_id ) {
+        $output = wp_terms_checklist(
+            0,
+            array(
+                'selected_cats' => array( $term_id ),
+                'echo'          => false,
+            )
+        );
+
+        $this->assertContains( "checked='checked'", $output );
+    }
+
+    /**
+     * @ticket 51147
+     * @dataProvider data_wp_terms_checklist_with_selected_cats
+     */
+    public function test_wp_terms_checklist_with_popular_cats( $term_id ) {
+        $output = wp_terms_checklist(
+            0,
+            array(
+                'popular_cats' => array( $term_id ),
+                'echo'         => false,
+            )
+        );
+
+        $this->assertContains( 'class="popular-category"', $output );
+    }
+
+    public function data_wp_terms_checklist_with_selected_cats() {
+        return array(
+            array( '1' ),
+            array( 1 ),
+        );
     }