Fixed #1133: Make shelljs imply node.

jshint · Jun 26, 2013 · 66f3e4c · 66f3e4c
1 parent 79dc812
commit 66f3e4c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/src/stable/jshint.js b/src/stable/jshint.js
@@ -330,6 +330,7 @@ var JSHINT = (function () {
 
 		if (state.option.shelljs) {
 			combine(predefined, vars.shelljs);
+			combine(predefined, vars.node);
 		}
 
 		if (state.option.phantom) {

diff --git a/tests/stable/unit/fixtures/shelljs.js b/tests/stable/unit/fixtures/shelljs.js
@@ -24,3 +24,8 @@ chmod();
 config.hi = 'hi';
 error();
 tempdir();
+
+// node stuff
+require('hi');
+module.exports = 'hi';
+process.exit();
diff --git a/tests/stable/unit/options.js b/tests/stable/unit/options.js
@@ -338,6 +338,9 @@ exports.shelljs = function (test) {
 		.addError(24, "'config' is not defined.")
 		.addError(25, "'error' is not defined.")
 		.addError(26, "'tempdir' is not defined.")
+		.addError(29, "'require' is not defined.")
+		.addError(30, "'module' is not defined.")
+		.addError(31, "'process' is not defined.")
 		.test(src, { undef: true });
 
 	TestRun(test, 2)