Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
02/27/2015 03:00:42 AM (9 years ago)
Author:
boonebgorges
Message:

Don't create a current user by default in most unit tests.

The default state of our tests should be as a logged-out user. BP does pretty
different things with different kinds of logged-in users, so individual tests
should decide for themselves what the current user status should be.

Moreover, the creation of fixture users is fairly resource-intensive. Not
creating users when they're not needed makes the test suite run 5-10% faster.

See #6009.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/testcases/friends/class-bp-friends-friendship.php

    r9525 r9561  
    55 */
    66class BP_Tests_BP_Friends_Friendship_TestCases extends BP_UnitTestCase {
    7     protected $old_current_user = 0;
    8 
    9     public function setUp() {
    10         parent::setUp();
    11 
    12         $this->old_current_user = get_current_user_id();
    13         $this->set_current_user( $this->factory->user->create( array( 'role' => 'subscriber' ) ) );
    14     }
    15 
    16     public function tearDown() {
    17         parent::tearDown();
    18         $this->set_current_user( $this->old_current_user );
    19     }
    20 
    217    public function test_search_friends() {
    228        $u1 = $this->factory->user->create();
Note: See TracChangeset for help on using the changeset viewer.