redis/node-redis
1
0
Fork 0
mirror of https://github.com/redis/node-redis.git synced 2025-08-06 02:15:48 +03:00
Code Activity

Do not wrap errors into other errors. The trace is going to be manipulated that way.

Browse Source
This commit is contained in:
Ruben Bridgewater
2015-09-07 14:55:57 +02:00
parent 4e42fb70f3
commit 0925885a88
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
index.js
View File

@@ -1055,7 +1055,7 @@ Multi.prototype.exec = function (callback) {
if (typeof cur[cur.length - 1] === "function") { if (typeof cur[cur.length - 1] === "function") {
cur[cur.length - 1](err); cur[cur.length - 1](err);
} else { } else {
errors.push(new Error(err)); errors.push(err);
} }
} }
}); });
@@ -1065,7 +1065,7 @@ Multi.prototype.exec = function (callback) {
return this._client.send_command("exec", [], function (err, replies) { return this._client.send_command("exec", [], function (err, replies) {
if (err) { if (err) {
if (callback) { if (callback) {
errors.push(new Error(err)); errors.push(err);
callback(errors); callback(errors);
return; return;
} else { } else {

4
test/commands/multi.spec.js
View File

@@ -231,8 +231,8 @@ describe("The 'multi' method", function () {
client.multi().set("foo").exec(function (err, reply) { client.multi().set("foo").exec(function (err, reply) {
assert(Array.isArray(err), "err should be an array"); assert(Array.isArray(err), "err should be an array");
assert.equal(2, err.length, "err should have 2 items"); assert.equal(2, err.length, "err should have 2 items");
assert(err[0].message.match(/ERR/), "First error message should contain ERR"); assert(err[0].message.match(/^ERR/), "First error message should begin with ERR");
assert(err[1].message.match(/EXECABORT/), "First error message should contain EXECABORT"); assert(err[1].message.match(/^EXECABORT/), "First error message should begin with EXECABORT");
return done(); return done();
}); });
}); });