WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/05/2019 08:41:12 PM (2 years ago)
Author:
SergeyBiryukov
Message:

REST API: Speed up pagination unit tests by creating less fixtures and reusing them where possible.

Includes minor documentation and code layout fixes for better readability.

See #30017, #48145.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/rest-api/rest-pages-controller.php

    r46586 r46657  
    126126            )
    127127        );
    128         // No parent
     128
     129        // No parent.
    129130        $request  = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    130131        $response = rest_get_server()->dispatch( $request );
    131132        $data     = $response->get_data();
    132133        $this->assertEquals( 2, count( $data ) );
    133         // Filter to parent
     134
     135        // Filter to parent.
    134136        $request->set_param( 'parent', $id1 );
    135137        $response = rest_get_server()->dispatch( $request );
     
    137139        $this->assertEquals( 1, count( $data ) );
    138140        $this->assertEquals( $id2, $data[0]['id'] );
    139         // Invalid parent should fail
     141
     142        // Invalid 'parent' should error.
    140143        $request->set_param( 'parent', 'some-slug' );
    141144        $response = rest_get_server()->dispatch( $request );
     
    170173            )
    171174        );
    172         // No parent
     175
     176        // No parent.
    173177        $request  = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    174178        $response = rest_get_server()->dispatch( $request );
    175179        $data     = $response->get_data();
    176180        $this->assertEquals( 4, count( $data ) );
    177         // Filter to parents
     181
     182        // Filter to parents.
    178183        $request->set_param( 'parent', array( $id1, $id3 ) );
    179184        $response = rest_get_server()->dispatch( $request );
     
    197202            )
    198203        );
    199         // No parent
     204
     205        // No parent.
    200206        $request  = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    201207        $response = rest_get_server()->dispatch( $request );
    202208        $data     = $response->get_data();
    203209        $this->assertEquals( 2, count( $data ) );
    204         // Filter to parent
     210
     211        // Filter to parent.
    205212        $request->set_param( 'parent_exclude', $id1 );
    206213        $response = rest_get_server()->dispatch( $request );
     
    208215        $this->assertEquals( 1, count( $data ) );
    209216        $this->assertEquals( $id1, $data[0]['id'] );
    210         // Invalid parent_exclude should error
     217
     218        // Invalid 'parent_exclude' should error.
    211219        $request->set_param( 'parent_exclude', 'some-slug' );
    212220        $response = rest_get_server()->dispatch( $request );
     
    242250            )
    243251        );
    244         // No parent
     252
     253        // No parent.
    245254        $request  = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    246255        $response = rest_get_server()->dispatch( $request );
    247256        $data     = $response->get_data();
    248257        $this->assertEqualSets( array( $id1, $id2, $id3, $id4 ), wp_list_pluck( $data, 'id' ) );
    249         // Filter to menu_order
     258
     259        // Filter to 'menu_order'.
    250260        $request->set_param( 'menu_order', 1 );
    251261        $response = rest_get_server()->dispatch( $request );
    252262        $data     = $response->get_data();
    253263        $this->assertEqualSets( array( $id4 ), wp_list_pluck( $data, 'id' ) );
    254         // Order by menu order
     264
     265        // Order by 'menu order'.
    255266        $request = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    256267        $request->set_param( 'order', 'asc' );
     
    262273        $this->assertEquals( $id2, $data[2]['id'] );
    263274        $this->assertEquals( $id3, $data[3]['id'] );
    264         // Invalid menu_order should fail
     275
     276        // Invalid 'menu_order' should error.
    265277        $request = new WP_REST_Request( 'GET', '/wp/v2/pages' );
    266278        $request->set_param( 'menu_order', 'top-first' );
Note: See TracChangeset for help on using the changeset viewer.