Changeset 14070 for trunk/tests/phpunit/testcases/groups/test-group-avatar-controller.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/test-group-avatar-controller.php (modified) (7 diffs)

trunk/tests/phpunit/testcases/groups/test-group-avatar-controller.php

@@ -47 +47 @@
      */
     public function test_get_item() {
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         $request = new WP_REST_Request( 'GET', sprintf( $this->endpoint_url . '/%d/avatar', $this->group_id ) );
@@ -108 +108 @@
         $reset_post  = $_POST;
 
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         add_filter( 'pre_move_uploaded_file', array( $this, 'copy_file' ), 10, 3 );
@@ -178 +178 @@
         $reset_post  = $_POST;
 
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         // Disabling group avatar upload.
@@ -208 +208 @@
      */
     public function test_create_item_empty_image() {
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         $request  = new WP_REST_Request( 'POST', sprintf( $this->endpoint_url . '/%d/avatar', $this->group_id ) );
@@ -230 +230 @@
         $u1 = $this->bp::factory()->user->create();
 
-        $this->bp::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         $request  = new WP_REST_Request( 'POST', sprintf( $this->endpoint_url . '/%d/avatar', REST_TESTS_IMPOSSIBLY_HIGH_NUMBER ) );
@@ -255 +255 @@
      */
     public function test_delete_item_failed() {
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         $request  = new WP_REST_Request( 'DELETE', sprintf( $this->endpoint_url . '/%d/avatar', $this->group_id ) );
@@ -276 +276 @@
      */
     public function test_delete_item_invalid_group() {
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         $request  = new WP_REST_Request( 'DELETE', sprintf( $this->endpoint_url . '/%d/avatar', REST_TESTS_IMPOSSIBLY_HIGH_NUMBER ) );