Changeset 3115 in products for quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests

Timestamp:

Apr 14, 2011 5:04:01 PM (13 years ago)

Author:

kroman0

Message:

Pyflakes and pep8 fixes #2

Location:

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests

Files:

• common.py (modified) (2 diffs)
• config.py (modified) (1 diff)
• testQPloneCommentsCommenting.py (modified) (2 diffs)
• testQPloneCommentsConfiglet.py (modified) (1 diff)
• testQPloneCommentsInstall.py (modified) (1 diff)

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/common.py

@@ -2 +2 @@
 # Common constants and methods
 #
+from Products.CMFCore.utils import getToolByName
 
-from Products.CMFCore.utils import getToolByName
 
 def addMembers(portal, users_map):
@@ -9 +9 @@
     membership = getToolByName(portal, 'portal_membership', None)
     for user_id in users_map.keys():
-        membership.addMember(user_id, users_map[user_id]['passw'] , users_map[user_id]['roles'], [], 
-                            {'email': '%s@test.com'%user_id,})
+        membership.addMember(user_id, users_map[user_id]['passw'],
+                             users_map[user_id]['roles'], [],
+                             {'email': '%s@test.com' % user_id,})
+
 
 def add2Group(portal, group, group_members):

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/config.py

@@ -16 +16 @@
 
 PERM_NAME = 'Moderate Discussion'
-USERS = {# Common Members
+USERS = {  # Common Members
          'admin':{'passw': 'secret_admin', 'roles': ['Manager']},
          'owner':{'passw': 'secret_owner', 'roles': ['Owner']},
          'member':{'passw': 'secret_member', 'roles': ['Member']},
          'reviewer':{'passw': 'secret_reviewer', 'roles': ['Reviewer']},
-         # Members for discussion manager group
+           # Members for discussion manager group
          'dm_admin':{'passw': 'secret_dm_admin', 'roles': ['Manager']},
          'dm_owner':{'passw': 'secret_dm_owner', 'roles': ['Owner']},

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/testQPloneCommentsCommenting.py

@@ -73 +73 @@
             "Anonymous user CAN'T really add comment in terned ON *Anonymous commenting mode*.")
 
+
     def testAddCommentToDocNotAnonymUsers(self):
 
@@ -176 +177 @@
             self.illegal_common_users = [u for u in COMMON_USERS_IDS if not u in self.allowable_common_users]
 
-    
     def testAddCommentToNotPublishedReplyDMUsers(self):
 

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/testQPloneCommentsConfiglet.py

@@ -78 +78 @@
             "role NOT added" %  actual_reply_permission)
         # Simulate switching OFF Anonymous Commenting
-        if self.request.form.has_key('enable_anonymous_commenting'):
-           del self.request.form['enable_anonymous_commenting']
+        if 'enable_anonymous_commenting' in self.request.form:
+            del self.request.form['enable_anonymous_commenting']
         self.portal.prefs_comments_setup()
         actual_reply_permission = getReplyRoles()

quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/testQPloneCommentsInstall.py

@@ -33 +33 @@
                 'There is no %s folder in portal_skins.' % PRODUCT_SKIN_NAME)
         for skin in skinstool.getSkinSelections():
-            path = skinstool.getSkinPath(skin)
+            path = skinstool.getSkinPath(skin)
             layers = map(str.strip, path.split(','))
             self.failUnless(PRODUCT_SKIN_NAME in layers,
-                '%s layer is not registered for %s.' % (PRODUCT_SKIN_NAME, skin))
+                '%s layer is not registered for %s.' % (PRODUCT_SKIN_NAME, skin))
 
     def test_layer_install(self):