aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-01-03 11:22:46 -0800
committerGravatar MarkKoz <[email protected]>2020-02-12 10:07:49 -0800
commit30ebb0184d12000db3ae5f276395fecd52d5dfa5 (patch)
tree6dd4cebd4df3e8b3788a1e431b80a2899aac936a
parentSync tests: test diff for updated users (diff)
Sync tests: test diff for new users
-rw-r--r--tests/bot/cogs/sync/test_users.py18
1 files changed, 10 insertions, 8 deletions
diff --git a/tests/bot/cogs/sync/test_users.py b/tests/bot/cogs/sync/test_users.py
index f1084fa98..c8ce7c04d 100644
--- a/tests/bot/cogs/sync/test_users.py
+++ b/tests/bot/cogs/sync/test_users.py
@@ -73,15 +73,17 @@ class UserSyncerDiffTests(unittest.TestCase):
self.assertEqual(actual_diff, expected_diff)
- def test_get_users_for_sync_returns_users_to_create_with_new_ids_on_guild(self):
- """When new users join the guild, they are returned as the first tuple element."""
- api_users = {43: fake_user()}
- guild_users = {43: fake_user(), 63: fake_user(id=63)}
+ def test_diff_for_new_users(self):
+ """Only new users should be added to the 'created' set of the diff."""
+ new_user = fake_user(id=99, name="new")
- self.assertEqual(
- get_users_for_sync(guild_users, api_users),
- ({fake_user(id=63)}, set())
- )
+ self.bot.api_client.get.return_value = [fake_user()]
+ guild = self.get_guild(fake_user(), new_user)
+
+ actual_diff = asyncio.run(self.syncer._get_diff(guild))
+ expected_diff = ({_User(**new_user)}, set(), None)
+
+ self.assertEqual(actual_diff, expected_diff)
def test_get_users_for_sync_updates_in_guild_field_on_user_leave(self):
"""When a user leaves the guild, the `in_guild` flag is updated to `False`."""