feat(admin): run perform_detect task on admin save

Author: Kl0ven

controller/sentry/admin.py

@@ -24,6 +24,7 @@
 from controller.sentry.inlines import AppEventInline, ProjectEventInline
 from controller.sentry.mixins import ChartMixin, PrettyTypeMixin, ProjectLinkMixin
 from controller.sentry.models import App, Event, Project
+from controller.sentry.tasks import perform_detect
 from controller.sentry.utils import invalidate_cache
 
 if TYPE_CHECKING:  # pragma: no cover  # pragma: no cover
@@ -114,6 +115,21 @@ def get_chart_data(self, sentry_id):
         }
         return data, options
 
+    def save_model(self, request: "HttpRequest", obj: Project, form: "ModelForm[Project]", change: dict) -> None:
+        """This method is responsible to save project in the admin.
+
+        We hook into it to start a :func:`perform_detect <controller.sentry.tasks.perform_detect>` task
+
+        Args:
+            request (HttpRequest): The request
+            obj (Project): The app to save
+            form (ModelForm[Project]): form
+            change (bool): change
+        """
+        if change:
+            perform_detect.delay(obj.sentry_id)
+        return super().save_model(request, obj, form, change)
+
 
 @admin.register(Event)
 class EventAdmin(

controller/sentry/tests/test_admin.py

@@ -480,3 +480,18 @@ def test_app_filter(admin_with_user):
     )
 
     assert list(filter_none.queryset(request, App.objects.all())) == list(App.objects.all())
+
+
+@patch("controller.sentry.admin.perform_detect")
+@pytest.mark.django_db
+@pytest.mark.parametrize("user_group", ["Developer"])
+@pytest.mark.admin_site(model_class=Project)
+def test_project_admin_save_model(perform_detect: Mock, admin_with_user):
+    site, request = admin_with_user
+    project = Project(sentry_id="test_project")
+    site.save_model(request, project, None, True)
+    perform_detect.delay.assert_called_once_with(project.sentry_id)
+
+    perform_detect.reset_mock()
+    site.save_model(request, project, None, None)
+    perform_detect.delay.assert_not_called()