Skip to content

Commit

Permalink
a bunch of additionnal queries, need a bit of review :-|
Browse files Browse the repository at this point in the history
  • Loading branch information
vincentporte committed Feb 10, 2025
1 parent 57bdfc1 commit c3cbf23
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions lacommunaute/forum/tests/tests_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,12 +188,12 @@ def test_cannot_submit_post(self, *args):
self.assertEqual(response.status_code, 200)
self.assertNotContains(response, f'id="collapsePost{self.topic.pk}')

def test_queries(self):
def test_numqueries(self):
ContentType.objects.clear_cache()

TopicFactory.create_batch(20, with_post=True)
self.client.force_login(self.user)
with self.assertNumQueries(22):
with self.assertNumQueries(28):
self.client.get(self.url)

def test_certified_post_display(self):
Expand Down
2 changes: 1 addition & 1 deletion lacommunaute/forum_conversation/tests/tests_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -757,7 +757,7 @@ def test_numqueries(self):
self.client.force_login(self.poster)

# note vincentporte : to be optimized
with self.assertNumQueries(39):
with self.assertNumQueries(45):
response = self.client.get(self.url)
self.assertEqual(response.status_code, 200)

Expand Down
2 changes: 1 addition & 1 deletion lacommunaute/forum_upvote/tests/test_upvotelistview.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,5 +78,5 @@ def test_numqueries(client, db, django_assert_num_queries):
PostFactory.create_batch(20, topic=TopicFactory(), upvoted_by=[user])

# vincentporte : to be optimized
with django_assert_num_queries(37):
with django_assert_num_queries(43):
client.get(url)

0 comments on commit c3cbf23

Please sign in to comment.