From 1f36a777ecc5a80b486444c95025d08ab55d19e3 Mon Sep 17 00:00:00 2001 From: Vidhi Singh Date: Sat, 10 Jan 2026 19:38:43 +0530 Subject: [PATCH 1/3] Fix 405 error when deleting community feeds --- aggregator/urls.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/aggregator/urls.py b/aggregator/urls.py index 1c49f35d0e..5ad550be61 100644 --- a/aggregator/urls.py +++ b/aggregator/urls.py @@ -19,5 +19,5 @@ path("/", views.FeedListView.as_view(), name="community-feed-list"), path("add//", views.add_feed, name="community-add-feed"), path("edit//", views.edit_feed, name="community-edit-feed"), - path("delete/", views.delete_feed, name="community-delete-feed"), + path("delete//", views.delete_feed, name="community-delete-feed"), ] From 66012b93bf3640380e345a1a0383852a6b0909ed Mon Sep 17 00:00:00 2001 From: Vidhi Singh Date: Sun, 11 Jan 2026 15:13:22 +0530 Subject: [PATCH 2/3] Added test for deleting community feeds --- aggregator/tests.py | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/aggregator/tests.py b/aggregator/tests.py index c83379d9b3..1772a50127 100644 --- a/aggregator/tests.py +++ b/aggregator/tests.py @@ -154,6 +154,29 @@ def test_feedtype_items_has_only_approved(self): ["Approved long URL Item", "Approved Item"], transform=attrgetter("title"), ) + def test_logged_in_user_can_delete_own_feed(self): + user = User.objects.create_user( + username="deleter", + password="pass123", + ) + + feed = models.Feed.objects.create( + title="My Feed", + feed_url="https://example.com/rss/", + public_url="https://example.com/", + approval_status=models.PENDING_FEED, + feed_type=self.feed_type, + owner=user, + ) + + self.client.login(username="deleter", password="pass123") + + response = self.client.post( + reverse("community-delete-feed", args=[feed.id]) + ) + + self.assertEqual(response.status_code, 302) + self.assertFalse(models.Feed.objects.filter(id=feed.id).exists()) class TestForms(SimpleTestCase): From f4a65dd56f1caa3acf2d2d492f4554b050ba89e4 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Sun, 11 Jan 2026 09:48:05 +0000 Subject: [PATCH 3/3] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- aggregator/tests.py | 43 +++++++++++++++++++++---------------------- 1 file changed, 21 insertions(+), 22 deletions(-) diff --git a/aggregator/tests.py b/aggregator/tests.py index 1772a50127..afe67b0c25 100644 --- a/aggregator/tests.py +++ b/aggregator/tests.py @@ -154,29 +154,28 @@ def test_feedtype_items_has_only_approved(self): ["Approved long URL Item", "Approved Item"], transform=attrgetter("title"), ) + def test_logged_in_user_can_delete_own_feed(self): - user = User.objects.create_user( - username="deleter", - password="pass123", - ) - - feed = models.Feed.objects.create( - title="My Feed", - feed_url="https://example.com/rss/", - public_url="https://example.com/", - approval_status=models.PENDING_FEED, - feed_type=self.feed_type, - owner=user, - ) - - self.client.login(username="deleter", password="pass123") - - response = self.client.post( - reverse("community-delete-feed", args=[feed.id]) - ) - - self.assertEqual(response.status_code, 302) - self.assertFalse(models.Feed.objects.filter(id=feed.id).exists()) + user = User.objects.create_user( + username="deleter", + password="pass123", + ) + + feed = models.Feed.objects.create( + title="My Feed", + feed_url="https://example.com/rss/", + public_url="https://example.com/", + approval_status=models.PENDING_FEED, + feed_type=self.feed_type, + owner=user, + ) + + self.client.login(username="deleter", password="pass123") + + response = self.client.post(reverse("community-delete-feed", args=[feed.id])) + + self.assertEqual(response.status_code, 302) + self.assertFalse(models.Feed.objects.filter(id=feed.id).exists()) class TestForms(SimpleTestCase):