Skip to content

Commit

Permalink
Removing order requirement for KEYS test
Browse files Browse the repository at this point in the history
Close #378.

Signed-off-by: DTrejo <david.daniel.trejo@gmail.com>
  • Loading branch information
brycebaril authored and DTrejo committed Feb 24, 2013
1 parent 2a12ca3 commit f3c298d
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions test.js
Original file line number Diff line number Diff line change
Expand Up @@ -171,12 +171,13 @@ tests.MULTI_2 = function () {
["set", "foo2", require_error(name)],
["incr", "multifoo", require_number(13, name)],
["incr", "multibar", require_number(23, name)]

]).exec(function (err, replies) {

if (server_version_at_least(client, [2, 6, 5])) {
assert.notEqual(err, null, name);
assert.equal(replies, undefined, name);
}
else {
} else {
assert.strictEqual(2, replies[0].length, name);
assert.strictEqual("12", replies[0][0].toString(), name);
assert.strictEqual("22", replies[0][1].toString(), name);
Expand Down Expand Up @@ -810,8 +811,8 @@ tests.KEYS = function () {
client.KEYS(["test keys*"], function (err, results) {
assert.strictEqual(null, err, "result sent back unexpected error: " + err);
assert.strictEqual(2, results.length, name);
assert.strictEqual("test keys 1", results[0].toString(), name);
assert.strictEqual("test keys 2", results[1].toString(), name);
assert.ok(~results.indexOf("test keys 1"));
assert.ok(~results.indexOf("test keys 2"));
next(name);
});
};
Expand Down

0 comments on commit f3c298d

Please sign in to comment.