From 3282da7c569c937d724e78df2c4b1e65baf24a4e Mon Sep 17 00:00:00 2001 From: lebaudantoine Date: Tue, 31 Dec 2024 11:32:21 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=A8(backend)=20fix=20Django=20Unordere?= =?UTF-8?q?dObjectListWarning=20on=20models?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Found this solution googling on Stack Overflow. Without a default ordering on a model, Django raises a warning, that pagination may yield inconsistent results. --- src/backend/core/models.py | 2 ++ src/backend/core/tests/test_api_users.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/backend/core/models.py b/src/backend/core/models.py index 4052a3f3..53653143 100644 --- a/src/backend/core/models.py +++ b/src/backend/core/models.py @@ -189,6 +189,7 @@ class User(AbstractBaseUser, BaseModel, auth_models.PermissionsMixin): class Meta: db_table = "meet_user" + ordering = ("-created_at",) verbose_name = _("user") verbose_name_plural = _("users") @@ -304,6 +305,7 @@ class ResourceAccess(BaseModel): class Meta: db_table = "meet_resource_access" + ordering = ("-created_at",) verbose_name = _("Resource access") verbose_name_plural = _("Resource accesses") constraints = [ diff --git a/src/backend/core/tests/test_api_users.py b/src/backend/core/tests/test_api_users.py index f57d5aa2..a8f182ad 100644 --- a/src/backend/core/tests/test_api_users.py +++ b/src/backend/core/tests/test_api_users.py @@ -66,7 +66,7 @@ def test_api_users_list_query_email(): assert response.status_code == 200 user_ids = [user["id"] for user in response.json()["results"]] - assert user_ids == [str(nicole.id), str(frank.id)] + assert user_ids == [str(frank.id), str(nicole.id)] def test_api_users_retrieve_me_anonymous():