Skip to content

Commit 59d4e08

Browse files
authored
Merge pull request #673 from visionmedia/v5.0.0-1
feat(v5.0.0): next release
2 parents 910a111 + f503048 commit 59d4e08

File tree

5 files changed

+1858
-1249
lines changed

5 files changed

+1858
-1249
lines changed

.eslintrc

-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
"consistent-return": [0],
1515

1616
// Disabled but may want to refactor code eventually
17-
"no-shadow": [1],
1817
"no-use-before-define": [2, "nofunc"],
1918
"no-underscore-dangle": [0],
2019

lib/test.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ Test.prototype.end = function(fn) {
147147
*/
148148

149149
Test.prototype.assert = function(resError, res, fn) {
150-
var error;
150+
var errorObj;
151151
var i;
152152

153153
// check for unexpected network errors or server not running/reachable errors
@@ -164,23 +164,23 @@ Test.prototype.assert = function(resError, res, fn) {
164164
if (!res && resError) {
165165
if (resError instanceof Error && resError.syscall === 'connect'
166166
&& Object.getOwnPropertyNames(sysErrors).indexOf(resError.code) >= 0) {
167-
error = new Error(resError.code + ': ' + sysErrors[resError.code]);
167+
errorObj = new Error(resError.code + ': ' + sysErrors[resError.code]);
168168
} else {
169-
error = resError;
169+
errorObj = resError;
170170
}
171171
}
172172

173173
// asserts
174-
for (i = 0; i < this._asserts.length && !error; i += 1) {
175-
error = this._assertFunction(this._asserts[i], res);
174+
for (i = 0; i < this._asserts.length && !errorObj; i += 1) {
175+
errorObj = this._assertFunction(this._asserts[i], res);
176176
}
177177

178178
// set unexpected superagent error if no other error has occurred.
179-
if (!error && resError instanceof Error && (!res || resError.status !== res.status)) {
180-
error = resError;
179+
if (!errorObj && resError instanceof Error && (!res || resError.status !== res.status)) {
180+
errorObj = resError;
181181
}
182182

183-
fn.call(this, error || null, res);
183+
fn.call(this, errorObj || null, res);
184184
};
185185

186186
/**

0 commit comments

Comments
 (0)