aboutsummaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-01-02 09:27:51 -0800
committerGravatar MarkKoz <[email protected]>2020-02-12 10:07:48 -0800
commitd212fb724be9ac6ab05671f28113318113a4bbe3 (patch)
tree9609b80c2c2d759b9f9d465fc55083ebcc842b39 /tests
parentSync tests: test diff for all 3 role changes simultaneously (diff)
Sync tests: remove diff test for updated and new roles together
Redundant since test_diff_for_new_updated_and_deleted_roles tests all 3 types together.
Diffstat (limited to 'tests')
-rw-r--r--tests/bot/cogs/sync/test_roles.py19
1 files changed, 0 insertions, 19 deletions
diff --git a/tests/bot/cogs/sync/test_roles.py b/tests/bot/cogs/sync/test_roles.py
index ccd617463..ca9df4305 100644
--- a/tests/bot/cogs/sync/test_roles.py
+++ b/tests/bot/cogs/sync/test_roles.py
@@ -72,25 +72,6 @@ class RoleSyncerTests(unittest.TestCase):
self.assertEqual(actual_diff, expected_diff)
- def test_get_roles_returns_roles_to_update_and_new_roles(self):
- """Newly created and updated roles should be returned together."""
- api_roles = {
- Role(id=41, name='old name', colour=35, permissions=0x8, position=1),
- }
- guild_roles = {
- Role(id=41, name='new name', colour=40, permissions=0x16, position=2),
- Role(id=53, name='other role', colour=55, permissions=0, position=3)
- }
-
- self.assertEqual(
- get_roles_for_sync(guild_roles, api_roles),
- (
- {Role(id=53, name='other role', colour=55, permissions=0, position=3)},
- {Role(id=41, name='new name', colour=40, permissions=0x16, position=2)},
- set(),
- )
- )
-
def test_diff_for_deleted_roles(self):
"""Only deleted roles should be added to the 'deleted' set of the diff."""
deleted_role = {"id": 61, "name": "delete", "colour": 99, "permissions": 0x9, "position": 2}