Add support for ioredis

[luin]

Hi, I'm the author of ioredis, a node.js redis client that supports Sentinel and Cluster.

I'm using this awesome module in my application, however this module doesn't support ioredis, which means I can't use it with my sentinel servers.

It's trivial to support ioredis since it's compatible with node_redis for most APIs. The only difference is the result of multi in ioredis is [[err1, res1], [err2, res2], [err3, res3]] and in node_redis is [res1, res2, res3]. Refer to https://github.com/luin/ioredis/wiki/Migrating-from-node_redis for more details.

The only change have to be made is changing (!res || !res[0]) to (!res || !res[0] || !res[0][1]), which works with both node_redis and ioredis. What's more, I added forEach wrapper around the original tests so that it will test this module against both node_redis and ioredis.

[noamshemesh]

The only problem that I see is that you rely on the fact that node_redis returns a string result.

I think that a better way would be to check if res[0] is an array, if so, check if res[0][1] exists.
what do you think?

[luin]

Hmm...yes, it would make more sense. Just submitted a commit for this.

[noamshemesh]

I really like this PR and want to merge it, just one more question, don't you want to check the response also for the multi() in decr() func?
I suggest to extract the response handling to another function and call it on every multi() callback. How does that sound to you?

[luin]

Yes, of course. Didn't notice there's another multi 😢

[noamshemesh]

thanks!