Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
08/12/2019 08:43:23 PM (6 years ago)
Author:
dcavins
Message:

Update BP_Groups_Member for Invitation API compatibility.

  • Update logic in BP_Groups_Member to avoid direct table lookups and instead use request and invitation functions to fetch and delete invitations.

See #6210.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php

    r12276 r12431  
    2121    }
    2222
    23     public static function invite_user_to_group( $user_id, $group_id, $inviter_id ) {
    24         $invite                = new BP_Groups_Member;
    25         $invite->group_id      = $group_id;
    26         $invite->user_id       = $user_id;
    27         $invite->date_modified = bp_core_current_time();
    28         $invite->inviter_id    = $inviter_id;
    29         $invite->is_confirmed  = 0;
    30         $invite->invite_sent   = 1;
    31 
    32         $invite->save();
    33         return $invite->id;
    34     }
    35 
    36     public static function create_group_membership_request( $user_id, $group_id ) {
    37         $request                = new BP_Groups_Member;
    38         $request->group_id      = $group_id;
    39         $request->user_id       = $user_id;
    40         $request->date_modified = bp_core_current_time();
    41         $request->inviter_id    = 0;
    42         $request->is_confirmed  = 0;
    43 
    44         $request->save();
    45         return $request->id;
    46     }
    47 
    4823    public function test_get_recently_joined_with_filter() {
    4924        $g1 = self::factory()->group->create( array(
    … …  
    134109
    135110    public function test_get_invites_with_exclude() {
     111        $u1 = self::factory()->user->create();
     112        $u2 = self::factory()->user->create();
    136113        $g1 = self::factory()->group->create( array(
    137             'name' => 'RC Cola',
     114            'status' => 'private',
     115            'creator_id' => $u1
    138116        ) );
    139117        $g2 = self::factory()->group->create( array(
    140             'name' => 'Pepsi',
    141         ) );
    142 
    143         $u1 = self::factory()->user->create();
    144         $u2 = self::factory()->user->create();
    145         self::add_user_to_group( $u1, $g1 );
    146         self::add_user_to_group( $u1, $g2 );
    147         self::invite_user_to_group( $u2, $g1, $u1 );
    148         self::invite_user_to_group( $u2, $g2, $u1 );
     118            'status' => 'private',
     119            'creator_id' => $u1
     120        ) );
     121
     122        groups_invite_user( array(
     123            'user_id' => $u2,
     124            'group_id' => $g1,
     125            'inviter_id' => $u1,
     126            'send_invite' => 1,
     127        ) );
     128        groups_invite_user( array(
     129            'user_id' => $u2,
     130            'group_id' => $g2,
     131            'inviter_id' => $u1,
     132            'send_invite' => 1,
     133        ) );
    149134
    150135        $groups = BP_Groups_Member::get_invites( $u2, false, false, array( 'awesome', $g1 ) );
    … …  
    283268
    284269        // Membership requests should be removed.
    285         self::create_group_membership_request( $u1, $g );
     270        groups_send_membership_request( array(
     271            'user_id' => $u1,
     272            'group_id' => $g
     273        ) );
     274
    286275        groups_reject_membership_request( null, $u1, $g );
    287276        $u1_has_request = groups_check_for_membership_request( $u1, $g );
    … …  
    326315
    327316        // Outstanding invitations should be left intact.
    328         self::invite_user_to_group( $u2, $g, $u1 );
     317        groups_invite_user( array(
     318            'user_id' => $u2,
     319            'group_id' => $g,
     320            'inviter_id' => $u1,
     321            'send_invite' => 1,
     322        ) );
    329323        groups_reject_membership_request( null, $u2, $g );
    330324        $u2_has_invite = groups_check_user_has_invite( $u2, $g );
    … …  
    344338
    345339        // Membership requests should be removed.
    346         self::create_group_membership_request( $u1, $g );
     340        groups_send_membership_request( array(
     341            'user_id' => $u1,
     342            'group_id' => $g
     343        ) );
    347344        groups_delete_membership_request( null, $u1, $g );
    348345        $u1_has_request = groups_check_for_membership_request( $u1, $g );
    … …  
    387384
    388385        // Outstanding invitations should be left intact.
    389         self::invite_user_to_group( $u2, $g, $u1 );
     386        groups_invite_user( array(
     387            'user_id' => $u2,
     388            'group_id' => $g,
     389            'inviter_id' => $u1,
     390            'send_invite' => 1,
     391        ) );
     392
    390393        groups_delete_membership_request( null, $u2, $g );
    391394        $u2_has_invite = groups_check_user_has_invite( $u2, $g );
    … …  
    411414
    412415        // The invitation should be removed.
    413         self::invite_user_to_group( $u2, $g, $u1 );
     416        groups_invite_user( array(
     417            'user_id' => $u2,
     418            'group_id' => $g,
     419            'inviter_id' => $u1,
     420            'send_invite' => 1,
     421        ) );
     422
    414423        groups_reject_invite( $u2, $g );
    415424        $u2_has_invite = groups_check_user_has_invite( $u2, $g, 'all' );
    … …  
    451460
    452461        // Membership requests should be left intact.
    453         self::create_group_membership_request( $u1, $g );
     462        groups_send_membership_request( array(
     463            'user_id' => $u1,
     464            'group_id' => $g
     465        ) );
    454466        groups_reject_invite( $u1, $g );
    455467        $u1_has_request = groups_check_for_membership_request( $u1, $g );
    … …  
    475487
    476488        // The invitation should be removed.
    477         self::invite_user_to_group( $u2, $g, $u1 );
     489        groups_invite_user( array(
     490            'user_id' => $u2,
     491            'group_id' => $g,
     492            'inviter_id' => $u1,
     493            'send_invite' => 1,
     494        ) );
     495
    478496        groups_delete_invite( $u2, $g );
    479497        $u2_has_invite = groups_check_user_has_invite( $u2, $g, 'all' );
    … …  
    548566
    549567        // Membership requests should be left intact.
    550         self::create_group_membership_request( $u1, $g );
     568        groups_send_membership_request( array(
     569            'user_id' => $u1,
     570            'group_id' => $g
     571        ) );
    551572        groups_delete_invite( $u1, $g );
    552573        $u1_has_request = groups_check_for_membership_request( $u1, $g );
    … …  
    572593
    573594        // The invitation should be removed.
    574         self::invite_user_to_group( $u2, $g, $u1 );
     595        groups_invite_user( array(
     596            'user_id' => $u2,
     597            'group_id' => $g,
     598            'inviter_id' => $u1,
     599            'send_invite' => 1,
     600        ) );
    575601        groups_uninvite_user( $u2, $g );
    576602        $u2_has_invite = groups_check_user_has_invite( $u2, $g, 'all' );
    … …  
    613639
    614640        // Membership requests should be left intact.
    615         self::create_group_membership_request( $u1, $g );
     641        groups_send_membership_request( array(
     642            'user_id' => $u1,
     643            'group_id' => $g
     644        ) );
    616645        groups_uninvite_user( $u1, $g );
    617646        $u1_has_request = groups_check_for_membership_request( $u1, $g );
    … …  
    673702        $m1->promote( 'admin' );
    674703
    675         self::invite_user_to_group( $u2, $g, $u1 );
    676 
     704        groups_invite_user( array(
     705            'user_id' => $u2,
     706            'group_id' => $g,
     707            'inviter_id' => $u1,
     708            'send_invite' => 1,
     709        ) );
    677710        groups_join_group( $g, $u2 );
    678711        // Upon joining the group, outstanding invitations should be cleaned up.
    … …  
    687720        $u1 = self::factory()->user->create();
    688721        $g = self::factory()->group->create();
    689         self::create_group_membership_request( $u1, $g );
     722
     723        groups_send_membership_request( array(
     724            'user_id' => $u1,
     725            'group_id' => $g
     726        ) );
    690727
    691728        groups_join_group( $g, $u1 );
    … …  
    826863        $u1 = self::factory()->user->create();
    827864        $u2 = self::factory()->user->create();
    828         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    829         $g2 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    830         $g3 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    831 
    832         self::invite_user_to_group( $u2, $g1, $u1 );
    833         self::invite_user_to_group( $u2, $g2, $u1 );
    834         self::invite_user_to_group( $u2, $g3, $u1 );
    835 
     865        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     866        $g2 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     867        $g3 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     868
     869        groups_invite_user( array(
     870            'user_id' => $u2,
     871            'group_id' => $g1,
     872            'inviter_id' => $u1,
     873            'send_invite' => 1,
     874        ) );
     875        groups_invite_user( array(
     876            'user_id' => $u2,
     877            'group_id' => $g2,
     878            'inviter_id' => $u1,
     879            'send_invite' => 1,
     880        ) );
     881        groups_invite_user( array(
     882            'user_id' => $u2,
     883            'group_id' => $g3,
     884            'inviter_id' => $u1,
     885            'send_invite' => 1,
     886        ) );
    836887        $groups = groups_get_invites_for_user( $u2 );
    837888
    … …  
    849900        $u1 = self::factory()->user->create();
    850901        $u2 = self::factory()->user->create();
    851         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    852         $g2 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    853         $g3 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    854 
    855         self::invite_user_to_group( $u2, $g1, $u1 );
    856         self::invite_user_to_group( $u2, $g2, $u1 );
    857         self::invite_user_to_group( $u2, $g3, $u1 );
     902        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     903        $g2 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     904        $g3 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     905
     906        groups_invite_user( array(
     907            'user_id' => $u2,
     908            'group_id' => $g1,
     909            'inviter_id' => $u1,
     910            'send_invite' => 1,
     911        ) );
     912        groups_invite_user( array(
     913            'user_id' => $u2,
     914            'group_id' => $g2,
     915            'inviter_id' => $u1,
     916            'send_invite' => 1,
     917        ) );
     918        groups_invite_user( array(
     919            'user_id' => $u2,
     920            'group_id' => $g3,
     921            'inviter_id' => $u1,
     922            'send_invite' => 1,
     923        ) );
    858924
    859925        $this->set_current_user( $u2 );
    … …  
    872938        $u1 = self::factory()->user->create();
    873939        $u2 = self::factory()->user->create();
    874         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    875         $g2 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    876         $g3 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    877 
    878         self::invite_user_to_group( $u2, $g1, $u1 );
    879         self::invite_user_to_group( $u2, $g2, $u1 );
    880         self::invite_user_to_group( $u2, $g3, $u1 );
     940        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     941        $g2 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     942        $g3 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     943
     944        groups_invite_user( array(
     945            'user_id' => $u2,
     946            'group_id' => $g1,
     947            'inviter_id' => $u1,
     948            'send_invite' => 1,
     949        ) );
     950        groups_invite_user( array(
     951            'user_id' => $u2,
     952            'group_id' => $g2,
     953            'inviter_id' => $u1,
     954            'send_invite' => 1,
     955        ) );
     956        groups_invite_user( array(
     957            'user_id' => $u2,
     958            'group_id' => $g3,
     959            'inviter_id' => $u1,
     960            'send_invite' => 1,
     961        ) );
    881962
    882963        $groups = groups_get_invites_for_user( $u2, false, false, array( $g2 ) );
    … …  
    892973        $u1 = self::factory()->user->create();
    893974        $u2 = self::factory()->user->create();
    894         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    895         $g2 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    896         $g3 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    897 
    898         self::invite_user_to_group( $u2, $g1, $u1 );
    899         self::invite_user_to_group( $u2, $g2, $u1 );
    900         self::invite_user_to_group( $u2, $g3, $u1 );
     975        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     976        $g2 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     977        $g3 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     978
     979        groups_invite_user( array(
     980            'user_id' => $u2,
     981            'group_id' => $g1,
     982            'inviter_id' => $u1,
     983            'send_invite' => 1,
     984        ) );
     985        groups_invite_user( array(
     986            'user_id' => $u2,
     987            'group_id' => $g2,
     988            'inviter_id' => $u1,
     989            'send_invite' => 1,
     990        ) );
     991        groups_invite_user( array(
     992            'user_id' => $u2,
     993            'group_id' => $g3,
     994            'inviter_id' => $u1,
     995            'send_invite' => 1,
     996        ) );
    901997
    902998        $this->assertEquals( 3, groups_get_invite_count_for_user( $u2 ) );
    … …  
    9111007        $u1 = self::factory()->user->create();
    9121008        $u2 = self::factory()->user->create();
    913         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1009        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    9141010
    9151011        // Create draft invitation.
    … …  
    9341030        $u1 = self::factory()->user->create();
    9351031        $u2 = self::factory()->user->create();
    936         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1032        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    9371033
    9381034        // Create draft invitation
    … …  
    9581054        $u1 = self::factory()->user->create();
    9591055        $u2 = self::factory()->user->create();
    960         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1056        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    9611057
    9621058        // Create draft invitation
    … …  
    9701066
    9711067        // Send the invitation
    972         groups_send_invites( $u1, $g1 );
     1068        groups_send_invites( array(
     1069            'group_id'   => $g1,
     1070            'inviter_id' => $u1,
     1071        ) );
    9731072
    9741073        // Check that the invitation has been sent.
    … …  
    9781077
    9791078    /**
    980      * @group groups_accept_invite
    981      * @group group_invitations
    982      * @group group_membership
    983      */
    984     public function test_groups_accept_invite() {
    985         $u1 = self::factory()->user->create();
    986         $u2 = self::factory()->user->create();
    987         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1079     * @group groups_send_invites
     1080     * @group group_invitations
     1081     * @group group_membership
     1082     * @expectedDeprecated groups_send_invites
     1083     */
     1084    public function test_groups_send_invites_deprecated_args() {
     1085        $u1 = self::factory()->user->create();
     1086        $u2 = self::factory()->user->create();
     1087        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    9881088
    9891089        // Create draft invitation
    … …  
    9981098        // Send the invitation
    9991099        groups_send_invites( $u1, $g1 );
     1100
     1101        // Check that the invitation has been sent.
     1102        $sent = groups_check_user_has_invite( $u2, $g1, $type = 'sent' );
     1103        $this->assertTrue( is_numeric( $sent ) && $sent > 0 );
     1104    }
     1105
     1106    /**
     1107     * @group groups_accept_invite
     1108     * @group group_invitations
     1109     * @group group_membership
     1110     */
     1111    public function test_groups_accept_invite() {
     1112        $u1 = self::factory()->user->create();
     1113        $u2 = self::factory()->user->create();
     1114        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     1115
     1116        // Create draft invitation
     1117        groups_invite_user( array(
     1118            'user_id'       => $u2,
     1119            'group_id'      => $g1,
     1120            'inviter_id'    => $u1,
     1121            'date_modified' => bp_core_current_time(),
     1122            'is_confirmed'  => 0,
     1123            'send_invite'   => 1
     1124        ) );
    10001125
    10011126        // Accept the invitation
    … …  
    10181143        $u1 = self::factory()->user->create();
    10191144        $u2 = self::factory()->user->create();
    1020         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1145        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    10211146
    10221147        // Create draft invitation
    … …  
    10301155
    10311156        // Create membership request
    1032         groups_send_membership_request( $u2, $g1 );
     1157        $request_id = groups_send_membership_request( array(
     1158            'user_id'       => $u2,
     1159            'group_id'      => $g1,
     1160        ) );
     1161
    10331162        $request = groups_check_for_membership_request( $u2, $g1 );
     1163
    10341164        $this->assertTrue( is_numeric( $request ) && $request > 0 );
    10351165
    10361166        // Send the invitation
    1037         groups_send_invites( $u1, $g1 );
     1167        groups_send_invites( array(
     1168            'group_id'   => $g1,
     1169            'inviter_id' => $u1,
     1170        ) );
    10381171
    10391172        // Accept the invitation
    … …  
    10531186        $u1 = self::factory()->user->create();
    10541187        $u2 = self::factory()->user->create();
    1055         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1188        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    10561189
    10571190        // Create draft invitation
    … …  
    10611194            'inviter_id'    => $u1,
    10621195            'date_modified' => bp_core_current_time(),
    1063             'is_confirmed'  => 0
    1064         ) );
    1065 
    1066         // Send the invitation
    1067         groups_send_invites( $u1, $g1 );
     1196            'is_confirmed'  => 0,
     1197            'send_invite'   => 1
     1198        ) );
    10681199
    10691200        // Create membership request
    1070         groups_send_membership_request( $u2, $g1 );
     1201        groups_send_membership_request( array(
     1202            'user_id' => $u2,
     1203            'group_id' => $g1
     1204        ) );
    10711205
    10721206        // User should now be a group member
    … …  
    10841218        $u2 = self::factory()->user->create();
    10851219        $u3 = self::factory()->user->create();
    1086         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
    1087 
    1088         self::invite_user_to_group( $u2, $g1, $u1 );
    1089         self::invite_user_to_group( $u3, $g1, $u1 );
     1220        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
     1221
     1222        groups_invite_user( array(
     1223            'user_id' => $u2,
     1224            'group_id' => $g1,
     1225            'inviter_id' => $u1,
     1226            'send_invite' => 1,
     1227        ) );
     1228        groups_invite_user( array(
     1229            'user_id' => $u3,
     1230            'group_id' => $g1,
     1231            'inviter_id' => $u1,
     1232            'send_invite' => 1,
     1233        ) );
    10901234
    10911235        groups_delete_all_group_invites( $g1 );
    10921236
    10931237        // Get group invitations of any type, from any user in the group.
    1094         $invitees = new BP_Group_Member_Query( array(
     1238
     1239        $invitees = groups_get_invites( array(
    10951240            'group_id'     => $g1,
    1096             'is_confirmed' => 0,
    1097             'invite_sent'  => null,
    1098             'inviter_id'   => 'any',
    1099         ) );
    1100 
    1101         $this->assertTrue( empty( $invitees->results ) );
     1241            'invite_sent'  => 'all',
     1242        ) );
     1243
     1244        $this->assertTrue( empty( $invitees ) );
    11021245    }
    11031246
    … …  
    11301273    public function test_groups_send_invites_fail_on_empty_user_id() {
    11311274        $u1 = self::factory()->user->create();
    1132         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1275        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    11331276
    11341277        // Create draft invitation with empty inviter_id
    … …  
    11521295        $u1 = self::factory()->user->create();
    11531296        $u2 = self::factory()->user->create();
    1154         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1297        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    11551298
    11561299        // Create draft invitation with empty inviter_id
    … …  
    11751318        $u1 = self::factory()->user->create();
    11761319        $u2 = self::factory()->user->create();
    1177         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1320        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    11781321
    11791322        // Create draft invitation
    … …  
    11831326            'inviter_id'    => $u1,
    11841327            'date_modified' => bp_core_current_time(),
    1185             'is_confirmed'  => 0
    1186         ) );
    1187 
    1188         // Send the invitation; this will set the 'invite_sent' value to 1.
    1189         groups_send_invites( $u1, $g1 );
     1328            'is_confirmed'  => 0,
     1329            'send_invite'   => 1
     1330        ) );
    11901331
    11911332        // Default groups_get_invites_for_group() call
    … …  
    12091350    public function test_groups_send_membership_request() {
    12101351        $u1 = self::factory()->user->create();
    1211         $g1 = self::factory()->group->create();
     1352        $g1 = self::factory()->group->create( array( 'status' => 'private' ) );
     1353
     1354        // Create membership request
     1355        groups_send_membership_request( array(
     1356            'user_id' => $u1,
     1357            'group_id' => $g1
     1358        ) );
     1359
     1360        $request = groups_check_for_membership_request( $u1, $g1 );
     1361        $this->assertTrue( is_numeric( $request ) && $request > 0 );
     1362    }
     1363
     1364    /**
     1365     * @group groups_send_membership_request
     1366     * @group group_membership_requests
     1367     * @group group_membership
     1368     * @expectedDeprecated groups_send_membership_request
     1369     */
     1370    public function test_groups_send_membership_request_deprecated_args() {
     1371        $u1 = self::factory()->user->create();
     1372        $g1 = self::factory()->group->create( array( 'status' => 'private' ) );
    12121373
    12131374        // Create membership request
    … …  
    12251386    public function test_groups_accept_membership_request_by_membership_id() {
    12261387        $u1 = self::factory()->user->create();
    1227         $g1 = self::factory()->group->create();
     1388        $g1 = self::factory()->group->create( array( 'status' => 'private' ) );
    12281389
    12291390        // Create membership request
    1230         groups_send_membership_request( $u1, $g1 );
     1391        groups_send_membership_request( array(
     1392            'user_id' => $u1,
     1393            'group_id' => $g1
     1394        ) );
    12311395
    12321396        // Get group invitations of any type, from any user in the group.
    12331397        $member = new BP_Groups_Member( $u1, $g1 );
    12341398
    1235         groups_accept_membership_request( $member->id );
     1399        groups_accept_membership_request( false, $u1, $g1 );
    12361400
    12371401        // User should now be a group member.
    … …  
    12481412    public function test_groups_accept_membership_request_by_user_id_group_id() {
    12491413        $u1 = self::factory()->user->create();
    1250         $g1 = self::factory()->group->create();
     1414        $g1 = self::factory()->group->create( array( 'status' => 'private' ) );
    12511415
    12521416        // Create membership request
    1253         groups_send_membership_request( $u1, $g1 );
     1417        groups_send_membership_request( array(
     1418            'user_id' => $u1,
     1419            'group_id' => $g1
     1420        ) );
    12541421
    12551422        groups_accept_membership_request( null, $u1, $g1 );
    … …  
    12691436        $u1 = self::factory()->user->create();
    12701437        $u2 = self::factory()->user->create();
    1271         $g1 = self::factory()->group->create( array( 'creator_id' => $u1 ) );
     1438        $g1 = self::factory()->group->create( array( 'creator_id' => $u1, 'status' => 'private' ) );
    12721439
    12731440        // Create membership request
    1274         groups_send_membership_request( $u2, $g1 );
     1441        groups_send_membership_request( array(
     1442            'user_id' => $u2,
     1443            'group_id' => $g1
     1444        ) );
    12751445
    12761446        // Create draft invitation
    … …  
    12801450            'inviter_id'    => $u1,
    12811451            'date_modified' => bp_core_current_time(),
    1282             'is_confirmed'  => 0
    1283         ) );
    1284 
    1285         // Send the invitation
    1286         groups_send_invites( $u1, $g1 );
     1452            'is_confirmed'  => 0,
     1453            'send_invite'   => 1
     1454        ) );
    12871455
    12881456        // User should now be a group member
    … …  
    13001468        $u2 = self::factory()->user->create();
    13011469        $u3 = self::factory()->user->create();
    1302         $g1 = self::factory()->group->create();
     1470        $g1 = self::factory()->group->create( array( 'status' => 'private' ) );
    13031471
    13041472        // Create membership request
    1305         groups_send_membership_request( $u1, $g1 );
    1306         groups_send_membership_request( $u2, $g1 );
    1307         groups_send_membership_request( $u3, $g1 );
     1473        groups_send_membership_request( array(
     1474            'user_id' => $u1,
     1475            'group_id' => $g1
     1476        ) );
     1477        groups_send_membership_request( array(
     1478            'user_id' => $u2,
     1479            'group_id' => $g1
     1480        ) );
     1481        groups_send_membership_request( array(
     1482            'user_id' => $u3,
     1483            'group_id' => $g1
     1484        ) );
    13081485
    13091486        groups_accept_all_pending_membership_requests( $g1 );
    … …  
    14011578        $this->assertCount( 1, $memberships );
    14021579        $this->assertSame( self::$group_ids[0], $memberships[0]->group_id );
    1403     }
    1404 
    1405     /**
    1406      * @ticket BP7859
    1407      */
    1408     public function test_get_user_memberships_type_pending_request() {
    1409         groups_join_group( self::$group_ids[0], self::$user_ids[0] );
    1410         groups_send_membership_request( self::$user_ids[0], self::$group_ids[1] );
    1411 
    1412         $memberships = BP_Groups_Member::get_user_memberships( self::$user_ids[0], array(
    1413             'type' => 'pending_request',
    1414         ) );
    1415 
    1416         $this->assertCount( 1, $memberships );
    1417         $this->assertSame( self::$group_ids[1], $memberships[0]->group_id );
    1418     }
    1419 
    1420     /**
    1421      * @ticket BP7859
    1422      */
    1423     public function test_get_user_memberships_type_pending_received_invitation() {
    1424         groups_join_group( self::$group_ids[0], self::$user_ids[0] );
    1425         groups_invite_user( array(
    1426             'user_id'    => self::$user_ids[0],
    1427             'group_id'   => self::$group_ids[1],
    1428             'inviter_id' => self::$user_ids[1],
    1429         ) );
    1430 
    1431         $memberships = BP_Groups_Member::get_user_memberships( self::$user_ids[0], array(
    1432             'type' => 'pending_received_invitation',
    1433         ) );
    1434 
    1435         $this->assertCount( 1, $memberships );
    1436         $this->assertSame( self::$group_ids[1], $memberships[0]->group_id );
    1437     }
    1438 
    1439     /**
    1440      * @ticket BP7859
    1441      */
    1442     public function test_get_user_memberships_type_pending_sent_invitation() {
    1443         groups_join_group( self::$group_ids[0], self::$user_ids[0] );
    1444         groups_invite_user( array(
    1445             'user_id'    => self::$user_ids[1],
    1446             'group_id'   => self::$group_ids[1],
    1447             'inviter_id' => self::$user_ids[0],
    1448         ) );
    1449 
    1450         $memberships = BP_Groups_Member::get_user_memberships( self::$user_ids[0], array(
    1451             'type' => 'pending_sent_invitation',
    1452         ) );
    1453 
    1454         $this->assertCount( 1, $memberships );
    1455         $this->assertSame( self::$group_ids[1], $memberships[0]->group_id );
    14561580    }
    14571581
Note: See TracChangeset for help on using the changeset viewer.