12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387 |
- import json
- from datetime import timedelta
- from django.utils import timezone
- from misago.categories.models import Category
- from misago.readtracker import poststracker
- from misago.threads import testutils
- from misago.threads.test import patch_category_acl, patch_other_category_acl
- from misago.threads.models import Thread
- from .test_threads_api import ThreadsApiTestCase
- class ThreadPatchApiTestCase(ThreadsApiTestCase):
- def patch(self, api_link, ops):
- return self.client.patch(api_link, json.dumps(ops), content_type="application/json")
- class ThreadAddAclApiTests(ThreadPatchApiTestCase):
- def test_add_acl_true(self):
- """api adds current thread's acl to response"""
- response = self.patch(self.api_link, [
- {
- 'op': 'add',
- 'path': 'acl',
- 'value': True,
- },
- ])
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertTrue(response_json['acl'])
- def test_add_acl_false(self):
- """if value is false, api won't add acl to the response, but will set empty key"""
- response = self.patch(self.api_link, [
- {
- 'op': 'add',
- 'path': 'acl',
- 'value': False,
- },
- ])
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertIsNone(response_json['acl'])
- class ThreadChangeTitleApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_edit_threads': 2})
- def test_change_thread_title(self):
- """api makes it possible to change thread title"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': "Lorem ipsum change!",
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['title'], "Lorem ipsum change!")
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['title'], "Lorem ipsum change!")
- @patch_category_acl({'can_edit_threads': 0})
- def test_change_thread_title_no_permission(self):
- """api validates permission to change title"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': "Lorem ipsum change!",
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(response_json['detail'][0], "You can't edit threads in this category.")
- @patch_category_acl({'can_edit_threads': 2, 'can_close_threads': 0})
- def test_change_thread_title_closed_category_no_permission(self):
- """api test permission to edit thread title in closed category"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': "Lorem ipsum change!",
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This category is closed. You can't edit threads in it."
- )
- @patch_category_acl({'can_edit_threads': 2, 'can_close_threads': 0})
- def test_change_thread_title_closed_thread_no_permission(self):
- """api test permission to edit closed thread title"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': "Lorem ipsum change!",
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This thread is closed. You can't edit it."
- )
- @patch_category_acl({'can_edit_threads': 1, 'thread_edit_time': 1})
- def test_change_thread_title_after_edit_time(self):
- """api cleans, validates and rejects too short title"""
- self.thread.started_on = timezone.now() - timedelta(minutes=10)
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': "Lorem ipsum change!",
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't edit threads that are older than 1 minute."
- )
- @patch_category_acl({'can_edit_threads': 2})
- def test_change_thread_title_invalid(self):
- """api cleans, validates and rejects too short title"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'title',
- 'value': 12,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0],
- "Thread title should be at least 5 characters long (it has 2)."
- )
- class ThreadPinGloballyApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_pin_threads': 2})
- def test_pin_thread(self):
- """api makes it possible to pin globally thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 2,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['weight'], 2)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 2)
- @patch_category_acl({'can_pin_threads': 2, 'can_close_threads': 0})
- def test_pin_thread_closed_category_no_permission(self):
- """api checks if category is closed"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 2,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This category is closed. You can't change threads weights in it."
- )
- @patch_category_acl({'can_pin_threads': 2, 'can_close_threads': 0})
- def test_pin_thread_closed_no_permission(self):
- """api checks if thread is closed"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 2,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This thread is closed. You can't change its weight."
- )
- @patch_category_acl({'can_pin_threads': 2})
- def test_unpin_thread(self):
- """api makes it possible to unpin thread"""
- self.thread.weight = 2
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 2)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 0,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['weight'], 0)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 0)
- @patch_category_acl({'can_pin_threads': 1})
- def test_pin_thread_no_permission(self):
- """api pin thread globally with no permission fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 2,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't pin threads globally in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 0)
- @patch_category_acl({'can_pin_threads': 1})
- def test_unpin_thread_no_permission(self):
- """api unpin thread with no permission fails"""
- self.thread.weight = 2
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 2)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 1,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't change globally pinned threads weights in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 2)
- class ThreadPinLocallyApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_pin_threads': 1})
- def test_pin_thread(self):
- """api makes it possible to pin locally thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 1,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['weight'], 1)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 1)
- @patch_category_acl({'can_pin_threads': 1})
- def test_unpin_thread(self):
- """api makes it possible to unpin thread"""
- self.thread.weight = 1
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 1)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 0,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['weight'], 0)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 0)
- @patch_category_acl({'can_pin_threads': 0})
- def test_pin_thread_no_permission(self):
- """api pin thread locally with no permission fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 1,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't change threads weights in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 0)
- @patch_category_acl({'can_pin_threads': 0})
- def test_unpin_thread_no_permission(self):
- """api unpin thread with no permission fails"""
- self.thread.weight = 1
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 1)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'weight',
- 'value': 0,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't change threads weights in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['weight'], 1)
- class ThreadMoveApiTests(ThreadPatchApiTestCase):
- def setUp(self):
- super().setUp()
- Category(
- name='Other category',
- slug='other-category',
- ).insert_at(
- self.category,
- position='last-child',
- save=True,
- )
- self.dst_category = Category.objects.get(slug='other-category')
- @patch_other_category_acl({'can_start_threads': 2})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_no_top(self):
- """api moves thread to other category, sets no top category"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'add',
- 'path': 'top-category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'replace',
- 'path': 'flatten-categories',
- 'value': None,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertEqual(reponse_json['category'], self.dst_category.pk)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.dst_category.pk)
- @patch_other_category_acl({'can_start_threads': 2})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_with_top(self):
- """api moves thread to other category, sets top"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'add',
- 'path': 'top-category',
- 'value': Category.objects.root_category().pk,
- },
- {
- 'op': 'replace',
- 'path': 'flatten-categories',
- 'value': None,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertEqual(reponse_json['category'], self.dst_category.pk)
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.dst_category.pk)
- @patch_other_category_acl({'can_start_threads': 2})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_reads(self):
- """api moves thread reads together with thread"""
- poststracker.save_read(self.user, self.thread.first_post)
- self.assertEqual(self.user.postread_set.count(), 1)
- self.user.postread_set.get(category=self.category)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'add',
- 'path': 'top-category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'replace',
- 'path': 'flatten-categories',
- 'value': None,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- # thread read was moved to new category
- postreads = self.user.postread_set.filter(post__is_event=False).order_by('id')
- self.assertEqual(postreads.count(), 1)
- postreads.get(category=self.dst_category)
- @patch_other_category_acl({'can_start_threads': 2})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_subscriptions(self):
- """api moves thread subscriptions together with thread"""
- self.user.subscription_set.create(
- thread=self.thread,
- category=self.thread.category,
- last_read_on=self.thread.last_post_on,
- send_email=False,
- )
- self.assertEqual(self.user.subscription_set.count(), 1)
- self.user.subscription_set.get(category=self.category)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'add',
- 'path': 'top-category',
- 'value': self.dst_category.pk,
- },
- {
- 'op': 'replace',
- 'path': 'flatten-categories',
- 'value': None,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- # thread read was moved to new category
- self.assertEqual(self.user.subscription_set.count(), 1)
- self.user.subscription_set.get(category=self.dst_category)
- @patch_category_acl({'can_move_threads': False})
- def test_move_thread_no_permission(self):
- """api move thread to other category with no permission fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't move threads in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.category.pk)
- @patch_other_category_acl({'can_close_threads': False})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_closed_category_no_permission(self):
- """api move thread from closed category with no permission fails"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This category is closed. You can't move it's threads."
- )
- @patch_other_category_acl({'can_close_threads': False})
- @patch_category_acl({'can_move_threads': True})
- def test_move_closed_thread_no_permission(self):
- """api move closed thread with no permission fails"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This thread is closed. You can't move it."
- )
- @patch_other_category_acl({'can_see': False})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_no_category_access(self):
- """api move thread to category with no access fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(response_json['detail'][0], 'NOT FOUND')
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.category.pk)
- @patch_other_category_acl({'can_browse': False})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_no_category_browse(self):
- """api move thread to category with no browsing access fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0],
- 'You don\'t have permission to browse "Other category" contents.'
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.category.pk)
- @patch_other_category_acl({'can_start_threads': False})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_no_category_start_threads(self):
- """api move thread to category with no posting access fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.dst_category.pk,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0],
- "You don't have permission to start new threads in this category."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.category.pk)
- @patch_other_category_acl({'can_start_threads': 2})
- @patch_category_acl({'can_move_threads': True})
- def test_move_thread_same_category(self):
- """api move thread to category it's already in fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'category',
- 'value': self.thread.category_id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't move thread to the category it's already in."
- )
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['category']['id'], self.category.pk)
- def test_thread_flatten_categories(self):
- """api flatten thread categories"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'flatten-categories',
- 'value': None,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertEqual(response_json['category'], self.category.pk)
- class ThreadCloseApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_close_threads': True})
- def test_close_thread(self):
- """api makes it possible to close thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-closed',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertTrue(response_json['is_closed'])
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['is_closed'])
- @patch_category_acl({'can_close_threads': True})
- def test_open_thread(self):
- """api makes it possible to open thread"""
- self.thread.is_closed = True
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['is_closed'])
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-closed',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertFalse(response_json['is_closed'])
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['is_closed'])
- @patch_category_acl({'can_close_threads': False})
- def test_close_thread_no_permission(self):
- """api close thread with no permission fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-closed',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You don't have permission to close this thread."
- )
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['is_closed'])
- @patch_category_acl({'can_close_threads': False})
- def test_open_thread_no_permission(self):
- """api open thread with no permission fails"""
- self.thread.is_closed = True
- self.thread.save()
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['is_closed'])
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-closed',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You don't have permission to open this thread."
- )
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['is_closed'])
- class ThreadApproveApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_approve_content': True})
- def test_approve_thread(self):
- """api makes it possible to approve thread"""
- self.thread.first_post.is_unapproved = True
- self.thread.first_post.save()
- self.thread.synchronize()
- self.thread.save()
- self.assertTrue(self.thread.is_unapproved)
- self.assertTrue(self.thread.has_unapproved_posts)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-unapproved',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- response_json = response.json()
- self.assertFalse(response_json['is_unapproved'])
- self.assertFalse(response_json['has_unapproved_posts'])
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['is_unapproved'])
- self.assertFalse(thread_json['has_unapproved_posts'])
- thread = Thread.objects.get(pk=self.thread.pk)
- self.assertFalse(thread.is_unapproved)
- self.assertFalse(thread.has_unapproved_posts)
- @patch_category_acl({'can_approve_content': True, 'can_close_threads': False})
- def test_approve_thread_category_closed_no_permission(self):
- """api checks permission for approving threads in closed categories"""
- self.thread.first_post.is_unapproved = True
- self.thread.first_post.save()
- self.thread.synchronize()
- self.thread.save()
- self.assertTrue(self.thread.is_unapproved)
- self.assertTrue(self.thread.has_unapproved_posts)
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-unapproved',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(response_json['detail'][0], "This category is closed. You can't approve threads in it.")
- @patch_category_acl({'can_approve_content': True, 'can_close_threads': False})
- def test_approve_thread_closed_no_permission(self):
- """api checks permission for approving posts in closed categories"""
- self.thread.first_post.is_unapproved = True
- self.thread.first_post.save()
- self.thread.synchronize()
- self.thread.save()
- self.assertTrue(self.thread.is_unapproved)
- self.assertTrue(self.thread.has_unapproved_posts)
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-unapproved',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(response_json['detail'][0], "This thread is closed. You can't approve it.")
- @patch_category_acl({'can_approve_content': True})
- def test_unapprove_thread(self):
- """api returns permission error on approval removal"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-unapproved',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(response_json['detail'][0], "Content approval can't be reversed.")
- class ThreadHideApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_hide_threads': True})
- def test_hide_thread(self):
- """api makes it possible to hide thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertTrue(reponse_json['is_hidden'])
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['is_hidden'])
- @patch_category_acl({'can_hide_threads': False})
- def test_hide_thread_no_permission(self):
- """api hide thread with no permission fails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't hide threads in this category."
- )
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['is_hidden'])
- @patch_category_acl({'can_hide_threads': False, 'can_hide_own_threads': True})
- def test_hide_non_owned_thread(self):
- """api forbids non-moderator from hiding other users threads"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't hide other users theads in this category."
- )
- @patch_category_acl({
- 'can_hide_threads': False,
- 'can_hide_own_threads': True,
- 'thread_edit_time': 1,
- })
- def test_hide_owned_thread_no_time(self):
- """api forbids non-moderator from hiding other users threads"""
- self.thread.started_on = timezone.now() - timedelta(minutes=5)
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "You can't hide threads that are older than 1 minute."
- )
- @patch_category_acl({'can_hide_threads': True, 'can_close_threads': False})
- def test_hide_closed_category_no_permission(self):
- """api test permission to hide thread in closed category"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This category is closed. You can't hide threads in it."
- )
- @patch_category_acl({'can_hide_threads': True, 'can_close_threads': False})
- def test_hide_closed_thread_no_permission(self):
- """api test permission to hide closed thread"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This thread is closed. You can't hide it."
- )
- class ThreadUnhideApiTests(ThreadPatchApiTestCase):
- def setUp(self):
- super().setUp()
- self.thread.is_hidden = True
- self.thread.save()
- @patch_category_acl({'can_hide_threads': True})
- def test_unhide_thread(self):
- """api makes it possible to unhide thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertFalse(reponse_json['is_hidden'])
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['is_hidden'])
- @patch_category_acl({'can_hide_threads': False})
- def test_unhide_thread_no_permission(self):
- """api unhide thread with no permission fails as thread is invisible"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': True,
- },
- ]
- )
- self.assertEqual(response.status_code, 404)
- @patch_category_acl({'can_hide_threads': True, 'can_close_threads': False})
- def test_unhide_closed_category_no_permission(self):
- """api test permission to unhide thread in closed category"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This category is closed. You can't reveal threads in it."
- )
- @patch_category_acl({'can_hide_threads': True, 'can_close_threads': False})
- def test_unhide_closed_thread_no_permission(self):
- """api test permission to unhide closed thread"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'is-hidden',
- 'value': False,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- response_json = response.json()
- self.assertEqual(
- response_json['detail'][0], "This thread is closed. You can't reveal it."
- )
- class ThreadSubscribeApiTests(ThreadPatchApiTestCase):
- def test_subscribe_thread(self):
- """api makes it possible to subscribe thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'subscription',
- 'value': 'notify',
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertFalse(reponse_json['subscription'])
- thread_json = self.get_thread_json()
- self.assertFalse(thread_json['subscription'])
- subscription = self.user.subscription_set.get(thread=self.thread)
- self.assertFalse(subscription.send_email)
- def test_subscribe_thread_with_email(self):
- """api makes it possible to subscribe thread with emails"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'subscription',
- 'value': 'email',
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertTrue(reponse_json['subscription'])
- thread_json = self.get_thread_json()
- self.assertTrue(thread_json['subscription'])
- subscription = self.user.subscription_set.get(thread=self.thread)
- self.assertTrue(subscription.send_email)
- def test_unsubscribe_thread(self):
- """api makes it possible to unsubscribe thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'subscription',
- 'value': 'remove',
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- reponse_json = response.json()
- self.assertIsNone(reponse_json['subscription'])
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['subscription'])
- self.assertEqual(self.user.subscription_set.count(), 0)
- def test_subscribe_as_guest(self):
- """api makes it impossible to subscribe thread"""
- self.logout_user()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'subscription',
- 'value': 'email',
- },
- ]
- )
- self.assertEqual(response.status_code, 403)
- def test_subscribe_nonexistant_thread(self):
- """api makes it impossible to subscribe nonexistant thread"""
- bad_api_link = self.api_link.replace(
- str(self.thread.pk), str(self.thread.pk + 9)
- )
- response = self.patch(
- bad_api_link, [
- {
- 'op': 'replace',
- 'path': 'subscription',
- 'value': 'email',
- },
- ]
- )
- self.assertEqual(response.status_code, 404)
- class ThreadMarkBestAnswerApiTests(ThreadPatchApiTestCase):
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer(self):
- """api makes it possible to mark best answer"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ['ok'],
- 'best_answer': best_answer.id,
- 'best_answer_is_protected': False,
- 'best_answer_marked_on': response.json()['best_answer_marked_on'],
- 'best_answer_marked_by': self.user.id,
- 'best_answer_marked_by_name': self.user.username,
- 'best_answer_marked_by_slug': self.user.slug,
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], best_answer.id)
- self.assertEqual(thread_json['best_answer_is_protected'], False)
- self.assertEqual(
- thread_json['best_answer_marked_on'], response.json()['best_answer_marked_on'])
- self.assertEqual(thread_json['best_answer_marked_by'], self.user.id)
- self.assertEqual(thread_json['best_answer_marked_by_name'], self.user.username)
- self.assertEqual(thread_json['best_answer_marked_by_slug'], self.user.slug)
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_anonymous(self):
- """api validates that user is authenticated before marking best answer"""
- self.logout_user()
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 403)
- self.assertEqual(response.json(), {
- 'detail': "This action is not available to guests.",
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 0})
- def test_mark_best_answer_no_permission(self):
- """api validates permission to mark best answers"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to mark best answers in the "First category" category.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 1})
- def test_mark_best_answer_not_thread_starter(self):
- """api validates permission to mark best answers in owned thread"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to mark best answer in this thread because you didn't "
- "start it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- # passing scenario is possible
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_close_threads': False})
- def test_mark_best_answer_category_closed_no_permission(self):
- """api validates permission to mark best answers in closed category"""
- best_answer = testutils.reply_thread(self.thread)
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to mark best answer in this thread because its '
- 'category "First category" is closed.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2, 'can_close_threads': True})
- def test_mark_best_answer_category_closed(self):
- """api validates permission to mark best answers in closed category"""
- best_answer = testutils.reply_thread(self.thread)
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_close_threads': False})
- def test_mark_best_answer_thread_closed_no_permission(self):
- """api validates permission to mark best answers in closed thread"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You can't mark best answer in this thread because it's closed and you don't have "
- "permission to open it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2, 'can_close_threads': True})
- def test_mark_best_answer_thread_closed(self):
- """api validates permission to mark best answers in closed thread"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
-
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_invalid_post_id(self):
- """api validates that post id is int"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': 'd7sd89a7d98sa',
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["A valid integer is required."],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_post_not_found(self):
- """api validates that post exists"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': self.thread.last_post_id + 1,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["NOT FOUND"],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_post_invisible(self):
- """api validates post visibility to action author"""
- unapproved_post = testutils.reply_thread(self.thread, is_unapproved=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': unapproved_post.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["NOT FOUND"],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_post_other_thread(self):
- """api validates post belongs to same thread"""
- other_thread = testutils.post_thread(self.category)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': other_thread.first_post_id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["NOT FOUND"],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_event_id(self):
- """api validates that post is not an event"""
- best_answer = testutils.reply_thread(self.thread)
- best_answer.is_event = True
- best_answer.save()
-
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["Events can't be marked as best answers."],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_first_post(self):
- """api validates that post is not a first post in thread"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': self.thread.first_post_id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["First post in a thread can't be marked as best answer."],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_hidden_post(self):
- """api validates that post is not hidden"""
- best_answer = testutils.reply_thread(self.thread, is_hidden=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["Hidden posts can't be marked as best answers."],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2})
- def test_mark_best_answer_unapproved_post(self):
- """api validates that post is not unapproved"""
- best_answer = testutils.reply_thread(self.thread, poster=self.user, is_unapproved=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["Unapproved posts can't be marked as best answers."],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2, 'can_protect_posts': False})
- def test_mark_best_answer_protected_post_no_permission(self):
- """api respects post protection"""
- best_answer = testutils.reply_thread(self.thread, is_protected=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to mark this post as best answer because a moderator "
- "has protected it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- @patch_category_acl({'can_mark_best_answers': 2, 'can_protect_posts': True})
- def test_mark_best_answer_protected_post(self):
- """api respects post protection"""
- best_answer = testutils.reply_thread(self.thread, is_protected=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- class ThreadChangeBestAnswerApiTests(ThreadPatchApiTestCase):
- def setUp(self):
- super().setUp()
- self.best_answer = testutils.reply_thread(self.thread)
- self.thread.set_best_answer(self.user, self.best_answer)
- self.thread.save()
- @patch_category_acl({'can_mark_best_answers': 2, 'can_change_marked_answers': 2})
- def test_change_best_answer(self):
- """api makes it possible to change best answer"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ['ok'],
- 'best_answer': best_answer.id,
- 'best_answer_is_protected': False,
- 'best_answer_marked_on': response.json()['best_answer_marked_on'],
- 'best_answer_marked_by': self.user.id,
- 'best_answer_marked_by_name': self.user.username,
- 'best_answer_marked_by_slug': self.user.slug,
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], best_answer.id)
- self.assertEqual(thread_json['best_answer_is_protected'], False)
- self.assertEqual(
- thread_json['best_answer_marked_on'], response.json()['best_answer_marked_on'])
- self.assertEqual(thread_json['best_answer_marked_by'], self.user.id)
- self.assertEqual(thread_json['best_answer_marked_by_name'], self.user.username)
- self.assertEqual(thread_json['best_answer_marked_by_slug'], self.user.slug)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_change_marked_answers': 2})
- def test_change_best_answer_same_post(self):
- """api validates if new best answer is same as current one"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["This post is already marked as thread's best answer."],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 2})
- def test_change_best_answer_no_permission_to_mark(self):
- """api validates permission to mark best answers before allowing answer change"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to mark best answers in the "First category" category.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_change_marked_answers': 0})
- def test_change_best_answer_no_permission(self):
- """api validates permission to change best answers"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to change this thread\'s marked answer because it\'s '
- 'in the "First category" category.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_change_marked_answers': 1})
- def test_change_best_answer_not_starter(self):
- """api validates permission to change best answers"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to change this thread's marked answer because you are "
- "not a thread starter."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- # passing scenario is possible
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 2,
- 'can_change_marked_answers': 1,
- 'best_answer_change_time': 5,
- })
- def test_change_best_answer_timelimit_out_of_time(self):
- """api validates permission for starter to change best answers within timelimit"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.best_answer_marked_on = timezone.now() - timedelta(minutes=6)
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to change best answer that was marked for more than "
- "5 minutes."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({
- 'can_mark_best_answers': 2,
- 'can_change_marked_answers': 1,
- 'best_answer_change_time': 5,
- })
- def test_change_best_answer_timelimit(self):
- """api validates permission for starter to change best answers within timelimit"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.best_answer_marked_on = timezone.now() - timedelta(minutes=1)
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 2,
- 'can_change_marked_answers': 2,
- 'can_protect_posts': False,
- })
- def test_change_best_answer_protected_no_permission(self):
- """api validates permission to change protected best answers"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.best_answer_is_protected = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to change this thread's best answer because a "
- "moderator has protected it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({
- 'can_mark_best_answers': 2,
- 'can_change_marked_answers': 2,
- 'can_protect_posts': True,
- })
- def test_change_best_answer_protected(self):
- """api validates permission to change protected best answers"""
- best_answer = testutils.reply_thread(self.thread)
- self.thread.best_answer_is_protected = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({'can_mark_best_answers': 2, 'can_change_marked_answers': 2})
- def test_change_best_answer_post_validation(self):
- """api validates new post'"""
- best_answer = testutils.reply_thread(self.thread, is_hidden=True)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'replace',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
-
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- class ThreadUnmarkBestAnswerApiTests(ThreadPatchApiTestCase):
- def setUp(self):
- super().setUp()
- self.best_answer = testutils.reply_thread(self.thread)
- self.thread.set_best_answer(self.user, self.best_answer)
- self.thread.save()
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 2})
- def test_unmark_best_answer(self):
- """api makes it possible to unmark best answer"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ['ok'],
- 'best_answer': None,
- 'best_answer_is_protected': False,
- 'best_answer_marked_on': None,
- 'best_answer_marked_by': None,
- 'best_answer_marked_by_name': None,
- 'best_answer_marked_by_slug': None,
- })
- thread_json = self.get_thread_json()
- self.assertIsNone(thread_json['best_answer'])
- self.assertFalse(thread_json['best_answer_is_protected'])
- self.assertIsNone(thread_json['best_answer_marked_on'])
- self.assertIsNone(thread_json['best_answer_marked_by'])
- self.assertIsNone(thread_json['best_answer_marked_by_name'])
- self.assertIsNone(thread_json['best_answer_marked_by_slug'])
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 2})
- def test_unmark_best_answer_invalid_post_id(self):
- """api validates that post id is int"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': 'd7sd89a7d98sa',
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["A valid integer is required."],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 2})
- def test_unmark_best_answer_post_not_found(self):
- """api validates that post to unmark exists"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id + 1,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': ["NOT FOUND"],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 2})
- def test_unmark_best_answer_wrong_post(self):
- """api validates if post given to unmark is best answer"""
- best_answer = testutils.reply_thread(self.thread)
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "This post can't be unmarked because it's not currently marked as best answer."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 0})
- def test_unmark_best_answer_no_permission(self):
- """api validates if user has permission to unmark best answers"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to unmark threads answers in the "First category" '
- 'category.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({'can_mark_best_answers': 0, 'can_change_marked_answers': 1})
- def test_unmark_best_answer_not_starter(self):
- """api validates if starter has permission to unmark best answers"""
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to unmark this best answer because you are not a "
- "thread starter."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- # passing scenario is possible
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 1,
- 'best_answer_change_time': 5,
- })
- def test_unmark_best_answer_timelimit(self):
- """api validates if starter has permission to unmark best answer within time limit"""
- self.thread.best_answer_marked_on = timezone.now() - timedelta(minutes=6)
- self.thread.starter = self.user
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to unmark best answer that was marked for more than "
- "5 minutes."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- # passing scenario is possible
- self.thread.best_answer_marked_on = timezone.now() - timedelta(minutes=2)
- self.thread.save()
-
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_close_threads': False,
- })
- def test_unmark_best_answer_closed_category_no_permission(self):
- """api validates if user has permission to unmark best answer in closed category"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- 'You don\'t have permission to unmark this best answer because its category '
- '"First category" is closed.'
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_close_threads': True,
- })
- def test_unmark_best_answer_closed_category(self):
- """api validates if user has permission to unmark best answer in closed category"""
- self.category.is_closed = True
- self.category.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_close_threads': False,
- })
- def test_unmark_best_answer_closed_thread_no_permission(self):
- """api validates if user has permission to unmark best answer in closed thread"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You can't unmark this thread's best answer because it's closed and you don't "
- "have permission to open it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_close_threads': True,
- })
- def test_unmark_best_answer_closed_thread(self):
- """api validates if user has permission to unmark best answer in closed thread"""
- self.thread.is_closed = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_protect_posts': 0,
- })
- def test_unmark_best_answer_protected_no_permission(self):
- """api validates permission to unmark protected best answers"""
- self.thread.best_answer_is_protected = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 400)
- self.assertEqual(response.json(), {
- 'id': self.thread.id,
- 'detail': [
- "You don't have permission to unmark this thread's best answer because a "
- "moderator has protected it."
- ],
- })
- thread_json = self.get_thread_json()
- self.assertEqual(thread_json['best_answer'], self.best_answer.id)
- @patch_category_acl({
- 'can_mark_best_answers': 0,
- 'can_change_marked_answers': 2,
- 'can_protect_posts': 1,
- })
- def test_unmark_best_answer_protected(self):
- """api validates permission to unmark protected best answers"""
- self.thread.best_answer_is_protected = True
- self.thread.save()
- response = self.patch(
- self.api_link, [
- {
- 'op': 'remove',
- 'path': 'best-answer',
- 'value': self.best_answer.id,
- },
- ]
- )
- self.assertEqual(response.status_code, 200)
|