Changeset 3134 in products for quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testUsageKeywords.py

Timestamp:

Apr 18, 2011 2:32:21 PM (13 years ago)

Author:

zidane

Message:

fixes pep8

File:

• quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testUsageKeywords.py (modified) (11 diffs)

quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testUsageKeywords.py

@@ -8 +8 @@
 KWSTMPL = '.*(<meta\s+(?:(?:name="keywords"\s*)|(?:content="%s"\s*)){2}/>)'
 
+
 class TestUsageKeywords(FunctionalTestCase):
 
@@ -13 +14 @@
         self.sp = self.portal.portal_properties.seo_properties
         self.pu = self.portal.plone_utils
-        self.basic_auth = ':'.join((portal_owner,default_password))
+        self.basic_auth = ':'.join((portal_owner, default_password))
         self.loginAsPortalOwner()
         #Preparation for functional testing
@@ -31 +32 @@
         for seokws in [('foo',), ('foo', 'bar')]:
             self.my_doc._updateProperty('qSEO_keywords', seokws)
-            html = str(self.publish(self.portal.id+'/my_doc', self.basic_auth))
+            html = str(self.publish(self.portal.id + '/my_doc',
+                                    self.basic_auth))
             expect = ',\s*'.join(seokws)
-            open('/tmp/testrender_SEOKeywords','w').write(html)
-            self.assert_(re.match(KWSTMPL % expect, html, re.S|re.M),
+            open('/tmp/testrender_SEOKeywords', 'w').write(html)
+            self.assert_(re.match(KWSTMPL % expect, html, re.S | re.M),
                          "No '%s' keyword found" % str(seokws))
 
@@ -40 +42 @@
         self.my_doc.setText('<p>local subject</p>')
         self.my_doc.setSubject('subject')
-        html = str(self.publish(self.portal.id+'/my_doc', self.basic_auth))
+        html = str(self.publish(self.portal.id + '/my_doc', self.basic_auth))
 
         expect = "subject"
-        self.assert_(re.match(KWSTMPL % expect, html, re.S|re.M),
+        self.assert_(re.match(KWSTMPL % expect, html, re.S | re.M),
                      "No '%s' keyword find" % expect)
 
@@ -51 +53 @@
         self.my_doc.setSubject('subject')
         self.my_doc.manage_addProperty('qSEO_keywords', SEOKWS, 'lines')
-        html = str(self.publish(self.portal.id+'/my_doc', self.basic_auth))
+        html = str(self.publish(self.portal.id + '/my_doc', self.basic_auth))
 
         expect = ',\s*'.join(SEOKWS)
-        self.assert_(re.match(KWSTMPL % expect, html, re.S|re.M),
+        self.assert_(re.match(KWSTMPL % expect, html, re.S | re.M),
                      "No '%s' keywords find" % SEOKWS)
 
     def testbehave_noSEOKeywordsNoSubject(self):
         """Nor seo keywords not subject added"""
-        html = str(self.publish(self.portal.id+'/my_doc', self.basic_auth))
+        html = str(self.publish(self.portal.id + '/my_doc', self.basic_auth))
         self.assertFalse(re.match('.*(<meta\s[^\>]*name="keywords"[^\>]*>)',
-                                  html, re.S|re.M), "'keyword' meta tag found")
+                                  html, re.S | re.M),
+                         "'keyword' meta tag found")
 
 
@@ -81 +84 @@
         self.chckView = queryMultiAdapter((self.my_doc, self.app.REQUEST),
             name="checkSEOKeywords")
-        
+
     def patchURLLib(self, fnc):
         self.orig_urlopen = urllib2.urlopen
         self.urlfd = StringIO()
         urllib2.urlopen = fnc
-    
+
     def unpatchURLLib(self):
         urllib2.urlopen = self.orig_urlopen
@@ -99 +102 @@
         def patch_urlopen(*args, **kwargs):
             if args[0] == self.my_doc.absolute_url():
-                self.urlfd.write(unicode(self.my_doc() + self.key).encode("utf-8"))
+                self.urlfd.write(unicode(self.my_doc() +
+                                 self.key).encode("utf-8"))
                 self.urlfd.seek(0)
                 return self.urlfd
@@ -110 +114 @@
         self.assertTrue('3' in self.chckView())
         # 2. Opened urllib file descriptor must be closed
-        self.assertTrue(self.urlfd.closed, "Opened file descriptor was not closed.")
+        self.assertTrue(self.urlfd.closed,
+                        "Opened file descriptor was not closed.")
         self.unpatchURLLib()
-        
+
     def test_ExternalURLError(self):
         def patch_urlopen(*args, **kwargs):
@@ -131 +136 @@
         # 2. Opened urllib file descriptor should not be closed because
         #    it even not returned to the view
-        self.assertFalse(self.urlfd.closed, "Opened file descriptor was closed.")
+        self.assertFalse(self.urlfd.closed,
+                         "Opened file descriptor was closed.")
         self.unpatchURLLib()
-    
+
     def test_ExternalIOError(self):
         def patch_urlopen(*args, **kwargs):
             if args[0] == self.my_doc.absolute_url():
-                self.urlfd.write(unicode(self.my_doc() + self.key).encode("utf-8"))
+                self.urlfd.write(unicode(self.my_doc() +
+                                 self.key).encode("utf-8"))
                 self.urlfd.seek(0)
                 return self.urlfd
             else:
                 return self.orig_urlopen(*args, **kwargs)
+
         def patch_read(*args, **kwargs):
             raise Exception("General exception")
         # Patch urllib2.urlopen to emulate external url retrieval
         self.patchURLLib(fnc=patch_urlopen)
-        # Patch opened by urllib2 file descriptor to emulate IOError during reading
+        # Patch opened by urllib2 file descriptor to emulate
+        # IOError during reading
         self.urlfd.read = patch_read
         self.seo._updateProperty("external_keywords_test", True)
@@ -153 +162 @@
         self.assertRaises(Exception, self.chckView)
         # 2. Opened urllib file descriptor must be closed
-        self.assertTrue(self.urlfd.closed, "Opened file descriptor was not closed.")
+        self.assertTrue(self.urlfd.closed,
+                        "Opened file descriptor was not closed.")
         self.unpatchURLLib()
-        
+
 
 def test_suite():
@@ -163 +173 @@
     suite.addTest(makeSuite(TestCalcKeywords))
     return suite
-