ajung/plone.app.discussion
ajung/plone.app.discussion
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use len(tuple()) instead of tuple() to make tests pass again.

Browse Source 
svn path=/plone.app.discussion/trunk/; revision=44706
This commit is contained in:
Timo Stollenwerk 2010-10-30 19:26:51 +00:00
parent 093eaa393a
commit c8cd370fc9
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
plone/app/discussion/tests/test_comments_viewlet.py
View File

@ -242,7 +242,7 @@ class TestCommentsViewlet(PloneTestCase):
conversation.addComment(comment) conversation.addComment(comment)
conversation.addComment(comment) conversation.addComment(comment)
replies = self.viewlet.get_replies() replies = self.viewlet.get_replies()
self.assertEquals(tuple(replies), 2) self.assertEquals(len(tuple(replies)), 2)
replies = self.viewlet.get_replies() replies = self.viewlet.get_replies()
replies.next() replies.next()
replies.next() replies.next()
@ -255,7 +255,7 @@ class TestCommentsViewlet(PloneTestCase):
conversation = IConversation(self.portal.doc1) conversation = IConversation(self.portal.doc1)
c1 = conversation.addComment(comment) c1 = conversation.addComment(comment)
self.assertEquals( self.assertEquals(
tuple(self.viewlet.get_replies(workflow_actions=True)), 1) len(tuple(self.viewlet.get_replies(workflow_actions=True))), 1)
# Enable moderation workflow # Enable moderation workflow
self.portal.portal_workflow.setChainForPortalTypes( self.portal.portal_workflow.setChainForPortalTypes(
('Discussion Item',), ('Discussion Item',),

8
plone/app/discussion/tests/test_conversation.py
View File

@ -54,7 +54,7 @@ class ConversationTest(PloneTestCase):
aq_base(conversation)) aq_base(conversation))
self.assertEquals(new_id, comment.comment_id) self.assertEquals(new_id, comment.comment_id)
self.assertEquals(len(list(conversation.getComments())), 1) self.assertEquals(len(list(conversation.getComments())), 1)
self.assertEquals(tuple(conversation.getThreads()), 1) self.assertEquals(len(tuple(conversation.getThreads())), 1)
self.assertEquals(conversation.total_comments, 1) self.assertEquals(conversation.total_comments, 1)
self.assert_(conversation.last_comment_date - datetime.utcnow() < self.assert_(conversation.last_comment_date - datetime.utcnow() <
timedelta(seconds=1)) timedelta(seconds=1))
@ -75,7 +75,7 @@ class ConversationTest(PloneTestCase):
# make sure the comment has been added # make sure the comment has been added
self.assertEquals(len(list(conversation.getComments())), 1) self.assertEquals(len(list(conversation.getComments())), 1)
self.assertEquals(tuple(conversation.getThreads()), 1) self.assertEquals(len(tuple(conversation.getThreads())), 1)
self.assertEquals(conversation.total_comments, 1) self.assertEquals(conversation.total_comments, 1)
# delete the comment we just created # delete the comment we just created
@ -83,7 +83,7 @@ class ConversationTest(PloneTestCase):
# make sure there is no comment left in the conversation # make sure there is no comment left in the conversation
self.assertEquals(len(list(conversation.getComments())), 0) self.assertEquals(len(list(conversation.getComments())), 0)
self.assertEquals(tuple(conversation.getThreads()), 0) self.assertEquals(len(tuple(conversation.getThreads())), 0)
self.assertEquals(conversation.total_comments, 0) self.assertEquals(conversation.total_comments, 0)
def test_delete_recursive(self): def test_delete_recursive(self):
@ -165,7 +165,7 @@ class ConversationTest(PloneTestCase):
# Make sure the comment has been deleted as well # Make sure the comment has been deleted as well
self.assertEquals(len(list(conversation.getComments())), 0) self.assertEquals(len(list(conversation.getComments())), 0)
self.assertEquals(tuple(conversation.getThreads()), 0) self.assertEquals(len(tuple(conversation.getThreads())), 0)
self.assertEquals(conversation.total_comments, 0) self.assertEquals(conversation.total_comments, 0)
def test_allow_discussion(self): def test_allow_discussion(self):