Changeset 49268 for trunk/tests/phpunit/tests/xmlrpc/wp/newComment.php

Timestamp:

10/22/2020 12:17:10 AM (4 years ago)

Author:

peterwilsoncc

Message:

Tests: Use shared fixtures in XML-RPC new comment unit tests.

See #51344.

File:

• trunk/tests/phpunit/tests/xmlrpc/wp/newComment.php (modified) (7 diffs)

trunk/tests/phpunit/tests/xmlrpc/wp/newComment.php

@@ -6 +6 @@
 class Tests_XMLRPC_wp_newComment extends WP_XMLRPC_UnitTestCase {
 
+    /**
+     * Post object for shared fixture.
+     *
+     * @var WP_Post
+     */
+    public static $post;
+
+    public static function wpSetUpBeforeClass( $factory ) {
+        self::make_user_by_role( 'administrator' );
+        self::$post = $factory->post->create_and_get();
+    }
+
     function test_valid_comment() {
-        $this->make_user_by_role( 'administrator' );
-        $post = self::factory()->post->create_and_get();
-
         $result = $this->myxmlrpcserver->wp_newComment(
             array(
@@ -15 +24 @@
                 'administrator',
                 'administrator',
-                $post->ID,
+                self::$post->ID,
                 array(
                     'content' => rand_str( 100 ),
@@ -26 +35 @@
 
     function test_empty_comment() {
-        $this->make_user_by_role( 'administrator' );
-        $post = self::factory()->post->create_and_get();
-
         $result = $this->myxmlrpcserver->wp_newComment(
             array(
@@ -34 +40 @@
                 'administrator',
                 'administrator',
-                $post->ID,
+                self::$post->ID,
                 array(
                     'content' => '',
@@ -46 +52 @@
 
     function test_new_comment_post_closed() {
-        $this->make_user_by_role( 'administrator' );
         $post = self::factory()->post->create_and_get(
             array(
@@ -72 +77 @@
 
     function test_new_comment_duplicated() {
-        $this->make_user_by_role( 'administrator' );
-        $post = self::factory()->post->create_and_get();
-
         $comment_args = array(
             1,
             'administrator',
             'administrator',
-            $post->ID,
+            self::$post->ID,
             array(
                 'content' => rand_str( 100 ),
@@ -95 +97 @@
         $this->assertSame( 403, $result->code );
     }
-
 }