ya, i think it might be because your fake cache is synchronous


On Mon, Jan 13, 2014 at 7:36 PM, Xiao Ma <notifications@github.com> wrote:

> In lib/CacheCluster.js:
>
> >
> >    var promises = []
> > -  for (var uri in servers) {
> > -    var server = servers[uri]
> > -    serverArr.push(server)
> > -    promises.push(server.instance.mget(server.keys).fail(returnEmptyArr))
> > +  for (var uri in keysPerInstance) {
> > +    var keysOnInstance = keysPerInstance[uri]
> > +    promises.push(this._servers[uri].mget(keysOnInstance)
> > +      .then(function (data) {
> > +        for (var i = 0; i < keysOnInstance.length; i++) values[keysOnInstance[i]] = data[i]
>
> Fixed. Weird that the unit test cases pass. Maybe because it runs too fast?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Obvious/zcache/pull/21/files#r8847165>
> .
>