Changeset 1671 in products for quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testCanonicalURL.py

Timestamp:

Feb 9, 2010 12:59:27 PM (14 years ago)

Author:

mylan

Message:

Added ICanonicalPath adapter and appropriate tests

File:

• quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testCanonicalURL.py (modified) (3 diffs)

quintagroup.seoptimizer/trunk/quintagroup/seoptimizer/tests/testCanonicalURL.py

@@ -1 +1 @@
 import re
 from zope.component import queryAdapter
+from Products.Archetypes.interfaces import IBaseContent
+
+from quintagroup.seoptimizer.interfaces import ICanonicalPath
 from base import getToolByName, FunctionalTestCase, newSecurityManager
 from config import *
@@ -22 +25 @@
         self.mydoc = self.portal['mydoc']
         self.mydoc_path = "/%s" % self.mydoc.absolute_url(1)
-        self.curl = re.compile('<link\srel\s*=\s*"canonical"\s+[^>]*href\s*=\s*\"([^\"]*)\"[^>]*>', re.S|re.M)
+        self.curl = re.compile('<link\srel\s*=\s*"canonical"\s+' \
+            '[^>]*href\s*=\s*\"([^\"]*)\"[^>]*>', re.S|re.M)
 
 
@@ -30 +34 @@
         mydoc_url = self.mydoc.absolute_url()
 
-        self.assert_([1 for curl in foundcurls if curl==mydoc_url],
-           "Wrong CANONICAL URL for document: %s, all must be: %s" % (foundcurls, mydoc_url))
+        self.assertTrue([1 for curl in foundcurls if curl==mydoc_url],
+           "Wrong CANONICAL URL for document: %s, all must be: %s" % (
+           foundcurls, mydoc_url))
 
     def test_updateCanonicalURL(self):
         mydoc_url_new = self.mydoc.absolute_url() + '.new'
         # Update canonical url property
-        self.publish(self.mydoc_path + '/@@seo-context-properties?seo_canonical_override=checked' \
-                     '&seo_canonical=%s&form.submitted=1' % mydoc_url_new, self.basic_auth)
+        self.publish(self.mydoc_path + '/@@seo-context-properties?' \
+           'seo_canonical_override=checked&seo_canonical=%s&' \
+           'form.submitted=1' % mydoc_url_new, self.basic_auth)
         # Test updated canonical url
         html = self.publish(self.mydoc_path, self.basic_auth).getBody()
         foundcurls = self.curl.findall(html)
 
-        self.assert_(self.mydoc.getProperty('qSEO_canonical','') == mydoc_url_new,
-                     "Not set 'qSEO_canonical' property")
-        self.assert_([1 for curl in foundcurls if curl==mydoc_url_new],
-           "Wrong CANONICAL URL for document: %s, all must be: %s" % (foundcurls, mydoc_url_new))
+        qseo_url = self.mydoc.getProperty('qSEO_canonical','')
+        self.assertTrue(qseo_url == mydoc_url_new,
+           "Not set 'qSEO_canonical' property")
+        self.assertTrue([1 for curl in foundcurls if curl==mydoc_url_new],
+           "Wrong CANONICAL URL for document: %s, all must be: %s" % (
+           foundcurls, mydoc_url_new))
 
+    def test_canonicalAdapterRegistration(self):
+        canonical = queryAdapter(self.mydoc, interface=ICanonicalPath,
+            name='qseo_canonical')
+        self.assertTrue(canonical is not None,
+            "Not registered 'qseo_canonical' adapter")
+
+    def test_canonicalAdapter(self):
+        purl = getToolByName(self.portal, 'portal_url')
+        mydoc_path_rel = '/'+'/'.join(purl.getRelativeContentPath(self.mydoc))
+
+        canonical = queryAdapter(self.mydoc, ICanonicalPath, name='qseo_canonical')
+        cpath = canonical.canonical_path()
+        self.assertTrue(cpath == mydoc_path_rel,
+            "By canonical path adapter got: '%s', must be: '%s'" % (
+             cpath, mydoc_path_rel))
+        
+        # Update canonical url property
+        mydoc_url_new = self.mydoc.absolute_url() + '.new'
+        self.publish(self.mydoc_path + '/@@seo-context-properties?' \
+            'seo_canonical_override=checked&seo_canonical=%s' \
+            '&form.submitted=1' % mydoc_url_new, self.basic_auth)
+
+        mydoc_path_rel_new = mydoc_path_rel + '.new'
+        newcpath = canonical.canonical_path()
+        self.assertTrue(newcpath == mydoc_path_rel_new,
+            "By canonical path adapter got: '%s', must be: '%s'" % (
+             newcpath, mydoc_path_rel_new))
+       
 
 def test_suite():