diff options
| author | 2020-01-03 10:53:45 -0800 | |
|---|---|---|
| committer | 2020-02-12 10:07:49 -0800 | |
| commit | 4912e94e3079b01b9481dee785c0b7f2552f7a1b (patch) | |
| tree | 0178809a73c68877811e6e42461a9d4d610b0366 | |
| parent | Sync tests: work around @everyone role being added by MockMember (diff) | |
Sync tests: test empty diff for identical users
| -rw-r--r-- | tests/bot/cogs/sync/test_users.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/bot/cogs/sync/test_users.py b/tests/bot/cogs/sync/test_users.py index 3dd2942b5..7a4a85c96 100644 --- a/tests/bot/cogs/sync/test_users.py +++ b/tests/bot/cogs/sync/test_users.py @@ -50,15 +50,15 @@ class UserSyncerDiffTests(unittest.TestCase): self.assertEqual(actual_diff, expected_diff) - def test_get_users_for_sync_returns_nothing_for_equal_users(self): - """When no users are updated, none are returned.""" - api_users = {43: fake_user()} - guild_users = {43: fake_user()} + def test_empty_diff_for_identical_users(self): + """No differences should be found if the users in the guild and DB are identical.""" + self.bot.api_client.get.return_value = [fake_user()] + guild = self.get_guild(fake_user()) - self.assertEqual( - get_users_for_sync(guild_users, api_users), - (set(), set()) - ) + actual_diff = asyncio.run(self.syncer._get_diff(guild)) + expected_diff = (set(), set(), None) + + self.assertEqual(actual_diff, expected_diff) def test_get_users_for_sync_returns_users_to_update_on_non_id_field_diff(self): """When a non-ID-field differs, the user to update is returned.""" |