Changeset 14070 for trunk/tests/phpunit/testcases/members/test-member-avatar-controller.php

Timestamp:

11/03/2024 06:19:06 PM (11 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/members/test-member-avatar-controller.php (modified) (6 diffs)

trunk/tests/phpunit/testcases/members/test-member-avatar-controller.php

@@ -39 +39 @@
         $u1 = $this->bp::factory()->user->create();
 
-        $this->bp::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         $request = new WP_REST_Request( 'GET', sprintf( $this->endpoint_url . '/%d/avatar', $u1 ) );
@@ -103 +103 @@
         $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 );
@@ -166 +166 @@
      */
     public function test_create_item_with_upload_disabled() {
-        $this->bp::set_current_user( $this->user );
+        wp_set_current_user( $this->user );
 
         // Disabling member avatar upload.
@@ -181 +181 @@
      */
     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->user ) );
@@ -205 +205 @@
         $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 ) );
@@ -231 +231 @@
      */
     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->user ) );