123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611 |
- from datetime import timedelta
- from django.urls import reverse
- from django.utils import timezone
- from misago.acl import add_acl
- from misago.core.utils import serialize_datetime
- from misago.threads.models import Poll
- from misago.threads.serializers.poll import MAX_POLL_OPTIONS
- from .test_thread_poll_api import ThreadPollApiTestCase
- class ThreadPollEditTests(ThreadPollApiTestCase):
- def setUp(self):
- super(ThreadPollEditTests, self).setUp()
- self.mock_poll()
- def test_anonymous(self):
- """api requires you to sign in to edit poll"""
- self.logout_user()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "This action is not available to guests.",
- })
- def test_invalid_thread_id(self):
- """api validates that thread id is integer"""
- api_link = reverse(
- 'misago:api:thread-poll-detail',
- kwargs={
- 'thread_pk': 'kjha6dsa687sa',
- 'pk': self.poll.pk,
- }
- )
- response = self.put(api_link)
- self.assertEqual(response.status_code, 404)
- self.assertEqual(response.json(), {'detail': 'NOT FOUND'})
- def test_nonexistant_thread_id(self):
- """api validates that thread exists"""
- api_link = reverse(
- 'misago:api:thread-poll-detail',
- kwargs={
- 'thread_pk': self.thread.pk + 1,
- 'pk': self.poll.pk,
- }
- )
- response = self.put(api_link)
- self.assertEqual(response.status_code, 404)
- self.assertEqual(response.json(), {'detail': 'NOT FOUND'})
- def test_invalid_poll_id(self):
- """api validates that poll id is integer"""
- api_link = reverse(
- 'misago:api:thread-poll-detail',
- kwargs={
- 'thread_pk': self.thread.pk,
- 'pk': 'sad98as7d97sa98',
- }
- )
- response = self.put(api_link)
- self.assertEqual(response.status_code, 404)
- self.assertEqual(response.json(), {'detail': 'NOT FOUND'})
- def test_nonexistant_poll_id(self):
- """api validates that poll exists"""
- api_link = reverse(
- 'misago:api:thread-poll-detail',
- kwargs={
- 'thread_pk': self.thread.pk,
- 'pk': self.poll.pk + 123,
- }
- )
- response = self.put(api_link)
- self.assertEqual(response.status_code, 404)
- self.assertEqual(response.json(), {'detail': 'NOT FOUND'})
- def test_no_permission(self):
- """api validates that user has permission to edit poll in thread"""
- self.override_acl({'can_edit_polls': 0})
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "You can't edit polls.",
- })
- def test_no_permission_timeout(self):
- """api validates that user's window to edit poll in thread has closed"""
- self.override_acl({'can_edit_polls': 1, 'poll_edit_time': 5})
- self.poll.posted_on = timezone.now() - timedelta(minutes=15)
- self.poll.save()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "You can't edit polls that are older than 5 minutes.",
- })
- def test_no_permission_poll_closed(self):
- """api validates that user's window to edit poll in thread has closed"""
- self.override_acl({'can_edit_polls': 1})
- self.poll.posted_on = timezone.now() - timedelta(days=15)
- self.poll.length = 5
- self.poll.save()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "This poll is over. You can't edit it.",
- })
- def test_no_permission_other_user_poll(self):
- """api validates that user has permission to edit other user poll in thread"""
- self.override_acl({'can_edit_polls': 1})
- self.poll.poster = None
- self.poll.save()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "You can't edit other users polls in this category.",
- })
- def test_no_permission_closed_thread(self):
- """api validates that user has permission to edit poll in closed thread"""
- self.override_acl(category={'can_close_threads': 0})
- self.thread.is_closed = True
- self.thread.save()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "This thread is closed. You can't edit polls in it.",
- })
- self.override_acl(category={'can_close_threads': 1})
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 400)
- def test_no_permission_closed_category(self):
- """api validates that user has permission to edit poll in closed category"""
- self.override_acl(category={'can_close_threads': 0})
- self.category.is_closed = True
- self.category.save()
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "This category is closed. You can't edit polls in it.",
- })
- self.override_acl(category={'can_close_threads': 1})
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 400)
- def test_empty_data(self):
- """api handles empty request data"""
- response = self.put(self.api_link)
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["This field is required."],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
- def test_length_validation(self):
- """api validates poll's length"""
- response = self.put(
- self.api_link, data={
- 'length': -1,
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["This field is required."],
- 'length': ["Ensure this value is greater than or equal to 0."],
- 'allowed_choices': ["This field is required."],
- })
- response = self.put(
- self.api_link, data={
- 'length': 200,
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["This field is required."],
- 'length': ["Ensure this value is less than or equal to 180."],
- 'allowed_choices': ["This field is required."],
- })
- def test_question_validation(self):
- """api validates question length"""
- response = self.put(
- self.api_link, data={
- 'question': 'abcd' * 255,
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["Ensure this field has no more than 255 characters."],
- 'choices': ["This field is required."],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
- def test_validate_choice_length(self):
- """api validates single choice length"""
- response = self.put(
- self.api_link, data={
- 'choices': [
- {
- 'hash': 'qwertyuiopas',
- 'label': '',
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["One or more poll choices are invalid."],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
- response = self.put(
- self.api_link,
- data={
- 'choices': [
- {
- 'hash': 'qwertyuiopas',
- 'label': 'abcd' * 255,
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["One or more poll choices are invalid."],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
-
- def test_validate_two_choices(self):
- """api validates that there are at least two choices in poll"""
- response = self.put(self.api_link, data={'choices': [{'label': 'Choice'}]})
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["You need to add at least two choices to a poll."],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
- def test_validate_max_choices(self):
- """api validates that there are no more choices in poll than allowed number"""
- response = self.put(
- self.api_link, data={
- 'choices': [
- {
- 'label': 'Choice',
- },
- ] * (MAX_POLL_OPTIONS + 1),
- }
- )
- self.assertEqual(response.status_code, 400)
- error_formats = (MAX_POLL_OPTIONS, MAX_POLL_OPTIONS + 1)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': [
- "You can't add more than %s options to a single poll (added %s)." % error_formats
- ],
- 'length': ["This field is required."],
- 'allowed_choices': ["This field is required."],
- })
- def test_allowed_choices_validation(self):
- """api validates allowed choices number"""
- response = self.put(self.api_link, data={'allowed_choices': 0})
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'question': ["This field is required."],
- 'choices': ["This field is required."],
- 'length': ["This field is required."],
- 'allowed_choices': ["Ensure this value is greater than or equal to 1."],
- })
- response = self.put(
- self.api_link,
- data={
- 'length': 0,
- 'question': "Lorem ipsum",
- 'allowed_choices': 3,
- 'choices': [
- {
- 'label': 'Choice',
- },
- {
- 'label': 'Choice',
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'non_field_errors': [
- "Number of allowed choices can't be greater than number of all choices."
- ],
- })
- def test_poll_all_choices_replaced(self):
- """api edits all poll choices out"""
- response = self.put(
- self.api_link,
- data={
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'is_public': True,
- 'choices': [
- {
- 'label': '\nRed ',
- },
- {
- 'label': 'Green',
- },
- {
- 'label': 'Blue',
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 200)
- poll = Poll.objects.all()[0]
- add_acl(self.user, poll)
- response_json = response.json()
- self.assertEqual(response_json['poster_name'], self.user.username)
- self.assertEqual(response_json['length'], 40)
- self.assertEqual(response_json['question'], "Select two best colors")
- self.assertEqual(response_json['allowed_choices'], 2)
- self.assertTrue(response_json['allow_revotes'])
- # you can't change poll's type after its posted
- self.assertFalse(response_json['is_public'])
- # choices were updated
- self.assertEqual(len(response_json['choices']), 3)
- self.assertEqual(len(set([c['hash'] for c in response_json['choices']])), 3)
- self.assertEqual([c['label'] for c in response_json['choices']], ['Red', 'Green', 'Blue'])
- self.assertEqual([c['votes'] for c in response_json['choices']], [0, 0, 0])
- self.assertEqual([c['selected'] for c in response_json['choices']], [False, False, False])
- # votes were removed
- self.assertEqual(response_json['votes'], 0)
- self.assertEqual(self.poll.pollvote_set.count(), 0)
- def test_poll_current_choices_edited(self):
- """api edits current poll choices"""
- response = self.put(
- self.api_link,
- data={
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'is_public': True,
- 'choices': [
- {
- 'hash': 'aaaaaaaaaaaa',
- 'label': '\nFirst ',
- 'votes': 5555,
- },
- {
- 'hash': 'bbbbbbbbbbbb',
- 'label': 'Second',
- 'votes': 5555,
- },
- {
- 'hash': 'gggggggggggg',
- 'label': 'Third',
- 'votes': 5555,
- },
- {
- 'hash': 'dddddddddddd',
- 'label': 'Fourth',
- 'votes': 5555,
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 200)
- poll = Poll.objects.all()[0]
- add_acl(self.user, poll)
- self.assertEqual(response.json(), {
- 'id': poll.id,
- 'poster_name': self.user.username,
- 'posted_on': serialize_datetime(poll.posted_on),
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'votes': 4,
- 'is_public': False,
- 'acl': poll.acl,
- 'choices': [
- {
- 'hash': 'aaaaaaaaaaaa',
- 'label': 'First',
- 'votes': 1,
- 'selected': False,
- },
- {
- 'hash': 'bbbbbbbbbbbb',
- 'label': 'Second',
- 'votes': 0,
- 'selected': False,
- },
- {
- 'hash': 'gggggggggggg',
- 'label': 'Third',
- 'votes': 2,
- 'selected': True,
- },
- {
- 'hash': 'dddddddddddd',
- 'label': 'Fourth',
- 'votes': 1,
- 'selected': True,
- },
- ],
- 'api': {
- 'index': poll.get_api_url(),
- 'votes': poll.get_votes_api_url(),
- },
- 'url': {
- 'poster': self.user.get_absolute_url(),
- },
- })
- # no votes were removed
- self.assertEqual(self.poll.pollvote_set.count(), 4)
- def test_poll_some_choices_edited(self):
- """api edits some poll choices"""
- response = self.put(
- self.api_link,
- data={
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'is_public': True,
- 'choices': [
- {
- 'hash': 'aaaaaaaaaaaa',
- 'label': '\nFirst ',
- 'votes': 5555,
- },
- {
- 'hash': 'bbbbbbbbbbbb',
- 'label': 'Second',
- 'votes': 5555,
- },
- {
- 'hash': 'dsadsadsa788',
- 'label': 'New Option',
- 'votes': 5555,
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 200)
- poll = Poll.objects.all()[0]
- add_acl(self.user, poll)
- self.assertEqual(response.json(), {
- 'id': poll.id,
- 'poster_name': self.user.username,
- 'posted_on': serialize_datetime(poll.posted_on),
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'votes': 1,
- 'is_public': False,
- 'acl': poll.acl,
- 'choices': [
- {
- 'hash': 'aaaaaaaaaaaa',
- 'label': 'First',
- 'votes': 1,
- 'selected': False,
- },
- {
- 'hash': 'bbbbbbbbbbbb',
- 'label': 'Second',
- 'votes': 0,
- 'selected': False,
- },
- {
- 'hash': poll.choices[2]['hash'],
- 'label': 'New Option',
- 'votes': 0,
- 'selected': False,
- },
- ],
- 'api': {
- 'index': poll.get_api_url(),
- 'votes': poll.get_votes_api_url(),
- },
- 'url': {
- 'poster': self.user.get_absolute_url(),
- },
- })
- # no votes were removed
- self.assertEqual(self.poll.pollvote_set.count(), 1)
- def test_moderate_user_poll(self):
- """api edits all poll choices out in other users poll, even if its over"""
- self.override_acl({'can_edit_polls': 2, 'poll_edit_time': 5})
- self.poll.poster = None
- self.poll.posted_on = timezone.now() - timedelta(days=15)
- self.poll.length = 5
- self.poll.save()
-
- response = self.put(
- self.api_link,
- data={
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'is_public': True,
- 'choices': [
- {
- 'label': '\nRed ',
- },
- {
- 'label': 'Green',
- },
- {
- 'label': 'Blue',
- },
- ],
- }
- )
- self.assertEqual(response.status_code, 200)
- poll = Poll.objects.all()[0]
- add_acl(self.user, poll)
- expected_choices = []
- for choice in poll.choices:
- self.assertIn(choice['label'], ["Red", "Green", "Blue"])
- expected_choices.append(choice.copy())
- expected_choices[-1]['selected'] = False
- self.assertEqual(response.json(), {
- 'id': poll.id,
- 'poster_name': self.user.username,
- 'posted_on': serialize_datetime(poll.posted_on),
- 'length': 40,
- 'question': "Select two best colors",
- 'allowed_choices': 2,
- 'allow_revotes': True,
- 'votes': 0,
- 'is_public': False,
- 'acl': poll.acl,
- 'choices': expected_choices,
- 'api': {
- 'index': poll.get_api_url(),
- 'votes': poll.get_votes_api_url(),
- },
- 'url': {
- 'poster': None,
- },
- })
-
- # votes were removed
- self.assertEqual(self.poll.pollvote_set.count(), 0)
|