Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
11/03/2017 07:44:10 PM (9 years ago)
Author:
boonebgorges
Message:

Use static factory method throughout PHPUnit tests.

See #7620.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/members/functions.php

    r11217 r11737  
    1515
    1616                // Create an admin for testing
    17                 $admin_user = $this->factory->user->create( array( 'role' => 'administrator' ) );
     17                $admin_user = self::factory()->user->create( array( 'role' => 'administrator' ) );
    1818                $this->grant_super_admin( $admin_user );
    1919
    2020                // 1. Admin can delete user account
    2121                $this->set_current_user( $admin_user );
    22                 $user1 = $this->factory->user->create( array( 'role' => 'subscriber' ) );
     22                $user1 = self::factory()->user->create( array( 'role' => 'subscriber' ) );
    2323                bp_core_delete_account( $user1 );
    2424                $maybe_user = new WP_User( $user1 );
    … …  
    2828
    2929                // 2. Admin cannot delete superadmin account
    30                 $user2 = $this->factory->user->create( array( 'role' => 'administrator' ) );
     30                $user2 = self::factory()->user->create( array( 'role' => 'administrator' ) );
    3131                $this->grant_super_admin( $user2 );
    3232                bp_core_delete_account( $user2 );
    … …  
    3636
    3737                // User cannot delete other's account
    38                 $user3 = $this->factory->user->create( array( 'role' => 'subscriber' ) );
    39                 $user4 = $this->factory->user->create( array( 'role' => 'subscriber' ) );
     38                $user3 = self::factory()->user->create( array( 'role' => 'subscriber' ) );
     39                $user4 = self::factory()->user->create( array( 'role' => 'subscriber' ) );
    4040                $this->set_current_user( $user3 );
    4141                bp_core_delete_account( $user4 );
    … …  
    5555        public function test_bp_core_get_user_domain_after_directory_page_update() {
    5656                // Generate user
    57                 $user_id = $this->factory->user->create( array( 'role' => 'subscriber' ) );
     57                $user_id = self::factory()->user->create( array( 'role' => 'subscriber' ) );
    5858
    5959                // Set object cache first for user domain
    … …  
    8484         */
    8585        public function test_bp_core_get_user_displayname_translate_username() {
    86                 $u = $this->factory->user->create();
     86                $u = self::factory()->user->create();
    8787
    8888                $user = new WP_User( $u );
    … …  
    108108                buddypress()->active_components['xprofile'] = '1';
    109109
    110                 $u = $this->factory->user->create( array(
     110                $u = self::factory()->user->create( array(
    111111                        'display_name' => 'Foo',
    112112                ) );
    … …  
    128128                buddypress()->active_components['xprofile'] = '1';
    129129
    130                 $u = $this->factory->user->create();
     130                $u = self::factory()->user->create();
    131131                xprofile_set_field_data( 1, $u, 'Foo Foo' );
    132132
    … …  
    145145                buddypress()->active_components['xprofile'] = '1';
    146146
    147                 $u = $this->factory->user->create();
     147                $u = self::factory()->user->create();
    148148                xprofile_set_field_data( 1, $u, 'Foo Foo' );
    149149
    … …  
    163163                $bp->active_components['xprofile'] = '1';
    164164
    165                 $u = $this->factory->user->create( array(
     165                $u = self::factory()->user->create( array(
    166166                        'display_name' => 'Foo Foo',
    167167                ) );
    … …  
    194194         */
    195195        public function test_bp_core_get_user_displaynames_all_uncached() {
    196                 $u1 = $this->factory->user->create();
    197                 $u2 = $this->factory->user->create();
     196                $u1 = self::factory()->user->create();
     197                $u2 = self::factory()->user->create();
    198198
    199199                xprofile_set_field_data( 1, $u1, 'Foo' );
    … …  
    212212         */
    213213        public function test_bp_core_get_user_displaynames_one_not_in_xprofile() {
    214                 $u1 = $this->factory->user->create();
    215                 $u2 = $this->factory->user->create( array(
     214                $u1 = self::factory()->user->create();
     215                $u2 = self::factory()->user->create( array(
    216216                        'display_name' => 'Bar',
    217217                ) );
    … …  
    239239         */
    240240        public function test_bp_core_get_user_displaynames_one_in_cache() {
    241                 $u1 = $this->factory->user->create();
     241                $u1 = self::factory()->user->create();
    242242                xprofile_set_field_data( 1, $u1, 'Foo' );
    243243
    … …  
    258258         */
    259259        public function test_bp_members_migrate_signups_standard() {
    260                 $u = $this->factory->user->create();
     260                $u = self::factory()->user->create();
    261261                $u_obj = new WP_User( $u );
    262262
    … …  
    292292         */
    293293        public function test_bp_members_migrate_signups_activation_key_but_user_status_0() {
    294                 $u = $this->factory->user->create();
     294                $u = self::factory()->user->create();
    295295                $u_obj = new WP_User( $u );
    296296
    … …  
    323323         */
    324324        public function test_bp_members_migrate_signups_no_activation_key_but_user_status_2() {
    325                 $u = $this->factory->user->create();
     325                $u = self::factory()->user->create();
    326326                $u_obj = new WP_User( $u );
    327327
    … …  
    353353                // We explicitly do not want last_activity created, so use the
    354354                // WP factory methods
    355                 $u1 = $this->factory->user->create();
    356                 $u2 = $this->factory->user->create();
    357                 $u3 = $this->factory->user->create();
     355                $u1 = self::factory()->user->create();
     356                $u2 = self::factory()->user->create();
     357                $u3 = self::factory()->user->create();
    358358
    359359                $time = time();
    … …  
    414414                // We explicitly do not want last_activity created, so use the
    415415                // WP factory methods
    416                 $user = $this->factory->user->create();
     416                $user = self::factory()->user->create();
    417417                $time = date( 'Y-m-d H:i:s', time() - 50 );
    418418
    … …  
    449449                $displayed_user = $bp->displayed_user;
    450450
    451                 $u1 = $this->factory->user->create();
     451                $u1 = self::factory()->user->create();
    452452                $bp->displayed_user->id = $u1;
    453453
    … …  
    477477                $displayed_user = $bp->displayed_user;
    478478
    479                 $u1 = $this->factory->user->create();
     479                $u1 = self::factory()->user->create();
    480480                $bp->displayed_user->id = $u1;
    481481
    … …  
    505505                $displayed_user = $bp->displayed_user;
    506506
    507                 $u1 = $this->factory->user->create();
     507                $u1 = self::factory()->user->create();
    508508                $bp->displayed_user->id = $u1;
    509509
    … …  
    528528                add_filter( 'make_spam_user', array( $this, 'notification_filter_callback' ) );
    529529
    530                 $u1 = $this->factory->user->create();
     530                $u1 = self::factory()->user->create();
    531531                $n = bp_core_process_spammer_status( $u1, 'spam' );
    532532
    … …  
    540540                add_filter( 'make_ham_user', array( $this, 'notification_filter_callback' ) );
    541541
    542                 $u1 = $this->factory->user->create();
     542                $u1 = self::factory()->user->create();
    543543                $n = bp_core_process_spammer_status( $u1, 'ham' );
    544544
    … …  
    552552                add_filter( 'bp_make_spam_user', array( $this, 'notification_filter_callback' ) );
    553553
    554                 $u1 = $this->factory->user->create();
     554                $u1 = self::factory()->user->create();
    555555                $n = bp_core_process_spammer_status( $u1, 'spam' );
    556556
    … …  
    564564                add_filter( 'bp_make_ham_user', array( $this, 'notification_filter_callback' ) );
    565565
    566                 $u1 = $this->factory->user->create();
     566                $u1 = self::factory()->user->create();
    567567                $n = bp_core_process_spammer_status( $u1, 'ham' );
    568568
    … …  
    597597
    598598                $signups = array( 'no-blog' =>
    599                         array( 'signup_id' => $this->factory->signup->create( array(
     599                        array( 'signup_id' => self::factory()->signup->create( array(
    600600                                        'user_login'     => 'noblog',
    601601                                        'user_email'     => 'noblog@example.com',
    … …  
    611611
    612612                if ( is_multisite() ) {
    613                         $signups['ms-blog'] = array( 'signup_id' => $this->factory->signup->create( array(
     613                        $signups['ms-blog'] = array( 'signup_id' => self::factory()->signup->create( array(
    614614                                        'user_login'     => 'msblog',
    615615                                        'user_email'     => 'msblog@example.com',
Note: See TracChangeset for help on using the changeset viewer.