aboutsummaryrefslogtreecommitdiffstats
path: root/pydis_site/apps/api/tests
diff options
context:
space:
mode:
authorGravatar RohanJnr <[email protected]>2021-06-10 02:45:55 +0530
committerGravatar RohanJnr <[email protected]>2021-06-10 02:47:37 +0530
commit8fdabfb4c08931b1e2352e98b307b3bfa3a121f1 (patch)
tree010340e8fee40e747f6771c15f0fc1bbcaa88422 /pydis_site/apps/api/tests
parentMerge branch 'main' into otn_softdel (diff)
Use sets to compare 2 un-ordered lists.
Diffstat (limited to 'pydis_site/apps/api/tests')
-rw-r--r--pydis_site/apps/api/tests/test_off_topic_channel_names.py19
1 files changed, 10 insertions, 9 deletions
diff --git a/pydis_site/apps/api/tests/test_off_topic_channel_names.py b/pydis_site/apps/api/tests/test_off_topic_channel_names.py
index ebb1224a..34dde7c6 100644
--- a/pydis_site/apps/api/tests/test_off_topic_channel_names.py
+++ b/pydis_site/apps/api/tests/test_off_topic_channel_names.py
@@ -81,13 +81,13 @@ class ListTests(APISubdomainTestCase):
response = self.client.get(url)
self.assertEqual(response.status_code, 200)
- self.assertListEqual(
- response.json(),
- [
+ self.assertEqual(
+ set(response.json()),
+ {
self.test_name.name,
self.test_name_2.name,
self.test_name_3.name
- ]
+ }
)
def test_returns_two_items_with_random_items_param_set_to_2(self):
@@ -97,7 +97,7 @@ class ListTests(APISubdomainTestCase):
self.assertEqual(response.status_code, 200)
self.assertEqual(len(response.json()), 2)
- self.assertEqual(response.json(), [self.test_name.name, self.test_name_2.name])
+ self.assertEqual(set(response.json()), {self.test_name.name, self.test_name_2.name})
def test_running_out_of_names_with_random_parameter(self):
"""Reset names `used` parameter to `False` when running out of names."""
@@ -105,8 +105,9 @@ class ListTests(APISubdomainTestCase):
response = self.client.get(f'{url}?random_items=3')
self.assertEqual(response.status_code, 200)
- self.assertListEqual(
- response.json(), [self.test_name.name, self.test_name_2.name, self.test_name_3.name]
+ self.assertEqual(
+ set(response.json()),
+ {self.test_name.name, self.test_name_2.name, self.test_name_3.name}
)
def test_returns_inactive_ot_names(self):
@@ -127,8 +128,8 @@ class ListTests(APISubdomainTestCase):
self.assertEqual(response.status_code, 200)
self.assertEqual(
- response.json(),
- [self.test_name.name, self.test_name_2.name]
+ set(response.json()),
+ {self.test_name.name, self.test_name_2.name}
)