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

Timestamp:

07/23/2022 03:40:10 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Revisions: Correct the function name for retrieving the last revision ID and total count.

Includes:

• Renaming the function to wp_get_last_revision_id_and_total_count().
• Making the default value for $post consistent with wp_get_post_revisions().
• Making WP_Error codes more specific and using them in test assertions.
• Adjusting the function description per the documentation standards.

Follow-up to [53759].

Props JustinSainton, SergeyBiryukov.
See #55857.

File:

• trunk/tests/phpunit/tests/post/revisions.php (modified) (3 diffs)

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

@@ -657 +657 @@
 
     /**
-     * Tests that wp_get_lastest_revision_id_and_total_count() returns last revision id and total count.
+     * Tests that wp_get_last_revision_id_and_total_count() returns the last revision ID and total count.
      *
      * @ticket 55857
@@ -675 +675 @@
         $post_revisions     = wp_get_post_revisions( $post_id );
         $last_post_revision = current( $post_revisions );
-        $revision           = wp_get_lastest_revision_id_and_total_count( $post_id );
+        $revision           = wp_get_last_revision_id_and_total_count( $post_id );
 
         $this->assertSame(
             $last_post_revision->ID,
             $revision['revision'],
-            'Failed asserting latest revision id.'
+            'The last revision ID does not match.'
         );
 
@@ -686 +686 @@
             count( $post_revisions ),
             $revision['count'],
-            'Failed asserting total count of revision.'
-        );
-    }
-
-    /**
-     * Tests that wp_get_lastest_revision_id_and_total_count() when no revisions.
+            'The total count of revisions does not match.'
+        );
+    }
+
+    /**
+     * Tests that wp_get_last_revision_id_and_total_count() returns a WP_Error when no revisions exist.
      *
      * @ticket 55857
      */
     public function test_wp_get_last_revision_id_and_total_count_no_revisions() {
-        $revision = wp_get_lastest_revision_id_and_total_count( null );
-        $this->assertWPError( $revision, 'Invalid Post, non existing revisions.' );
-        $this->assertSame( $revision->get_error_message(), 'Invalid post.' );
+        $revision = wp_get_last_revision_id_and_total_count( null );
+
+        $this->assertWPError( $revision, 'Invalid post, no revisions should exist.' );
+        $this->assertSame( $revision->get_error_code(), 'invalid_post' );
 
         add_filter( 'wp_revisions_to_keep', '__return_zero' );
         $post_id  = self::factory()->post->create();
-        $revision = wp_get_lastest_revision_id_and_total_count( $post_id );
-        $this->assertWPError( $revision, 'Revisions should be not enabled.' );
-        $this->assertSame( $revision->get_error_message(), 'Revisions not enabled.' );
+        $revision = wp_get_last_revision_id_and_total_count( $post_id );
+
+        $this->assertWPError( $revision, 'Revisions should not be enabled.' );
+        $this->assertSame( $revision->get_error_code(), 'revisions_not_enabled' );
     }