feat: supprimer les traces d'envoi d'emails de plus de 90 jours

clevercloud/cron.json

@@ -8,5 +8,6 @@
     "*/15 7-21 * * * $ROOT/clevercloud/run_management_command.sh send_messages_notifications asap",
     "20 6 * * * $ROOT/clevercloud/run_management_command.sh send_messages_notifications day",
     "10 6-22 * * * $ROOT/clevercloud/run_management_command.sh add_user_to_list_when_register",
+    "0 12  * * 1 $ROOT/clevercloud/run_management_command.sh delete_old_email_sent_tracks",
     "30 13 * * 1-5 $ROOT/clevercloud/run_management_command.sh send_notifs_on_unanswered_topics"
 ]

config/settings/base.py

@@ -323,6 +323,10 @@
             "handlers": ["console"],
             "level": os.getenv("DJANGO_LOG_LEVEL", "INFO"),
         },
+        "commands": {
+            "handlers": ["console"],
+            "level": os.getenv("COMMANDS_LOG_LEVEL", "INFO"),
+        },
     },
 }
 

lacommunaute/notification/admin.py

@@ -7,7 +7,7 @@
 @admin.register(EmailSentTrack)
 class EmailSentTrackAdmin(admin.ModelAdmin):
     list_display = ("kind", "created", "status_code")
-    list_filter = ("kind",)
+    list_filter = ("kind", "status_code")
 
 
 class BaseNotificationListFilter(admin.SimpleListFilter):

lacommunaute/notification/factories.py

@@ -9,14 +9,17 @@
 from lacommunaute.forum_conversation.factories import AnonymousPostFactory, TopicFactory
 from lacommunaute.notification.enums import EmailSentTrackKind, NotificationDelay
 from lacommunaute.notification.models import EmailSentTrack, Notification
+from lacommunaute.utils.factory_boy import AutoNowAddOverrideMixin
 
 
 faker = Faker()
 
 
-class EmailSentTrackFactory(factory.django.DjangoModelFactory):
-    status_code = faker.pyint()
-    response = faker.text()
+class EmailSentTrackFactory(AutoNowAddOverrideMixin, factory.django.DjangoModelFactory):
+    created = factory.LazyFunction(timezone.now)
+    updated = factory.LazyFunction(timezone.now)
+    status_code = factory.Faker("random_int", min=200, max=500)
+    response = factory.Faker("text")
     datas = {"text": faker.text()}
     kind = EmailSentTrackKind.FIRST_REPLY
 

lacommunaute/notification/management/commands/delete_old_email_sent_tracks.py

@@ -0,0 +1,16 @@
+from logging import getLogger
+
+from django.core.management.base import BaseCommand
+
+from lacommunaute.notification.models import EmailSentTrack
+
+
+logger = getLogger("commands")
+
+
+class Command(BaseCommand):
+    help = "Supprimer les anciens enregistrements EmailSentTrack"
+
+    def handle(self, *args, **options):
+        nb_deleted = EmailSentTrack.objects.delete_old_records()
+        logger.info("%s enregistrements EmailSentTrack supprimés", nb_deleted)

lacommunaute/notification/models.py

@@ -2,14 +2,22 @@
 from itertools import groupby
 from operator import attrgetter
 
+from dateutil.relativedelta import relativedelta
 from django.db import models
 from django.db.models import F
+from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 from machina.models.abstract_models import DatedModel
 
 from lacommunaute.notification.enums import EmailSentTrackKind, NotificationDelay
 
 
+class EmailSentTrackQuerySet(models.QuerySet):
+    def delete_old_records(self):
+        nb, _ = self.filter(created__lt=timezone.now() - relativedelta(days=90)).delete()
+        return nb
+
+
 class EmailSentTrack(DatedModel):
     status_code = models.IntegerField(verbose_name="code de retour de l'API")
     response = models.TextField(verbose_name="réponse de l'API")
@@ -18,6 +26,8 @@ class EmailSentTrack(DatedModel):
         verbose_name="type", choices=EmailSentTrackKind.choices, max_length=20, null=False, blank=False
     )
 
+    objects = EmailSentTrackQuerySet().as_manager()
+
     class Meta:
         verbose_name = "trace des emails envoyés"
         verbose_name_plural = "traces des emails envoyés"

lacommunaute/notification/tests/tests_management_commands.py

@@ -0,0 +1,12 @@
+from dateutil.relativedelta import relativedelta
+from django.core.management import call_command
+from django.utils import timezone
+
+from lacommunaute.notification.factories import EmailSentTrackFactory
+from lacommunaute.notification.models import EmailSentTrack
+
+
+def test_delete_old_email_sent_tracks(db):
+    EmailSentTrackFactory(created=timezone.now() - relativedelta(days=90))
+    call_command("delete_old_email_sent_tracks")
+    assert EmailSentTrack.objects.count() == 0

lacommunaute/notification/tests/tests_models.py

@@ -1,16 +1,28 @@
 import pytest
+from dateutil.relativedelta import relativedelta
 from django.contrib.auth.models import AnonymousUser
 from django.db import IntegrityError
 from django.db.models import F
 from django.test import TestCase
+from django.utils import timezone
 
 from lacommunaute.forum_conversation.factories import TopicFactory
 from lacommunaute.notification.enums import EmailSentTrackKind
-from lacommunaute.notification.factories import NotificationFactory
+from lacommunaute.notification.factories import EmailSentTrackFactory, NotificationFactory
 from lacommunaute.notification.models import EmailSentTrack, Notification
 from lacommunaute.users.factories import UserFactory
 
 
+class TestEmailSentTrackQuerySet:
+    def test_delete_old_records(self, db):
+        _ = [EmailSentTrackFactory(created=timezone.now() - relativedelta(days=nb_days)) for nb_days in range(89, 92)]
+
+        EmailSentTrack.objects.delete_old_records()
+
+        email_sent_track = EmailSentTrack.objects.get()
+        assert email_sent_track.created >= timezone.now() - relativedelta(days=90)
+
+
 class EmailSentTrackModelTest(TestCase):
     def test_str(self):
         track = EmailSentTrack(status_code=200)

lacommunaute/utils/factory_boy.py

@@ -0,0 +1,15 @@
+class AutoNowAddOverrideMixin:
+    """This mixin allows you to override fields with `auto_now=True`"""
+
+    @classmethod
+    def _create(cls, model_class, *args, **kwargs):
+        auto_now_add_desactivated = []
+        for field in model_class._meta.get_fields():
+            if getattr(field, "auto_now_add", False) and kwargs.get(field.name):
+                field.auto_now_add = False
+                auto_now_add_desactivated.append(field)
+        try:
+            return super()._create(model_class, *args, **kwargs)
+        finally:
+            for field in auto_now_add_desactivated:
+                field.auto_now_add = True