123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304 |
- from unittest.mock import Mock
- from rest_framework import serializers
- from .. import test
- from ...acl import useracl
- from ...acl.test import patch_user_acl
- from ...categories.models import Category
- from ...conf import settings
- from ...conftest import get_cache_versions
- from ...users.test import AuthenticatedUserTestCase
- from ..api.postingendpoint import PostingEndpoint
- from ..api.postingendpoint.attachments import (
- AttachmentsMiddleware,
- validate_attachments_count,
- )
- from ..models import Attachment, AttachmentType
- cache_versions = get_cache_versions()
- def patch_attachments_acl(acl_patch=None):
- acl_patch = acl_patch or {}
- acl_patch.setdefault("max_attachment_size", 1024)
- return patch_user_acl(acl_patch)
- class AttachmentsMiddlewareTests(AuthenticatedUserTestCase):
- def setUp(self):
- super().setUp()
- self.category = Category.objects.get(slug="first-category")
- self.thread = test.post_thread(category=self.category)
- self.post = self.thread.first_post
- self.post.update_fields = []
- self.filetype = AttachmentType.objects.order_by("id").last()
- def mock_attachment(self, user=True, post=None):
- return Attachment.objects.create(
- secret=Attachment.generate_new_secret(),
- filetype=self.filetype,
- post=post,
- size=1000,
- uploader=self.user if user else None,
- uploader_name=self.user.username,
- uploader_slug=self.user.slug,
- filename="testfile_%s.zip" % (Attachment.objects.count() + 1),
- )
- def test_use_this_middleware(self):
- """use_this_middleware returns False if we can't upload attachments"""
- with patch_user_acl({"max_attachment_size": 0}):
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(user=self.user, user_acl=user_acl)
- self.assertFalse(middleware.use_this_middleware())
- with patch_user_acl({"max_attachment_size": 1024}):
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(user=self.user, user_acl=user_acl)
- self.assertTrue(middleware.use_this_middleware())
- @patch_attachments_acl()
- def test_middleware_is_optional(self):
- """middleware is optional"""
- INPUTS = [{}, {"attachments": []}]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- for test_input in INPUTS:
- middleware = AttachmentsMiddleware(
- request=Mock(data=test_input),
- mode=PostingEndpoint.START,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertTrue(serializer.is_valid())
- @patch_attachments_acl()
- def test_middleware_validates_ids(self):
- """middleware validates attachments ids"""
- INPUTS = [
- "none",
- ["a", "b", 123],
- range(settings.MISAGO_POST_ATTACHMENTS_LIMIT + 1),
- ]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- for test_input in INPUTS:
- middleware = AttachmentsMiddleware(
- request=Mock(data={"attachments": test_input}),
- mode=PostingEndpoint.START,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertFalse(
- serializer.is_valid(), "%r shouldn't validate" % test_input
- )
- @patch_attachments_acl()
- def test_get_initial_attachments(self):
- """get_initial_attachments returns list of attachments already existing on post"""
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- attachments = serializer.get_initial_attachments(
- middleware.mode, middleware.user, middleware.post
- )
- self.assertEqual(attachments, [])
- attachment = self.mock_attachment(post=self.post)
- attachments = serializer.get_initial_attachments(
- middleware.mode, middleware.user_acl, middleware.post
- )
- self.assertEqual(attachments, [attachment])
- @patch_attachments_acl()
- def test_get_new_attachments(self):
- """get_initial_attachments returns list of attachments already existing on post"""
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- attachments = serializer.get_new_attachments(middleware.user, [1, 2, 3])
- self.assertEqual(attachments, [])
- attachment = self.mock_attachment()
- attachments = serializer.get_new_attachments(middleware.user, [attachment.pk])
- self.assertEqual(attachments, [attachment])
- # only own orphaned attachments may be assigned to posts
- other_user_attachment = self.mock_attachment(user=False)
- attachments = serializer.get_new_attachments(
- middleware.user, [other_user_attachment.pk]
- )
- self.assertEqual(attachments, [])
- @patch_attachments_acl({"can_delete_other_users_attachments": False})
- def test_cant_delete_attachment(self):
- """middleware validates if we have permission to delete other users attachments"""
- attachment = self.mock_attachment(user=False, post=self.post)
- self.assertIsNone(attachment.uploader)
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- serializer = AttachmentsMiddleware(
- request=Mock(data={"attachments": []}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- ).get_serializer()
- self.assertFalse(serializer.is_valid())
- @patch_attachments_acl()
- def test_add_attachments(self):
- """middleware adds attachments to post"""
- attachments = [self.mock_attachment(), self.mock_attachment()]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={"attachments": [a.pk for a in attachments]}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertTrue(serializer.is_valid())
- middleware.save(serializer)
- # attachments were associated with post
- self.assertEqual(self.post.update_fields, ["attachments_cache"])
- self.assertEqual(self.post.attachment_set.count(), 2)
- attachments_filenames = list(reversed([a.filename for a in attachments]))
- self.assertEqual(
- [a["filename"] for a in self.post.attachments_cache], attachments_filenames
- )
- @patch_attachments_acl()
- def test_remove_attachments(self):
- """middleware removes attachment from post and db"""
- attachments = [
- self.mock_attachment(post=self.post),
- self.mock_attachment(post=self.post),
- ]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={"attachments": [attachments[0].pk]}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertTrue(serializer.is_valid())
- middleware.save(serializer)
- # attachments were associated with post
- self.assertEqual(self.post.update_fields, ["attachments_cache"])
- self.assertEqual(self.post.attachment_set.count(), 1)
- self.assertEqual(Attachment.objects.count(), 1)
- attachments_filenames = [attachments[0].filename]
- self.assertEqual(
- [a["filename"] for a in self.post.attachments_cache], attachments_filenames
- )
- @patch_attachments_acl()
- def test_steal_attachments(self):
- """middleware validates if attachments are already assigned to other posts"""
- other_post = test.reply_thread(self.thread)
- attachments = [self.mock_attachment(post=other_post), self.mock_attachment()]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={"attachments": [attachments[0].pk, attachments[1].pk]}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertTrue(serializer.is_valid())
- middleware.save(serializer)
- # only unassociated attachment was associated with post
- self.assertEqual(self.post.update_fields, ["attachments_cache"])
- self.assertEqual(self.post.attachment_set.count(), 1)
- self.assertEqual(Attachment.objects.get(pk=attachments[0].pk).post, other_post)
- self.assertEqual(Attachment.objects.get(pk=attachments[1].pk).post, self.post)
- @patch_attachments_acl()
- def test_edit_attachments(self):
- """middleware removes and adds attachments to post"""
- attachments = [
- self.mock_attachment(post=self.post),
- self.mock_attachment(post=self.post),
- self.mock_attachment(),
- ]
- user_acl = useracl.get_user_acl(self.user, cache_versions)
- middleware = AttachmentsMiddleware(
- request=Mock(data={"attachments": [attachments[0].pk, attachments[2].pk]}),
- mode=PostingEndpoint.EDIT,
- user=self.user,
- user_acl=user_acl,
- post=self.post,
- )
- serializer = middleware.get_serializer()
- self.assertTrue(serializer.is_valid())
- middleware.save(serializer)
- # attachments were associated with post
- self.assertEqual(self.post.update_fields, ["attachments_cache"])
- self.assertEqual(self.post.attachment_set.count(), 2)
- attachments_filenames = [attachments[2].filename, attachments[0].filename]
- self.assertEqual(
- [a["filename"] for a in self.post.attachments_cache], attachments_filenames
- )
- class ValidateAttachmentsCountTests(AuthenticatedUserTestCase):
- def test_validate_attachments_count(self):
- """too large count of attachments is rejected"""
- validate_attachments_count(range(settings.MISAGO_POST_ATTACHMENTS_LIMIT))
- with self.assertRaises(serializers.ValidationError):
- validate_attachments_count(
- range(settings.MISAGO_POST_ATTACHMENTS_LIMIT + 1)
- )
|