Merge pull request #603 from python-discord/decrease-batch-size-for-user-list-endpoint

Decrease page size for user list endpoint.

Author: jchristgit

pydis_site/apps/api/tests/test_users.py

@@ -6,6 +6,7 @@
 from .base import AuthenticatedAPITestCase
 from ..models import Role, User
 from ..models.bot.metricity import NotFoundError
+from ..viewsets.bot.user import UserListPagination
 
 
 class UnauthedUserAPITests(AuthenticatedAPITestCase):
@@ -357,7 +358,7 @@ class UserPaginatorTests(AuthenticatedAPITestCase):
     @classmethod
     def setUpTestData(cls):
         users = []
-        for i in range(1, 10_001):
+        for i in range(1, UserListPagination.page_size + 1):
             users.append(User(
                 id=i,
                 name=f"user{i}",
@@ -373,9 +374,10 @@ def test_returns_single_page_response(self):
         self.assertIsNone(response["previous_page_no"])
 
     def test_returns_next_page_number(self):
+        user_id = UserListPagination.page_size + 1
         User.objects.create(
-            id=10_001,
-            name="user10001",
+            id=user_id,
+            name=f"user{user_id}",
             discriminator=1111,
             in_guild=True
         )
@@ -384,9 +386,10 @@ def test_returns_next_page_number(self):
         self.assertEqual(2, response["next_page_no"])
 
     def test_returns_previous_page_number(self):
+        user_id = UserListPagination.page_size + 1
         User.objects.create(
-            id=10_001,
-            name="user10001",
+            id=user_id,
+            name=f"user{user_id}",
             discriminator=1111,
             in_guild=True
         )

pydis_site/apps/api/viewsets/bot/user.py

@@ -19,7 +19,7 @@
 class UserListPagination(PageNumberPagination):
     """Custom pagination class for the User Model."""
 
-    page_size = 10000
+    page_size = 2500
     page_size_query_param = "page_size"
 
     def get_next_page_number(self) -> typing.Optional[int]: