kombu.tests.test_transport_pyredis.test_Redis.test_publish__get fails (with cjson)

[fladi]

Running on python-2.6.6 in Debian Squeeze (again inside build-chroot) this unit test fails because the message body is not decoded. Looking at the test with pdb shows that self.queue(channel).get().payload returns u'{"hello": "world"}' instead of the expected dictionary. Running it outside the chroot does not trigger this problem.

FAIL: test_publish__get (kombu.tests.test_transport_pyredis.test_Redis)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/buildd/kombu-1.0.2/kombu/tests/test_transport_pyredis.py", line 213, in test_publish__get
    {"hello": "world"})
AssertionError: False is not True : First argument is not a dictionary
    'False is not True : First argument is not a dictionary' = self._formatMessage('False is not True : First argument is not a dictionary', "%s is not True" % safe_repr(False))
>>  raise self.failureException('False is not True : First argument is not a dictionary')

[ask]

Could you give me the value of

 >>> import anyjson
 >>> anyjson.implementation.name

inside the chroot?

[fladi]

>>> anyjson.implementation.name
'cjson'

[fladi]

This seems to be the culprit, outside the chroot it's simplejson:
>>> anyjson.implementation.name
'simplejson'

I forced the installation of simplejson during build--time inside the chroot and now the test works. Still it would be great if kombu worked with cjson too.

[fladi]

Just an update on this issue, it's not a bug in kombu but in cjson which fails to properly decode a once encoded json string. This affects the decoding of the content_type of a message, resulting in "application/json" for which no decoder can then be found in _decoders. It is documented in Debian BTS: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=534709

This unittest shows the culprit:
import unittest2
from cjson import encode, decode
class TestDecoding(unittest2.TestCase):
def test_encode_decode(self):
original = 'application/json'
self.assertEqual(original, decode(encode(original)))

[ask]

cjson is not recommended anymore, and anyjson will show a warning if it is the current implementation.