123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624 |
- import json
- from django.contrib.auth import get_user_model
- from django.core import mail
- from misago.acl.testutils import override_acl
- from misago.threads import testutils
- from misago.threads.models import Thread, ThreadParticipant
- from .test_privatethreads import PrivateThreadsTestCase
- UserModel = get_user_model()
- class PrivateThreadPatchApiTestCase(PrivateThreadsTestCase):
- def setUp(self):
- super(PrivateThreadPatchApiTestCase, self).setUp()
- self.thread = testutils.post_thread(self.category, poster=self.user)
- self.api_link = self.thread.get_api_url()
- self.other_user = UserModel.objects.create_user(
- 'BobBoberson', 'bob@boberson.com', 'pass123')
- def patch(self, api_link, ops):
- return self.client.patch(
- api_link, json.dumps(ops), content_type="application/json")
- class PrivateThreadAddParticipantApiTests(PrivateThreadPatchApiTestCase):
- def test_add_participant_not_owner(self):
- """non-owner can't add participant"""
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.user.username}
- ])
- self.assertContains(
- response, "be thread owner to add new participants to it", status_code=400)
- def test_add_empty_username(self):
- """path validates username"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': ''}
- ])
- self.assertContains(
- response, "You have to enter new participant's username.", status_code=400)
- def test_add_nonexistant_user(self):
- """can't user two times"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': 'InvalidUser'}
- ])
- self.assertContains(response, "No user with such name exists.", status_code=400)
- def test_add_already_participant(self):
- """can't add user that is already participant"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.user.username}
- ])
- self.assertContains(
- response, "This user is already thread participant", status_code=400)
- def test_add_blocking_user(self):
- """can't add user that is already participant"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- self.other_user.blocks.add(self.user)
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- self.assertContains(response, "BobBoberson is blocking you.", status_code=400)
- def test_add_no_perm_user(self):
- """can't add user that has no permission to use private threads"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- override_acl(self.other_user, {
- 'can_use_private_threads': 0
- })
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- self.assertContains(response, "BobBoberson can't participate", status_code=400)
- def test_add_too_many_users(self):
- """can't add user that is already participant"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- for i in range(self.user.acl_cache['max_private_thread_participants']):
- user = UserModel.objects.create_user(
- 'User{}'.format(i), 'user{}@example.com'.format(i), 'Pass.123')
- ThreadParticipant.objects.add_participants(self.thread, [user])
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- self.assertContains(
- response, "You can't add any more new users to this thread.", status_code=400)
- def test_add_user_closed_thread(self):
- """adding user to closed thread fails for non-moderator"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- self.assertContains(
- response, "Only moderators can add participants to closed threads.", status_code=400)
- def test_add_user(self):
- """adding user to thread add user to thread as participant, sets event and emails him"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- # event was set on thread
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'added_participant')
- # notification about new private thread was sent to other user
- self.assertEqual(len(mail.outbox), 1)
- email = mail.outbox[-1]
- self.assertIn(self.user.username, email.subject)
- self.assertIn(self.thread.title, email.subject)
- def test_add_user_to_other_user_thread_moderator(self):
- """moderators can add users to other users threads"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- self.thread.has_reported_posts = True
- self.thread.save()
- override_acl(self.user, {
- 'can_moderate_private_threads': 1
- })
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.user.username}
- ])
- # event was set on thread
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'entered_thread')
- # notification about new private thread wasn't send because we invited ourselves
- self.assertEqual(len(mail.outbox), 0)
- def test_add_user_to_closed_moderator(self):
- """moderators can add users to closed threads"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- self.thread.is_closed = True
- self.thread.save()
- override_acl(self.user, {
- 'can_moderate_private_threads': 1
- })
- response = self.patch(self.api_link, [
- {'op': 'add', 'path': 'participants', 'value': self.other_user.username}
- ])
- # event was set on thread
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'added_participant')
- # notification about new private thread was sent to other user
- self.assertEqual(len(mail.outbox), 1)
- email = mail.outbox[-1]
- self.assertIn(self.user.username, email.subject)
- self.assertIn(self.thread.title, email.subject)
- class PrivateThreadRemoveParticipantApiTests(PrivateThreadPatchApiTestCase):
- def test_remove_empty(self):
- """api handles empty user id"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': 'string'}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_remove_invalid(self):
- """api validates user id type"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': 'string'}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_remove_nonexistant(self):
- """removed user has to be participant"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.other_user.pk}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_remove_not_owner(self):
- """api validates if user trying to remove other user is an owner"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.other_user.pk}
- ])
- self.assertContains(
- response, "be thread owner to remove participants from it", status_code=400)
- def test_owner_remove_user_closed_thread(self):
- """api disallows owner to remove other user from closed thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.other_user.pk}
- ])
- self.assertContains(
- response, "moderators can remove participants from closed threads", status_code=400)
- def test_user_leave_thread(self):
- """api allows user to remove himself from thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- self.user.subscription_set.create(
- category=self.category,
- thread=self.thread,
- )
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertFalse(response.json()['deleted'])
- # thread still exists
- self.assertTrue(Thread.objects.get(pk=self.thread.pk))
- # leave event has valid type
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'participant_left')
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # user was removed from participation
- self.assertEqual(self.thread.participants.count(), 1)
- self.assertEqual(self.thread.participants.filter(pk=self.user.pk).count(), 0)
- # thread was removed from user subscriptions
- self.assertEqual(self.user.subscription_set.count(), 0)
- def test_user_leave_closed_thread(self):
- """api allows user to remove himself from closed thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertFalse(response.json()['deleted'])
- # thread still exists
- self.assertTrue(Thread.objects.get(pk=self.thread.pk))
- # leave event has valid type
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'participant_left')
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # user was removed from participation
- self.assertEqual(self.thread.participants.count(), 1)
- self.assertEqual(self.thread.participants.filter(pk=self.user.pk).count(), 0)
- def test_moderator_remove_user(self):
- """api allows moderator to remove other user"""
- removed_user = UserModel.objects.create_user(
- 'Vigilante', 'test@test.com', 'pass123')
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user, removed_user])
- override_acl(self.user, {
- 'can_moderate_private_threads': True
- })
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': removed_user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertFalse(response.json()['deleted'])
- # thread still exists
- self.assertTrue(Thread.objects.get(pk=self.thread.pk))
- # leave event has valid type
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'participant_removed')
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=removed_user.pk).sync_unread_private_threads)
- # user was removed from participation
- self.assertEqual(self.thread.participants.count(), 2)
- self.assertEqual(self.thread.participants.filter(pk=removed_user.pk).count(), 0)
- def test_owner_remove_user(self):
- """api allows owner to remove other user"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.other_user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertFalse(response.json()['deleted'])
- # thread still exists
- self.assertTrue(Thread.objects.get(pk=self.thread.pk))
- # leave event has valid type
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'participant_removed')
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # user was removed from participation
- self.assertEqual(self.thread.participants.count(), 1)
- self.assertEqual(self.thread.participants.filter(pk=self.other_user.pk).count(), 0)
- def test_owner_leave_thread(self):
- """api allows owner to remove hisemf from thread, causing thread to close"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertFalse(response.json()['deleted'])
- # thread still exists and is closed
- self.assertTrue(Thread.objects.get(pk=self.thread.pk).is_closed)
- # leave event has valid type
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'owner_left')
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # user was removed from participation
- self.assertEqual(self.thread.participants.count(), 1)
- self.assertEqual(self.thread.participants.filter(pk=self.user.pk).count(), 0)
- def test_last_user_leave_thread(self):
- """api allows last user leave thread, causing thread to delete"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'remove', 'path': 'participants', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- self.assertTrue(response.json()['deleted'])
- # thread is gone
- with self.assertRaises(Thread.DoesNotExist):
- Thread.objects.get(pk=self.thread.pk)
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- class PrivateThreadTakeOverApiTests(PrivateThreadPatchApiTestCase):
- def test_empty_user_id(self):
- """api handles empty user id"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': ''}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_invalid_user_id(self):
- """api handles invalid user id"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': 'dsadsa'}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_nonexistant_user_id(self):
- """api handles nonexistant user id"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.other_user.pk}
- ])
- self.assertContains(response, "Participant doesn't exist.", status_code=400)
- def test_no_permission(self):
- """non-moderator/owner can't change owner"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.user.pk}
- ])
- self.assertContains(
- response, "thread owner and moderators can change threads owners", status_code=400)
- def test_no_change(self):
- """api validates that new owner id is same as current owner"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.user.pk}
- ])
- self.assertContains(response, "This user already is thread owner.", status_code=400)
- def test_change_closed_thread_owner(self):
- """non-moderator can't change owner in closed thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.other_user.pk}
- ])
- self.assertContains(
- response, "Only moderators can change closed threads owners.", status_code=400)
- def test_owner_change_thread_owner(self):
- """owner can pass thread ownership to other participant"""
- ThreadParticipant.objects.set_owner(self.thread, self.user)
- ThreadParticipant.objects.add_participants(self.thread, [self.other_user])
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.other_user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- # valid users were flagged for sync
- self.assertFalse(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # ownership was transfered
- self.assertEqual(self.thread.participants.count(), 2)
- self.assertTrue(ThreadParticipant.objects.get(user=self.other_user).is_owner)
- self.assertFalse(ThreadParticipant.objects.get(user=self.user).is_owner)
- # change was recorded in event
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'changed_owner')
- def test_moderator_change_owner(self):
- """moderator can change thread owner to other user"""
- new_owner = UserModel.objects.create_user(
- 'NewOwner', 'new@owner.com', 'pass123')
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user, new_owner])
- override_acl(self.user, {
- 'can_moderate_private_threads': 1
- })
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': new_owner.pk}
- ])
- self.assertEqual(response.status_code, 200)
- # valid users were flagged for sync
- self.assertTrue(UserModel.objects.get(pk=new_owner.pk).sync_unread_private_threads)
- self.assertFalse(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # ownership was transfered
- self.assertEqual(self.thread.participants.count(), 3)
- self.assertTrue(ThreadParticipant.objects.get(user=new_owner).is_owner)
- self.assertFalse(ThreadParticipant.objects.get(user=self.user).is_owner)
- self.assertFalse(ThreadParticipant.objects.get(user=self.other_user).is_owner)
- # change was recorded in event
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'changed_owner')
- def test_moderator_takeover(self):
- """moderator can takeover the thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- override_acl(self.user, {
- 'can_moderate_private_threads': 1
- })
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- # valid users were flagged for sync
- self.assertFalse(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # ownership was transfered
- self.assertEqual(self.thread.participants.count(), 2)
- self.assertTrue(ThreadParticipant.objects.get(user=self.user).is_owner)
- self.assertFalse(ThreadParticipant.objects.get(user=self.other_user).is_owner)
- # change was recorded in event
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'tookover')
- def test_moderator_closed_thread_takeover(self):
- """moderator can takeover closed thread thread"""
- ThreadParticipant.objects.set_owner(self.thread, self.other_user)
- ThreadParticipant.objects.add_participants(self.thread, [self.user])
- self.thread.is_closed = True
- self.thread.save()
- override_acl(self.user, {
- 'can_moderate_private_threads': 1
- })
- response = self.patch(self.api_link, [
- {'op': 'replace', 'path': 'owner', 'value': self.user.pk}
- ])
- self.assertEqual(response.status_code, 200)
- # valid users were flagged for sync
- self.assertFalse(UserModel.objects.get(pk=self.user.pk).sync_unread_private_threads)
- self.assertTrue(UserModel.objects.get(pk=self.other_user.pk).sync_unread_private_threads)
- # ownership was transfered
- self.assertEqual(self.thread.participants.count(), 2)
- self.assertTrue(ThreadParticipant.objects.get(user=self.user).is_owner)
- self.assertFalse(ThreadParticipant.objects.get(user=self.other_user).is_owner)
- # change was recorded in event
- event = self.thread.post_set.order_by('id').last()
- self.assertTrue(event.is_event)
- self.assertTrue(event.event_type, 'tookover')
|