123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578 |
- import json
- from django.urls import reverse
- from misago.acl.testutils import override_acl
- from misago.categories.models import Category
- from misago.readtracker import poststracker
- from misago.threads import testutils
- from misago.threads.models import Post, Thread
- from misago.threads.serializers.moderation import POSTS_LIMIT
- from misago.users.testutils import AuthenticatedUserTestCase
- class ThreadPostMergeApiTestCase(AuthenticatedUserTestCase):
- def setUp(self):
- super().setUp()
- self.category = Category.objects.get(slug='first-category')
- self.thread = testutils.post_thread(category=self.category)
- self.post = testutils.reply_thread(self.thread, poster=self.user)
- self.api_link = reverse(
- 'misago:api:thread-post-merge', kwargs={
- 'thread_pk': self.thread.pk,
- }
- )
- self.override_acl()
- def refresh_thread(self):
- self.thread = Thread.objects.get(pk=self.thread.pk)
- def override_acl(self, extra_acl=None):
- new_acl = self.user.acl_cache
- new_acl['categories'][self.category.pk].update({
- 'can_see': 1,
- 'can_browse': 1,
- 'can_start_threads': 0,
- 'can_reply_threads': 0,
- 'can_edit_posts': 1,
- 'can_approve_content': 0,
- 'can_merge_posts': 1,
- })
- if extra_acl:
- new_acl['categories'][self.category.pk].update(extra_acl)
- override_acl(self.user, new_acl)
- def test_anonymous_user(self):
- """you need to authenticate to merge posts"""
- self.logout_user()
- response = self.client.post(
- self.api_link,
- json.dumps({}),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 403)
- def test_no_permission(self):
- """api validates permission to merge"""
- self.override_acl({'can_merge_posts': 0})
- response = self.client.post(
- self.api_link,
- json.dumps({}),
- content_type="application/json",
- )
- self.assertContains(response, "You can't merge posts in this thread.", status_code=403)
- def test_empty_data_json(self):
- """api handles empty json data"""
- response = self.client.post(
- self.api_link, json.dumps({}), content_type="application/json"
- )
- self.assertContains(
- response, "You have to select at least two posts to merge.", status_code=400
- )
- def test_empty_data_form(self):
- """api handles empty form data"""
- response = self.client.post(self.api_link, {})
- self.assertContains(
- response, "You have to select at least two posts to merge.", status_code=400
- )
- def test_invalid_data(self):
- """api handles post that is invalid type"""
- self.override_acl()
- response = self.client.post(self.api_link, '[]', content_type="application/json")
- self.assertContains(response, "Invalid data. Expected a dictionary", status_code=400)
- self.override_acl()
- response = self.client.post(self.api_link, '123', content_type="application/json")
- self.assertContains(response, "Invalid data. Expected a dictionary", status_code=400)
- self.override_acl()
- response = self.client.post(self.api_link, '"string"', content_type="application/json")
- self.assertContains(response, "Invalid data. Expected a dictionary", status_code=400)
- self.override_acl()
- response = self.client.post(self.api_link, 'malformed', content_type="application/json")
- self.assertContains(response, "JSON parse error", status_code=400)
- def test_no_posts_ids(self):
- """api rejects no posts ids"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': []
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "You have to select at least two posts to merge.", status_code=400
- )
- def test_invalid_posts_data(self):
- """api handles invalid data"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': 'string'
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Expected a list of items but got type", status_code=400
- )
- def test_invalid_posts_ids(self):
- """api handles invalid post id"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [1, 2, 'string']
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "One or more post ids received were invalid.", status_code=400
- )
- def test_one_post_id(self):
- """api rejects one post id"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [1]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "You have to select at least two posts to merge.", status_code=400
- )
- def test_merge_limit(self):
- """api rejects more posts than merge limit"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': list(range(POSTS_LIMIT + 1))
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "No more than {} posts can be merged".format(POSTS_LIMIT), status_code=400
- )
- def test_merge_event(self):
- """api recjects events"""
- event = testutils.reply_thread(self.thread, is_event=True, poster=self.user)
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [self.post.pk, event.pk]
- }),
- content_type="application/json",
- )
- self.assertContains(response, "Events can't be merged.", status_code=400)
- def test_merge_notfound_pk(self):
- """api recjects nonexistant pk's"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [self.post.pk, self.post.pk * 1000]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "One or more posts to merge could not be found.", status_code=400
- )
- def test_merge_cross_threads(self):
- """api recjects attempt to merge with post made in other thread"""
- other_thread = testutils.post_thread(category=self.category)
- other_post = testutils.reply_thread(other_thread, poster=self.user)
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [self.post.pk, other_post.pk]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "One or more posts to merge could not be found.", status_code=400
- )
- def test_merge_authenticated_with_guest_post(self):
- """api recjects attempt to merge with post made by deleted user"""
- other_post = testutils.reply_thread(self.thread)
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [self.post.pk, other_post.pk]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Posts made by different users can't be merged.", status_code=400
- )
- def test_merge_guest_with_authenticated_post(self):
- """api recjects attempt to merge with post made by deleted user"""
- other_post = testutils.reply_thread(self.thread)
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [other_post.pk, self.post.pk]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Posts made by different users can't be merged.", status_code=400
- )
- def test_merge_guest_posts_different_usernames(self):
- """api recjects attempt to merge posts made by different guests"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster="Bob").pk,
- testutils.reply_thread(self.thread, poster="Miku").pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Posts made by different users can't be merged.", status_code=400
- )
- def test_merge_different_visibility(self):
- """api recjects attempt to merge posts with different visibility"""
- self.override_acl({'can_hide_posts': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster=self.user, is_hidden=True).pk,
- testutils.reply_thread(self.thread, poster=self.user, is_hidden=False).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Posts with different visibility can't be merged.", status_code=400
- )
- def test_merge_different_approval(self):
- """api recjects attempt to merge posts with different approval"""
- self.override_acl({'can_approve_content': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster=self.user, is_unapproved=True).pk,
- testutils.reply_thread(self.thread, poster=self.user, is_unapproved=False).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertContains(
- response, "Posts with different visibility can't be merged.", status_code=400
- )
- def test_closed_thread(self):
- """api validates permission to merge in closed thread"""
- self.thread.is_closed = True
- self.thread.save()
- posts = [
- testutils.reply_thread(self.thread, poster=self.user).pk,
- testutils.reply_thread(self.thread, poster=self.user).pk,
- ]
- response = self.client.post(
- self.api_link,
- json.dumps({'posts': posts}),
- content_type="application/json",
- )
- self.assertContains(
- response,
- "This thread is closed. You can't merge posts in it.",
- status_code=400,
- )
- # allow closing threads
- self.override_acl({'can_close_threads': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({'posts': posts}),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_closed_category(self):
- """api validates permission to merge in closed category"""
- self.category.is_closed = True
- self.category.save()
- posts = [
- testutils.reply_thread(self.thread, poster=self.user).pk,
- testutils.reply_thread(self.thread, poster=self.user).pk,
- ]
- response = self.client.post(
- self.api_link,
- json.dumps({'posts': posts}),
- content_type="application/json",
- )
- self.assertContains(
- response,
- "This category is closed. You can't merge posts in it.",
- status_code=400,
- )
- # allow closing threads
- self.override_acl({'can_close_threads': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({'posts': posts}),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_merge_best_answer_first_post(self):
- """api recjects attempt to merge best_answer with first post"""
- self.thread.first_post.poster = self.user
- self.thread.first_post.save()
- self.post.poster = self.user
- self.post.save()
- self.thread.set_best_answer(self.user, self.post)
- self.thread.save()
-
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- self.thread.first_post.pk,
- self.post.pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'detail': "Post marked as best answer can't be merged with thread's first post."
- })
- def test_merge_posts(self):
- """api merges two posts"""
- post_a = testutils.reply_thread(self.thread, poster=self.user, message="Battęry")
- post_b = testutils.reply_thread(self.thread, poster=self.user, message="Hórse")
- thread_replies = self.thread.replies
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [post_a.pk, post_b.pk]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- self.refresh_thread()
- self.assertEqual(self.thread.replies, thread_replies - 1)
- with self.assertRaises(Post.DoesNotExist):
- Post.objects.get(pk=post_b.pk)
- merged_post = Post.objects.get(pk=post_a.pk)
- self.assertEqual(merged_post.parsed, '{}\n{}'.format(post_a.parsed, post_b.parsed))
- def test_merge_guest_posts(self):
- """api recjects attempt to merge posts made by same guest"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster="Bob").pk,
- testutils.reply_thread(self.thread, poster="Bob").pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_merge_hidden_posts(self):
- """api merges two hidden posts"""
- self.override_acl({'can_hide_posts': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster=self.user, is_hidden=True).pk,
- testutils.reply_thread(self.thread, poster=self.user, is_hidden=True).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_merge_unapproved_posts(self):
- """api merges two unapproved posts"""
- self.override_acl({'can_approve_content': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster=self.user, is_unapproved=True).pk,
- testutils.reply_thread(self.thread, poster=self.user, is_unapproved=True).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_merge_with_hidden_thread(self):
- """api excludes thread's first post from visibility checks"""
- self.thread.first_post.is_hidden = True
- self.thread.first_post.poster = self.user
- self.thread.first_post.save()
- post_visible = testutils.reply_thread(self.thread, poster=self.user, is_hidden=False)
- self.override_acl({'can_hide_threads': 1})
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [self.thread.first_post.pk, post_visible.pk]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- def test_merge_protected(self):
- """api preserves protected status after merge"""
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- testutils.reply_thread(self.thread, poster="Bob", is_protected=True).pk,
- testutils.reply_thread(self.thread, poster="Bob", is_protected=False).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- merged_post = self.thread.post_set.order_by('-id')[0]
- self.assertTrue(merged_post.is_protected)
- def test_merge_best_answer(self):
- """api merges best answer with other post"""
- best_answer = testutils.reply_thread(self.thread, poster="Bob")
- self.thread.set_best_answer(self.user, best_answer)
- self.thread.save()
-
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- best_answer.pk,
- testutils.reply_thread(self.thread, poster="Bob").pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- self.refresh_thread()
- self.assertEqual(self.thread.best_answer, best_answer)
- def test_merge_best_answer_in(self):
- """api merges best answer into other post"""
- other_post = testutils.reply_thread(self.thread, poster="Bob")
- best_answer = testutils.reply_thread(self.thread, poster="Bob")
- self.thread.set_best_answer(self.user, best_answer)
- self.thread.save()
-
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- best_answer.pk,
- other_post.pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- self.refresh_thread()
- self.assertEqual(self.thread.best_answer, other_post)
- def test_merge_best_answer_in_protected(self):
- """api merges best answer into protected post"""
- best_answer = testutils.reply_thread(self.thread, poster="Bob")
-
- self.thread.set_best_answer(self.user, best_answer)
- self.thread.save()
-
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [
- best_answer.pk,
- testutils.reply_thread(self.thread, poster="Bob", is_protected=True).pk,
- ]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- self.refresh_thread()
- self.assertEqual(self.thread.best_answer, best_answer)
- self.assertTrue(self.thread.best_answer.is_protected)
- self.assertTrue(self.thread.best_answer_is_protected)
- def test_merge_remove_reads(self):
- """two posts merge removes read tracker from post"""
- post_a = testutils.reply_thread(self.thread, poster=self.user, message="Battęry")
- post_b = testutils.reply_thread(self.thread, poster=self.user, message="Hórse")
- poststracker.save_read(self.user, post_a)
- poststracker.save_read(self.user, post_b)
- response = self.client.post(
- self.api_link,
- json.dumps({
- 'posts': [post_a.pk, post_b.pk]
- }),
- content_type="application/json",
- )
- self.assertEqual(response.status_code, 200)
- # both post's were removed from readtracker
- self.assertEqual(self.user.postread_set.count(), 0)
|