aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-02-10 17:02:02 -0800
committerGravatar MarkKoz <[email protected]>2020-02-12 10:07:58 -0800
commite8b1fa52daf5950ad253e52c3b386a9d4967e739 (patch)
tree8b166854d772534f4c81875fd16f1d8b5cc9ba3c
parentSync tests: test sync commands require the admin permission (diff)
Sync tests: assert that listeners are actually added as listeners
-rw-r--r--tests/bot/cogs/sync/test_cog.py14
1 files changed, 14 insertions, 0 deletions
diff --git a/tests/bot/cogs/sync/test_cog.py b/tests/bot/cogs/sync/test_cog.py
index b1f586a5b..f7e86f063 100644
--- a/tests/bot/cogs/sync/test_cog.py
+++ b/tests/bot/cogs/sync/test_cog.py
@@ -147,6 +147,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_guild_role_create(self):
"""A POST request should be sent with the new role's data."""
+ self.assertTrue(self.cog.on_guild_role_create.__cog_listener__)
+
role_data = {
"colour": 49,
"id": 777,
@@ -161,6 +163,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_guild_role_delete(self):
"""A DELETE request should be sent."""
+ self.assertTrue(self.cog.on_guild_role_delete.__cog_listener__)
+
role = helpers.MockRole(id=99)
asyncio.run(self.cog.on_guild_role_delete(role))
@@ -168,6 +172,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_guild_role_update(self):
"""A PUT request should be sent if the colour, name, permissions, or position changes."""
+ self.assertTrue(self.cog.on_guild_role_update.__cog_listener__)
+
role_data = {
"colour": 49,
"id": 777,
@@ -203,6 +209,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_member_remove(self):
"""A PUT request should be sent to set in_guild as False and update other fields."""
+ 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)
@@ -220,6 +228,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_member_update_roles(self):
"""Members should be patched if their roles have changed."""
+ self.assertTrue(self.cog.on_member_update.__cog_listener__)
+
before_roles = [helpers.MockRole(id=12), helpers.MockRole(id=30)]
before_member = helpers.MockMember(roles=before_roles)
after_member = helpers.MockMember(roles=before_roles[1:])
@@ -231,6 +241,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_member_update_other(self):
"""Members should not be patched if other attributes have changed."""
+ self.assertTrue(self.cog.on_member_update.__cog_listener__)
+
subtests = (
("activities", discord.Game("Pong"), discord.Game("Frogger")),
("nick", "old nick", "new nick"),
@@ -250,6 +262,8 @@ class SyncCogListenerTests(SyncCogTestCase):
def test_sync_cog_on_user_update(self):
"""A user should be patched only if the name, discriminator, or avatar changes."""
+ self.assertTrue(self.cog.on_user_update.__cog_listener__)
+
before_data = {
"name": "old name",
"discriminator": "1234",