WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/05/2019 08:41:12 PM (6 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-tags-controller.php

    r46586 r46657  
    1717    protected static $subscriber;
    1818
     19    protected static $tag_ids    = array();
     20    protected static $total_tags = 30;
     21    protected static $per_page   = 50;
     22
    1923    public static function wpSetUpBeforeClass( $factory ) {
    2024        self::$superadmin    = $factory->user->create(
    … …  
    4448            )
    4549        );
     50
    4651        if ( is_multisite() ) {
    4752            update_site_option( 'site_admins', array( 'superadmin' ) );
     53        }
     54
     55        // Set up tags for pagination tests.
     56        for ( $i = 0; $i < self::$total_tags; $i++ ) {
     57            $tag_ids[] = $factory->tag->create(
     58                array(
     59                    'name' => "Tag {$i}",
     60                )
     61            );
    4862        }
    4963    }
    … …  
    5468        self::delete_user( self::$editor );
    5569        self::delete_user( self::$subscriber );
     70
     71        // Remove tags for pagination tests.
     72        foreach ( self::$tag_ids as $tag_id ) {
     73            wp_delete_term( $tag_id, 'post_tag' );
     74        }
    5675    }
    5776
    … …  
    155174    public function test_get_items() {
    156175        $this->factory->tag->create();
    157         $request  = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     176
     177        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     178        $request->set_param( 'per_page', self::$per_page );
    158179        $response = rest_get_server()->dispatch( $request );
    159180        $this->check_get_taxonomy_terms_response( $response );
    … …  
    162183    public function test_get_items_invalid_permission_for_context() {
    163184        wp_set_current_user( 0 );
     185
    164186        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    165187        $request->set_param( 'context', 'edit' );
    … …  
    172194        $tag1    = $this->factory->tag->create( array( 'name' => 'Season 5' ) );
    173195        $tag2    = $this->factory->tag->create( array( 'name' => 'The Be Sharps' ) );
     196
    174197        wp_set_object_terms( $post_id, array( $tag1, $tag2 ), 'post_tag' );
     198
    175199        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    176200        $request->set_param( 'hide_empty', true );
    … …  
    180204        $this->assertEquals( 'Season 5', $data[0]['name'] );
    181205        $this->assertEquals( 'The Be Sharps', $data[1]['name'] );
    182         // invalid value should fail
     206
     207        // Invalid 'hide_empty' should error.
    183208        $request->set_param( 'hide_empty', 'nothanks' );
    184209        $response = rest_get_server()->dispatch( $request );
    … …  
    187212
    188213    public function test_get_items_include_query() {
    189         $id1     = $this->factory->tag->create();
    190         $id2     = $this->factory->tag->create();
    191         $id3     = $this->factory->tag->create();
    192         $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    193         // Orderby=>asc
    194         $request->set_param( 'include', array( $id3, $id1 ) );
     214        $id1 = $this->factory->tag->create();
     215        $id2 = $this->factory->tag->create();
     216
     217        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     218
     219        // 'orderby' => 'asc'.
     220        $request->set_param( 'include', array( $id2, $id1 ) );
    195221        $response = rest_get_server()->dispatch( $request );
    196222        $data     = $response->get_data();
    197223        $this->assertEquals( 2, count( $data ) );
    198224        $this->assertEquals( $id1, $data[0]['id'] );
    199         // Orderby=>include
     225
     226        // 'orderby' => 'include'.
    200227        $request->set_param( 'orderby', 'include' );
    201228        $response = rest_get_server()->dispatch( $request );
    202229        $data     = $response->get_data();
    203230        $this->assertEquals( 2, count( $data ) );
    204         $this->assertEquals( $id3, $data[0]['id'] );
    205         // Include invalid value shoud fail
     231        $this->assertEquals( $id2, $data[0]['id'] );
     232
     233        // Invalid 'include' should error.
    206234        $request->set_param( 'include', array( 'myterm' ) );
    207235        $response = rest_get_server()->dispatch( $request );
    … …  
    210238
    211239    public function test_get_items_exclude_query() {
    212         $id1      = $this->factory->tag->create();
    213         $id2      = $this->factory->tag->create();
    214         $request  = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     240        $id1 = $this->factory->tag->create();
     241        $id2 = $this->factory->tag->create();
     242
     243        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     244        $request->set_param( 'per_page', self::$per_page );
    215245        $response = rest_get_server()->dispatch( $request );
    216246        $data     = $response->get_data();
    217         $this->assertTrue( in_array( $id1, wp_list_pluck( $data, 'id' ), true ) );
    218         $this->assertTrue( in_array( $id2, wp_list_pluck( $data, 'id' ), true ) );
     247        $ids      = wp_list_pluck( $data, 'id' );
     248        $this->assertTrue( in_array( $id1, $ids, true ) );
     249        $this->assertTrue( in_array( $id2, $ids, true ) );
     250
    219251        $request->set_param( 'exclude', array( $id2 ) );
    220252        $response = rest_get_server()->dispatch( $request );
    221253        $data     = $response->get_data();
    222         $this->assertTrue( in_array( $id1, wp_list_pluck( $data, 'id' ), true ) );
    223         $this->assertFalse( in_array( $id2, wp_list_pluck( $data, 'id' ), true ) );
    224         // Invalid exclude value should fail
     254        $ids      = wp_list_pluck( $data, 'id' );
     255        $this->assertTrue( in_array( $id1, $ids, true ) );
     256        $this->assertFalse( in_array( $id2, $ids, true ) );
     257
     258        // Invalid 'exclude' should error.
    225259        $request->set_param( 'exclude', array( 'invalid' ) );
    226260        $response = rest_get_server()->dispatch( $request );
    … …  
    229263
    230264    public function test_get_items_offset_query() {
    231         $id1     = $this->factory->tag->create();
    232         $id2     = $this->factory->tag->create();
    233         $id3     = $this->factory->tag->create();
    234         $id4     = $this->factory->tag->create();
    235         $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     265        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     266        $request->set_param( 'per_page', self::$per_page );
    236267        $request->set_param( 'offset', 1 );
    237268        $response = rest_get_server()->dispatch( $request );
    238         $this->assertCount( 3, $response->get_data() );
    239         // 'offset' works with 'per_page'
     269        $this->assertCount( self::$total_tags - 1, $response->get_data() );
     270
     271        // 'offset' works with 'per_page'.
    240272        $request->set_param( 'per_page', 2 );
    241273        $response = rest_get_server()->dispatch( $request );
    242274        $this->assertCount( 2, $response->get_data() );
    243         // 'offset' takes priority over 'page'
     275
     276        // 'offset' takes priority over 'page'.
    244277        $request->set_param( 'page', 3 );
    245278        $response = rest_get_server()->dispatch( $request );
    246279        $this->assertCount( 2, $response->get_data() );
    247         // 'offset' invalid value shoudl fail
     280
     281        // Invalid 'offset' should error.
    248282        $request->set_param( 'offset', 'moreplease' );
    249283        $response = rest_get_server()->dispatch( $request );
    … …  
    254288    public function test_get_items_orderby_args() {
    255289        $tag1 = $this->factory->tag->create( array( 'name' => 'Apple' ) );
    256         $tag2 = $this->factory->tag->create( array( 'name' => 'Banana' ) );
     290        $tag2 = $this->factory->tag->create( array( 'name' => 'Zucchini' ) );
     291
    257292        /*
    258293         * Tests:
    … …  
    269304        $data = $response->get_data();
    270305        $this->assertEquals( 1, count( $data ) );
    271         $this->assertEquals( 'Banana', $data[0]['name'] );
     306        $this->assertEquals( 'Zucchini', $data[0]['name'] );
     307
    272308        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    273309        $request->set_param( 'orderby', 'name' );
    … …  
    279315        $this->assertEquals( 2, count( $data ) );
    280316        $this->assertEquals( 'Apple', $data[0]['name'] );
    281         // Invalid orderby should fail.
     317
     318        // Invalid 'orderby' should error.
    282319        $request->set_param( 'orderby', 'invalid' );
    283320        $response = rest_get_server()->dispatch( $request );
    … …  
    289326        $tag1 = $this->factory->tag->create( array( 'name' => 'Apple' ) );
    290327        $tag2 = $this->factory->tag->create( array( 'name' => 'Banana' ) );
    291         // defaults to orderby=name, order=asc
     328
     329        // Defaults to 'orderby' => 'name', 'order' => 'asc'.
    292330        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    293331        $response = rest_get_server()->dispatch( $request );
    … …  
    297335        $this->assertEquals( 'Banana', $data[1]['name'] );
    298336        $this->assertEquals( 'Cantaloupe', $data[2]['name'] );
    299         // orderby=id, with default order=asc
     337
     338        // 'orderby' => 'id', with default 'order' => 'asc'.
    300339        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    301340        $request->set_param( 'orderby', 'id' );
    … …  
    303342        $this->assertEquals( 200, $response->get_status() );
    304343        $data = $response->get_data();
    305         $this->assertEquals( 'Cantaloupe', $data[0]['name'] );
    306         $this->assertEquals( 'Apple', $data[1]['name'] );
    307         $this->assertEquals( 'Banana', $data[2]['name'] );
    308         // orderby=id, order=desc
     344        $this->assertEquals( 'Tag 0', $data[0]['name'] );
     345        $this->assertEquals( 'Tag 1', $data[1]['name'] );
     346        $this->assertEquals( 'Tag 2', $data[2]['name'] );
     347
     348        // 'orderby' => 'id', 'order' => 'desc'.
    309349        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    310350        $request->set_param( 'orderby', 'id' );
    … …  
    339379        $tag2    = $this->factory->tag->create( array( 'name' => 'Marvel' ) );
    340380        $this->factory->tag->create( array( 'name' => 'Dark Horse' ) );
     381
    341382        wp_set_object_terms( $post_id, array( $tag1, $tag2 ), 'post_tag' );
    342383
    … …  
    350391        $this->assertEquals( 'DC', $data[0]['name'] );
    351392
    352         // Invalid post should error.
     393        // Invalid 'post' should error.
    353394        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    354395        $request->set_param( 'post', 'invalid-post' );
    … …  
    359400    public function test_get_terms_post_args_paging() {
    360401        $post_id = $this->factory->post->create();
    361         $tag_ids = array();
    362 
    363         for ( $i = 0; $i < 30; $i++ ) {
    364             $tag_ids[] = $this->factory->tag->create(
    365                 array(
    366                     'name' => "Tag {$i}",
    367                 )
    368             );
    369         }
    370         wp_set_object_terms( $post_id, $tag_ids, 'post_tag' );
     402
     403        wp_set_object_terms( $post_id, self::$tag_ids, 'post_tag' );
    371404
    372405        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    … …  
    433466        );
    434467        $post_id = $this->factory->post->create();
     468
    435469        wp_set_object_terms( $post_id, array( $term1, $term2 ), 'batman' );
    436470
    … …  
    448482        $tag1 = $this->factory->tag->create( array( 'name' => 'Apple' ) );
    449483        $tag2 = $this->factory->tag->create( array( 'name' => 'Banana' ) );
     484
    450485        /*
    451486         * Tests:
    … …  
    459494        $this->assertEquals( 1, count( $data ) );
    460495        $this->assertEquals( 'Apple', $data[0]['name'] );
     496
    461497        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    462498        $request->set_param( 'search', 'Garbage' );
    … …  
    468504
    469505    public function test_get_items_slug_arg() {
    470         $tag1    = $this->factory->tag->create( array( 'name' => 'Apple' ) );
    471         $tag2    = $this->factory->tag->create( array( 'name' => 'Banana' ) );
     506        $tag1 = $this->factory->tag->create( array( 'name' => 'Apple' ) );
     507        $tag2 = $this->factory->tag->create( array( 'name' => 'Banana' ) );
     508
    472509        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    473510        $request->set_param( 'slug', 'apple' );
    … …  
    484521        $id3 = $this->factory->tag->create( array( 'name' => 'Burrito' ) );
    485522        $this->factory->tag->create( array( 'name' => 'Pizza' ) );
     523
    486524        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    487525        $request->set_param(
    … …  
    506544        $id3 = $this->factory->tag->create( array( 'name' => 'Burrito' ) );
    507545        $this->factory->tag->create( array( 'name' => 'Pizza' ) );
     546
    508547        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    509548        $request->set_param( 'slug', 'taco,burrito, enchilada' );
    … …  
    537576
    538577    public function test_get_terms_pagination_headers() {
    539         // Start of the index
    540         for ( $i = 0; $i < 50; $i++ ) {
    541             $this->factory->tag->create(
    542                 array(
    543                     'name' => "Tag {$i}",
    544                 )
    545             );
    546         }
     578        $total_tags  = self::$total_tags;
     579        $total_pages = (int) ceil( $total_tags / 10 );
     580
     581        // Start of the index.
    547582        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    548583        $response = rest_get_server()->dispatch( $request );
    549584        $headers  = $response->get_headers();
    550         $this->assertEquals( 50, $headers['X-WP-Total'] );
    551         $this->assertEquals( 5, $headers['X-WP-TotalPages'] );
     585        $this->assertEquals( $total_tags, $headers['X-WP-Total'] );
     586        $this->assertEquals( $total_pages, $headers['X-WP-TotalPages'] );
    552587        $next_link = add_query_arg(
    553588            array(
    … …  
    558593        $this->assertFalse( stripos( $headers['Link'], 'rel="prev"' ) );
    559594        $this->assertContains( '<' . $next_link . '>; rel="next"', $headers['Link'] );
    560         // 3rd page
    561         $this->factory->tag->create(
    562             array(
    563                 'name' => 'Tag 51',
    564             )
    565         );
     595
     596        // 3rd page.
     597        $this->factory->tag->create();
     598        $total_tags++;
     599        $total_pages++;
    566600        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    567601        $request->set_param( 'page', 3 );
    568602        $response = rest_get_server()->dispatch( $request );
    569603        $headers  = $response->get_headers();
    570         $this->assertEquals( 51, $headers['X-WP-Total'] );
    571         $this->assertEquals( 6, $headers['X-WP-TotalPages'] );
     604        $this->assertEquals( $total_tags, $headers['X-WP-Total'] );
     605        $this->assertEquals( $total_pages, $headers['X-WP-TotalPages'] );
    572606        $prev_link = add_query_arg(
    573607            array(
    … …  
    584618        );
    585619        $this->assertContains( '<' . $next_link . '>; rel="next"', $headers['Link'] );
    586         // Last page
    587         $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    588         $request->set_param( 'page', 6 );
     620
     621        // Last page.
     622        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     623        $request->set_param( 'page', $total_pages );
    589624        $response = rest_get_server()->dispatch( $request );
    590625        $headers  = $response->get_headers();
    591         $this->assertEquals( 51, $headers['X-WP-Total'] );
    592         $this->assertEquals( 6, $headers['X-WP-TotalPages'] );
     626        $this->assertEquals( $total_tags, $headers['X-WP-Total'] );
     627        $this->assertEquals( $total_pages, $headers['X-WP-TotalPages'] );
    593628        $prev_link = add_query_arg(
    594629            array(
    595                 'page' => 5,
     630                'page' => $total_pages - 1,
    596631            ),
    597632            rest_url( 'wp/v2/tags' )
    … …  
    599634        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
    600635        $this->assertFalse( stripos( $headers['Link'], 'rel="next"' ) );
    601         // Out of bounds
    602         $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
    603         $request->set_param( 'page', 8 );
     636
     637        // Out of bounds.
     638        $request = new WP_REST_Request( 'GET', '/wp/v2/tags' );
     639        $request->set_param( 'page', 100 );
    604640        $response = rest_get_server()->dispatch( $request );
    605641        $headers  = $response->get_headers();
    606         $this->assertEquals( 51, $headers['X-WP-Total'] );
    607         $this->assertEquals( 6, $headers['X-WP-TotalPages'] );
     642        $this->assertEquals( $total_tags, $headers['X-WP-Total'] );
     643        $this->assertEquals( $total_pages, $headers['X-WP-TotalPages'] );
    608644        $prev_link = add_query_arg(
    609645            array(
    610                 'page' => 6,
     646                'page' => $total_pages,
    611647            ),
    612648            rest_url( 'wp/v2/tags' )
    … …  
    624660
    625661    public function test_get_item() {
    626         $id       = $this->factory->tag->create();
     662        $id = $this->factory->tag->create();
     663
    627664        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $id );
    628665        $response = rest_get_server()->dispatch( $request );
    … …  
    634671     */
    635672    public function test_get_item_meta() {
    636         $id       = $this->factory->tag->create();
     673        $id = $this->factory->tag->create();
     674
    637675        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $id );
    638676        $response = rest_get_server()->dispatch( $request );
    … …  
    655693     */
    656694    public function test_get_item_meta_registered_for_different_taxonomy() {
    657         $id       = $this->factory->tag->create();
     695        $id = $this->factory->tag->create();
     696
    658697        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $id );
    659698        $response = rest_get_server()->dispatch( $request );
    … …  
    673712    public function test_get_item_invalid_permission_for_context() {
    674713        $id = $this->factory->tag->create();
     714
    675715        wp_set_current_user( 0 );
     716
    676717        $request = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $id );
    677718        $request->set_param( 'context', 'edit' );
    … …  
    696737    public function test_get_item_incorrect_taxonomy() {
    697738        register_taxonomy( 'robin', 'post' );
    698         $term1    = $this->factory->term->create(
     739        $term1 = $this->factory->term->create(
    699740            array(
    700741                'name'     => 'Cape',
    … …  
    702743            )
    703744        );
     745
    704746        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $term1 );
    705747        $response = rest_get_server()->dispatch( $request );
    … …  
    709751    public function test_create_item() {
    710752        wp_set_current_user( self::$administrator );
     753
    711754        $request = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    712755        $request->set_param( 'name', 'My Awesome Term' );
    … …  
    725768    public function test_create_item_contributor() {
    726769        wp_set_current_user( self::$contributor );
     770
    727771        $request = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    728772        $request->set_param( 'name', 'My Awesome Term' );
    … …  
    741785    public function test_create_item_incorrect_permissions() {
    742786        wp_set_current_user( self::$subscriber );
     787
    743788        $request = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    744789        $request->set_param( 'name', 'Incorrect permissions' );
    … …  
    749794    public function test_create_item_missing_arguments() {
    750795        wp_set_current_user( self::$administrator );
     796
    751797        $request  = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    752798        $response = rest_get_server()->dispatch( $request );
    … …  
    766812    public function test_create_item_with_meta() {
    767813        wp_set_current_user( self::$administrator );
     814
    768815        $request = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    769816        $request->set_param( 'name', 'My Awesome Term' );
    … …  
    780827    public function test_create_item_with_meta_wrong_id() {
    781828        wp_set_current_user( self::$administrator );
     829
    782830        $existing_tag_id = $this->factory->tag->create( array( 'name' => 'My Not So Awesome Term' ) );
    783         $request         = new WP_REST_Request( 'POST', '/wp/v2/tags' );
     831
     832        $request = new WP_REST_Request( 'POST', '/wp/v2/tags' );
    784833        $request->set_param( 'name', 'My Awesome Term' );
    785834        $request->set_param( 'meta', array( 'test_tag_single' => 'hello' ) );
    … …  
    797846    public function test_update_item() {
    798847        wp_set_current_user( self::$administrator );
     848
    799849        $orig_args = array(
    800850            'name'        => 'Original Name',
    … …  
    802852            'slug'        => 'original-slug',
    803853        );
    804         $term      = get_term_by( 'id', $this->factory->tag->create( $orig_args ), 'post_tag' );
    805         $request   = new WP_REST_Request( 'POST', '/wp/v2/tags/' . $term->term_id );
     854
     855        $term = get_term_by( 'id', $this->factory->tag->create( $orig_args ), 'post_tag' );
     856
     857        $request = new WP_REST_Request( 'POST', '/wp/v2/tags/' . $term->term_id );
    806858        $request->set_param( 'name', 'New Name' );
    807859        $request->set_param( 'description', 'New Description' );
    … …  
    828880    public function test_update_item_no_change() {
    829881        wp_set_current_user( self::$administrator );
     882
    830883        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
    831884
    832         $request = new WP_REST_Request( 'PUT', '/wp/v2/tags/' . $term->term_id );
    833 
     885        $request  = new WP_REST_Request( 'PUT', '/wp/v2/tags/' . $term->term_id );
    834886        $response = rest_get_server()->dispatch( $request );
    835887        $this->assertEquals( 200, $response->get_status() );
    … …  
    847899    public function test_update_item_invalid_term() {
    848900        wp_set_current_user( self::$administrator );
     901
    849902        $request = new WP_REST_Request( 'POST', '/wp/v2/tags/' . REST_TESTS_IMPOSSIBLY_HIGH_NUMBER );
    850903        $request->set_param( 'name', 'Invalid Term' );
    … …  
    855908    public function test_update_item_incorrect_permissions() {
    856909        wp_set_current_user( self::$subscriber );
    857         $term    = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     910
     911        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     912
    858913        $request = new WP_REST_Request( 'POST', '/wp/v2/tags/' . $term->term_id );
    859914        $request->set_param( 'name', 'Incorrect permissions' );
    … …  
    867922    public function test_update_item_with_edit_term_cap_granted() {
    868923        wp_set_current_user( self::$subscriber );
    869         $term    = $this->factory->tag->create_and_get();
     924
     925        $term = $this->factory->tag->create_and_get();
     926
    870927        $request = new WP_REST_Request( 'POST', '/wp/v2/tags/' . $term->term_id );
    871928        $request->set_param( 'name', 'New Name' );
    … …  
    892949    public function test_update_item_with_edit_term_cap_revoked() {
    893950        wp_set_current_user( self::$administrator );
    894         $term    = $this->factory->tag->create_and_get();
     951
     952        $term = $this->factory->tag->create_and_get();
     953
    895954        $request = new WP_REST_Request( 'POST', '/wp/v2/tags/' . $term->term_id );
    896955        $request->set_param( 'name', 'New Name' );
    … …  
    912971    public function test_update_item_parent_non_hierarchical_taxonomy() {
    913972        wp_set_current_user( self::$administrator );
     973
    914974        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
    915975
    … …  
    9601020    public function test_tag_roundtrip_as_editor() {
    9611021        wp_set_current_user( self::$editor );
     1022
    9621023        $this->assertEquals( ! is_multisite(), current_user_can( 'unfiltered_html' ) );
    9631024        $this->verify_tag_roundtrip(
    … …  
    9751036    public function test_tag_roundtrip_as_editor_html() {
    9761037        wp_set_current_user( self::$editor );
     1038
    9771039        if ( is_multisite() ) {
    9781040            $this->assertFalse( current_user_can( 'unfiltered_html' ) );
    … …  
    10041066    public function test_tag_roundtrip_as_superadmin() {
    10051067        wp_set_current_user( self::$superadmin );
     1068
    10061069        $this->assertTrue( current_user_can( 'unfiltered_html' ) );
    10071070        $this->verify_tag_roundtrip(
    … …  
    10191082    public function test_tag_roundtrip_as_superadmin_html() {
    10201083        wp_set_current_user( self::$superadmin );
     1084
    10211085        $this->assertTrue( current_user_can( 'unfiltered_html' ) );
    10221086        $this->verify_tag_roundtrip(
    … …  
    10341098    public function test_delete_item() {
    10351099        wp_set_current_user( self::$administrator );
    1036         $term    = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1100
     1101        $term = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1102
    10371103        $request = new WP_REST_Request( 'DELETE', '/wp/v2/tags/' . $term->term_id );
    10381104        $request->set_param( 'force', true );
    … …  
    10461112    public function test_delete_item_no_trash() {
    10471113        wp_set_current_user( self::$administrator );
     1114
    10481115        $term = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
    10491116
    … …  
    10591126    public function test_delete_item_invalid_term() {
    10601127        wp_set_current_user( self::$administrator );
     1128
    10611129        $request  = new WP_REST_Request( 'DELETE', '/wp/v2/tags/' . REST_TESTS_IMPOSSIBLY_HIGH_NUMBER );
    10621130        $response = rest_get_server()->dispatch( $request );
    … …  
    10661134    public function test_delete_item_incorrect_permissions() {
    10671135        wp_set_current_user( self::$subscriber );
    1068         $term     = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     1136
     1137        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     1138
    10691139        $request  = new WP_REST_Request( 'DELETE', '/wp/v2/tags/' . $term->term_id );
    10701140        $response = rest_get_server()->dispatch( $request );
    … …  
    10771147    public function test_delete_item_with_delete_term_cap_granted() {
    10781148        wp_set_current_user( self::$subscriber );
    1079         $term    = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1149
     1150        $term = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1151
    10801152        $request = new WP_REST_Request( 'DELETE', '/wp/v2/tags/' . $term->term_id );
    10811153        $request->set_param( 'force', true );
    … …  
    11031175    public function test_delete_item_with_delete_term_cap_revoked() {
    11041176        wp_set_current_user( self::$administrator );
    1105         $term    = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1177
     1178        $term = get_term_by( 'id', $this->factory->tag->create( array( 'name' => 'Deleted Tag' ) ), 'post_tag' );
     1179
    11061180        $request = new WP_REST_Request( 'DELETE', '/wp/v2/tags/' . $term->term_id );
    11071181        $request->set_param( 'force', true );
    … …  
    11221196
    11231197    public function test_prepare_item() {
    1124         $term     = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     1198        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
     1199
    11251200        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $term->term_id );
    11261201        $response = rest_get_server()->dispatch( $request );
    … …  
    11961271        $this->assertEquals( $schema, $data['schema']['properties']['my_custom_int'] );
    11971272
    1198         $tag_id  = $this->factory->tag->create();
    1199         $request = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $tag_id );
    1200 
     1273        $tag_id = $this->factory->tag->create();
     1274
     1275        $request  = new WP_REST_Request( 'GET', '/wp/v2/tags/' . $tag_id );
    12011276        $response = rest_get_server()->dispatch( $request );
    12021277        $this->assertArrayHasKey( 'my_custom_int', $response->data );
    … …  
    12251300
    12261301        wp_set_current_user( self::$administrator );
     1302
    12271303        $tag_id = $this->factory->tag->create();
     1304
    12281305        // Check for error on update.
    12291306        $request = new WP_REST_Request( 'POST', sprintf( '/wp/v2/tags/%d', $tag_id ) );
Note: See TracChangeset for help on using the changeset viewer.