Merge pull request #1492 from ae-utbm/fix-pagination

fix: broken matmat pagination

matmat/templates/matmat/search_form.jinja

@@ -25,7 +25,7 @@
       {% endfor %}
     </div>
     {% if page_obj.has_other_pages() %}
-      {{ paginate_htmx(page_obj, paginator) }}
+      {{ paginate_htmx(request, page_obj, paginator) }}
     {% endif %}
     <hr>
   {% endif %}

matmat/tests.py

@@ -58,3 +58,13 @@ class TestMatmatronch(TestCase):
         assert list(response.context_data["object_list"]) == []
         assert not response.context_data["form"].is_valid()
         assert "Recherche vide" in response.context_data["form"].non_field_errors()
+
+    def test_search_many_users(self):
+        """Test that the pagination works when a lot of users are returned."""
+
+        baker.make(User, promo=17, _quantity=40, _bulk_create=True)
+        self.client.force_login(subscriber_user.make())
+        response = self.client.get(reverse("matmat:search", query={"promo": 17}))
+        assert response.status_code == 200
+        assert response.context_data["paginator"].count == 43
+        assert response.context_data["paginator"].num_pages == 3

matmat/views.py

@@ -20,7 +20,6 @@
 # Place - Suite 330, Boston, MA 02111-1307, USA.
 #
 #
-
 from django.db.models import F
 from django.views.generic import ListView
 from django.views.generic.edit import FormMixin