Changeset 12429 for trunk/tests/phpunit/testcases/groups/functions.php

Timestamp:

08/12/2019 08:42:49 PM (7 years ago)

Author:

dcavins

Message:

Introduce BP_Groups_Invitation_Manager.

• Add BP_Groups_Invitation_Manager, a new class that extends BP_Invitation_Manager for the purpose of using the new API to manage group invitations and requests.
• Migrate group invitations to the new table on update, via bp_groups_migrate_invitations().
• Update group invitation function logic to make use of BP_Groups_Invitation_Manager.

See #6210.

File:

• trunk/tests/phpunit/testcases/groups/functions.php (modified) (10 diffs)

trunk/tests/phpunit/testcases/groups/functions.php

@@ -13 +13 @@
                 self::$group_ids = $factory->group->create_many( 2, array(
                         'creator_id' => self::$user_ids[2],
+                        'status'     => 'private'
                 ) );
         }
@@ -115 +116 @@
                 $u1 = self::factory()->user->create();
                 $u2 = self::factory()->user->create();
-                $g = self::factory()->group->create();
+                $g = self::factory()->group->create( array( 'status' => 'private', 'creator_id' => $u2 ) );
+
                 groups_invite_user( array(
                         'user_id' => $u1,
                         'group_id' => $g,
                         'inviter_id' => $u2,
-                ) );
-
-                groups_accept_invite( $u2, $g );
-
-                $this->assertEquals( 1, bp_get_user_meta( $u2, 'total_group_count', true ) );
+                        'send_invite' => 1,
+                ) );
+
+                groups_accept_invite( $u1, $g );
+
+                $this->assertEquals( 1, bp_get_user_meta( $u1, 'total_group_count', true ) );
         }
 
@@ -138 +141 @@
                 $this->set_current_user( $u2 );
 
-                $g = self::factory()->group->create();
-                groups_send_membership_request( $u1, $g );
+                $g = self::factory()->group->create( array( 'status' => 'private' ) );
+                groups_send_membership_request( array(
+                        'user_id'       => $u1,
+                        'group_id'      => $g,
+                ) );
 
                 groups_accept_membership_request( 0, $u1, $g );
@@ -242 +248 @@
                 $u1 = self::factory()->user->create();
                 $u2 = self::factory()->user->create();
-                $g = self::factory()->group->create( array( 'creator_id' => $u1 ) );
+                $g = self::factory()->group->create( array( 'status' => 'private', 'creator_id' => $u1 ) );
                 groups_invite_user( array(
-                        'user_id' => $u1,
-                        'group_id' => $g,
-                        'inviter_id' => $u2,
+                        'user_id'     => $u2,
+                        'group_id'    => $g,
+                        'inviter_id'  => $u1,
+                        'send_invite' => 1,
                 ) );
 
@@ -261 +268 @@
                 $u1 = self::factory()->user->create();
                 $u2 = self::factory()->user->create();
-                $g = self::factory()->group->create( array( 'creator_id' => $u1 ) );
-
-                groups_send_membership_request( $u2, $g );
+                $g = self::factory()->group->create( array( 'status' => 'private', 'creator_id' => $u1 ) );
+
+                groups_send_membership_request( array(
+                        'user_id'       => $u2,
+                        'group_id'      => $g,
+                ) );
                 groups_accept_membership_request( 0, $u2, $g );
 
@@ -633 +643 @@
                 $u1 = self::factory()->user->create();
                 $u2 = self::factory()->user->create();
-                $g = self::factory()->group->create( array( 'creator_id' => $u1 ) );
+                $g = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
 
                 // create invitation
@@ -640 +650 @@
                         'group_id'   => $g,
                         'inviter_id' => $u1,
-                ) );
-
-                // send the invite
-                // this function is imperative to set the 'invite_sent' flag in the DB
-                // why is this separated from groups_invite_user()?
-                // @see groups_screen_group_invite()
-                groups_send_invites( $u1, $g );
+                        'send_invite' => 1
+                ) );
 
                 // assert invite count
@@ -891 +896 @@
          */
         public function test_bp_groups_pending_requests_personal_data_exporter() {
-                groups_send_membership_request( self::$user_ids[0], self::$group_ids[0] );
+                groups_send_membership_request( array(
+                        'user_id'       => self::$user_ids[0],
+                        'group_id'      => self::$group_ids[0],
+                ) );
 
                 $test_user = new WP_User( self::$user_ids[0] );
@@ -908 +916 @@
         public function test_bp_groups_pending_sent_invitations_personal_data_exporter() {
                 groups_invite_user( array(
-                        'user_id'    => self::$user_ids[1],
-                        'group_id'   => self::$group_ids[0],
-                        'inviter_id' => self::$user_ids[0],
-                ) );
-
-                $test_user = new WP_User( self::$user_ids[0] );
+                        'user_id'     => self::$user_ids[0],
+                        'group_id'    => self::$group_ids[0],
+                        'inviter_id'  => self::$user_ids[2],
+                        'send_invite' => 1,
+                ) );
+
+                $test_user = new WP_User( self::$user_ids[2] );
 
                 $actual = bp_groups_pending_sent_invitations_personal_data_exporter( $test_user->user_email, 1 );
@@ -930 +939 @@
                         'user_id'    => self::$user_ids[0],
                         'group_id'   => self::$group_ids[0],
-                        'inviter_id' => self::$user_ids[1],
+                        'inviter_id' => self::$user_ids[2],
                 ) );