redis/node-redis
1
0
Fork 0
mirror of https://github.com/redis/node-redis.git synced 2025-08-07 13:22:56 +03:00
Code Activity

Explicitly check for empty Array instead of comparing to []

Browse Source
This commit is contained in:
Matt Ranney
2010-11-30 13:37:53 -08:00
parent 275030148b
commit f30e08235f
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
test.js
View File

@@ -74,6 +74,10 @@ function require_error(label) {
}; };
} }
function is_empty_array(obj) {
return Array.isArray(obj) && obj.length === 0;
}
function last(name, fn) { function last(name, fn) {
return function (err, results) { return function (err, results) {
fn(err, results); fn(err, results);
@@ -160,7 +164,7 @@ tests.MULTI_3 = function () {
]) ])
.scard("some set") .scard("some set")
.exec(function (err, replies) { .exec(function (err, replies) {
assert.deepEqual(replies[2][0], [], name); assert.strictEqual(true, is_empty_array(replies[2][0]), name);
next(name); next(name);
}); });
}; };
@@ -366,7 +370,7 @@ tests.MULTIBULK_ZERO_LENGTH = function () {
var name = "MULTIBULK_ZERO_LENGTH"; var name = "MULTIBULK_ZERO_LENGTH";
client.KEYS(['users:*'], function (err, results) { client.KEYS(['users:*'], function (err, results) {
assert.strictEqual(null, err, 'error on empty multibulk reply'); assert.strictEqual(null, err, 'error on empty multibulk reply');
assert.deepEqual([], results); assert.strictEqual(true, is_empty_array(results), "not an empty array");
next(name); next(name);
}); });
}; };