Changeset 55406 for trunk/tests/phpunit/tests/post/revisions.php

Timestamp:

02/22/2023 08:47:38 PM (2 years ago)

Author:

johnbillion

Message:

Revisions: Remove an unnecessary call to _doing_it_wrong() and corresponding new text string from the implementation of the new wp_save_post_revision_revisions_before_deletion filter.

While the guard condition was technically correct, it's not practical or necessary to provide this protection for every use of every filter, and it adds unnecessary burden to translators to provide translations for strings that will likely not be seen.

Follow up to [55254].

Fixes #57320

File:

• trunk/tests/phpunit/tests/post/revisions.php (modified) (1 diff)

trunk/tests/phpunit/tests/post/revisions.php

@@ -929 +929 @@
         );
     }
-
-    /**
-     * Tests that wp_save_post_revision() ignores an invalid return value
-     * from the 'wp_save_post_revision_revisions_before_deletion' filter
-     * and throws _doing_it_wrong().
-     *
-     * @ticket 57320
-     *
-     * @covers ::wp_save_post_revision
-     *
-     * @expectedIncorrectUsage wp_save_post_revision
-     */
-    public function test_wp_save_post_revision_should_ignore_invalid_revisions_before_deletion_filter() {
-        $post_id = self::factory()->post->create( array( 'post_title' => 'Test 57320' ) );
-
-        add_filter(
-            'wp_revisions_to_keep',
-            static function() {
-                return 1;
-            }
-        );
-
-        add_filter( 'wp_save_post_revision_revisions_before_deletion', '__return_null' );
-
-        for ( $update = 1; $update < 4; ++$update ) {
-            wp_update_post(
-                array(
-                    'ID'         => $post_id,
-                    'post_title' => 'Test 57320 Update ' . $update,
-                )
-            );
-        }
-
-        $actual = wp_get_post_revisions( $post_id );
-
-        $this->assertCount(
-            1,
-            $actual,
-            'There should only be one revision.'
-        );
-
-        $first = reset( $actual );
-
-        $this->assertSame(
-            'Test 57320 Update 3',
-            $first->post_title,
-            'The title of the first revision was incorrect.'
-        );
-    }
 }