Implement key expiration via individual setTimeouts and remove expire

faeldt / redis-mock

Node.js redis client mock

112 stars 157 forks source link

Closed cmawhorter closed 9 years ago

cmawhorter commented 10 years ago

I noticed there was a bug in the expire loop interval, which was mirrored in the tests (which is why they were passing).

1s was specified as 100ms, when it should be 1000.

Instead of just fixing that, I swapped in something that should be significantly more accurate when expiring keys. I also fixed the tests.

faeldt commented 9 years ago

Very cool. Thank you :)