[CMF-checkins] SVN: CMF/trunk/CMFCore/tests/ fixed tests added in r69191: they expected the wrong behavior and just didn't fail because they were broken

Yvo Schubbe y.2006_ at wcm-solutions.de
Mon Oct 2 07:34:15 EDT 2006


Log message for revision 70472:
  fixed tests added in r69191: they expected the wrong behavior and just didn't fail because they were broken

Changed:
  U   CMF/trunk/CMFCore/tests/base/dummy.py
  U   CMF/trunk/CMFCore/tests/test_PortalContent.py

-=-
Modified: CMF/trunk/CMFCore/tests/base/dummy.py
===================================================================
--- CMF/trunk/CMFCore/tests/base/dummy.py	2006-10-02 11:33:53 UTC (rev 70471)
+++ CMF/trunk/CMFCore/tests/base/dummy.py	2006-10-02 11:34:15 UTC (rev 70472)
@@ -98,31 +98,6 @@
         return True
 
 
-    def listActions(self, info=None, object=None):
-        rs = []
-        for k,v in self._actions.items():
-           rs.append( DummyAction( k,v ) )
-        return rs
-
-class DummyAction:
-
-    def __init__( self, id, target, permissions=() ):
-        self._id = id
-        self.target = target
-        self.permissions = permissions
-
-    def getId( self ):
-        return self._id
-
-    # can this be right? e.g. utils._getViewFor calls action
-    # attribute directly, which is not part of API but no other way
-    # to do it...
-    def action( self, context ):
-        return self.target
-
-    def getPermissions( self ):
-        return self.permissions
-
 class DummyContent( PortalContent, Item ):
     """
     A Dummy piece of PortalContent

Modified: CMF/trunk/CMFCore/tests/test_PortalContent.py
===================================================================
--- CMF/trunk/CMFCore/tests/test_PortalContent.py	2006-10-02 11:33:53 UTC (rev 70471)
+++ CMF/trunk/CMFCore/tests/test_PortalContent.py	2006-10-02 11:34:15 UTC (rev 70472)
@@ -23,14 +23,16 @@
 from OFS.Folder import Folder
 from zope.testing.cleanup import cleanUp
 
+from Products.CMFCore.exceptions import NotFound
+from Products.CMFCore.tests.base.dummy import DummyContent
+from Products.CMFCore.tests.base.dummy import DummyObject
 from Products.CMFCore.tests.base.dummy import DummySite
+from Products.CMFCore.tests.base.dummy import DummyTool
 from Products.CMFCore.tests.base.dummy import DummyUserFolder
 from Products.CMFCore.tests.base.testcase import SecurityRequestTest
 from Products.CMFCore.tests.base.testcase import setUpEvents
-from Products.CMFCore.tests.base.dummy import DummyTool
-from Products.CMFCore.tests.base.dummy import DummyContent
-from Products.CMFCore.tests.base.dummy import DummyObject
 
+
 class PortalContentTests(unittest.TestCase):
 
     def test_z2interfaces(self):
@@ -59,7 +61,7 @@
 
         # set up dummy type info with problematic double-default alias
         root._setObject( 'portal_types', DummyTool() )
-        root.portal_types.view_actions = aliases
+        root.portal_types._type_actions = aliases
 
         # dummy content and skin
         root._setObject( 'dummycontent', DummyContent() )
@@ -71,18 +73,16 @@
                          ('view', 'dummy_view'),
                        )
         ob = self._setupCallTests(test_aliases)
-        # in unfixed version fail here with AttributeError
-        # can end up with this arrangement using _getAliases though
-        # in fixed version, falls through to _getViewFor, which is BBB
-        self.assertEqual( ob(), 'dummy' )
+        # PortalContent no longer supports the BBB '(Default)' alias
+        self.assertRaises(NotFound, ob)
 
     def test_BlankDefaultAlias(self):
         test_aliases = ( ('(Default)', ''),
                          ('view', 'dummy_view'),
                        )
         ob = self._setupCallTests(test_aliases)
-        # blank default is BBB
-        self.assertEqual( ob(), 'dummy' )
+        # blank values are not valid
+        self.assertRaises(NotFound, ob)
 
     def test_SpecificAlias(self):
         test_aliases = ( ('(Default)', 'dummy_view'),
@@ -90,6 +90,7 @@
         ob = self._setupCallTests(test_aliases)
         self.assertEqual( ob(), 'dummy' )
 
+
 class TestContentCopyPaste(SecurityRequestTest):
 
     # Tests related to http://www.zope.org/Collectors/CMF/205



More information about the CMF-checkins mailing list