Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

node test/test.js -q FAILED (nodejs 0.1.102 + redis 1.3.17 + redis-node 0.3.5) #34

Closed
ogoudron opened this issue Jul 31, 2010 · 5 comments

Comments

@ogoudron
Copy link

The result is :

assert:80
throw new assert.AssertionError({
^
AssertionError: "testZINTER" "ERR unknown command 'zinter'"
at /home/ogoudron/fictorial-redis-node-client-30e9f41/test/test.js:121:25
at Client.onReply_ (/home/ogoudron/fictorial-redis-node-client-30e9f41/lib/redis-client.js:400:34)
at /home/ogoudron/fictorial-redis-node-client-30e9f41/lib/redis-client.js:143:30
at ReplyParser.feed (/home/ogoudron/fictorial-redis-node-client-30e9f41/lib/redis-client.js:160:55)
at Stream. (/home/ogoudron/fictorial-redis-node-client-30e9f41/lib/redis-client.js:337:28)
at Stream.emit (events:26:26)
at IOWatcher.callback (net:512:16)
at node.js:265:9

Tested on Ubuntu Desktop 10.04 (VIA C7) and Debian 5.0 (Celeron 220).

Bests regards, Olivier Goudron

@ogoudron
Copy link
Author

after some search (i am new to redis) i found that ZINTER/ZUNION have been renamed to ZINTERSTORE/ZUNIONSTORE.
Maybe the test.js should be aware of this.

@matthewford
Copy link

patch as a gist: http://gist.github.com/503966

@fictorial
Copy link
Owner

branch redis-2.0

@matthewford
Copy link

3am coding for you; I thought I had the 2.0 branch, but L had just created a local branch called radis-2.0. doh

@ogoudron
Copy link
Author

ogoudron commented Aug 2, 2010

Thanks :-)

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants