test_attachments_middleware.py 9.9 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275
  1. from rest_framework import serializers
  2. from misago.acl.testutils import override_acl
  3. from misago.categories.models import Category
  4. from misago.conf import settings
  5. from misago.threads import testutils
  6. from misago.threads.api.postingendpoint import PostingEndpoint
  7. from misago.threads.api.postingendpoint.attachments import (
  8. AttachmentsMiddleware, validate_attachments_count)
  9. from misago.threads.models import Attachment, AttachmentType
  10. from misago.users.testutils import AuthenticatedUserTestCase
  11. class RequestMock(object):
  12. def __init__(self, data=None):
  13. self.data = data or {}
  14. class AttachmentsMiddlewareTests(AuthenticatedUserTestCase):
  15. def setUp(self):
  16. super(AttachmentsMiddlewareTests, self).setUp()
  17. self.category = Category.objects.get(slug='first-category')
  18. self.thread = testutils.post_thread(category=self.category)
  19. self.post = self.thread.first_post
  20. self.post.update_fields = []
  21. self.override_acl()
  22. self.filetype = AttachmentType.objects.order_by('id').last()
  23. def override_acl(self, new_acl=None):
  24. override_acl(self.user, new_acl or {'max_attachment_size': 1024})
  25. def mock_attachment(self, user=True, post=None):
  26. return Attachment.objects.create(
  27. secret=Attachment.generate_new_secret(),
  28. filetype=self.filetype,
  29. post=post,
  30. size=1000,
  31. uploader=self.user if user else None,
  32. uploader_name=self.user.username,
  33. uploader_slug=self.user.slug,
  34. uploader_ip='127.0.0.1',
  35. filename='testfile_{}.zip'.format(Attachment.objects.count() + 1),
  36. )
  37. def test_use_this_middleware(self):
  38. """use_this_middleware returns False if we can't upload attachments"""
  39. middleware = AttachmentsMiddleware(user=self.user)
  40. self.override_acl({'max_attachment_size': 0})
  41. self.assertFalse(middleware.use_this_middleware())
  42. self.override_acl({'max_attachment_size': 1024})
  43. self.assertTrue(middleware.use_this_middleware())
  44. def test_middleware_is_optional(self):
  45. """middleware is optional"""
  46. INPUTS = ({}, {'attachments': []})
  47. for test_input in INPUTS:
  48. middleware = AttachmentsMiddleware(
  49. request=RequestMock(test_input),
  50. mode=PostingEndpoint.START,
  51. user=self.user,
  52. post=self.post
  53. )
  54. serializer = middleware.get_serializer()
  55. self.assertTrue(serializer.is_valid())
  56. def test_middleware_validates_ids(self):
  57. """middleware validates attachments ids"""
  58. INPUTS = ('none', ['a', 'b', 123], range(settings.MISAGO_POST_ATTACHMENTS_LIMIT + 1))
  59. for test_input in INPUTS:
  60. middleware = AttachmentsMiddleware(
  61. request=RequestMock({
  62. 'attachments': test_input
  63. }),
  64. mode=PostingEndpoint.START,
  65. user=self.user,
  66. post=self.post
  67. )
  68. serializer = middleware.get_serializer()
  69. self.assertFalse(serializer.is_valid(), "%r shouldn't validate" % test_input)
  70. def test_get_initial_attachments(self):
  71. """get_initial_attachments returns list of attachments already existing on post"""
  72. middleware = AttachmentsMiddleware(
  73. request=RequestMock(), mode=PostingEndpoint.EDIT, user=self.user, post=self.post
  74. )
  75. serializer = middleware.get_serializer()
  76. attachments = serializer.get_initial_attachments(
  77. middleware.mode, middleware.user, middleware.post
  78. )
  79. self.assertEqual(attachments, [])
  80. attachment = self.mock_attachment(post=self.post)
  81. attachments = serializer.get_initial_attachments(
  82. middleware.mode, middleware.user, middleware.post
  83. )
  84. self.assertEqual(attachments, [attachment])
  85. def test_get_new_attachments(self):
  86. """get_initial_attachments returns list of attachments already existing on post"""
  87. middleware = AttachmentsMiddleware(
  88. request=RequestMock(), mode=PostingEndpoint.EDIT, user=self.user, post=self.post
  89. )
  90. serializer = middleware.get_serializer()
  91. attachments = serializer.get_new_attachments(middleware.user, [1, 2, 3])
  92. self.assertEqual(attachments, [])
  93. attachment = self.mock_attachment()
  94. attachments = serializer.get_new_attachments(middleware.user, [attachment.pk])
  95. self.assertEqual(attachments, [attachment])
  96. # only own orphaned attachments may be assigned to posts
  97. other_user_attachment = self.mock_attachment(user=False)
  98. attachments = serializer.get_new_attachments(middleware.user, [other_user_attachment.pk])
  99. self.assertEqual(attachments, [])
  100. def test_cant_delete_attachment(self):
  101. """middleware validates if we have permission to delete other users attachments"""
  102. self.override_acl({
  103. 'max_attachment_size': 1024,
  104. 'can_delete_other_users_attachments': False
  105. })
  106. attachment = self.mock_attachment(user=False, post=self.post)
  107. self.assertIsNone(attachment.uploader)
  108. serializer = AttachmentsMiddleware(
  109. request=RequestMock({
  110. 'attachments': []
  111. }),
  112. mode=PostingEndpoint.EDIT,
  113. user=self.user,
  114. post=self.post
  115. ).get_serializer()
  116. self.assertFalse(serializer.is_valid())
  117. def test_add_attachments(self):
  118. """middleware adds attachments to post"""
  119. attachments = [
  120. self.mock_attachment(),
  121. self.mock_attachment(),
  122. ]
  123. middleware = AttachmentsMiddleware(
  124. request=RequestMock({
  125. 'attachments': [a.pk for a in attachments]
  126. }),
  127. mode=PostingEndpoint.EDIT,
  128. user=self.user,
  129. post=self.post
  130. )
  131. serializer = middleware.get_serializer()
  132. self.assertTrue(serializer.is_valid())
  133. middleware.save(serializer)
  134. # attachments were associated with post
  135. self.assertEqual(self.post.update_fields, ['attachments_cache'])
  136. self.assertEqual(self.post.attachment_set.count(), 2)
  137. attachments_filenames = list(reversed([a.filename for a in attachments]))
  138. self.assertEqual([a['filename'] for a in self.post.attachments_cache],
  139. attachments_filenames)
  140. def test_remove_attachments(self):
  141. """middleware removes attachment from post and db"""
  142. attachments = [
  143. self.mock_attachment(post=self.post),
  144. self.mock_attachment(post=self.post),
  145. ]
  146. middleware = AttachmentsMiddleware(
  147. request=RequestMock({
  148. 'attachments': [attachments[0].pk]
  149. }),
  150. mode=PostingEndpoint.EDIT,
  151. user=self.user,
  152. post=self.post
  153. )
  154. serializer = middleware.get_serializer()
  155. self.assertTrue(serializer.is_valid())
  156. middleware.save(serializer)
  157. # attachments were associated with post
  158. self.assertEqual(self.post.update_fields, ['attachments_cache'])
  159. self.assertEqual(self.post.attachment_set.count(), 1)
  160. self.assertEqual(Attachment.objects.count(), 1)
  161. attachments_filenames = [attachments[0].filename]
  162. self.assertEqual([a['filename'] for a in self.post.attachments_cache],
  163. attachments_filenames)
  164. def test_steal_attachments(self):
  165. """middleware validates if attachments are already assigned to other posts"""
  166. other_post = testutils.reply_thread(self.thread)
  167. attachments = [
  168. self.mock_attachment(post=other_post),
  169. self.mock_attachment(),
  170. ]
  171. middleware = AttachmentsMiddleware(
  172. request=RequestMock({
  173. 'attachments': [attachments[0].pk, attachments[1].pk]
  174. }),
  175. mode=PostingEndpoint.EDIT,
  176. user=self.user,
  177. post=self.post
  178. )
  179. serializer = middleware.get_serializer()
  180. self.assertTrue(serializer.is_valid())
  181. middleware.save(serializer)
  182. # only unassociated attachment was associated with post
  183. self.assertEqual(self.post.update_fields, ['attachments_cache'])
  184. self.assertEqual(self.post.attachment_set.count(), 1)
  185. self.assertEqual(Attachment.objects.get(pk=attachments[0].pk).post, other_post)
  186. self.assertEqual(Attachment.objects.get(pk=attachments[1].pk).post, self.post)
  187. def test_edit_attachments(self):
  188. """middleware removes and adds attachments to post"""
  189. attachments = [
  190. self.mock_attachment(post=self.post),
  191. self.mock_attachment(post=self.post),
  192. self.mock_attachment(),
  193. ]
  194. middleware = AttachmentsMiddleware(
  195. request=RequestMock({
  196. 'attachments': [attachments[0].pk, attachments[2].pk]
  197. }),
  198. mode=PostingEndpoint.EDIT,
  199. user=self.user,
  200. post=self.post
  201. )
  202. serializer = middleware.get_serializer()
  203. self.assertTrue(serializer.is_valid())
  204. middleware.save(serializer)
  205. # attachments were associated with post
  206. self.assertEqual(self.post.update_fields, ['attachments_cache'])
  207. self.assertEqual(self.post.attachment_set.count(), 2)
  208. attachments_filenames = [attachments[2].filename, attachments[0].filename]
  209. self.assertEqual([a['filename'] for a in self.post.attachments_cache],
  210. attachments_filenames)
  211. class ValidateAttachmentsCountTests(AuthenticatedUserTestCase):
  212. def test_validate_attachments_count(self):
  213. """too large count of attachments is rejected"""
  214. validate_attachments_count(range(settings.MISAGO_POST_ATTACHMENTS_LIMIT))
  215. with self.assertRaises(serializers.ValidationError):
  216. validate_attachments_count(range(settings.MISAGO_POST_ATTACHMENTS_LIMIT + 1))