Changeset 14070 for trunk/tests/phpunit/testcases/groups/functions.php

Timestamp:

11/03/2024 06:19:06 PM (5 months ago)

Author:

espellcaste

Message:

A user is no longer de-authenticated when making REST API requests.

We are introducing a new BP_LoggedIn_User class to fetch data about a BuddyPress logged-in user. This new addition fixes an issue where a user could be de-authenticated when making REST API requests.

Props dcavins, DJPaul, johnjamesjacoby, and imath.

Closes ​https://github.com/buddypress/buddypress/pull/395
See #9229 and #9145
Fixes #7658

File:

• trunk/tests/phpunit/testcases/groups/functions.php (modified) (11 diffs)

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

@@ -61 +61 @@
 
         // Set the current user so the leave group request goes through.
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
         groups_leave_group( $g1, $u2 );
         $this->assertEquals( 1, bp_get_user_meta( $u2, 'total_group_count', true ) );
@@ -79 +79 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -100 +100 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -140 +140 @@
 
         $current_user = bp_loggedin_user_id();
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
 
         $g = self::factory()->group->create( array( 'status' => 'private' ) );
@@ -152 +152 @@
         $this->assertEquals( 1, bp_get_user_meta( $u1, 'total_group_count', true ) );
 
-        self::set_current_user( $current_user );
+        wp_set_current_user( $current_user );
     }
 
@@ -168 +168 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -227 +227 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -248 +248 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -311 +311 @@
 
         // Fool the admin check
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         buddypress()->is_item_admin = true;
 
@@ -382 +382 @@
         ) );
 
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         $g1 = self::factory()->group->create();
 
@@ -389 +389 @@
         $this->assertEquals( 2, groups_get_total_member_count( $g1 ) );
 
-        self::set_current_user( $current_user );
+        wp_set_current_user( $current_user );
     }