WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
09/06/2022 10:09:49 PM (3 years ago)
Author:
SergeyBiryukov
Message:

Tests: Use the factory method instead of the property.

This replaces all references to the WP_UnitTestCase_Base::$factory property with static function calls to the WP_UnitTestCase_Base::factory() method.

This is a consistency improvement for the test suite.

Follow up to [35225], [35242], [49603], [54087], [54088].

Props jrf.
See #55652.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/rest-api/rest-users-controller.php

    r53948 r54090  
    341341
    342342        // 3rd page.
    343         $this->factory->user->create();
     343        self::factory()->user->create();
    344344        $total_users++;
    345345        $total_pages++;
    … …  
    433433        wp_set_current_user( self::$user );
    434434
    435         $low_id  = $this->factory->user->create( array( 'display_name' => 'AAAAA' ) );
    436         $mid_id  = $this->factory->user->create( array( 'display_name' => 'NNNNN' ) );
    437         $high_id = $this->factory->user->create( array( 'display_name' => 'ZZZZ' ) );
     435        $low_id  = self::factory()->user->create( array( 'display_name' => 'AAAAA' ) );
     436        $mid_id  = self::factory()->user->create( array( 'display_name' => 'NNNNN' ) );
     437        $high_id = self::factory()->user->create( array( 'display_name' => 'ZZZZ' ) );
    438438
    439439        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    457457        wp_set_current_user( self::$user );
    458458
    459         $low_id  = $this->factory->user->create( array( 'user_url' => 'http://a.com' ) );
    460         $high_id = $this->factory->user->create( array( 'user_url' => 'http://b.com' ) );
     459        $low_id  = self::factory()->user->create( array( 'user_url' => 'http://a.com' ) );
     460        $high_id = self::factory()->user->create( array( 'user_url' => 'http://b.com' ) );
    461461
    462462        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    482482        wp_set_current_user( self::$user );
    483483
    484         $high_id = $this->factory->user->create( array( 'user_nicename' => 'blogin' ) );
    485         $low_id  = $this->factory->user->create( array( 'user_nicename' => 'alogin' ) );
     484        $high_id = self::factory()->user->create( array( 'user_nicename' => 'blogin' ) );
     485        $low_id  = self::factory()->user->create( array( 'user_nicename' => 'alogin' ) );
    486486
    487487        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    507507        wp_set_current_user( self::$user );
    508508
    509         $this->factory->user->create( array( 'user_nicename' => 'burrito' ) );
    510         $this->factory->user->create( array( 'user_nicename' => 'taco' ) );
    511         $this->factory->user->create( array( 'user_nicename' => 'chalupa' ) );
     509        self::factory()->user->create( array( 'user_nicename' => 'burrito' ) );
     510        self::factory()->user->create( array( 'user_nicename' => 'taco' ) );
     511        self::factory()->user->create( array( 'user_nicename' => 'chalupa' ) );
    512512
    513513        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    525525        wp_set_current_user( self::$user );
    526526
    527         $high_id = $this->factory->user->create( array( 'user_email' => 'bemail@gmail.com' ) );
    528         $low_id  = $this->factory->user->create( array( 'user_email' => 'aemail@gmail.com' ) );
     527        $high_id = self::factory()->user->create( array( 'user_email' => 'bemail@gmail.com' ) );
     528        $low_id  = self::factory()->user->create( array( 'user_email' => 'aemail@gmail.com' ) );
    529529
    530530        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    605605        wp_set_current_user( self::$user );
    606606
    607         $id1 = $this->factory->user->create();
    608         $id2 = $this->factory->user->create();
     607        $id1 = self::factory()->user->create();
     608        $id2 = self::factory()->user->create();
    609609
    610610        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    641641        wp_set_current_user( self::$user );
    642642
    643         $id1 = $this->factory->user->create();
    644         $id2 = $this->factory->user->create();
     643        $id1 = self::factory()->user->create();
     644        $id2 = self::factory()->user->create();
    645645
    646646        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    673673        $this->assertCount( 0, $response->get_data() );
    674674
    675         $yolo_id = $this->factory->user->create( array( 'display_name' => 'yololololo' ) );
     675        $yolo_id = self::factory()->user->create( array( 'display_name' => 'yololololo' ) );
    676676
    677677        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    … …  
    680680        $this->assertCount( 1, $response->get_data() );
    681681        // Default to wildcard search.
    682         $adam_id = $this->factory->user->create(
     682        $adam_id = self::factory()->user->create(
    683683            array(
    684684                'role'          => 'author',
    … …  
    698698        wp_set_current_user( self::$user );
    699699
    700         $this->factory->user->create(
     700        self::factory()->user->create(
    701701            array(
    702702                'display_name' => 'foo',
    … …  
    704704            )
    705705        );
    706         $id2 = $this->factory->user->create(
     706        $id2 = self::factory()->user->create(
    707707            array(
    708708                'display_name' => 'Moo',
    … …  
    722722        wp_set_current_user( self::$user );
    723723
    724         $id1 = $this->factory->user->create(
     724        $id1 = self::factory()->user->create(
    725725            array(
    726726                'display_name' => 'Taco',
    … …  
    728728            )
    729729        );
    730         $id2 = $this->factory->user->create(
     730        $id2 = self::factory()->user->create(
    731731            array(
    732732                'display_name' => 'Enchilada',
    … …  
    734734            )
    735735        );
    736         $id3 = $this->factory->user->create(
     736        $id3 = self::factory()->user->create(
    737737            array(
    738738                'display_name' => 'Burrito',
    … …  
    740740            )
    741741        );
    742         $this->factory->user->create(
     742        self::factory()->user->create(
    743743            array(
    744744                'display_name' => 'Hon Pizza',
    … …  
    768768        wp_set_current_user( self::$user );
    769769
    770         $id1 = $this->factory->user->create(
     770        $id1 = self::factory()->user->create(
    771771            array(
    772772                'display_name' => 'Taco',
    … …  
    774774            )
    775775        );
    776         $id2 = $this->factory->user->create(
     776        $id2 = self::factory()->user->create(
    777777            array(
    778778                'display_name' => 'Enchilada',
    … …  
    780780            )
    781781        );
    782         $id3 = $this->factory->user->create(
     782        $id3 = self::factory()->user->create(
    783783            array(
    784784                'display_name' => 'Burrito',
    … …  
    786786            )
    787787        );
    788         $this->factory->user->create(
     788        self::factory()->user->create(
    789789            array(
    790790                'display_name' => 'Hon Pizza',
    … …  
    963963
    964964    public function test_get_item() {
    965         $user_id = $this->factory->user->create();
     965        $user_id = self::factory()->user->create();
    966966
    967967        wp_set_current_user( self::$user );
    … …  
    10301030        $this->allow_user_to_manage_multisite();
    10311031
    1032         $lolz = $this->factory->user->create(
     1032        $lolz = self::factory()->user->create(
    10331033            array(
    10341034                'display_name' => 'lolz',
    … …  
    11151115
    11161116    public function test_get_item_published_author_post() {
    1117         $author_id = $this->factory->user->create(
     1117        $author_id = self::factory()->user->create(
    11181118            array(
    11191119                'role' => 'author',
    … …  
    11211121        );
    11221122
    1123         $post_id = $this->factory->post->create(
     1123        $post_id = self::factory()->post->create(
    11241124            array(
    11251125                'post_author' => $author_id,
    … …  
    11351135
    11361136    public function test_get_item_published_author_pages() {
    1137         $author_id = $this->factory->user->create(
     1137        $author_id = self::factory()->user->create(
    11381138            array(
    11391139                'role' => 'author',
    … …  
    11471147        $this->assertSame( 401, $response->get_status() );
    11481148
    1149         $post_id = $this->factory->post->create(
     1149        $post_id = self::factory()->post->create(
    11501150            array(
    11511151                'post_author' => $author_id,
    … …  
    11591159
    11601160    public function test_get_user_with_edit_context() {
    1161         $user_id = $this->factory->user->create();
     1161        $user_id = self::factory()->user->create();
    11621162
    11631163        $this->allow_user_to_manage_multisite();
    … …  
    11701170
    11711171    public function test_get_item_published_author_wrong_context() {
    1172         $author_id = $this->factory->user->create(
     1172        $author_id = self::factory()->user->create(
    11731173            array(
    11741174                'role' => 'author',
    … …  
    11761176        );
    11771177
    1178         $post_id = $this->factory->post->create(
     1178        $post_id = self::factory()->post->create(
    11791179            array(
    11801180                'post_author' => $author_id,
    … …  
    15551555
    15561556    public function test_update_item() {
    1557         $user_id = $this->factory->user->create(
     1557        $user_id = self::factory()->user->create(
    15581558            array(
    15591559                'user_email' => 'test@example.com',
    … …  
    16211621
    16221622    public function test_update_item_existing_email() {
    1623         $user1 = $this->factory->user->create(
     1623        $user1 = self::factory()->user->create(
    16241624            array(
    16251625                'user_login' => 'test_json_user',
    … …  
    16271627            )
    16281628        );
    1629         $user2 = $this->factory->user->create(
     1629        $user2 = self::factory()->user->create(
    16301630            array(
    16311631                'user_login' => 'test_json_user2',
    … …  
    16851685
    16861686    public function test_update_item_invalid_locale() {
    1687         $user1 = $this->factory->user->create(
     1687        $user1 = self::factory()->user->create(
    16881688            array(
    16891689                'user_login' => 'test_json_user',
    … …  
    17041704
    17051705    public function test_update_item_en_US_locale() {
    1706         $user_id = $this->factory->user->create(
     1706        $user_id = self::factory()->user->create(
    17071707            array(
    17081708                'user_login' => 'test_json_user',
    … …  
    17281728     */
    17291729    public function test_update_item_empty_locale() {
    1730         $user_id = $this->factory->user->create(
     1730        $user_id = self::factory()->user->create(
    17311731            array(
    17321732                'user_login' => 'test_json_user',
    … …  
    17521752
    17531753    public function test_update_item_username_attempt() {
    1754         $user1 = $this->factory->user->create(
     1754        $user1 = self::factory()->user->create(
    17551755            array(
    17561756                'user_login' => 'test_json_user',
    … …  
    17581758            )
    17591759        );
    1760         $user2 = $this->factory->user->create(
     1760        $user2 = self::factory()->user->create(
    17611761            array(
    17621762                'user_login' => 'test_json_user2',
    … …  
    17771777
    17781778    public function test_update_item_existing_nicename() {
    1779         $user1 = $this->factory->user->create(
     1779        $user1 = self::factory()->user->create(
    17801780            array(
    17811781                'user_login' => 'test_json_user',
    … …  
    17831783            )
    17841784        );
    1785         $user2 = $this->factory->user->create(
     1785        $user2 = self::factory()->user->create(
    17861786            array(
    17871787                'user_login' => 'test_json_user2',
    … …  
    18021802
    18031803    public function test_json_update_user() {
    1804         $user_id = $this->factory->user->create(
     1804        $user_id = self::factory()->user->create(
    18051805            array(
    18061806                'user_email' => 'testjson2@example.com',
    … …  
    18471847
    18481848    public function test_update_user_role() {
    1849         $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
     1849        $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
    18501850
    18511851        wp_set_current_user( self::$user );
    … …  
    18681868
    18691869    public function test_update_user_multiple_roles() {
    1870         $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
     1870        $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
    18711871
    18721872        wp_set_current_user( self::$user );
    … …  
    19141914     */
    19151915    public function test_update_user_role_invalid_privilege_deescalation() {
    1916         $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
     1916        $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
    19171917
    19181918        wp_set_current_user( $user_id );
    … …  
    19431943     */
    19441944    public function test_update_user_role_privilege_deescalation_multisite() {
    1945         $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
     1945        $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
    19461946
    19471947        wp_set_current_user( $user_id );
    … …  
    19571957        $this->assertNotEquals( 'administrator', $new_data['roles'][0] );
    19581958
    1959         $user_id = $this->factory->user->create( array( 'role' => 'administrator' ) );
     1959        $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
    19601960
    19611961        wp_set_current_user( $user_id );
    … …  
    20472047     */
    20482048    public function test_update_item_only_roles_as_editor() {
    2049         $user_id = $this->factory->user->create(
     2049        $user_id = self::factory()->user->create(
    20502050            array(
    20512051                'role' => 'author',
    … …  
    20652065     */
    20662066    public function test_update_item_only_roles_as_site_administrator() {
    2067         $user_id = $this->factory->user->create(
     2067        $user_id = self::factory()->user->create(
    20682068            array(
    20692069                'role' => 'author',
    … …  
    20862086     */
    20872087    public function test_update_item_including_roles_and_other_params() {
    2088         $user_id = $this->factory->user->create(
     2088        $user_id = self::factory()->user->create(
    20892089            array(
    20902090                'role' => 'author',
    … …  
    23422342
    23432343    public function test_delete_item() {
    2344         $user_id = $this->factory->user->create( array( 'display_name' => 'Deleted User' ) );
     2344        $user_id = self::factory()->user->create( array( 'display_name' => 'Deleted User' ) );
    23452345
    23462346        $this->allow_user_to_manage_multisite();
    … …  
    23672367
    23682368    public function test_delete_item_no_trash() {
    2369         $user_id = $this->factory->user->create( array( 'display_name' => 'Deleted User' ) );
     2369        $user_id = self::factory()->user->create( array( 'display_name' => 'Deleted User' ) );
    23702370
    23712371        $this->allow_user_to_manage_multisite();
    … …  
    23972397
    23982398    public function test_delete_current_item() {
    2399         $user_id = $this->factory->user->create(
     2399        $user_id = self::factory()->user->create(
    24002400            array(
    24012401                'role'         => 'administrator',
    … …  
    24262426
    24272427    public function test_delete_current_item_no_trash() {
    2428         $user_id = $this->factory->user->create(
     2428        $user_id = self::factory()->user->create(
    24292429            array(
    24302430                'role'         => 'administrator',
    … …  
    24592459
    24602460    public function test_delete_user_without_permission() {
    2461         $user_id = $this->factory->user->create();
     2461        $user_id = self::factory()->user->create();
    24622462
    24632463        $this->allow_user_to_manage_multisite();
    … …  
    24972497
    24982498        // Test with a new user, to avoid any complications.
    2499         $user_id     = $this->factory->user->create();
    2500         $reassign_id = $this->factory->user->create();
    2501         $test_post   = $this->factory->post->create(
     2499        $user_id     = self::factory()->user->create();
     2500        $reassign_id = self::factory()->user->create();
     2501        $test_post   = self::factory()->post->create(
    25022502            array(
    25032503                'post_author' => $user_id,
    … …  
    25312531
    25322532    public function test_delete_user_invalid_reassign_id() {
    2533         $user_id = $this->factory->user->create();
     2533        $user_id = self::factory()->user->create();
    25342534
    25352535        $this->allow_user_to_manage_multisite();
    … …  
    25522552
    25532553    public function test_delete_user_invalid_reassign_passed_as_string() {
    2554         $user_id = $this->factory->user->create();
     2554        $user_id = self::factory()->user->create();
    25552555
    25562556        $this->allow_user_to_manage_multisite();
    … …  
    25672567
    25682568    public function test_delete_user_reassign_passed_as_boolean_false_trashes_post() {
    2569         $user_id = $this->factory->user->create();
    2570 
    2571         $this->allow_user_to_manage_multisite();
    2572 
    2573         wp_set_current_user( self::$user );
    2574 
    2575         $test_post = $this->factory->post->create(
     2569        $user_id = self::factory()->user->create();
     2570
     2571        $this->allow_user_to_manage_multisite();
     2572
     2573        wp_set_current_user( self::$user );
     2574
     2575        $test_post = self::factory()->post->create(
    25762576            array(
    25772577                'post_author' => $user_id,
    … …  
    25952595
    25962596    public function test_delete_user_reassign_passed_as_string_false_trashes_post() {
    2597         $user_id = $this->factory->user->create();
    2598 
    2599         $this->allow_user_to_manage_multisite();
    2600 
    2601         wp_set_current_user( self::$user );
    2602 
    2603         $test_post = $this->factory->post->create(
     2597        $user_id = self::factory()->user->create();
     2598
     2599        $this->allow_user_to_manage_multisite();
     2600
     2601        wp_set_current_user( self::$user );
     2602
     2603        $test_post = self::factory()->post->create(
    26042604            array(
    26052605                'post_author' => $user_id,
    … …  
    26232623
    26242624    public function test_delete_user_reassign_passed_as_empty_string_trashes_post() {
    2625         $user_id = $this->factory->user->create();
    2626 
    2627         $this->allow_user_to_manage_multisite();
    2628 
    2629         wp_set_current_user( self::$user );
    2630 
    2631         $test_post = $this->factory->post->create(
     2625        $user_id = self::factory()->user->create();
     2626
     2627        $this->allow_user_to_manage_multisite();
     2628
     2629        wp_set_current_user( self::$user );
     2630
     2631        $test_post = self::factory()->post->create(
    26322632            array(
    26332633                'post_author' => $user_id,
    … …  
    26512651
    26522652    public function test_delete_user_reassign_passed_as_0_reassigns_author() {
    2653         $user_id = $this->factory->user->create();
    2654 
    2655         $this->allow_user_to_manage_multisite();
    2656 
    2657         wp_set_current_user( self::$user );
    2658 
    2659         $test_post = $this->factory->post->create(
     2653        $user_id = self::factory()->user->create();
     2654
     2655        $this->allow_user_to_manage_multisite();
     2656
     2657        wp_set_current_user( self::$user );
     2658
     2659        $test_post = self::factory()->post->create(
    26602660            array(
    26612661                'post_author' => $user_id,
    … …  
    28322832    public function test_get_item_from_different_site_as_site_administrator() {
    28332833        switch_to_blog( self::$site );
    2834         $user_id = $this->factory->user->create(
     2834        $user_id = self::factory()->user->create(
    28352835            array(
    28362836                'role' => 'author',
    … …  
    28522852    public function test_get_item_from_different_site_as_network_administrator() {
    28532853        switch_to_blog( self::$site );
    2854         $user_id = $this->factory->user->create(
     2854        $user_id = self::factory()->user->create(
    28552855            array(
    28562856                'role' => 'author',
    … …  
    28722872    public function test_update_item_from_different_site_as_site_administrator() {
    28732873        switch_to_blog( self::$site );
    2874         $user_id = $this->factory->user->create(
     2874        $user_id = self::factory()->user->create(
    28752875            array(
    28762876                'role' => 'author',
    … …  
    28942894    public function test_update_item_from_different_site_as_network_administrator() {
    28952895        switch_to_blog( self::$site );
    2896         $user_id = $this->factory->user->create(
     2896        $user_id = self::factory()->user->create(
    28972897            array(
    28982898                'role' => 'author',
    … …  
    29162916    public function test_delete_item_from_different_site_as_site_administrator() {
    29172917        switch_to_blog( self::$site );
    2918         $user_id = $this->factory->user->create(
     2918        $user_id = self::factory()->user->create(
    29192919            array(
    29202920                'role' => 'author',
    … …  
    29382938    public function test_delete_item_from_different_site_as_network_administrator() {
    29392939        switch_to_blog( self::$site );
    2940         $user_id = $this->factory->user->create(
     2940        $user_id = self::factory()->user->create(
    29412941            array(
    29422942                'role' => 'author',
Note: See TracChangeset for help on using the changeset viewer.