Changeset 53909 for trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php

Timestamp:

08/19/2022 04:13:38 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Tests: Assign created fixtures to the dedicated class properties in some test classes.

This affects:

• WP_Test_REST_Categories_Controller
• WP_Test_REST_Comments_Controller
• WP_Test_REST_Tags_Controller

and brings consistency with:

• WP_Test_REST_Posts_Controller
• WP_Test_REST_Users_Controller

These test classes were previously updated to improve performance by creating less fixtures and reusing them where possible. While the pagination tests for categories and comments still passed due to enough items being created, the pagination test for tags did not work as expected and did not perform any assertions due to trying to iterate over an empty array of results.

This is now corrected by assigning the properties as intended and adding more assertions to the affected test.

Follow-up to [46657].

Props johnregan3, costdev, johnbillion.
See #54662.

File:

• trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php (modified) (3 diffs)

trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php

@@ -55 +55 @@
         // Set up tags for pagination tests.
         for ( $i = 0; $i < self::$total_tags; $i++ ) {
-            $tag_ids[] = $factory->tag->create(
+            self::$tag_ids[] = $factory->tag->create(
                 array(
                     'name' => "Tag {$i}",
@@ -413 +413 @@
         $tags     = $response->get_data();
 
+        $this->assertNotEmpty( $tags );
+
         $i = 0;
         foreach ( $tags as $tag ) {
@@ -426 +428 @@
         $response = rest_get_server()->dispatch( $request );
         $tags     = $response->get_data();
+
+        $this->assertNotEmpty( $tags );
 
         foreach ( $tags as $tag ) {