incrby rejects string amounts

[eswald]

redis.StrictRedis.incrby accepts strings for its amount parameter, because the network protocol ends up turning it into a string anyway. However, FakeStrictRedis does not, requiring that it be an actual int instance.

Granted, being stricter guarantees that Redis will work whenever FakeRedis does, but it's still a surprising difference.

[amw]

I've been bitten by this as well. FakeRedis dies on a sane algorithm that is more readable in redis-py without explicit conversions:

# delay operation
r.incr('counter', 12)

# perform operation
value = r.get('counter')
do_something(value)
r.decr('counter', value)  # TypeError: unsupported operand type(s) for -: 'int' and 'str'

[bmerry]

Should be fixed in 1.0rc1.