Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
07/27/2024 04:30:23 PM (10 months ago)
Author:
espellcaste
Message:

Unit Tests: Conducted a thorough review of existing unit tests files and improved them to ensure robustness.

Props imath.

Closes ​https://github.com/buddypress/buddypress/pull/295
Fixes #9081

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/core/invitations.php

    r13082 r13980  
    1111        $old_current_user = get_current_user_id();
    1212
    13         $u1 = $this->factory->user->create();
    14         $u2 = $this->factory->user->create();
    15         $u3 = $this->factory->user->create();
    16         $this->set_current_user( $u1 );
     13        $u1 = self::factory()->user->create();
     14        $u2 = self::factory()->user->create();
     15        $u3 = self::factory()->user->create();
     16        self::set_current_user( $u1 );
    1717
    1818        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    3636        $this->assertEqualSets( array( $i1, $i2 ), $invites );
    3737
    38         $this->set_current_user( $old_current_user );
     38        self::set_current_user( $old_current_user );
    3939    }
    4040
    … …  
    4242        $old_current_user = get_current_user_id();
    4343
    44         $u1 = $this->factory->user->create();
    45         $u2 = $this->factory->user->create();
    46         $this->set_current_user( $u1 );
     44        $u1 = self::factory()->user->create();
     45        $u2 = self::factory()->user->create();
     46        self::set_current_user( $u1 );
    4747
    4848        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    6060        $this->assertEquals( $i1, $i2 );
    6161
    62         $this->set_current_user( $old_current_user );
     62        self::set_current_user( $old_current_user );
    6363    }
    6464
    … …  
    6666        $old_current_user = get_current_user_id();
    6767
    68         $u1 = $this->factory->user->create();
    69         $u2 = $this->factory->user->create();
    70         $u3 = $this->factory->user->create();
    71         $this->set_current_user( $u1 );
     68        $u1 = self::factory()->user->create();
     69        $u2 = self::factory()->user->create();
     70        $u3 = self::factory()->user->create();
     71        self::set_current_user( $u1 );
    7272
    7373        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    9696        $this->assertEqualSets( array( $i1 ), wp_list_pluck( $invites, 'id' ) );
    9797
    98         $this->set_current_user( $old_current_user );
     98        self::set_current_user( $old_current_user );
    9999    }
    100100
    … …  
    102102        $old_current_user = get_current_user_id();
    103103
    104         $u1 = $this->factory->user->create();
    105         $u2 = $this->factory->user->create();
    106         $u3 = $this->factory->user->create();
    107         $this->set_current_user( $u1 );
     104        $u1 = self::factory()->user->create();
     105        $u2 = self::factory()->user->create();
     106        $u3 = self::factory()->user->create();
     107        self::set_current_user( $u1 );
    108108
    109109        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    132132        $this->assertEqualSets( array(), wp_list_pluck( $invites, 'id' ) );
    133133
    134         $this->set_current_user( $old_current_user );
     134        self::set_current_user( $old_current_user );
    135135    }
    136136
    … …  
    138138        $old_current_user = get_current_user_id();
    139139
    140         $u1 = $this->factory->user->create();
    141         $u2 = $this->factory->user->create();
    142         $u3 = $this->factory->user->create();
    143         $this->set_current_user( $u1 );
     140        $u1 = self::factory()->user->create();
     141        $u2 = self::factory()->user->create();
     142        $u3 = self::factory()->user->create();
     143        self::set_current_user( $u1 );
    144144
    145145        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    173173        $this->assertEqualSets( array( $i1, $r1 ), $invites );
    174174
    175         $this->set_current_user( $old_current_user );
     175        self::set_current_user( $old_current_user );
    176176    }
    177177
    … …  
    179179        $old_current_user = get_current_user_id();
    180180
    181         $u1 = $this->factory->user->create();
    182         $this->set_current_user( $u1 );
     181        $u1 = self::factory()->user->create();
     182        self::set_current_user( $u1 );
    183183
    184184        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    200200        $this->assertEqualSets( array( $r1, $r2 ), $requests );
    201201
    202         $this->set_current_user( $old_current_user );
     202        self::set_current_user( $old_current_user );
    203203    }
    204204
    … …  
    208208        $invites_class = new BPTest_Invitation_Manager_Extension();
    209209
    210         $u1 = $this->factory->user->create();
    211         $this->set_current_user( $u1 );
     210        $u1 = self::factory()->user->create();
     211        self::set_current_user( $u1 );
    212212
    213213        // Create a couple of requests.
    … …  
    220220        $this->assertFalse( $invites_class->add_request( $request_args ) );
    221221
    222         $this->set_current_user( $old_current_user );
     222        self::set_current_user( $old_current_user );
    223223    }
    224224
    … …  
    226226        $old_current_user = get_current_user_id();
    227227
    228         $u1 = $this->factory->user->create();
    229         $u2 = $this->factory->user->create();
    230         $this->set_current_user( $u1 );
     228        $u1 = self::factory()->user->create();
     229        $u2 = self::factory()->user->create();
     230        self::set_current_user( $u1 );
    231231
    232232        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    258258        $this->assertEqualSets( array( $r1, $i1 ), $invites );
    259259
    260         $this->set_current_user( $old_current_user );
     260        self::set_current_user( $old_current_user );
    261261    }
    262262
    … …  
    264264        $old_current_user = get_current_user_id();
    265265
    266         $u1 = $this->factory->user->create();
    267         $u2 = $this->factory->user->create();
    268         $this->set_current_user( $u1 );
     266        $u1 = self::factory()->user->create();
     267        $u2 = self::factory()->user->create();
     268        self::set_current_user( $u1 );
    269269
    270270        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    286286        $this->assertEquals( 1, $invite->invite_sent );
    287287
    288         $this->set_current_user( $old_current_user );
     288        self::set_current_user( $old_current_user );
    289289    }
    290290
    … …  
    292292        $old_current_user = get_current_user_id();
    293293
    294         $u1 = $this->factory->user->create();
    295         $u2 = $this->factory->user->create();
    296         $this->set_current_user( $u1 );
     294        $u1 = self::factory()->user->create();
     295        $u2 = self::factory()->user->create();
     296        self::set_current_user( $u1 );
    297297
    298298        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    331331        $this->assertEqualSets( array( $i2 ), $invites );
    332332
    333         $this->set_current_user( $old_current_user );
     333        self::set_current_user( $old_current_user );
    334334    }
    335335
    … …  
    337337        $old_current_user = get_current_user_id();
    338338
    339         $u1 = $this->factory->user->create();
    340         $this->set_current_user( $u1 );
     339        $u1 = self::factory()->user->create();
     340        self::set_current_user( $u1 );
    341341
    342342        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    353353        $this->assertEquals( $time, $req->date_modified );
    354354
    355         $this->set_current_user( $old_current_user );
     355        self::set_current_user( $old_current_user );
    356356    }
    357357
    … …  
    359359        $old_current_user = get_current_user_id();
    360360
    361         $u1 = $this->factory->user->create();
    362         $u2 = $this->factory->user->create();
    363         $this->set_current_user( $u1 );
     361        $u1 = self::factory()->user->create();
     362        $u2 = self::factory()->user->create();
     363        self::set_current_user( $u1 );
    364364
    365365        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    379379        $this->assertEquals( $time, $inv->date_modified );
    380380
    381         $this->set_current_user( $old_current_user );
     381        self::set_current_user( $old_current_user );
    382382    }
    383383
    … …  
    385385        $old_current_user = get_current_user_id();
    386386
    387         $u1 = $this->factory->user->create();
    388         $u2 = $this->factory->user->create();
    389         $u3 = $this->factory->user->create();
    390         $this->set_current_user( $u1 );
     387        $u1 = self::factory()->user->create();
     388        $u2 = self::factory()->user->create();
     389        $u3 = self::factory()->user->create();
     390        self::set_current_user( $u1 );
    391391
    392392        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    429429        $this->assertEquals( array( $i3, $i1, $i2 ), $invites );
    430430
    431         $this->set_current_user( $old_current_user );
     431        self::set_current_user( $old_current_user );
    432432    }
    433433}
Note: See TracChangeset for help on using the changeset viewer.