aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-02-10 17:06:18 -0800
committerGravatar MarkKoz <[email protected]>2020-02-12 10:07:58 -0800
commit5c385da1a41b2a6463b38b1973e13fd4590d61cb (patch)
tree565941e878adbe3dc951069a473a61de663d76fd
parentSync tests: assert that listeners are actually added as listeners (diff)
Sync tests: fix on_member_remove listener test
The listener was changed earlier to simply set in_guild to False. This commit accounts for that in the test.
-rw-r--r--tests/bot/cogs/sync/test_cog.py19
1 files changed, 6 insertions, 13 deletions
diff --git a/tests/bot/cogs/sync/test_cog.py b/tests/bot/cogs/sync/test_cog.py
index f7e86f063..a8c79e0d3 100644
--- a/tests/bot/cogs/sync/test_cog.py
+++ b/tests/bot/cogs/sync/test_cog.py
@@ -208,23 +208,16 @@ class SyncCogListenerTests(SyncCogTestCase):
self.bot.api_client.put.assert_not_called()
def test_sync_cog_on_member_remove(self):
- """A PUT request should be sent to set in_guild as False and update other fields."""
+ """Member should patched to set in_guild as False."""
self.assertTrue(self.cog.on_member_remove.__cog_listener__)
- roles = [helpers.MockRole(id=i) for i in (57, 22, 43)] # purposefully unsorted
- member = helpers.MockMember(roles=roles)
-
+ member = helpers.MockMember()
asyncio.run(self.cog.on_member_remove(member))
- json_data = {
- "avatar_hash": member.avatar,
- "discriminator": int(member.discriminator),
- "id": member.id,
- "in_guild": False,
- "name": member.name,
- "roles": sorted(role.id for role in member.roles)
- }
- self.bot.api_client.put.assert_called_once_with(f"bot/users/{member.id}", json=json_data)
+ self.cog.patch_user.assert_called_once_with(
+ member.id,
+ updated_information={"in_guild": False}
+ )
def test_sync_cog_on_member_update_roles(self):
"""Members should be patched if their roles have changed."""