2010-09-16 16:58:11 +02:00
|
|
|
# -*- coding: utf-8 -*-
|
|
|
|
from plone.app.discussion.browser.moderation import BulkActionsView
|
2016-09-19 17:06:17 +02:00
|
|
|
from plone.app.discussion.browser.moderation import DeleteComment
|
|
|
|
from plone.app.discussion.browser.moderation import PublishComment
|
2015-05-03 08:16:39 +02:00
|
|
|
from plone.app.discussion.browser.moderation import View
|
2010-07-13 12:45:53 +02:00
|
|
|
from plone.app.discussion.interfaces import IConversation
|
2016-09-19 17:06:17 +02:00
|
|
|
from plone.app.discussion.interfaces import IDiscussionSettings
|
2015-05-03 08:41:56 +02:00
|
|
|
from plone.app.discussion.testing import PLONE_APP_DISCUSSION_INTEGRATION_TESTING # noqa
|
2015-05-03 08:16:39 +02:00
|
|
|
from plone.app.testing import setRoles
|
|
|
|
from plone.app.testing import TEST_USER_ID
|
2016-09-19 17:06:17 +02:00
|
|
|
from plone.registry.interfaces import IRegistry
|
2016-02-05 01:39:53 +01:00
|
|
|
from Products.CMFCore.utils import getToolByName
|
2015-05-03 08:16:39 +02:00
|
|
|
from zope.component import createObject
|
2016-09-19 17:06:17 +02:00
|
|
|
from zope.component import queryUtility
|
2015-05-03 08:16:39 +02:00
|
|
|
|
|
|
|
import unittest
|
2009-06-18 22:57:47 +02:00
|
|
|
|
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
class ModerationViewTest(unittest.TestCase):
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
def setUp(self):
|
|
|
|
self.app = self.layer['app']
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
self.request = self.layer['request']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
2009-10-18 15:12:52 +02:00
|
|
|
self.portal_discussion = getToolByName(self.portal,
|
|
|
|
'portal_discussion',
|
|
|
|
None)
|
2011-04-17 10:50:34 +02:00
|
|
|
self.membership_tool = getToolByName(self.portal,
|
2009-10-18 15:12:52 +02:00
|
|
|
'portal_membership')
|
2009-06-18 22:57:47 +02:00
|
|
|
self.memberdata = self.portal.portal_memberdata
|
|
|
|
request = self.app.REQUEST
|
|
|
|
context = getattr(self.portal, 'doc1')
|
|
|
|
self.view = View(context, request)
|
2010-01-22 17:28:00 +01:00
|
|
|
self.view.__of__(context)
|
2009-10-18 15:12:52 +02:00
|
|
|
self.portal.portal_workflow.setChainForPortalTypes(
|
|
|
|
('Discussion Item',), 'comment_review_workflow')
|
2009-06-27 11:35:52 +02:00
|
|
|
self.wf_tool = self.portal.portal_workflow
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2009-10-18 15:12:52 +02:00
|
|
|
def test_moderation_enabled(self):
|
2010-12-16 00:52:56 +01:00
|
|
|
"""Make sure that moderation_enabled returns true if the comment
|
2010-10-08 12:22:40 +02:00
|
|
|
workflow implements a 'pending' state.
|
|
|
|
"""
|
2011-04-27 19:41:07 +02:00
|
|
|
# If workflow is not set, enabled must return False
|
2012-01-09 16:43:54 +01:00
|
|
|
self.wf_tool.setChainForPortalTypes(('Discussion Item',), ())
|
2011-04-27 19:41:07 +02:00
|
|
|
self.assertEqual(self.view.moderation_enabled(), False)
|
2010-10-08 12:22:40 +02:00
|
|
|
# The one_state_workflow does not have a 'pending' state
|
2009-10-18 15:12:52 +02:00
|
|
|
self.wf_tool.setChainForPortalTypes(('Discussion Item',),
|
2010-10-08 12:22:40 +02:00
|
|
|
('one_state_workflow,'))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(self.view.moderation_enabled(), False)
|
2010-10-08 12:22:40 +02:00
|
|
|
# The comment_review_workflow does have a 'pending' state
|
|
|
|
self.wf_tool.setChainForPortalTypes(('Discussion Item',),
|
|
|
|
('comment_review_workflow,'))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(self.view.moderation_enabled(), True)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
class ModerationBulkActionsViewTest(unittest.TestCase):
|
2010-09-16 16:58:11 +02:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
2010-09-16 16:58:11 +02:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
def setUp(self):
|
|
|
|
self.app = self.layer['app']
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
self.request = self.layer['request']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
2010-12-16 00:52:56 +01:00
|
|
|
self.wf = getToolByName(self.portal,
|
2010-09-16 17:12:59 +02:00
|
|
|
'portal_workflow',
|
|
|
|
None)
|
2010-09-16 16:58:11 +02:00
|
|
|
self.context = self.portal
|
|
|
|
self.portal.portal_workflow.setChainForPortalTypes(
|
|
|
|
('Discussion Item',), 'comment_review_workflow')
|
|
|
|
self.wf_tool = self.portal.portal_workflow
|
|
|
|
# Add a conversation with three comments
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.title = 'Comment 1'
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
comment1.Creator = 'Jim'
|
|
|
|
new_id_1 = conversation.addComment(comment1)
|
2013-04-17 19:04:45 +02:00
|
|
|
self.comment1 = self.portal.doc1.restrictedTraverse(
|
2016-02-05 01:39:53 +01:00
|
|
|
'++conversation++default/{0}'.format(new_id_1)
|
2013-04-17 19:04:45 +02:00
|
|
|
)
|
2010-09-16 16:58:11 +02:00
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.title = 'Comment 2'
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
comment2.Creator = 'Joe'
|
|
|
|
new_id_2 = conversation.addComment(comment2)
|
2013-04-17 19:04:45 +02:00
|
|
|
self.comment2 = self.portal.doc1.restrictedTraverse(
|
2016-02-05 01:39:53 +01:00
|
|
|
'++conversation++default/{0}'.format(new_id_2)
|
2013-04-17 19:04:45 +02:00
|
|
|
)
|
2010-09-16 16:58:11 +02:00
|
|
|
comment3 = createObject('plone.Comment')
|
|
|
|
comment3.title = 'Comment 3'
|
|
|
|
comment3.text = 'Comment text'
|
|
|
|
comment3.Creator = 'Emma'
|
|
|
|
new_id_3 = conversation.addComment(comment3)
|
2013-04-17 19:04:45 +02:00
|
|
|
self.comment3 = self.portal.doc1.restrictedTraverse(
|
2016-02-05 01:39:53 +01:00
|
|
|
'++conversation++default/{0}'.format(new_id_3)
|
2013-04-17 19:04:45 +02:00
|
|
|
)
|
2010-09-16 16:58:11 +02:00
|
|
|
self.conversation = conversation
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2010-09-16 17:42:27 +02:00
|
|
|
def test_default_bulkaction(self):
|
2010-09-16 17:44:07 +02:00
|
|
|
# Make sure no error is raised when no bulk actions has been supplied
|
2010-09-16 17:42:27 +02:00
|
|
|
self.request.set('form.select.BulkAction', '-1')
|
2010-09-18 19:02:37 +02:00
|
|
|
self.request.set('paths', ['/'.join(self.comment1.getPhysicalPath())])
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
view = BulkActionsView(self.portal, self.request)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertFalse(view())
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
def test_retract(self):
|
|
|
|
self.request.set('form.select.BulkAction', 'retract')
|
2010-09-18 19:02:37 +02:00
|
|
|
self.request.set('paths', ['/'.join(self.comment1.getPhysicalPath())])
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
view = BulkActionsView(self.portal, self.request)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
self.assertRaises(NotImplementedError,
|
2010-09-18 19:02:37 +02:00
|
|
|
view)
|
2010-09-16 16:58:11 +02:00
|
|
|
|
|
|
|
def test_publish(self):
|
|
|
|
self.request.set('form.select.BulkAction', 'publish')
|
2010-12-16 00:52:56 +01:00
|
|
|
self.request.set('paths', ['/'.join(self.comment1.getPhysicalPath())])
|
2011-04-17 10:50:34 +02:00
|
|
|
view = BulkActionsView(self.portal, self.request)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 17:12:59 +02:00
|
|
|
view()
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 17:12:59 +02:00
|
|
|
# Count published comments
|
|
|
|
published_comments = 0
|
|
|
|
for r in self.conversation.getThreads():
|
|
|
|
comment_obj = r['comment']
|
|
|
|
workflow_status = self.wf.getInfoFor(comment_obj, 'review_state')
|
|
|
|
if workflow_status == 'published':
|
|
|
|
published_comments += 1
|
2010-09-16 17:42:27 +02:00
|
|
|
# Make sure the comment has been published
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(published_comments, 1)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
def test_mark_as_spam(self):
|
|
|
|
self.request.set('form.select.BulkAction', 'mark_as_spam')
|
2010-09-18 19:02:37 +02:00
|
|
|
self.request.set('paths', ['/'.join(self.comment1.getPhysicalPath())])
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2011-04-17 10:50:34 +02:00
|
|
|
view = BulkActionsView(self.portal, self.request)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
self.assertRaises(NotImplementedError,
|
2010-09-18 19:02:37 +02:00
|
|
|
view)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
def test_delete(self):
|
|
|
|
# Initially we have three comments
|
2012-06-13 13:17:22 +02:00
|
|
|
self.assertEqual(len(self.conversation.objectIds()), 3)
|
2010-09-16 16:58:11 +02:00
|
|
|
# Delete two comments with bulk actions
|
|
|
|
self.request.set('form.select.BulkAction', 'delete')
|
|
|
|
self.request.set('paths', ['/'.join(self.comment1.getPhysicalPath()),
|
2010-12-16 00:52:56 +01:00
|
|
|
'/'.join(self.comment3.getPhysicalPath())])
|
2011-04-17 10:50:34 +02:00
|
|
|
view = BulkActionsView(self.app, self.request)
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
view()
|
2012-01-09 16:43:54 +01:00
|
|
|
|
2010-09-16 16:58:11 +02:00
|
|
|
# Make sure that the two comments have been deleted
|
2012-06-13 13:17:22 +02:00
|
|
|
self.assertEqual(len(self.conversation.objectIds()), 1)
|
2010-09-16 16:58:11 +02:00
|
|
|
comment = self.conversation.getComments().next()
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(comment)
|
|
|
|
self.assertEqual(comment, self.comment2)
|
2016-09-19 17:06:17 +02:00
|
|
|
|
|
|
|
|
|
|
|
class RedirectionTest(unittest.TestCase):
|
|
|
|
|
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
# Update settings.
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
self.request = self.layer['request']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
|
|
|
# applyProfile(self.portal, 'plone.app.discussion:default')
|
|
|
|
registry = queryUtility(IRegistry)
|
|
|
|
settings = registry.forInterface(IDiscussionSettings)
|
|
|
|
settings.globally_enabled = True
|
|
|
|
self.portal.portal_workflow.setChainForPortalTypes(
|
|
|
|
('Discussion Item',),
|
|
|
|
('comment_review_workflow',))
|
|
|
|
# Create page plus comment.
|
|
|
|
self.portal.invokeFactory(
|
|
|
|
id='page',
|
|
|
|
title='Page 1',
|
|
|
|
type_name='Document'
|
|
|
|
)
|
|
|
|
self.page = self.portal.page
|
|
|
|
self.conversation = IConversation(self.page)
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
|
|
|
self.comment_id = self.conversation.addComment(comment)
|
|
|
|
self.comment = list(self.conversation.getComments())[0]
|
|
|
|
|
|
|
|
def test_regression(self):
|
|
|
|
page_url = self.page.absolute_url()
|
|
|
|
self.request['HTTP_REFERER'] = page_url
|
|
|
|
for Klass in (DeleteComment, PublishComment):
|
|
|
|
view = Klass(self.comment, self.request)
|
|
|
|
view.__parent__ = self.comment
|
|
|
|
self.assertEqual(page_url, view())
|
|
|
|
|
|
|
|
def test_valid_next_url(self):
|
|
|
|
self.request['HTTP_REFERER'] = 'http://attacker.com'
|
|
|
|
for Klass in (DeleteComment, PublishComment):
|
|
|
|
view = Klass(self.comment, self.request)
|
|
|
|
view.__parent__ = self.comment
|
|
|
|
self.assertNotEqual('http://attacker.com', view())
|