[buddypress-trac] [BuddyPress Trac] #6209: Add tests for group invitations and membership request functions

[buddypress-trac]

#6209: Add tests for group invitations and membership request functions
--------------------------------+----------------------------------------
 Reporter:  dcavins             |      Owner:
     Type:  task                |     Status:  new
 Priority:  normal              |  Milestone:  Awaiting Review
Component:  Component - Groups  |    Version:
 Severity:  normal              |   Keywords:  needs-unit-tests has-patch
--------------------------------+----------------------------------------
 As the first step toward developing a new invitations API, I want to track
 down the actual and expected behavior of functions related to group
 invitations and membership requests.

 I've started with the delete/reject functions, because they don't behave
 as I would expect. For example, using `groups_delete_membership_request()`
 or `groups_reject_membership_request()` will also remove outstanding
 invitations and confirmed group memberships for that user id. Using
 `groups_reject_invite()` or `groups_uninvite_user()` will also remove any
 membership requests or confirmed group memberships for that user id.

 These oddities can be fixed pretty easily by using a more scoped method
 like `BP_Groups_Member::delete_invite()`or
 `BP_Groups_Member::delete_request()` rather than the more burn-notice-like
 `BP_Groups_Member::delete()`.

 I will finish writing tests for the other invite/request functions very
 soon. Please offer any thoughts or feedback.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6209>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac