[Zope-Checkins] SVN: Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py Supporting empty path elements seems a lost use case, as explained

[Philipp von Weitershausen]

Log message for revision 68287:
  Supporting empty path elements seems a lost use case, as explained
  in http://mail.zope.org/pipermail/zope-dev/2006-May/027493.html.
  Disable the corresponding tests therefore.
  

Changed:
  U   Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py

-=-
Modified: Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py
===================================================================
--- Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py	2006-05-25 20:09:48 UTC (rev 68286)
+++ Zope/branches/ajung-zpt-end-game/lib/python/Products/PageTemplates/tests/testExpressions.py	2006-05-25 20:25:06 UTC (rev 68287)
@@ -50,26 +50,11 @@
         '''Test advanced expression evaluation 1'''
         ec = self.ec
         assert ec.evaluate('x | nothing') is None
-
-        # XXX The following test fails because Zope 3 doesn't allow
-        # empty path elements.  My idea is to eventually disallow
-        # blank path elements, but to allow them for a limited
-        # deprecation period.  We do that by implementing our own
-        # SubPathExpr that just looks for empty path elements and
-        # replaces them with a call to a TALESNamespace adapter that
-        # explicitly does the empty element lookup.  Then we hand off
-        # to the standard Zope 3 SubPathExpr implementation.
-        # Something along these lines (pseudo-ish code!):
-        #
-        # class Zope2SubPathExpr(SubPathExpr):
-        #     def __init__(self, text, ...):
-        #         text = text.replace('//', 'blank:element')
-        #         return super(Zope2SubPathExpr, self).__init__(text ...)
-        #
-        assert ec.evaluate('d/') == 'blank'
-
+        # empty path elements aren't supported anymore, for the lack
+        # of a use case
+        #assert ec.evaluate('d/') == 'blank'
         assert ec.evaluate('d/_') == 'under'
-        assert ec.evaluate('d/ | nothing') == 'blank'
+        #assert ec.evaluate('d/ | nothing') == 'blank'
         assert ec.evaluate('d/?blank') == 'blank'
 
     def testHybrid(self):