Skip to content

Commit b6df174

Browse files
committed
Retain error message from jslint problems
1 parent dd83d4f commit b6df174

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

jones-test/lib/LintTest.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,20 +46,24 @@ var lintOptions = {
4646
"regexp" : true // allow . and [^ ...] in regular expressions
4747
};
4848

49+
var jslintLoaderError;
50+
4951
try {
5052
lintModule = require("jslint/lib/linter");
5153
linter = lintModule.lint;
5254
assert(typeof linter === 'function');
5355
haveJsLint = true; // older node-jslint
5456
}
5557
catch(e1) {
58+
jslintLoaderError = e1.message;
5659
try {
5760
lintModule = require("jslint");
5861
linter = lintModule.load("es5");
5962
assert(typeof linter === 'function');
6063
haveJsLint = true; // newer node-jslint
6164
}
6265
catch(e2) {
66+
jslintLoaderError = e2.message;
6367
}
6468
}
6569
if(! haveJsLint) {
@@ -92,11 +96,8 @@ LintSmokeTest.prototype = new Test.Test();
9296

9397
LintSmokeTest.prototype.run = function() {
9498
if(skipTests) {
95-
this.fail("linter is not available");
96-
} else if (typeof linter !== 'function') {
97-
this.fail("incompatible linter");
98-
}
99-
else {
99+
this.fail("linter is not available: " + jslintLoaderError);
100+
} else {
100101
this.pass();
101102
}
102103
};

0 commit comments

Comments
 (0)