Changeset 13980 for trunk/tests/phpunit/testcases/core/suggestions.php

Timestamp:

07/27/2024 04:30:23 PM (14 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/core/suggestions.php (modified) (5 diffs)

trunk/tests/phpunit/testcases/core/suggestions.php

@@ -112 +112 @@
     public function set_up() {
         parent::set_up();
-        $this->set_current_user( self::$current_user );
+        self::set_current_user( self::$current_user );
     }
 
     public function tear_down() {
+        self::set_current_user( self::$old_user_id );
         parent::tear_down();
-        $this->set_current_user( self::$old_user_id );
     }
 
@@ -269 +269 @@
 
         // "alpaca red" is in the hidden group
-        $this->set_current_user( self::$user_ids['alpaca red'] );
+        self::set_current_user( self::$user_ids['alpaca red'] );
         $suggestions = bp_core_get_suggestions( array(
             'group_id' => self::$group_ids['hidden'],
@@ -289 +289 @@
 
         // "caterpillar" is in the private group
-        $this->set_current_user( self::$user_ids['caterpillar'] );
+        self::set_current_user( self::$user_ids['caterpillar'] );
         $suggestions = bp_core_get_suggestions( array(
             'group_id' => self::$group_ids['private'],
@@ -328 +328 @@
 
 
-        $this->set_current_user( self::$user_ids['caterpillar'] );
+        self::set_current_user( self::$user_ids['caterpillar'] );
 
         // "cat" is in the private group, so won't show up here.
@@ -359 +359 @@
 
 
-        $this->set_current_user( self::$user_ids['alpaca red'] );
+        self::set_current_user( self::$user_ids['alpaca red'] );
 
         // "alpaca red" is in the hidden group, so won't show up here.