Changeset 9139 for trunk/tests/phpunit/testcases/core/nav.php

Timestamp:

11/14/2014 02:01:18 PM (10 years ago)

Author:

boonebgorges

Message:

In automated tests, move user creation to a proper factory method.

BP user creation requires that a couple of extra pieces of data be set up
(last activity, display name, etc). So we previously had a wrapper in
BP_UnitTestCase called create_user() that performed the extra setup.
However, the wrapper made it impossible to use create_user() statically,
because the user_login and user_email iterator was not persistent from call to
call. (The create_user() syntax is also a break with the rest of our unit
tests, which is not ideal.)

This changeset introduces BP_UnitTest_Factory_For_User, which reproduces the
customizations of create_user(), but in a proper factory method. All
instances of create_user() throughout the test suite have also been replaced.

See #6009.

File:

• trunk/tests/phpunit/testcases/core/nav.php (modified) (5 diffs)

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

@@ -198 +198 @@
      */
     public function test_maybe_hook_new_subnav_screen_function_user_has_access_false_user_logged_in_my_profile() {
-        $u = $this->create_user();
+        $u = $this->factory->user->create();
         $old_current_user = get_current_user_id();
         $this->set_current_user( $u );
@@ -220 +220 @@
      */
     public function test_maybe_hook_new_subnav_screen_function_user_has_access_false_user_logged_in_others_profile_default_component_accessible() {
-        $u1 = $this->create_user();
-        $u2 = $this->create_user();
+        $u1 = $this->factory->user->create();
+        $u2 = $this->factory->user->create();
         $old_current_user = get_current_user_id();
         $this->set_current_user( $u1 );
@@ -255 +255 @@
      */
     public function test_maybe_hook_new_subnav_screen_function_user_has_access_false_user_logged_in_others_profile_default_component_not_accessible() {
-        $u1 = $this->create_user();
-        $u2 = $this->create_user();
+        $u1 = $this->factory->user->create();
+        $u2 = $this->factory->user->create();
         $old_current_user = get_current_user_id();
         $this->set_current_user( $u1 );
@@ -290 +290 @@
      */
     public function test_maybe_hook_new_subnav_screen_function_user_has_access_false_user_logged_in_group() {
-        $u = $this->create_user();
+        $u = $this->factory->user->create();
         $g = $this->factory->group->create();
         $old_current_user = get_current_user_id();
@@ -319 +319 @@
      */
     public function test_maybe_hook_new_subnav_screen_function_user_has_access_false_user_logged_in_group_no_redirect_url_provided() {
-        $u = $this->create_user();
+        $u = $this->factory->user->create();
         $g = $this->factory->group->create();
         $old_current_user = get_current_user_id();