Skip to content

Commit 6ad6097

Browse files
committed
Optimize use of test accounts when testing deleting memberships
Use a generic / reusable ’named test account’ instead of a temp_account. This same test_account can be reused when testing team memberships.
1 parent a4efcc2 commit 6ad6097

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

tests/api/test_memberships.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,9 @@ def test_update_membership_make_moderator(self,
205205
assert is_valid_membership(make_me_group_room_moderator)
206206
assert make_me_group_room_moderator.isModerator
207207

208-
def test_delete_membership(self, api, group_room, temp_person):
209-
membership = add_person_to_room_by_id(api, group_room, temp_person)
208+
def test_delete_membership(self, api, group_room, test_people):
209+
person = test_people["delete_membership_test_person"]
210+
membership = add_person_to_room_by_id(api, group_room, person)
210211
assert is_valid_membership(membership)
211212
delete_membership(api, membership)
212213
assert not membership_exists(api, membership)

0 commit comments

Comments
 (0)