Changeset 13980 for trunk/tests/phpunit/testcases/activity/template.php

Timestamp:

07/27/2024 04:30:23 PM (23 months ago)

Author:

espellcaste

Message:

Unit Tests: Conducted a thorough review of existing unit tests files and improved them to ensure robustness.

Props imath.

Closes ​https://github.com/buddypress/buddypress/pull/295
Fixes #9081

File:

• trunk/tests/phpunit/testcases/activity/template.php (modified) (10 diffs)

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

@@ -12 +12 @@
         $u = self::factory()->user->create();
         $original_user = bp_loggedin_user_id();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
 
         $a = self::factory()->activity->create( array(
@@ -24 +24 @@
 
         // Logged-out user can't delete
-        $this->set_current_user( 0 );
+        self::set_current_user( 0 );
         $this->assertFalse( bp_activity_user_can_delete( $activity ) );
 
         // Miscellaneous user can't delete
         $misc_user = self::factory()->user->create( array( 'role' => 'subscriber' ) );
-        $this->set_current_user( $misc_user );
+        self::set_current_user( $misc_user );
         $this->assertFalse( bp_activity_user_can_delete( $activity ) );
 
@@ -43 +43 @@
         $bp->is_single_item = $is_single_item;
         $bp->is_item_admin = $is_item_admin;
-        $this->set_current_user( $original_user );
+        self::set_current_user( $original_user );
     }
 
@@ -53 +53 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
 
         // create an activity update for the user
@@ -70 +70 @@
 
         // reset
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -96 +96 @@
 
         $current_user = bp_loggedin_user_id();
-        $this->set_current_user( $user_id );
+        self::set_current_user( $user_id );
 
         bp_activity_add_user_favorite( $a1, $user_id );
         bp_activity_add_user_favorite( $a2, $user_id );
 
-        $this->set_current_user( $current_user );
+        self::set_current_user( $current_user );
 
         // groan. It sucks that you have to invoke the global
@@ -146 +146 @@
         // save the current user and override logged-in user
         $old_user = get_current_user_id();
-        $this->set_current_user( $u1 );
+        self::set_current_user( $u1 );
 
         $now = time();
@@ -188 +188 @@
         // clean up!
         $activities_template = null;
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -1007 +1007 @@
         $u3 = self::factory()->user->create();
 
-        $this->set_current_user( $u1 );
+        self::set_current_user( $u1 );
 
         $g = self::factory()->group->create( array(
@@ -1061 +1061 @@
         $u3 = self::factory()->user->create();
 
-        $this->set_current_user( $u1 );
+        self::set_current_user( $u1 );
 
         $g = self::factory()->group->create( array(