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/groups/functions.php

    r9471 r9561  
    66 */
    77class BP_Tests_Groups_Functions extends BP_UnitTestCase {
    8     protected $old_current_user_id = 0;
    9 
    10     public function setUp() {
    11         parent::setUp();
    12         $this->old_current_user = get_current_user_id();
    13     }
    14 
    15     public function tearDown() {
    16         parent::tearDown();
    17         $this->set_current_user( $this->old_current_user );
    18     }
    19 
    208    public function test_creating_new_group_as_authenticated_user() {
    219        $u = $this->factory->user->create();
Note: See TracChangeset for help on using the changeset viewer.