Changeset 9562

Timestamp:

02/27/2015 03:18:45 AM (10 years ago)

Author:

boonebgorges

Message:

Ensure a current user is set in test_total_group_count_groups_accept_membership_request().

In versions of WP prior to 4.1, the failure to have a current user was causing
a PHP notice to be thrown.

See #6009.

File:

• trunk/tests/phpunit/testcases/groups/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/groups/functions.php

@@ -110 +110 @@
      */
     public function test_total_group_count_groups_accept_membership_request() {
-        $u = $this->factory->user->create();
-        $g = $this->factory->group->create();
-        groups_send_membership_request( $u, $g );
-
-        groups_accept_membership_request( 0, $u, $g );
-
-        $this->assertEquals( 1, bp_get_user_meta( $u, 'total_group_count', true ) );
+        $u1 = $this->factory->user->create();
+        $u2 = $this->factory->user->create();
+
+        $current_user = bp_loggedin_user_id();
+        $this->set_current_user( $u2 );
+
+        $g = $this->factory->group->create();
+        groups_send_membership_request( $u1, $g );
+
+        groups_accept_membership_request( 0, $u1, $g );
+
+        $this->assertEquals( 1, bp_get_user_meta( $u1, 'total_group_count', true ) );
+
+        $this->set_current_user( $current_user );
     }