WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
09/06/2022 10:09:49 PM (4 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.