Test fails for Genshi-0.5.1 and Python 2.6.2

[neurogeek@…]

Hello there,

When running python setup.py test in Genshi-0.5.1 using python-2.6.2, I get a test failed for genshi.template.eval.Undefined in here:

"""Represents a reference to an undefined variable.
221	   
227	    >>> foo = Undefined('foo')
228	    >>> bool(foo)
229	    False
230	    >>> list(foo) -----> This line fail test
231	    []
232	    >>> print foo
233	    undefined

Thanks,

[lmacken@…]

When tweaking the class a little and running the tests::

    python-2.4.3-27.el5
    -------------------
    Failed example: list(foo)
    Expected: []
    Got:
        Undefined.__iter__
        []

    python-2.6.2-2.fc12
    ------------------------
    Failed example: list(foo)
    Expected: []
    Got:
        Undefined.__iter__
        __getattr__(('__length_hint__',))
        []

Since the Undefined class overloads sets __getattr__ = _die, this causes some unexpected behavior on Python 2.6.2

I'm thinking that this behavior was introduced in:  http://bugs.python.org/issue1242657 Another example of this behavior can be found in the test case that came along with it:  http://svn.python.org/view/python/trunk/Lib/test/test_iterlen.py?r1=69227&r2=69226&pathrev=69227

I threw together the following patch for Fedora that gets the test suite running again:

--- genshi/template/eval.py.orig	2009-09-11 17:09:13.589445262 -0400
+++ genshi/template/eval.py	2009-09-11 17:11:17.818691489 -0400
@@ -279,6 +279,10 @@
         raise UndefinedError(self._name, self._owner)
     __call__ = __getattr__ = __getitem__ = _die
 
+    # Hack around some behavior introduced in Python 2.6.2
+    # http://genshi.edgewall.org/ticket/324
+    __length_hint__ = None
+
 
 class LookupBase(object):
     """Abstract base class for variable lookup implementations."""

[lmacken@…]

python-genshi-0.5.1-py2.6.2-length_hint.patch

[rblank]

Another (much less elegant) patch is in comment:6:ticket:370.

[cmlenz]

Issue raised again in #370.

[cmlenz]

Applied in [1101]. Thanks and sorry for the wait!