Changeset 14070 for trunk/tests/phpunit/testcases/activity/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/activity/functions.php (modified) (9 diffs)

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

@@ -1416 +1416 @@
                 // bp_activity_add_user_favorite() requires a logged-in user.
                 $current_user = bp_loggedin_user_id();
-                self::set_current_user( $u );
+                wp_set_current_user( $u );
 
                 $this->assertTrue( bp_activity_add_user_favorite( $a, $u ) );
@@ -1424 +1424 @@
                 $this->assertEquals( 1, bp_activity_get_meta( $a, 'favorite_count' ) );
 
-                self::set_current_user( $current_user );
+                wp_set_current_user( $current_user );
         }
 
@@ -1437 +1437 @@
                 // bp_activity_add_user_favorite() requires a logged-in user.
                 $current_user = bp_loggedin_user_id();
-                self::set_current_user( $u );
+                wp_set_current_user( $u );
                 $this->assertTrue( bp_activity_add_user_favorite( $a, $u ) );
 
-                self::set_current_user( $current_user );
+                wp_set_current_user( $current_user );
         }
 
@@ -1454 +1454 @@
                 // bp_activity_add_user_favorite() requires a logged-in user.
                 $current_user = bp_loggedin_user_id();
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
 
                 // Only favorite for user 1
@@ -1463 +1463 @@
                 $this->assertEquals( 1, bp_activity_get_meta( $a, 'favorite_count' ) );
 
-                self::set_current_user( $current_user );
+                wp_set_current_user( $current_user );
         }
 
@@ -1476 +1476 @@
                 // bp_activity_add_user_favorite() requires a logged-in user.
                 $current_user = bp_loggedin_user_id();
-                self::set_current_user( $u1 );
+                wp_set_current_user( $u1 );
 
                 // Only favorite for user 1
@@ -1490 +1490 @@
                 $this->assertEquals( 1, bp_activity_get_meta( $a, 'favorite_count' ) );
 
-                self::set_current_user( $current_user );
+                wp_set_current_user( $current_user );
         }
 
@@ -1621 +1621 @@
                 $this->assertTrue( bp_activity_user_can_read( $o, $u ) );
 
-                self::set_current_user( $u2 );
+                wp_set_current_user( $u2 );
                 $this->assertTrue( bp_activity_user_can_read( $o, $u2 ) );
         }
@@ -1659 +1659 @@
                 bp_activity_mark_as_spam( $o );
 
-                self::set_current_user( $u2 );
+                wp_set_current_user( $u2 );
                 $this->assertTrue( bp_activity_user_can_read( $o, $u2 ) );
         }