Skip to content

Commit

Permalink
Used assertCountEqual() in contrib.postgres.search tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
timgraham committed Nov 29, 2018
1 parent 9fa0d37 commit 41db8cb
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions tests/postgres_tests/test_search.py
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ def test_search_with_null(self):
searched = Line.objects.annotate(
search=SearchVector('scene__setting', 'dialogue'),
).filter(search='bedemir')
self.assertEqual(set(searched), {self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck})
self.assertCountEqual(searched, [self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck])

def test_search_with_non_text(self):
searched = Line.objects.annotate(
Expand Down Expand Up @@ -186,7 +186,7 @@ def test_phrase_search_with_config(self):
def test_raw_search(self):
line_qs = Line.objects.annotate(search=SearchVector('dialogue'))
searched = line_qs.filter(search=SearchQuery('Robin', search_type='raw'))
self.assertEqual(set(searched), {self.verse0, self.verse1})
self.assertCountEqual(searched, [self.verse0, self.verse1])
searched = line_qs.filter(search=SearchQuery("Robin & !'Camelot'", search_type='raw'))
self.assertSequenceEqual(searched, [self.verse1])

Expand Down Expand Up @@ -233,7 +233,7 @@ def test_vector_add(self):
searched = Line.objects.annotate(
search=SearchVector('scene__setting') + SearchVector('character__name'),
).filter(search='bedemir')
self.assertEqual(set(searched), {self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck})
self.assertCountEqual(searched, [self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck])

def test_vector_add_multi(self):
searched = Line.objects.annotate(
Expand All @@ -243,7 +243,7 @@ def test_vector_add_multi(self):
SearchVector('dialogue')
),
).filter(search='bedemir')
self.assertEqual(set(searched), {self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck})
self.assertCountEqual(searched, [self.bedemir0, self.bedemir1, self.crowd, self.witch, self.duck])

def test_query_and(self):
searched = Line.objects.annotate(
Expand All @@ -264,17 +264,17 @@ def test_query_multiple_and(self):

def test_query_or(self):
searched = Line.objects.filter(dialogue__search=SearchQuery('kneecaps') | SearchQuery('nostrils'))
self.assertEqual(set(searched), {self.verse1, self.verse2})
self.assertCountEqual(searched, [self.verse1, self.verse2])

def test_query_multiple_or(self):
searched = Line.objects.filter(
dialogue__search=SearchQuery('kneecaps') | SearchQuery('nostrils') | SearchQuery('Sir Robin')
)
self.assertEqual(set(searched), {self.verse1, self.verse2, self.verse0})
self.assertCountEqual(searched, [self.verse1, self.verse2, self.verse0])

def test_query_invert(self):
searched = Line.objects.filter(character=self.minstrel, dialogue__search=~SearchQuery('kneecaps'))
self.assertEqual(set(searched), {self.verse0, self.verse2})
self.assertCountEqual(searched, [self.verse0, self.verse2])

def test_query_config_mismatch(self):
with self.assertRaisesMessage(TypeError, "SearchQuery configs don't match."):
Expand Down

0 comments on commit 41db8cb

Please sign in to comment.