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

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

@@ -20 +20 @@
         $this->u1 = self::factory()->user->create();
         $this->u2 = self::factory()->user->create();
-        self::set_current_user( $this->u1 );
+        wp_set_current_user( $this->u1 );
 
         /**
@@ -32 +32 @@
 
     public function tear_down() {
-        self::set_current_user( $this->current_user );
+        wp_set_current_user( $this->current_user );
         $this->set_permalink_structure( $this->permalink_structure );
 
@@ -90 +90 @@
 
         // Log out
-        self::set_current_user( 0 );
+        wp_set_current_user( 0 );
 
         // Go to the activity permalink page
@@ -110 +110 @@
         $this->assertEquals( array( $this->a1 ), wp_list_pluck( $notifications, 'item_id' ) );
 
-        self::set_current_user( $this->u1 );
+        wp_set_current_user( $this->u1 );
     }
 
@@ -129 +129 @@
 
         // Switch user
-        self::set_current_user( $this->u2 );
+        wp_set_current_user( $this->u2 );
 
         // Go to the activity permalink page
@@ -149 +149 @@
         $this->assertEquals( array( $this->a1 ), wp_list_pluck( $notifications, 'item_id' ) );
 
-        self::set_current_user( $this->u1 );
+        wp_set_current_user( $this->u1 );
     }
 
@@ -202 +202 @@
 
         // Log out
-        self::set_current_user( 0 );
+        wp_set_current_user( 0 );
 
         // Go to the My Activity page
@@ -223 +223 @@
 
         // clean up
-        self::set_current_user( $this->u1 );
+        wp_set_current_user( $this->u1 );
     }
 
@@ -242 +242 @@
 
         // Log out
-        self::set_current_user( $this->u2 );
+        wp_set_current_user( $this->u2 );
 
         // Go to the My Activity page
@@ -263 +263 @@
 
         // clean up
-        self::set_current_user( $this->u1 );
+        wp_set_current_user( $this->u1 );
     }
 
@@ -444 +444 @@
 
         // Attempt to mark 'comment_reply' notifications as read for user 2.
-        self::set_current_user( $this->u2 );
+        wp_set_current_user( $this->u2 );
         foreach ( $u2_notifications as $i => $n ) {
             $n = bp_activity_format_notifications( $n->component_action, $n->item_id, $n->secondary_item_id, 1, 'array', $n->id );
@@ -471 +471 @@
         $u3 = self::factory()->user->create();
 
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         $userdata = get_userdata( $u1 );
 
@@ -487 +487 @@
         ) );
 
-        self::set_current_user( $u2 );
+        wp_set_current_user( $u2 );
         $userdata = get_userdata( $u2 );
 
@@ -503 +503 @@
         self::factory()->comment->update_object( $c1, array( 'comment_approved' => 1 ) );
 
-        self::set_current_user( $u3 );
+        wp_set_current_user( $u3 );
         $userdata = get_userdata( $u3 );
 
@@ -543 +543 @@
 
         // Reset.
-        self::set_current_user( $old_user );
+        wp_set_current_user( $old_user );
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
         remove_filter( 'comment_flood_filter', '__return_false' );