WordPress.org
Get WordPress

Make WordPress Core


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