2011-04-16 11:27:15 +02:00
|
|
|
import unittest2 as unittest
|
2009-05-18 17:15:36 +02:00
|
|
|
from datetime import datetime, timedelta
|
|
|
|
|
2011-04-15 06:29:46 +02:00
|
|
|
from zope import interface
|
2010-07-12 15:47:53 +02:00
|
|
|
from zope.component import createObject, queryUtility
|
2011-04-14 18:17:29 +02:00
|
|
|
from zope.annotation.interfaces import IAnnotations
|
2009-05-18 17:15:36 +02:00
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
from Acquisition import aq_base, aq_parent
|
2009-05-18 17:15:36 +02:00
|
|
|
|
2009-05-31 19:54:14 +02:00
|
|
|
from plone.app.vocabularies.types import BAD_TYPES
|
|
|
|
|
2009-06-11 12:14:44 +02:00
|
|
|
from plone.registry.interfaces import IRegistry
|
|
|
|
|
2009-06-04 17:38:12 +02:00
|
|
|
from Products.CMFCore.utils import getToolByName
|
2011-04-16 11:27:15 +02:00
|
|
|
|
|
|
|
from plone.app.testing import TEST_USER_ID, setRoles
|
|
|
|
|
2012-01-14 07:26:01 +01:00
|
|
|
from plone.app.discussion.testing import \
|
|
|
|
PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
2009-05-18 17:15:36 +02:00
|
|
|
|
2011-04-15 06:29:46 +02:00
|
|
|
from plone.app.discussion import interfaces
|
2010-08-28 19:06:53 +02:00
|
|
|
from plone.app.discussion.interfaces import IConversation
|
|
|
|
from plone.app.discussion.interfaces import IComment
|
|
|
|
from plone.app.discussion.interfaces import IReplies
|
|
|
|
from plone.app.discussion.interfaces import IDiscussionSettings
|
|
|
|
|
2012-07-12 18:56:10 +02:00
|
|
|
try:
|
|
|
|
import plone.dexterity
|
|
|
|
DEXTERITY = True
|
|
|
|
except:
|
|
|
|
DEXTERITY = False
|
|
|
|
|
2009-05-24 14:34:15 +02:00
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
class ConversationTest(unittest.TestCase):
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
def setUp(self):
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
2011-04-15 06:29:46 +02:00
|
|
|
interface.alsoProvides(
|
|
|
|
self.portal.REQUEST, interfaces.IDiscussionLayer)
|
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
typetool = self.portal.portal_types
|
|
|
|
typetool.constructContent('Document', self.portal, 'doc1')
|
2009-06-18 22:57:47 +02:00
|
|
|
self.typetool = typetool
|
2010-12-16 00:52:56 +01:00
|
|
|
self.portal_discussion = getToolByName(self.portal,
|
|
|
|
'portal_discussion',
|
2010-08-28 19:06:53 +02:00
|
|
|
None)
|
2010-12-14 18:58:46 +01:00
|
|
|
# Allow discussion
|
|
|
|
registry = queryUtility(IRegistry)
|
|
|
|
settings = registry.forInterface(IDiscussionSettings)
|
|
|
|
settings.globally_enabled = True
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_add_comment(self):
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2010-12-16 00:52:56 +01:00
|
|
|
# Add a comment. Note: in real life, we always create comments via the
|
2010-08-28 19:06:53 +02:00
|
|
|
# factory to allow different factories to be swapped in
|
2009-05-18 17:15:36 +02:00
|
|
|
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
new_id = conversation.addComment(comment)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
# Check that the conversation methods return the correct data
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(isinstance(comment.comment_id, long))
|
|
|
|
self.assertTrue(IComment.providedBy(conversation[new_id]))
|
|
|
|
self.assertEqual(aq_base(conversation[new_id].__parent__),
|
2013-03-28 14:28:22 +01:00
|
|
|
aq_base(conversation))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(new_id, comment.comment_id)
|
|
|
|
self.assertEqual(len(list(conversation.getComments())), 1)
|
|
|
|
self.assertEqual(len(tuple(conversation.getThreads())), 1)
|
|
|
|
self.assertEqual(conversation.total_comments, 1)
|
|
|
|
self.assertTrue(conversation.last_comment_date - datetime.utcnow() <
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(seconds=1))
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2013-03-28 14:28:22 +01:00
|
|
|
def test_private_comment(self):
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.author_username = "nobody"
|
|
|
|
conversation.addComment(comment)
|
|
|
|
comment.manage_permission("View", roles=tuple())
|
|
|
|
self.assertEquals(0, conversation.total_comments)
|
|
|
|
self.assertEquals(None, conversation.last_comment_date)
|
|
|
|
self.assertEquals(["nobody"], list(conversation.commentators))
|
|
|
|
self.assertEquals([], list(conversation.public_commentators))
|
|
|
|
|
2009-05-23 13:52:57 +02:00
|
|
|
def test_delete_comment(self):
|
2009-05-23 16:18:35 +02:00
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2010-12-16 00:52:56 +01:00
|
|
|
# Add a comment. Note: in real life, we always create comments via the
|
2010-08-28 19:06:53 +02:00
|
|
|
# factory to allow different factories to be swapped in
|
2009-05-23 16:18:35 +02:00
|
|
|
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
|
|
|
|
|
|
|
new_id = conversation.addComment(comment)
|
|
|
|
|
2009-05-23 16:24:24 +02:00
|
|
|
# make sure the comment has been added
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(list(conversation.getComments())), 1)
|
|
|
|
self.assertEqual(len(tuple(conversation.getThreads())), 1)
|
|
|
|
self.assertEqual(conversation.total_comments, 1)
|
2009-05-23 16:24:24 +02:00
|
|
|
|
2009-05-23 16:18:35 +02:00
|
|
|
# delete the comment we just created
|
2009-05-24 16:19:06 +02:00
|
|
|
del conversation[new_id]
|
2009-05-23 16:18:35 +02:00
|
|
|
|
|
|
|
# make sure there is no comment left in the conversation
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(list(conversation.getComments())), 0)
|
|
|
|
self.assertEqual(len(tuple(conversation.getThreads())), 0)
|
|
|
|
self.assertEqual(conversation.total_comments, 0)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
def test_delete_recursive(self):
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
IReplies(conversation)
|
2009-05-24 18:19:21 +02:00
|
|
|
|
|
|
|
# Create a nested comment structure:
|
|
|
|
#
|
|
|
|
# Conversation
|
|
|
|
# +- Comment 1
|
|
|
|
# +- Comment 1_1
|
|
|
|
# | +- Comment 1_1_1
|
|
|
|
# +- Comment 1_2
|
|
|
|
# +- Comment 2
|
|
|
|
# +- Comment 2_1
|
|
|
|
|
|
|
|
# Create all comments
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_2 = createObject('plone.Comment')
|
|
|
|
comment1_2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2_1 = createObject('plone.Comment')
|
|
|
|
comment2_1.text = 'Comment text'
|
|
|
|
|
|
|
|
# Create the nested comment structure
|
|
|
|
new_id_1 = conversation.addComment(comment1)
|
|
|
|
new_id_2 = conversation.addComment(comment2)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
comment1_1.in_reply_to = new_id_1
|
|
|
|
new_id_1_1 = conversation.addComment(comment1_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
comment1_1_1.in_reply_to = new_id_1_1
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment1_1_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
comment1_2.in_reply_to = new_id_1
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment1_2)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
comment2_1.in_reply_to = new_id_2
|
|
|
|
new_id_2_1 = conversation.addComment(comment2_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:19:21 +02:00
|
|
|
del conversation[new_id_1]
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(
|
2009-05-24 18:19:21 +02:00
|
|
|
[{'comment': comment2, 'depth': 0, 'id': new_id_2},
|
|
|
|
{'comment': comment2_1, 'depth': 1, 'id': new_id_2_1},
|
|
|
|
], list(conversation.getThreads()))
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2010-03-16 12:09:40 +01:00
|
|
|
def test_delete_comment_when_content_object_is_deleted(self):
|
2010-12-16 00:52:56 +01:00
|
|
|
# Make sure all comments of a content object are deleted when the
|
2010-08-28 19:06:53 +02:00
|
|
|
# object itself is deleted.
|
2010-03-16 12:09:40 +01:00
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment)
|
2010-03-16 12:09:40 +01:00
|
|
|
|
|
|
|
# Delete the content object
|
|
|
|
self.portal.manage_delObjects(['doc1'])
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2010-03-16 12:09:40 +01:00
|
|
|
# Make sure the comment has been deleted as well
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(list(conversation.getComments())), 0)
|
|
|
|
self.assertEqual(len(tuple(conversation.getThreads())), 0)
|
|
|
|
self.assertEqual(conversation.total_comments, 0)
|
2010-03-16 12:09:40 +01:00
|
|
|
|
2009-06-16 13:51:25 +02:00
|
|
|
def test_allow_discussion(self):
|
|
|
|
# This is not a real test! It's only there to understand the
|
|
|
|
# allow discussion attribute. Maybe we should remove this at
|
|
|
|
# some point.
|
|
|
|
|
2009-06-19 17:51:50 +02:00
|
|
|
# 1) allow_discussion attribute: Every content object in Plone
|
|
|
|
# has a allow_discussion attribute. By default it is set to None.
|
|
|
|
|
2009-06-16 13:51:25 +02:00
|
|
|
# Create a conversation.
|
2010-07-13 12:45:53 +02:00
|
|
|
IConversation(self.portal.doc1)
|
2009-06-16 13:51:25 +02:00
|
|
|
|
|
|
|
# By default, discussion is disabled for all content types
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
for type in list(portal_types):
|
|
|
|
type_fti = getattr(portal_types, type)
|
|
|
|
if type not in BAD_TYPES:
|
|
|
|
if type != 'Discussion Item':
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertFalse(type_fti.allowDiscussion())
|
2009-06-16 13:51:25 +02:00
|
|
|
|
|
|
|
# By default, allow_discussion on newly created content objects is
|
|
|
|
# set to False
|
|
|
|
portal_discussion = getToolByName(self.portal, 'portal_discussion')
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(portal_discussion.isDiscussionAllowedFor(
|
2010-08-28 19:06:53 +02:00
|
|
|
self.portal.doc1), False)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(self.portal.doc1.getTypeInfo().allowDiscussion(),
|
2010-08-28 19:06:53 +02:00
|
|
|
False)
|
2009-06-19 17:51:50 +02:00
|
|
|
|
|
|
|
# The allow discussion flag is None by default
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertFalse(getattr(self.portal.doc1, 'allow_discussion', None))
|
2009-06-16 13:51:25 +02:00
|
|
|
|
2010-12-16 00:52:56 +01:00
|
|
|
# But isDiscussionAllowedFor, also checks if discussion is allowed on
|
|
|
|
# the content type. So we allow discussion on the Document content
|
2010-08-28 19:06:53 +02:00
|
|
|
# type and check if the Document object allows discussion now.
|
2009-06-16 13:51:25 +02:00
|
|
|
document_fti = getattr(portal_types, 'Document')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(portal_discussion.isDiscussionAllowedFor(
|
2010-08-28 19:06:53 +02:00
|
|
|
self.portal.doc1), True)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(self.portal.doc1.getTypeInfo().allowDiscussion(),
|
2010-08-28 19:06:53 +02:00
|
|
|
True)
|
2009-06-16 13:51:25 +02:00
|
|
|
|
|
|
|
# We can also override the allow_discussion locally
|
|
|
|
self.portal_discussion.overrideDiscussionFor(self.portal.doc1, False)
|
|
|
|
# Check if the Document discussion is disabled
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(portal_discussion.isDiscussionAllowedFor(
|
2010-08-28 19:06:53 +02:00
|
|
|
self.portal.doc1), False)
|
2010-12-16 00:52:56 +01:00
|
|
|
# Check that the local allow_discussion flag is now explicitly set to
|
2010-08-28 19:06:53 +02:00
|
|
|
# False
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(getattr(self.portal.doc1, 'allow_discussion', None),
|
2010-08-28 19:06:53 +02:00
|
|
|
False)
|
2009-06-16 13:51:25 +02:00
|
|
|
|
|
|
|
# Disallow discussion on the Document content type again
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=False)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(portal_discussion.isDiscussionAllowedFor(
|
2010-08-28 19:06:53 +02:00
|
|
|
self.portal.doc1), False)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(self.portal.doc1.getTypeInfo().allowDiscussion(),
|
2010-08-28 19:06:53 +02:00
|
|
|
False)
|
2009-06-16 13:51:25 +02:00
|
|
|
|
|
|
|
# Now we override allow_discussion again (True) for the Document
|
|
|
|
# content object
|
|
|
|
self.portal_discussion.overrideDiscussionFor(self.portal.doc1, True)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(portal_discussion.isDiscussionAllowedFor(
|
2010-08-28 19:06:53 +02:00
|
|
|
self.portal.doc1), True)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(getattr(self.portal.doc1, 'allow_discussion', None),
|
2010-08-28 19:06:53 +02:00
|
|
|
True)
|
2009-06-16 13:51:25 +02:00
|
|
|
|
2010-08-20 13:52:05 +02:00
|
|
|
def test_comments_enabled_on_doc_in_subfolder(self):
|
|
|
|
typetool = self.portal.portal_types
|
|
|
|
typetool.constructContent('Folder', self.portal, 'folder1')
|
|
|
|
typetool.constructContent('Document', self.portal.folder1, 'doc2')
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2010-08-20 13:52:05 +02:00
|
|
|
folder = self.portal.folder1
|
|
|
|
folder.allowDiscussion(False)
|
|
|
|
self.assertFalse(hasattr(aq_base(folder), 'allow_discussion'))
|
|
|
|
folder.allowDiscussion(True)
|
|
|
|
self.assertTrue(aq_base(folder).allow_discussion)
|
|
|
|
folder.allowDiscussion(False)
|
|
|
|
self.assertFalse(aq_base(folder).allow_discussion)
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2010-08-20 13:52:05 +02:00
|
|
|
doc = self.portal.folder1.doc2
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = doc.restrictedTraverse('@@conversation_view')
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2010-12-16 00:52:56 +01:00
|
|
|
|
2010-08-20 13:52:05 +02:00
|
|
|
# We have to allow discussion on Document content type, since
|
|
|
|
# otherwise allow_discussion will always return False
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'Document')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2010-08-20 13:52:05 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2010-08-20 13:52:05 +02:00
|
|
|
|
2009-06-11 12:14:44 +02:00
|
|
|
def test_disable_commenting_globally(self):
|
|
|
|
|
|
|
|
# Create a conversation.
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = self.portal.doc1.restrictedTraverse(
|
|
|
|
'@@conversation_view')
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# We have to allow discussion on Document content type, since
|
|
|
|
# otherwise allow_discussion will always return False
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'Document')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Check if conversation is enabled now
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Disable commenting in the registry
|
2010-07-12 15:47:53 +02:00
|
|
|
registry = queryUtility(IRegistry)
|
2009-07-12 21:13:42 +02:00
|
|
|
settings = registry.forInterface(IDiscussionSettings)
|
2009-06-11 12:14:44 +02:00
|
|
|
settings.globally_enabled = False
|
|
|
|
|
|
|
|
# Check if commenting is disabled on the conversation
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Enable discussion again
|
|
|
|
settings.globally_enabled = True
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
2009-07-07 16:48:39 +02:00
|
|
|
def test_allow_discussion_for_news_items(self):
|
|
|
|
|
|
|
|
self.typetool.constructContent('News Item', self.portal, 'newsitem')
|
|
|
|
newsitem = self.portal.newsitem
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = newsitem.restrictedTraverse('@@conversation_view')
|
2009-07-07 16:48:39 +02:00
|
|
|
|
|
|
|
# We have to allow discussion on Document content type, since
|
|
|
|
# otherwise allow_discussion will always return False
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'News Item')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2009-07-07 16:48:39 +02:00
|
|
|
|
|
|
|
# Check if conversation is enabled now
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-07-07 16:48:39 +02:00
|
|
|
|
|
|
|
# Disable commenting in the registry
|
2010-07-12 15:47:53 +02:00
|
|
|
registry = queryUtility(IRegistry)
|
2009-07-12 21:13:42 +02:00
|
|
|
settings = registry.forInterface(IDiscussionSettings)
|
2009-07-07 16:48:39 +02:00
|
|
|
settings.globally_enabled = False
|
|
|
|
|
|
|
|
# Check if commenting is disabled on the conversation
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-07-07 16:48:39 +02:00
|
|
|
|
|
|
|
# Enable discussion again
|
|
|
|
settings.globally_enabled = True
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-07-07 16:48:39 +02:00
|
|
|
|
2009-06-11 12:14:44 +02:00
|
|
|
def test_disable_commenting_for_content_type(self):
|
|
|
|
|
|
|
|
# Create a conversation.
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = self.portal.doc1.restrictedTraverse(
|
|
|
|
'@@conversation_view')
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# The Document content type is disabled by default
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Allow discussion on Document content type
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'Document')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Check if conversation is enabled now
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Disallow discussion on Document content type
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'Document')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=False)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Check if conversation is enabled now
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
2009-06-18 22:57:47 +02:00
|
|
|
def test_allow_discussion_on_folder(self):
|
2009-06-18 23:12:38 +02:00
|
|
|
# The enabled method should always return False for the folder
|
|
|
|
# itself.
|
|
|
|
|
2011-04-15 06:29:46 +02:00
|
|
|
# Create a folderp
|
2009-06-18 23:12:38 +02:00
|
|
|
self.typetool.constructContent('Folder', self.portal, 'f1')
|
|
|
|
f1 = self.portal.f1
|
|
|
|
# Usually we don't create a conversation on a folder
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = self.portal.f1.restrictedTraverse('@@conversation_view')
|
2009-06-18 23:12:38 +02:00
|
|
|
|
|
|
|
# Allow discussion for the folder
|
|
|
|
self.portal_discussion.overrideDiscussionFor(f1, True)
|
|
|
|
|
|
|
|
# Allow discussion on Folder content type
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, 'Folder')
|
2012-01-14 07:26:01 +01:00
|
|
|
document_fti.manage_changeProperties(allow_discussion=True)
|
2009-06-18 23:12:38 +02:00
|
|
|
|
|
|
|
# Always return False
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertFalse(conversation.enabled())
|
2009-06-11 12:14:44 +02:00
|
|
|
|
2009-06-18 22:57:47 +02:00
|
|
|
def test_is_discussion_allowed_for_folder(self):
|
|
|
|
# When a content item provides IFolderish from CMF and
|
|
|
|
# does not provide INonStructuralFolder from Plone,
|
|
|
|
# allow_discussion acts as an on/off flag for all items
|
|
|
|
# in that folder, overriding settings for any parent folders,
|
|
|
|
# and the for the FTI, but is overridden by child items and
|
|
|
|
# folders further down.
|
|
|
|
|
|
|
|
# Create a folder
|
|
|
|
self.typetool.constructContent('Folder', self.portal, 'f1')
|
|
|
|
f1 = self.portal.f1
|
|
|
|
|
2009-06-19 17:51:50 +02:00
|
|
|
# Create a document inside the folder
|
2009-06-18 22:57:47 +02:00
|
|
|
self.typetool.constructContent('Document', f1, 'doc1')
|
|
|
|
doc1 = self.portal.f1.doc1
|
2011-04-15 06:29:46 +02:00
|
|
|
doc1_conversation = doc1.restrictedTraverse('@@conversation_view')
|
2009-06-19 17:51:50 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(doc1_conversation.enabled(), False)
|
2009-06-18 22:57:47 +02:00
|
|
|
|
|
|
|
# Allow commenting for the folder
|
|
|
|
self.portal_discussion.overrideDiscussionFor(f1, True)
|
|
|
|
|
|
|
|
# Check if the content objects allows discussion
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(doc1_conversation.enabled(), True)
|
2009-06-18 22:57:47 +02:00
|
|
|
|
|
|
|
# Turn commenting for the folder off
|
2009-06-19 17:51:50 +02:00
|
|
|
self.portal_discussion.overrideDiscussionFor(f1, False)
|
2009-06-18 22:57:47 +02:00
|
|
|
|
|
|
|
# Check if content objects do not allow discussion anymore
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(doc1_conversation.enabled(), False)
|
2009-06-18 22:57:47 +02:00
|
|
|
|
2009-06-11 12:14:44 +02:00
|
|
|
def test_is_discussion_allowed_on_content_object(self):
|
|
|
|
# Allow discussion on a single content object
|
|
|
|
|
|
|
|
# Create a conversation.
|
2011-04-15 06:29:46 +02:00
|
|
|
conversation = self.portal.doc1.restrictedTraverse(
|
|
|
|
'@@conversation_view')
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Discussion is disallowed by default
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
# Allow discussion on content object
|
|
|
|
self.portal_discussion.overrideDiscussionFor(self.portal.doc1, True)
|
|
|
|
|
|
|
|
# Check if discussion is now allowed on the content object
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), True)
|
2009-06-11 12:14:44 +02:00
|
|
|
|
|
|
|
self.portal_discussion.overrideDiscussionFor(self.portal.doc1, False)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.enabled(), False)
|
2009-05-24 18:19:21 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_dict_operations(self):
|
2009-05-20 17:39:45 +02:00
|
|
|
# test dict operations and acquisition wrapping
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2010-12-16 00:52:56 +01:00
|
|
|
# Add a comment. Note: in real life, we always create comments via the
|
2010-08-28 19:06:53 +02:00
|
|
|
# factory to allow different factories to be swapped in
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
|
|
|
|
new_id1 = conversation.addComment(comment1)
|
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
|
|
|
|
new_id2 = conversation.addComment(comment2)
|
|
|
|
|
|
|
|
# check if get returns a comment object, and None if the key
|
|
|
|
# can not be found
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(IComment.providedBy(conversation.get(new_id1)))
|
|
|
|
self.assertTrue(IComment.providedBy(conversation.get(new_id2)))
|
|
|
|
self.assertEqual(conversation.get(123), None)
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if keys return the ids of all comments
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(conversation.keys()), 2)
|
|
|
|
self.assertTrue(new_id1 in conversation.keys())
|
|
|
|
self.assertTrue(new_id2 in conversation.keys())
|
|
|
|
self.assertFalse(123 in conversation.keys())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if items returns (key, comment object) pairs
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(conversation.items()), 2)
|
|
|
|
self.assertTrue((new_id1, comment1) in conversation.items())
|
|
|
|
self.assertTrue((new_id2, comment2) in conversation.items())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if values returns the two comment objects
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(conversation.values()), 2)
|
|
|
|
self.assertTrue(comment1 in conversation.values())
|
|
|
|
self.assertTrue(comment2 in conversation.values())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if comment ids are in iterkeys
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(new_id1 in conversation.iterkeys())
|
|
|
|
self.assertTrue(new_id2 in conversation.iterkeys())
|
|
|
|
self.assertFalse(123 in conversation.iterkeys())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if comment objects are in itervalues
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(comment1 in conversation.itervalues())
|
|
|
|
self.assertTrue(comment2 in conversation.itervalues())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# check if iteritems returns (key, comment object) pairs
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue((new_id1, comment1) in conversation.iteritems())
|
|
|
|
self.assertTrue((new_id2, comment2) in conversation.iteritems())
|
2009-05-24 12:53:13 +02:00
|
|
|
|
|
|
|
# TODO test acquisition wrapping
|
2011-04-15 18:23:38 +02:00
|
|
|
#self.assertTrue(aq_base(aq_parent(comment1)) is conversation)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_total_comments(self):
|
2009-05-20 17:39:45 +02:00
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2009-05-20 18:07:29 +02:00
|
|
|
# Add a three comments. Note: in real life, we always create
|
|
|
|
# comments via the factory to allow different factories to be
|
|
|
|
# swapped in
|
2009-05-20 17:39:45 +02:00
|
|
|
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment3 = createObject('plone.Comment')
|
|
|
|
comment3.text = 'Comment text'
|
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment1)
|
|
|
|
conversation.addComment(comment2)
|
|
|
|
conversation.addComment(comment3)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.total_comments, 3)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_commentators(self):
|
2009-05-20 17:39:45 +02:00
|
|
|
# add and remove a few comments to make sure the commentators
|
2009-05-18 17:15:36 +02:00
|
|
|
# property returns a true set
|
2009-05-23 18:28:10 +02:00
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.total_comments, 0)
|
2009-07-02 19:50:20 +02:00
|
|
|
|
2009-05-23 19:17:06 +02:00
|
|
|
# Add a four comments from three different users
|
2009-05-23 18:28:10 +02:00
|
|
|
# Note: in real life, we always create
|
|
|
|
# comments via the factory to allow different factories to be
|
|
|
|
# swapped in
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
comment1.author_username = "Jim"
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment1)
|
2009-05-23 18:28:10 +02:00
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
comment2.author_username = "Joe"
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment2)
|
2009-05-23 18:28:10 +02:00
|
|
|
|
|
|
|
comment3 = createObject('plone.Comment')
|
|
|
|
comment3.text = 'Comment text'
|
|
|
|
comment3.author_username = "Jack"
|
|
|
|
new_comment3_id = conversation.addComment(comment3)
|
|
|
|
|
2009-05-23 19:17:06 +02:00
|
|
|
comment4 = createObject('plone.Comment')
|
|
|
|
comment4.text = 'Comment text'
|
|
|
|
comment4.author_username = "Jack"
|
|
|
|
new_comment4_id = conversation.addComment(comment4)
|
|
|
|
|
2009-05-23 18:28:10 +02:00
|
|
|
# check if all commentators are in the commentators list
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.total_comments, 4)
|
|
|
|
self.assertTrue('Jim' in conversation.commentators)
|
|
|
|
self.assertTrue('Joe' in conversation.commentators)
|
|
|
|
self.assertTrue('Jack' in conversation.commentators)
|
2009-05-23 19:17:06 +02:00
|
|
|
|
|
|
|
# remove the comment from Jack
|
|
|
|
del conversation[new_comment3_id]
|
|
|
|
|
|
|
|
# check if Jack is still in the commentators list (since
|
|
|
|
# he had added two comments)
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue('Jim' in conversation.commentators)
|
|
|
|
self.assertTrue('Joe' in conversation.commentators)
|
|
|
|
self.assertTrue('Jack' in conversation.commentators)
|
|
|
|
self.assertEqual(conversation.total_comments, 3)
|
2009-05-23 18:28:10 +02:00
|
|
|
|
2009-05-23 19:17:06 +02:00
|
|
|
# remove the second comment from Jack
|
|
|
|
del conversation[new_comment4_id]
|
2009-05-23 18:28:10 +02:00
|
|
|
|
2009-05-23 19:17:06 +02:00
|
|
|
# check if Jack has been removed from the commentators list
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue('Jim' in conversation.commentators)
|
|
|
|
self.assertTrue('Joe' in conversation.commentators)
|
|
|
|
self.assertFalse('Jack' in conversation.commentators)
|
|
|
|
self.assertEqual(conversation.total_comments, 2)
|
2009-05-23 18:28:10 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_last_comment_date(self):
|
2009-05-23 20:04:39 +02:00
|
|
|
# add and remove some comments and check if last_comment_date
|
|
|
|
# is properly updated
|
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
# Add a three comments that are at least one day old
|
|
|
|
# Note: in real life, we always create
|
|
|
|
# comments via the factory to allow different factories to be
|
|
|
|
# swapped in
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
2010-09-07 14:03:07 +02:00
|
|
|
comment1.creation_date = datetime.utcnow() - timedelta(4)
|
2010-07-13 12:45:53 +02:00
|
|
|
conversation.addComment(comment1)
|
2009-05-23 20:04:39 +02:00
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
2010-09-07 14:03:07 +02:00
|
|
|
comment2.creation_date = datetime.utcnow() - timedelta(2)
|
2009-05-23 20:04:39 +02:00
|
|
|
new_comment2_id = conversation.addComment(comment2)
|
|
|
|
|
|
|
|
comment3 = createObject('plone.Comment')
|
|
|
|
comment3.text = 'Comment text'
|
2010-09-07 14:03:07 +02:00
|
|
|
comment3.creation_date = datetime.utcnow() - timedelta(1)
|
2009-05-23 20:04:39 +02:00
|
|
|
new_comment3_id = conversation.addComment(comment3)
|
|
|
|
|
|
|
|
# check if the latest comment is exactly one day old
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date < datetime.utcnow() -
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(hours=23, minutes=59, seconds=59))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date >
|
2010-09-07 14:03:07 +02:00
|
|
|
datetime.utcnow() - timedelta(days=1, seconds=1))
|
2009-05-23 20:04:39 +02:00
|
|
|
|
|
|
|
# remove the latest comment
|
|
|
|
del conversation[new_comment3_id]
|
|
|
|
|
|
|
|
# check if the latest comment has been updated
|
|
|
|
# the latest comment should be exactly two days old
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date < datetime.utcnow() -
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(days=1, hours=23, minutes=59, seconds=59))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date > datetime.utcnow() -
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(days=2, seconds=1))
|
2009-05-23 20:04:39 +02:00
|
|
|
|
|
|
|
# remove the latest comment again
|
|
|
|
del conversation[new_comment2_id]
|
|
|
|
|
|
|
|
# check if the latest comment has been updated
|
|
|
|
# the latest comment should be exactly four days old
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date < datetime.utcnow() -
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(days=3, hours=23, minutes=59, seconds=59))
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.last_comment_date > datetime.utcnow() -
|
2010-08-28 19:06:53 +02:00
|
|
|
timedelta(days=4, seconds=2))
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
def test_get_comments_full(self):
|
2009-05-18 17:15:36 +02:00
|
|
|
pass
|
|
|
|
|
|
|
|
def test_get_comments_batched(self):
|
2009-05-23 06:55:06 +02:00
|
|
|
pass
|
2009-05-18 17:15:36 +02:00
|
|
|
|
|
|
|
def test_get_threads(self):
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
IReplies(conversation)
|
2009-05-24 18:04:49 +02:00
|
|
|
|
|
|
|
# Create a nested comment structure:
|
|
|
|
#
|
|
|
|
# Conversation
|
|
|
|
# +- Comment 1
|
|
|
|
# +- Comment 1_1
|
|
|
|
# | +- Comment 1_1_1
|
|
|
|
# +- Comment 1_2
|
|
|
|
# +- Comment 2
|
|
|
|
# +- Comment 2_1
|
|
|
|
|
|
|
|
# Create all comments
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_2 = createObject('plone.Comment')
|
|
|
|
comment1_2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2_1 = createObject('plone.Comment')
|
|
|
|
comment2_1.text = 'Comment text'
|
|
|
|
|
|
|
|
# Create the nested comment structure
|
|
|
|
new_id_1 = conversation.addComment(comment1)
|
|
|
|
new_id_2 = conversation.addComment(comment2)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
comment1_1.in_reply_to = new_id_1
|
|
|
|
new_id_1_1 = conversation.addComment(comment1_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
comment1_1_1.in_reply_to = new_id_1_1
|
|
|
|
new_id_1_1_1 = conversation.addComment(comment1_1_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
comment1_2.in_reply_to = new_id_1
|
|
|
|
new_id_1_2 = conversation.addComment(comment1_2)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
comment2_1.in_reply_to = new_id_2
|
|
|
|
new_id_2_1 = conversation.addComment(comment2_1)
|
2009-05-25 12:38:58 +02:00
|
|
|
|
2009-05-24 18:04:49 +02:00
|
|
|
# Get threads
|
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(
|
2009-05-24 18:04:49 +02:00
|
|
|
[{'comment': comment1, 'depth': 0, 'id': new_id_1},
|
|
|
|
{'comment': comment1_1, 'depth': 1, 'id': new_id_1_1},
|
|
|
|
{'comment': comment1_1_1, 'depth': 2, 'id': new_id_1_1_1},
|
|
|
|
{'comment': comment1_2, 'depth': 1, 'id': new_id_1_2},
|
|
|
|
{'comment': comment2, 'depth': 0, 'id': new_id_2},
|
|
|
|
{'comment': comment2_1, 'depth': 1, 'id': new_id_2_1},
|
|
|
|
], list(conversation.getThreads()))
|
2009-05-18 17:15:36 +02:00
|
|
|
|
|
|
|
def test_get_threads_batched(self):
|
2009-05-24 18:04:49 +02:00
|
|
|
# TODO: test start, size, root and depth arguments to getThreads()
|
|
|
|
# - may want to split this into multiple tests
|
2009-05-18 17:15:36 +02:00
|
|
|
pass
|
2009-05-23 16:18:35 +02:00
|
|
|
|
2009-05-23 13:52:57 +02:00
|
|
|
def test_traversal(self):
|
|
|
|
# make sure we can traverse to conversations and get a URL and path
|
2009-05-23 16:18:35 +02:00
|
|
|
|
2010-08-28 19:06:53 +02:00
|
|
|
conversation = self.portal.doc1.restrictedTraverse(
|
|
|
|
'++conversation++default')
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(IConversation.providedBy(conversation))
|
2009-05-23 16:18:35 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(('', 'plone', 'doc1', '++conversation++default'),
|
2010-08-28 19:06:53 +02:00
|
|
|
conversation.getPhysicalPath())
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual('http://nohost/plone/doc1/++conversation++default',
|
2011-02-08 10:28:51 +01:00
|
|
|
conversation.absolute_url())
|
2009-05-18 17:15:36 +02:00
|
|
|
|
2012-11-16 10:28:00 +01:00
|
|
|
def test_unconvertible_id(self):
|
|
|
|
# make sure the conversation view doesn't break when given comment id
|
|
|
|
# can't be converted to long
|
|
|
|
|
|
|
|
conversation = self.portal.doc1.restrictedTraverse(
|
|
|
|
'++conversation++default/ThisCantBeRight')
|
|
|
|
self.assertEqual(conversation, None)
|
|
|
|
|
2009-06-11 12:14:44 +02:00
|
|
|
def test_parent(self):
|
|
|
|
# Check that conversation has a content object as parent
|
|
|
|
|
|
|
|
# Create a conversation.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
# Check the parent
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(conversation.__parent__)
|
|
|
|
self.assertTrue(aq_parent(conversation))
|
2009-06-18 22:57:47 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.__parent__.getId(), 'doc1')
|
2009-06-11 12:14:44 +02:00
|
|
|
|
2009-05-31 19:54:14 +02:00
|
|
|
def test_discussion_item_not_in_bad_types(self):
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertFalse('Discussion Item' in BAD_TYPES)
|
2009-05-31 19:54:14 +02:00
|
|
|
|
2011-04-14 18:17:29 +02:00
|
|
|
def test_no_comment(self):
|
2012-01-14 07:26:01 +01:00
|
|
|
IConversation(self.portal.doc1)
|
2011-04-14 18:17:29 +02:00
|
|
|
# Make sure no conversation has been created
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue('plone.app.discussion:conversation' not in
|
2011-04-14 18:17:29 +02:00
|
|
|
IAnnotations(self.portal.doc1))
|
|
|
|
|
2009-06-04 17:38:12 +02:00
|
|
|
|
2012-06-30 10:59:49 +02:00
|
|
|
class ConversationEnabledForDexterityTypesTest(unittest.TestCase):
|
|
|
|
|
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
|
|
|
|
|
|
|
def setUp(self):
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
|
|
|
interface.alsoProvides(
|
|
|
|
self.portal.REQUEST,
|
|
|
|
interfaces.IDiscussionLayer)
|
|
|
|
|
|
|
|
typetool = self.portal.portal_types
|
|
|
|
typetool.constructContent('Document', self.portal, 'doc1')
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
from plone.dexterity.interfaces import IDexterityContent
|
|
|
|
interface.alsoProvides(
|
|
|
|
self.portal.doc1,
|
|
|
|
IDexterityContent)
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
def _makeOne(self, *args, **kw):
|
|
|
|
return self.portal.doc1.restrictedTraverse('@@conversation_view')
|
|
|
|
|
|
|
|
def _globally_enable_discussion(self, value):
|
|
|
|
registry = queryUtility(IRegistry)
|
|
|
|
settings = registry.forInterface(IDiscussionSettings)
|
|
|
|
settings.globally_enabled = value
|
|
|
|
|
|
|
|
def _enable_discussion_on_portal_type(self, portal_type, allow_discussion):
|
|
|
|
portal_types = getToolByName(self.portal, 'portal_types')
|
|
|
|
document_fti = getattr(portal_types, portal_type)
|
|
|
|
document_fti.manage_changeProperties(allow_discussion=allow_discussion)
|
|
|
|
|
|
|
|
def test_conversation_is_not_enabled_by_default(self):
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
conversation = self._makeOne(self.portal.doc1)
|
|
|
|
self.assertFalse(conversation.enabled())
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
def test_conversation_is_not_enabled_by_default_on_portal_type(self):
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
self._globally_enable_discussion(True)
|
|
|
|
conversation = self._makeOne(self.portal.doc1)
|
|
|
|
self.assertFalse(conversation.enabled())
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
def test_conversation_needs_to_be_enabled_globally_and_for_type(self):
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
self._globally_enable_discussion(True)
|
|
|
|
self._enable_discussion_on_portal_type('Document', True)
|
|
|
|
conversation = self._makeOne(self.portal.doc1)
|
|
|
|
self.assertTrue(conversation.enabled())
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
def test_disable_discussion(self):
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
self._globally_enable_discussion(True)
|
|
|
|
self._enable_discussion_on_portal_type('Document', True)
|
|
|
|
self.portal.doc1.allow_discussion = False
|
|
|
|
conversation = self._makeOne(self.portal.doc1)
|
|
|
|
self.assertFalse(conversation.enabled())
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
def test_enable_discussion(self):
|
2012-07-12 18:56:10 +02:00
|
|
|
if DEXTERITY:
|
|
|
|
self._globally_enable_discussion(True)
|
|
|
|
self._enable_discussion_on_portal_type('Document', True)
|
|
|
|
self.portal.doc1.allow_discussion = True
|
|
|
|
conversation = self._makeOne(self.portal.doc1)
|
|
|
|
self.assertTrue(conversation.enabled())
|
2012-06-30 10:59:49 +02:00
|
|
|
|
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
class RepliesTest(unittest.TestCase):
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
# test the IReplies adapter on a conversation
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
layer = PLONE_APP_DISCUSSION_INTEGRATION_TESTING
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2011-04-16 11:27:15 +02:00
|
|
|
def setUp(self):
|
|
|
|
self.portal = self.layer['portal']
|
|
|
|
setRoles(self.portal, TEST_USER_ID, ['Manager'])
|
2009-05-18 17:15:36 +02:00
|
|
|
typetool = self.portal.portal_types
|
|
|
|
typetool.constructContent('Document', self.portal, 'doc1')
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_add_comment(self):
|
2009-05-24 14:34:15 +02:00
|
|
|
# Add comments to a ConversationReplies adapter
|
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
replies = IReplies(conversation)
|
|
|
|
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
|
|
|
|
|
|
|
new_id = replies.addComment(comment)
|
|
|
|
|
|
|
|
# check that replies provides the IReplies interface
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(IReplies.providedBy(replies))
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2009-05-24 16:19:06 +02:00
|
|
|
# Make sure our comment was added
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(new_id in replies)
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2009-05-24 16:19:06 +02:00
|
|
|
# Make sure it is also reflected in the conversation
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertTrue(new_id in conversation)
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation[new_id].comment_id, new_id)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_delete_comment(self):
|
2009-05-24 18:23:49 +02:00
|
|
|
# Create and remove a comment and check if the replies adapter
|
|
|
|
# has been updated accordingly
|
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
replies = IReplies(conversation)
|
|
|
|
|
|
|
|
# Add a comment.
|
|
|
|
comment = createObject('plone.Comment')
|
|
|
|
comment.text = 'Comment text'
|
|
|
|
|
|
|
|
new_id = replies.addComment(comment)
|
|
|
|
|
|
|
|
# make sure the comment has been added
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(replies), 1)
|
2009-05-24 18:23:49 +02:00
|
|
|
|
|
|
|
# delete the comment we just created
|
|
|
|
del replies[new_id]
|
|
|
|
|
|
|
|
# make sure there is no comment left in the conversation
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(len(replies), 0)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_dict_api(self):
|
2009-05-25 12:38:58 +02:00
|
|
|
# This test is for the ConversationReplies as well as the
|
|
|
|
# CommentReplies adapter.
|
|
|
|
#
|
2009-05-24 17:25:49 +02:00
|
|
|
# Ensure all operations use only top-level comments. Add some
|
2009-05-24 16:19:06 +02:00
|
|
|
# deeper children and ensure that these are not exposed through the
|
|
|
|
# IReplies dict.
|
2009-05-24 17:25:49 +02:00
|
|
|
|
|
|
|
# Create a conversation. In this case we doesn't assign it to an
|
|
|
|
# object, as we just want to check the Conversation object API.
|
|
|
|
conversation = IConversation(self.portal.doc1)
|
|
|
|
|
|
|
|
replies = IReplies(conversation)
|
|
|
|
|
|
|
|
# Create a nested comment structure:
|
|
|
|
#
|
|
|
|
# Conversation
|
|
|
|
# +- Comment 1
|
|
|
|
# +- Comment 1_1
|
|
|
|
# | +- Comment 1_1_1
|
|
|
|
# +- Comment 1_2
|
|
|
|
# +- Comment 2
|
|
|
|
# +- Comment 2_1
|
|
|
|
|
|
|
|
# Create all comments
|
|
|
|
comment1 = createObject('plone.Comment')
|
|
|
|
comment1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_1_1 = createObject('plone.Comment')
|
|
|
|
comment1_1_1.text = 'Comment text'
|
|
|
|
|
|
|
|
comment1_2 = createObject('plone.Comment')
|
|
|
|
comment1_2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2 = createObject('plone.Comment')
|
|
|
|
comment2.text = 'Comment text'
|
|
|
|
|
|
|
|
comment2_1 = createObject('plone.Comment')
|
|
|
|
comment2_1.text = 'Comment text'
|
|
|
|
|
|
|
|
# Create the nested comment structure
|
|
|
|
new_id_1 = replies.addComment(comment1)
|
2010-08-28 19:06:53 +02:00
|
|
|
comment1 = self.portal.doc1.restrictedTraverse(
|
|
|
|
'++conversation++default/%s' % new_id_1)
|
2009-05-24 17:25:49 +02:00
|
|
|
replies_to_comment1 = IReplies(comment1)
|
|
|
|
new_id_2 = replies.addComment(comment2)
|
2010-08-28 19:06:53 +02:00
|
|
|
comment2 = self.portal.doc1.restrictedTraverse(
|
|
|
|
'++conversation++default/%s' % new_id_2)
|
2009-05-24 17:25:49 +02:00
|
|
|
replies_to_comment2 = IReplies(comment2)
|
|
|
|
|
|
|
|
new_id_1_1 = replies_to_comment1.addComment(comment1_1)
|
2010-08-28 19:06:53 +02:00
|
|
|
comment1_1 = self.portal.doc1.restrictedTraverse(
|
|
|
|
'++conversation++default/%s' % new_id_1_1)
|
2009-05-24 17:25:49 +02:00
|
|
|
replies_to_comment1_1 = IReplies(comment1_1)
|
2010-07-13 12:45:53 +02:00
|
|
|
replies_to_comment1_1.addComment(comment1_1_1)
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
replies_to_comment1.addComment(comment1_2)
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2010-07-13 12:45:53 +02:00
|
|
|
replies_to_comment2.addComment(comment2_1)
|
2009-05-24 17:25:49 +02:00
|
|
|
|
2009-05-24 17:36:31 +02:00
|
|
|
# check that replies only contain the direct comments
|
|
|
|
# and no comments deeper than 1
|
2011-04-15 18:23:38 +02:00
|
|
|
self.assertEqual(conversation.total_comments, 6)
|
|
|
|
self.assertEqual(len(replies), 2)
|
|
|
|
self.assertEqual(len(replies_to_comment1), 2)
|
|
|
|
self.assertEqual(len(replies_to_comment1_1), 1)
|
|
|
|
self.assertEqual(len(replies_to_comment2), 1)
|
2009-05-20 17:39:45 +02:00
|
|
|
|
2012-01-14 07:26:01 +01:00
|
|
|
|
2009-05-18 17:15:36 +02:00
|
|
|
def test_suite():
|
2010-08-20 13:52:05 +02:00
|
|
|
return unittest.defaultTestLoader.loadTestsFromName(__name__)
|