faeldt / redis-mock

Node.js redis client mock
112 stars 156 forks source link

Missing "options" property from RedisClient #58

Open ksmithut opened 6 years ago

ksmithut commented 6 years ago

The redis-mock library is missing the "options" property from the RedisClient. I'm using a library called cache-manager-redis-store that uses redis to manage a cache. On this line, they access the options property and subsequently options that were normalized upon instantiation, which I assume is a normalized options object based on what was passed into the createClient method taking into account that people might call createClient with varying function signatures, e.g. .createClient(redisUrl, redisOptions), or .createClient(redisOptions). Here are the relevant lines (1) (2) in the node_redis project where node_redis is normalizing these options.