Changeset 48328 for trunk/tests/phpunit/tests/post/query.php

Timestamp:

07/05/2020 09:32:26 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Query: Make sure the found_posts property of WP_Query is always an integer, to match the documented type.

This makes the property consistent with similar properties of other classes:

• WP_Comment_Query::$found_comments
• WP_Network_Query::$found_networks
• WP_Site_Query::$found_sites
• WP_User_Query::$total_users

Props birgire, PressLabs.
Fixes #42469.

File:

• trunk/tests/phpunit/tests/post/query.php (modified) (5 diffs)

trunk/tests/phpunit/tests/post/query.php

@@ -653 +653 @@
 
         add_filter( 'split_the_query', '__return_true' );
+
         $q = new WP_Query(
             array(
@@ -659 +660 @@
             )
         );
+
         remove_filter( 'split_the_query', '__return_true' );
 
@@ -678 +680 @@
         // ! $split_the_query
         add_filter( 'split_the_query', '__return_false' );
+
         $q = new WP_Query(
             array(
@@ -684 +687 @@
             )
         );
+
         remove_filter( 'split_the_query', '__return_false' );
 
@@ -722 +726 @@
     }
 
+    /**
+     * @ticket 42469
+     */
+    public function test_found_posts_should_be_integer_not_string() {
+        $this->post_id = self::factory()->post->create();
+
+        $q = new WP_Query(
+            array(
+                'posts_per_page' => 1,
+            )
+        );
+
+        $this->assertInternalType( 'int', $q->found_posts );
+    }
+
+    /**
+     * @ticket 42469
+     */
+    public function test_found_posts_should_be_integer_even_if_found_posts_filter_returns_string_value() {
+        $this->post_id = self::factory()->post->create();
+
+        add_filter( 'found_posts', '__return_empty_string' );
+
+        $q = new WP_Query(
+            array(
+                'posts_per_page' => 1,
+            )
+        );
+
+        remove_filter( 'found_posts', '__return_empty_string' );
+
+        $this->assertInternalType( 'int', $q->found_posts );
+    }
 }