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

Timestamp:

07/05/2020 09:32:26 PM (6 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 );
+        }
 }