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

Timestamp:

11/03/2024 06:19:06 PM (2 years 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/friends/functions.php (modified) (11 diffs)

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

@@ -67 +67 @@
                 // accept friendship
                 $old_user = get_current_user_id();
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
                 friends_accept_friendship( friends_get_friendship_id( $u2, $u1 ) );
 
@@ -74 +74 @@
                 $this->assertEquals( array( $u3 ), $requests );
 
-                self::set_current_user( $old_user );
+                wp_set_current_user( $old_user );
         }
 
@@ -124 +124 @@
                 // user 2 withdraws friendship
                 $old_user = get_current_user_id();
-                self::set_current_user( $u2 );
+                wp_set_current_user( $u2 );
                 friends_withdraw_friendship( $u2, $u1 );
 
@@ -131 +131 @@
                 $this->assertEquals( array(), $requests );
 
-                self::set_current_user( $old_user );
+                wp_set_current_user( $old_user );
         }
 
@@ -152 +152 @@
                 // user 1 rejects friendship
                 $old_user = get_current_user_id();
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
                 friends_reject_friendship( friends_get_friendship_id( $u2, $u1 ) );
 
@@ -159 +159 @@
                 $this->assertEquals( array(), $requests );
 
-                self::set_current_user( $old_user );
+                wp_set_current_user( $old_user );
         }
 
@@ -198 +198 @@
 
                 $old_user = get_current_user_id();
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
 
                 $found = array();
@@ -217 +217 @@
                 // clean up
                 $GLOBALS['members_template'] = null;
-                self::set_current_user( $old_user );
+                wp_set_current_user( $old_user );
         }
 
@@ -300 +300 @@
                  * @see bp_friends_filter_user_query_populate_extras()
                  */
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
                 $this->go_to( bp_get_members_directory_permalink() );
                 ob_start();
@@ -315 +315 @@
                  * properly.
                  */
-                self::set_current_user( $u2 );
+                wp_set_current_user( $u2 );
                 friends_accept_friendship( friends_get_friendship_id( $u1, $u2 ) );
 
                 // Afterwards, user 1 decides to cancel friendship.
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
                 friends_remove_friend( $u1, $u2 );
 
@@ -325 +325 @@
                 $this->assertEquals( 'not_friends', BP_Friends_Friendship::check_is_friend( $u1, $u2 ) );
 
-                self::set_current_user( $old_user );
+                wp_set_current_user( $old_user );
         }