mirrors/bookwyrm
1
0
Fork 1
mirror of https://github.com/bookwyrm-social/bookwyrm.git synced 2024-11-29 21:11:16 +00:00
Code Issues Projects Releases Wiki Activity

Handle invalid status urls with 404

Browse source
This commit is contained in:
Mouse Reeve 2021-03-30 10:46:02 -07:00
parent d021666f2b
commit 0365a57307
2 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
bookwyrm/tests/views/test_feed.py
View file

@ -66,6 +66,18 @@ class FeedViews(TestCase):
self.assertIsInstance(result, ActivitypubResponse) self.assertIsInstance(result, ActivitypubResponse)
self.assertEqual(result.status_code, 200) self.assertEqual(result.status_code, 200)
def test_status_page_not_found(self, *_):
""" there are so many views, this just makes sure it LOADS """
view = views.Status.as_view()
request = self.factory.get("")
request.user = self.local_user
with patch("bookwyrm.views.feed.is_api_request") as is_api:
is_api.return_value = False
result = view(request, "mouse", 12345)
self.assertEqual(result.status_code, 404)
def test_status_page_with_image(self, *_): def test_status_page_with_image(self, *_):
""" there are so many views, this just makes sure it LOADS """ """ there are so many views, this just makes sure it LOADS """
view = views.Status.as_view() view = views.Status.as_view()

2
bookwyrm/views/feed.py
View file

@ -115,7 +115,7 @@ class Status(View):
status = models.Status.objects.select_subclasses().get( status = models.Status.objects.select_subclasses().get(
id=status_id, deleted=False id=status_id, deleted=False
) )
except ValueError: except (ValueError, models.Status.DoesNotExist):
return HttpResponseNotFound() return HttpResponseNotFound()
# the url should have the poster's username in it # the url should have the poster's username in it