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

Timestamp:

07/27/2024 04:30:23 PM (11 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/blogs/activity.php (modified) (16 diffs)

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

@@ -300 +300 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -338 +338 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -348 +348 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -386 +386 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -396 +396 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -446 +446 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -456 +456 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -503 +503 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -513 +513 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -560 +560 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -570 +570 @@
         $old_user = get_current_user_id();
         $u = self::factory()->user->create();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $userdata = get_userdata( $u );
 
@@ -614 +614 @@
         remove_action( 'bp_activity_before_save', array( $this, 'set_activity_to_spam' ) );
 
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -626 +626 @@
         $reset_at = isset( $GLOBALS['activities_template'] ) ? $GLOBALS['activities_template'] : null;
 
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
 
         // let's use activity comments instead of single "new_blog_comment" activity items
@@ -673 +673 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
         $GLOBALS['activities_template'] = $reset_at;
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }
 
@@ -685 +685 @@
         $reset_at = isset( $GLOBALS['activities_template'] ) ? $GLOBALS['activities_template'] : null;
 
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
 
         // let's use activity comments instead of single "new_blog_comment" activity items
@@ -731 +731 @@
         remove_filter( 'bp_disable_blogforum_comments', '__return_false' );
         $GLOBALS['activities_template'] = $reset_at;
-        $this->set_current_user( $old_user );
+        self::set_current_user( $old_user );
     }