From e55468862d6bf4c6ab23446151891cc1158d9cfb Mon Sep 17 00:00:00 2001 From: Sabrina Demagny Date: Wed, 12 Feb 2025 10:24:13 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=91=E2=80=8D=F0=9F=92=BB(domains)=20ch?= =?UTF-8?q?ange=20default=20ordering?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Order by last created models MailDomain, Mailbox and MailDomainAccess. --- src/backend/mailbox_manager/models.py | 3 +++ src/backend/mailbox_manager/tests/test_utils_dimail_client.py | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/backend/mailbox_manager/models.py b/src/backend/mailbox_manager/models.py index d2905fc..185cf25 100644 --- a/src/backend/mailbox_manager/models.py +++ b/src/backend/mailbox_manager/models.py @@ -41,6 +41,7 @@ class MailDomain(BaseModel): db_table = "people_mail_domain" verbose_name = _("Mail domain") verbose_name_plural = _("Mail domains") + ordering = ["-created_at"] def __str__(self): return self.name @@ -109,6 +110,7 @@ class MailDomainAccess(BaseModel): verbose_name = _("User/mail domain relation") verbose_name_plural = _("User/mail domain relations") unique_together = ("user", "domain") + ordering = ["-created_at"] def __str__(self): return f"Access of user {self.user} on domain {self.domain}." @@ -206,6 +208,7 @@ class Mailbox(BaseModel): verbose_name = _("Mailbox") verbose_name_plural = _("Mailboxes") unique_together = ("local_part", "domain") + ordering = ["-created_at"] def __str__(self): return f"{self.local_part!s}@{self.domain.name:s}" diff --git a/src/backend/mailbox_manager/tests/test_utils_dimail_client.py b/src/backend/mailbox_manager/tests/test_utils_dimail_client.py index 445db3f..3e0db8f 100644 --- a/src/backend/mailbox_manager/tests/test_utils_dimail_client.py +++ b/src/backend/mailbox_manager/tests/test_utils_dimail_client.py @@ -398,7 +398,7 @@ def test_dimail__enable_pending_mailboxes(caplog): ) assert ( caplog.records[2].message - == f"Information for mailbox mock@{domain.name} sent to {mailbox1.secondary_email}." + == f"Information for mailbox mock@{domain.name} sent to {mailbox2.secondary_email}." ) assert ( caplog.records[4].message @@ -406,5 +406,5 @@ def test_dimail__enable_pending_mailboxes(caplog): ) assert ( caplog.records[5].message - == f"Information for mailbox mock@{domain.name} sent to {mailbox2.secondary_email}." + == f"Information for mailbox mock@{domain.name} sent to {mailbox1.secondary_email}." )