Changeset 54271 for trunk/tests/phpunit/tests/post/getPageByTitle.php

Timestamp:

09/21/2022 05:27:12 AM (3 years ago)

Author:

peterwilsoncc

Message:

Posts, Post Types: Fix WP_Query parameter used by get_page_by_title().

Fixes the call to WP_Query within get_page_by_title() by using the correct title parameter, title.

Modify the orderby parameter to prioritize the oldest published date over the smallest post ID. This ensures the behaviour matches that of the previous version of get_page_by_title().

The tests have been modified to include a populated post table to ensure the posts returned are matched by design rather than coincidence.

Follow up to [54234].

Props dd32, timothyblynjacobs, peterwilsoncc.
Fixes #56609.
See #36905.

File:

• trunk/tests/phpunit/tests/post/getPageByTitle.php (modified) (2 diffs)

trunk/tests/phpunit/tests/post/getPageByTitle.php

@@ -6 +6 @@
  */
 class Tests_Post_GetPageByTitle extends WP_UnitTestCase {
+
+    /**
+     * Generate shared fixtures.
+     *
+     * These are not used in the tests but are rather used to populate the
+     * posts table and ensure that the tests return the correct post object
+     * by design rather than through chance.
+     */
+    public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
+        // Fill the database with some pages.
+        $factory->post->create_many(
+            2,
+            array(
+                'post_type' => 'page',
+            )
+        );
+
+        // Fill the database with some attachments.
+        $factory->post->create_many(
+            2,
+            array(
+                'post_type' => 'attachment',
+            )
+        );
+
+        // Fill the database with some test post types.
+        register_post_type( 'wptests_pt' );
+        $factory->post->create_many(
+            2,
+            array(
+                'post_type' => 'wptests_pt',
+            )
+        );
+    }
+
     /**
      * @ticket 36905
@@ -42 +77 @@
 
         $this->assertSame( $page, $found->ID );
+    }
+
+    /**
+     * @ticket 36905
+     * @ticket 56609
+     */
+    public function test_should_be_case_insensitive_match() {
+        $page = self::factory()->post->create(
+            array(
+                'post_type'  => 'page',
+                'post_title' => 'Foo',
+            )
+        );
+
+        $found = get_page_by_title( 'foo' );
+
+        $this->assertSame( $page, $found->ID );
+    }
+
+    /**
+     * Test the oldest published post is matched first.
+     *
+     * Per the docs: in case of more than one post having the same title,
+     * it will check the oldest publication date, not the smallest ID.
+     *
+     * @ticket 36905
+     * @ticket 56609
+     */
+    public function test_should_match_oldest_published_date_when_titles_match() {
+        self::factory()->post->create(
+            array(
+                'post_type'  => 'page',
+                'post_title' => 'foo',
+            )
+        );
+
+        $old_page = self::factory()->post->create(
+            array(
+                'post_type'  => 'page',
+                'post_title' => 'foo',
+                'post_date'  => '1984-01-11 05:00:00',
+            )
+        );
+
+        $found = get_page_by_title( 'foo' );
+
+        $this->assertSame( $old_page, $found->ID );
     }