fix: Delete feature external resources when GitHub integration was deleted

api/integrations/github/models.py

@@ -2,6 +2,7 @@
 
 from core.models import SoftDeleteExportableModel
 from django.db import models
+from django_lifecycle import BEFORE_DELETE, LifecycleModelMixin, hook
 
 from organisations.models import Organisation
 
@@ -21,7 +22,7 @@ def has_github_configuration(organisation_id: int) -> bool:
         ).exists()
 
 
-class GithubRepository(SoftDeleteExportableModel):
+class GithubRepository(LifecycleModelMixin, SoftDeleteExportableModel):
     github_configuration = models.ForeignKey(
         GithubConfiguration, related_name="repository_config", on_delete=models.CASCADE
     )
@@ -47,3 +48,19 @@ class Meta:
                 name="unique_repository_data",
             )
         ]
+
+    @hook(BEFORE_DELETE)
+    def delete_feature_external_respurces(
+        self,
+    ) -> None:
+        from features.feature_external_resources.models import (
+            FeatureExternalResource,
+        )
+
+        FeatureExternalResource.objects.filter(
+            feature__in=self.project.features.all(),
+            type__in=[
+                FeatureExternalResource.ResourceType.GITHUB_ISSUE,
+                FeatureExternalResource.ResourceType.GITHUB_PR,
+            ],
+        ).delete()

api/tests/unit/integrations/github/test_unit_github_views.py

@@ -4,6 +4,7 @@
 from rest_framework import status
 from rest_framework.test import APIClient
 
+from features.feature_external_resources.models import FeatureExternalResource
 from integrations.github.models import GithubConfiguration, GithubRepository
 from organisations.models import Organisation
 from projects.models import Project
@@ -191,18 +192,28 @@ def test_cannot_create_github_repository_due_to_unique_constraint(
 def test_github_delete_repository(
     client: APIClient,
     organisation: Organisation,
+    feature_external_resource: FeatureExternalResource,
     github_configuration: GithubConfiguration,
     github_repository: GithubRepository,
+    mocker,
 ) -> None:
     # Given
+    mock_generate_token = mocker.patch(
+        "integrations.github.github.generate_token",
+    )
+    mock_generate_token.return_value = "mocked_token"
     url = reverse(
         "api-v1:organisations:repositories-detail",
         args=[organisation.id, github_configuration.id, github_repository.id],
     )
+    for feature in github_repository.project.features.all():
+        assert FeatureExternalResource.objects.filter(feature=feature).exists()
     # When
     response = client.delete(url)
     # Then
     assert response.status_code == status.HTTP_204_NO_CONTENT
+    for feature in github_repository.project.features.all():
+        assert not FeatureExternalResource.objects.filter(feature=feature).exists()
 
 
 def mocked_requests_get(*args, **kwargs):