Tests failing for redis

[varadharajan]

I got a failed test redis for the string "should find filtered set of records". The complete stack trace is:

AssertionError: false == true
at Object.ok (/usr/local/lib/node_modules/nodeunit/lib/types.js:83:39)
at /projects/jugglingdb/test/common_test.js:324:18
at /projects/jugglingdb/lib/abstract-class.js:214:13
at /projects/jugglingdb/lib/adapters/redis.js:158:13
at /projects/jugglingdb/node_modules/redis/index.js:922:13
at try_callback (/projects/jugglingdb/node_modules/redis/index.js:466:9)
at RedisClient.return_reply (/projects/jugglingdb/node_modules/redis/index.js:503:13)
at RedisReplyParser. (/projects/jugglingdb/node_modules/redis/index.js:259:14)
at RedisReplyParser.emit (events.js:64:17)
at RedisReplyParser.add_multi_bulk_reply (/projects/jugglingdb/node_modules/redis/lib/parser/javascript.js:306:18)

[1602]

Fixed in [master a8db2ad] Do not store null values, thanks you.
This bug was related to saving nulls in redis datastore. When you saving
some key with null value it saved as 'null' string, weird! Solution: remove
all null-data from object before saving, it works because undefined == null.

On Fri, Jan 13, 2012 at 8:31 PM, Varadharajan <
reply@reply.github.com

wrote:

I got a failed test redis for the string "should find filtered set of
records". The complete stack trace is:

AssertionError: false == true
at Object.ok (/usr/local/lib/node_modules/nodeunit/lib/types.js:83:39)
at /projects/jugglingdb/test/common_test.js:324:18
at /projects/jugglingdb/lib/abstract-class.js:214:13
at /projects/jugglingdb/lib/adapters/redis.js:158:13
at /projects/jugglingdb/node_modules/redis/index.js:922:13
at try_callback (/projects/jugglingdb/node_modules/redis/index.js:466:9)
at RedisClient.return_reply
(/projects/jugglingdb/node_modules/redis/index.js:503:13)
at RedisReplyParser.
(/projects/jugglingdb/node_modules/redis/index.js:259:14)
at RedisReplyParser.emit (events.js:64:17)
at RedisReplyParser.add_multi_bulk_reply
(/projects/jugglingdb/node_modules/redis/lib/parser/javascript.js:306:18)

---

Reply to this email directly or view it on GitHub:
#20

Thanks,
Anatoliy Chakkaev