search

limpyd / redis-limpyd

Provide an easy way to store python objects in Redis, without losing the power and the control of the Redis API
https://redis-limpyd.readthedocs.org/
Do What The F*ck You Want To Public License
72 stars 11 forks source link

python3.5 test failure on IndexableListFieldTest #94

Closed stuaxo closed 7 years ago

stuaxo commented 8 years ago 

======================================================================
FAIL: test_pop_commands_should_correctly_deindex_one_value (tests.fields.list.IndexableListFieldTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/stu/projects/external/redis-limpyd/tests/fields/list.py", line 65, in test_pop_commands_should_correctly_deindex_one_value
    bar = obj.field.lpop()
  File "/home/stu/projects/external/redis-limpyd/tests/base.py", line 103, in __exit__
    executed, self.num
AssertionError: 9 != 8 : 9 commands executed, 8 expected

----------------------------------------------------------------------
Ran 301 tests in 4.720s

FAILED (failures=1)
twidi commented 8 years ago

Hello @stuaxo

It will be a pleasure to review your pull request on the subject :p

Seriously: I really have no time to focus on this for now, so if you have some, feel free to dig on this.

stuaxo commented 8 years ago

NP - at the moment, I'm seeing about the typed fields thing.

twidi commented 7 years ago

I added support for python 3.5 and 3.6 without any change in a development branch recently without encountering this error, so I close ;)