Changeset 14070 for trunk/tests/phpunit/testcases/groups/class-bp-groups-member.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/class-bp-groups-member.php (modified) (12 diffs)

trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php

@@ -222 +222 @@
         $this->assertTrue( bp_groups_user_can_send_invites( $g, $u_siteadmin ) );
         // Falling back to current user
-        self::set_current_user( $u_members );
+        wp_set_current_user( $u_members );
         $this->assertTrue( bp_groups_user_can_send_invites( $g, null ) );
 
@@ -233 +233 @@
         $this->assertTrue( bp_groups_user_can_send_invites( $g, $u_siteadmin ) );
         // Falling back to current user
-        self::set_current_user( $u_members );
+        wp_set_current_user( $u_members );
         $this->assertFalse( bp_groups_user_can_send_invites( $g, null ) );
-        self::set_current_user( $u_mods );
+        wp_set_current_user( $u_mods );
         $this->assertTrue( bp_groups_user_can_send_invites( $g, null ) );
 
@@ -246 +246 @@
         $this->assertTrue( bp_groups_user_can_send_invites( $g, $u_siteadmin ) );
         // Falling back to current user
-        self::set_current_user( $u_mods );
+        wp_set_current_user( $u_mods );
         $this->assertFalse( bp_groups_user_can_send_invites( $g, null ) );
-        self::set_current_user( $u_admins );
+        wp_set_current_user( $u_admins );
         $this->assertTrue( bp_groups_user_can_send_invites( $g, null ) );
 
@@ -264 +264 @@
         $this->assertTrue( bp_groups_user_can_send_invites( null, $u_mods ) );
 
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -680 +680 @@
         $g = self::factory()->group->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         groups_join_group( $g );
         $membership_id = groups_is_user_member( $u1, $g );
         $this->assertTrue( is_numeric( $membership_id ) && $membership_id > 0 );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -754 +754 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
         groups_leave_group( $g, $u2 );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before - 1, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -774 +774 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
         groups_leave_group( $g );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before - 1, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -794 +794 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         groups_leave_group( $g, $u2 );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before - 1, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -816 +816 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         groups_leave_group( $g, $u2 );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before - 1, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -836 +836 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         groups_leave_group( $g, $u1 );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -858 +858 @@
 
         $before = groups_get_total_member_count( $g );
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         groups_leave_group( $g, $u1 );
         $after = groups_get_total_member_count( $g );
 
         $this->assertEquals( $before - 1, $after );
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -935 +935 @@
         ) );
 
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
         $groups = groups_get_invites_for_user();
         $this->assertEqualSets( array( $g1, $g2, $g3 ), wp_list_pluck( $groups['groups'], 'id' ) );
 
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }