Changeset 1929 in products

Timestamp:

Mar 17, 2010 4:42:39 PM (14 years ago)

Author:

mylan

Message:

#177: Extend indexer registration tests, add tests for default ICanonicalLink adapter

Location:

quintagroup.canonicalpath/trunk/quintagroup/canonicalpath

Files:

• configure.zcml (modified) (1 diff)
• tests.py (modified) (7 diffs)

quintagroup.canonicalpath/trunk/quintagroup/canonicalpath/configure.zcml

@@ -6 +6 @@
   <adapter factory=".adapters.DefaultCanonicalPathAdapter" />
 
+  <!-- Add general canonical_link adapter -->
+  <adapter factory=".adapters.DefaultCanonicalLinkAdapter" />
+
   <!-- Register canonical_path adapter for catalog -->
   <adapter name="canonical_path" factory=".catalog.canonical_path" />
 
+  <!-- Register canonical_link adapter for catalog -->
+  <adapter name="canonical_link" factory=".catalog.canonical_link" />
+
 </configure>

quintagroup.canonicalpath/trunk/quintagroup/canonicalpath/tests.py

@@ -17 +17 @@
 
 from quintagroup.canonicalpath.interfaces import ICanonicalPath
-from quintagroup.canonicalpath.adapters import PROPERTY
+from quintagroup.canonicalpath.interfaces import ICanonicalLink
+from quintagroup.canonicalpath.adapters import PROPERTY_PATH
+from quintagroup.canonicalpath.adapters import PROPERTY_LINK
 
 class TestCase(ptc.PloneTestCase):
@@ -45 +47 @@
         wrapper = queryMultiAdapter((self.my_doc, self.catalog), IIndexableObject)
         self.assertFalse(wrapper is None, "No indexer registered for document object")
-        
-        
-class TestDefaultAdapter(TestCase):
+
+    def testCanonicalPathForPortal(self):
+        wrapper = queryMultiAdapter((self.portal, self.catalog), IIndexableObject)
+        self.assertTrue(hasattr(wrapper, 'canonical_path'),
+            "'canonical_path' attribute not accessible with indexer wrapper for portal object")
+
+    def testCanonicalPathForAT(self):
+        wrapper = queryMultiAdapter((self.my_doc, self.catalog), IIndexableObject)
+        self.assertTrue(hasattr(wrapper, 'canonical_path'),
+            "'canonical_path' attribute not accessible with indexer wrapper for Document object")
+
+    def testCanonicalLinkForPortal(self):
+        wrapper = queryMultiAdapter((self.portal, self.catalog), IIndexableObject)
+        self.assertTrue(hasattr(wrapper, 'canonical_link'),
+            "'canonical_link' attribute not accessible with indexer wrapper for portal object")
+
+    def testCanonicalLinkForAT(self):
+        wrapper = queryMultiAdapter((self.my_doc, self.catalog), IIndexableObject)
+        self.assertTrue(hasattr(wrapper, 'canonical_link'),
+            "'canonical_link' attribute not accessible with indexer wrapper for Document object")
+
+
+        
+class TestDefaultCanonicalPathAdapter(TestCase):
 
 
@@ -89 +112 @@
 
         cpadapter.canonical_path = newcp
-        prop = self.portal.getProperty(PROPERTY, None)
+        prop = self.portal.getProperty(PROPERTY_PATH, None)
         self.assertTrue(prop == newcp,
             "Canonical path adapter setter NOT SET new '%s' value to '%s' " \
-            "propery for the portal" % (newcp, PROPERTY) )
+            "propery for the portal" % (newcp, PROPERTY_PATH) )
 
         self.assertTrue(cpadapter.canonical_path == newcp,
@@ -104 +127 @@
 
         cpadapter.canonical_path = newcp
-        prop = self.my_doc.getProperty(PROPERTY, None)
+        prop = self.my_doc.getProperty(PROPERTY_PATH, None)
         self.assertTrue(prop == newcp,
             "Canonical path adapter setter NOT SET new '%s' value to '%s' " \
-            "propery for the Document" % (newcp, PROPERTY) )
+            "propery for the Document" % (newcp, PROPERTY_PATH) )
 
         self.assertTrue(cpadapter.canonical_path == newcp,
@@ -114 +137 @@
 
 
-    def testValidation(self):
+    def testValidationWrong(self):
         cpadapter = queryAdapter(self.my_doc, ICanonicalPath)
         for wrong in ['new\nline','s p a c e','with\ttabs']:
@@ -122 +145 @@
                 continue
             else:
+                raise self.failureException, "InvalidValue not raised when '%s' wrong value try to set" % wrong
+        
+    def testValidationGood(self):
+        cpadapter = queryAdapter(self.my_doc, ICanonicalPath)
+        for good in ['./good','../good','/good', 'good']:
+            cpadapter.canonical_path = good
+
+
+class TestDefaultCanonicalLinkAdapter(TestCase):
+
+
+    def afterSetUp(self):
+        self.loginAsPortalOwner()
+        self.purl = getToolByName(self.portal, 'portal_url')
+        self.my_doc = makeContent(self.portal, portal_type='Document', id='my_doc')
+        self.logout()
+
+        self.mydoc_cl = self.my_doc.absolute_url()
+
+    def testRegistration4Portal(self):
+        cladapter = queryAdapter(self.portal, ICanonicalLink)
+        self.assertFalse(cladapter is None,
+            "Can't get canonical link adapter for the plone site object")
+
+    def testRegistration4AT(self):
+        cladapter = queryAdapter(self.my_doc, ICanonicalLink)
+        self.assertFalse(cladapter is None,
+            "Can't get canonical link adapter for the Document object")
+        
+
+    def testGetDefault4Portal(self):
+        cladapter = queryAdapter(self.portal, ICanonicalLink)
+        self.assertTrue(cladapter.canonical_link == self.purl(),
+            "Canonical link adapter return '%s' for portal, must be: '%s'" % (
+            cladapter.canonical_link, self.purl()) )
+
+
+    def testGetDefault4AT(self):
+        cladapter = queryAdapter(self.my_doc, ICanonicalLink)
+        self.assertTrue(cladapter.canonical_link == self.mydoc_cl,
+            "Canonical link adapter return '%s' for document, must be: '%s'" % (
+            cladapter.canonical_link, self.mydoc_cl) )
+
+
+    def testSet4Portal(self):
+        cladapter = queryAdapter(self.portal, ICanonicalLink)
+        newcl = self.purl() + '/new_portal_canonical'
+
+        cladapter.canonical_link = newcl
+        prop = self.portal.getProperty(PROPERTY_LINK, None)
+        self.assertTrue(prop == newcl,
+            "Canonical link adapter setter NOT SET new '%s' value to '%s' " \
+            "propery for the portal" % (newcl, PROPERTY_LINK) )
+
+        self.assertTrue(cladapter.canonical_link == newcl,
+            "Canonical link adapter GET '%s' canonical_link, for portal, " \
+            "must be: '%s'" % (cladapter.canonical_link, newcl) )
+
+
+    def testSet4AT(self):
+        cladapter = queryAdapter(self.my_doc, ICanonicalLink)
+        newcl = self.mydoc_cl + '/new_mydoc_canonical'
+
+        cladapter.canonical_link = newcl
+        prop = self.my_doc.getProperty(PROPERTY_LINK, None)
+        self.assertTrue(prop == newcl,
+            "Canonical link adapter setter NOT SET new '%s' value to '%s' " \
+            "propery for the Document" % (newcl, PROPERTY_LINK) )
+
+        self.assertTrue(cladapter.canonical_link == newcl,
+            "Canonical link adapter GET '%s' canonical_link, for Document, " \
+            "must be: '%s'" % (cladapter.canonical_link, newcl) )
+
+    def testValidationWrong(self):
+        cladapter = queryAdapter(self.my_doc, ICanonicalLink)
+        for wrong in ['http://new\nline','s p a c e','with\ttabs']:
+            try:
+                cladapter.canonical_link = wrong
+            except InvalidValue:
+                continue
+            else:
                 raise self.failureException, "InvalidValue not raised when " \
-                      "wrong value set"
+                    "'%s' wrong value try to set" % wrong
+        
+    def testValidationGood(self):
+        cladapter = queryAdapter(self.my_doc, ICanonicalLink)
+        for good in ['http://', './good','../good','/good', 'good']:
+            cladapter.canonical_link = good
 
 
@@ -129 +238 @@
     return unittest.TestSuite([
         unittest.makeSuite(TestIndexerRegistration),
-        unittest.makeSuite(TestDefaultAdapter),
+        unittest.makeSuite(TestDefaultCanonicalPathAdapter),
+        unittest.makeSuite(TestDefaultCanonicalLinkAdapter),
         ])