Changeset 219

Timestamp:

01/31/06 06:23:18

Author:

mylan

Message:

Raise publishing reliability.

Files:

• qPloneComments/trunk/skins/qplonecomments/discussion_publish_comment.py (modified) (1 diff)
• qPloneComments/trunk/tests/testQPloneCommentsModeration.py (modified) (2 diffs)

qPloneComments/trunk/skins/qplonecomments/discussion_publish_comment.py

@@ -5 +5 @@
-##bind script=script
-##bind subpath=traverse_subpath
-reply
+obj
-##title=
-##
-from Products.CMFPlone import transaction_note
-from Products.qPloneComments.utils import publishDiscussion
-from Products.qPloneComments.utils import publishDiscussion, send_email
-
-reply
-reply
+obj
+obj
-
-
-
+
+
+
+
+
-
-
+
+
+
-send_notification_message = send_email(reply, container, state="published")
-
-# Inform user about awaiting moderation
-portal_status_message='Comment+successfully+published'
-

qPloneComments/trunk/tests/testQPloneCommentsModeration.py

@@ -149 +149 @@
-
-
-
+__
-        # Check whether perform real publishing
-        for u in DM_USERS_IDS:
@@ -165 +165 @@
-            self.logout()
-            self.assert_(getReplies(), "%s - member of DiscussionManager group NOT PUBLISH reply" % u)
+
+
+    def testPublishing(self):
+        # Check whether perform real publishing
+        # Pattern for publish button presence checking
+        pattern = re.compile('.*<input\\s*class="standalone"\\s*type="submit"\\s*value="Publish This Discussion"\\s*/>',\
+                             re.S|re.M)
+        for u in DM_USERS_IDS:
+            doc_id = "doc_%s" % u
+            doc_obj = getattr(self.portal, doc_id)
+            getReplies = self.discussion.getDiscussionFor(doc_obj).getReplies
+            # Check whether anonymous get no reply
+            self.logout()
+            self.assert_(not getReplies(), "View not published reply ALLOW for Anonymous")
+            # Login with actual (tested) user with DiscussionManager role and publish discussion
+            self.login(u)
+            self.assert_(getReplies(), "%s - member of DiscussionManager group NOT VIEW not published reply" % u)
+            getReplies()[0].discussion_publish_comment()
+            # Check if Publish button still present in document view page
+            auth = "%s:" % u
+            if not u=='anonym':
+                auth = '%s:%s' % (u,USERS[u]['passw'])
+            html = str(self.publish(self.portal.id+'/%s' % doc_id, auth))
+            m = pattern.match(html)
+            self.assert_(not m, "Publish button present for %s - DiscussionManager role user after publishing" % u)
+            # Check whether Anonym view published reply
+            self.logout()
+            self.assert_(getReplies(), "%s - member of DiscussionManager group NOT PUBLISH reply" % u)
+
-
-