Changeset 13980 for trunk/tests/phpunit/testcases/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.php

Timestamp:

07/27/2024 04:30:23 PM (6 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/nav/bpCoreMaybeHookNewSubnavScreenFunction.php (modified) (12 diffs)

trunk/tests/phpunit/testcases/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.php

@@ -49 +49 @@
     public function test_user_has_access_false_user_logged_out() {
         $old_current_user = get_current_user_id();
-        $this->set_current_user( 0 );
+        self::set_current_user( 0 );
 
         $subnav_item = array(
@@ -62 +62 @@
         $this->assertSame( $expected, bp_core_maybe_hook_new_subnav_screen_function( $subnav_item ) );
 
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
     }
 
@@ -68 +68 @@
         $u = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -82 +82 @@
         $this->assertSame( bp_members_get_user_url( $u ), $found['redirect_args']['root'] );
 
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
     }
 
@@ -89 +89 @@
         $u2 = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        $this->set_current_user( $u1 );
+        self::set_current_user( $u1 );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -113 +113 @@
 
         // Clean up
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
         buddypress()->default_component = $old_default_component;
         buddypress()->bp_nav = $old_bp_nav;
@@ -125 +125 @@
         $u2 = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        $this->set_current_user( $u1 );
+        self::set_current_user( $u1 );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -150 +150 @@
 
         // Clean up
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
         buddypress()->default_component = $old_default_component;
         buddypress()->bp_nav = $old_bp_nav;
@@ -170 +170 @@
         $g = self::factory()->group->create();
         $old_current_user = get_current_user_id();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -188 +188 @@
 
         // Clean up
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
     }
 
@@ -195 +195 @@
         $g = self::factory()->group->create();
         $old_current_user = get_current_user_id();
-        $this->set_current_user( $u );
+        self::set_current_user( $u );
 
         $group = groups_get_group( $g );
@@ -211 +211 @@
 
         // Clean up
-        $this->set_current_user( $old_current_user );
+        self::set_current_user( $old_current_user );
     }
 }