linting

src/community_drives/tests.py

@@ -223,7 +223,9 @@ def test_delete_participation_invalid_method(self):
         self.assertEqual(response.json()["error"], "Invalid request method")
 
     def test_delete_drive_success(self):
-        url = reverse("community_drives:delete_drive", kwargs={"drive_id": self.drive.drive_id})
+        url = reverse(
+            "community_drives:delete_drive", kwargs={"drive_id": self.drive.drive_id}
+        )
 
         response = self.client.post(url)
 
@@ -237,11 +239,15 @@ def test_delete_drive_success(self):
 
     def test_delete_drive_nonexistent(self):
         invalid_drive_id = uuid4()
-        url = reverse("community_drives:delete_drive", kwargs={"drive_id": invalid_drive_id})
+        url = reverse(
+            "community_drives:delete_drive", kwargs={"drive_id": invalid_drive_id}
+        )
 
         response = self.client.post(url)
 
-        self.assertEqual(response.status_code, 302)  # Expect redirect even if deletion fails
+        self.assertEqual(
+            response.status_code, 302
+        )  # Expect redirect even if deletion fails
         messages = list(response.wsgi_request._messages)
         self.assertEqual(len(messages), 1)
         self.assertEqual(str(messages[0]), "Failed to delete community drive.")

src/community_drives/urls.py

@@ -32,5 +32,5 @@
         "delete-drive/<uuid:drive_id>/",
         views.delete_drive,
         name="delete_drive",
-    )
+    ),
 ]

src/community_drives/views.py

@@ -256,6 +256,7 @@ def delete_participation(request, organization_id, drive_id):
         {"success": False, "error": "Invalid request method"}, status=405
     )
 
+
 def delete_drive(request, drive_id):
     if request.method == "POST":
         try:
@@ -268,8 +269,10 @@ def delete_drive(request, drive_id):
                 drive_org.modified_at = timezone.now()
                 drive_org.save()
 
-            messages.success(request, f"Community drive '{drive}' successfully deleted.")
+            messages.success(
+                request, f"Community drive '{drive}' successfully deleted."
+            )
+            return redirect("/community_drives")
+        except CommunityDrive.DoesNotExist:
+            messages.error(request, "Failed to delete community drive. Couldn't find the drive.")
             return redirect("/community_drives")
-        except:
-            messages.error(request, "Failed to delete community drive.")
-            return redirect("/community_drives")