add extra tests for .jshintignore

Author: vladikoff

.jshintignore

@@ -1 +1,2 @@
-test/fixtures/dontlint.txt
+test/fixtures/dontlint.txt
+node_modules/**

Gruntfile.js

@@ -36,8 +36,16 @@ module.exports = function(grunt) {
           'test/fixtures/wrule.js'
         ],
       },
-      ignoresSupport: {
-        src: ['test/fixtures/dontlint.txt'],
+      ignoresSupportGruntFile: {
+        src: ['test/fixtures/dontlint_gruntfile.txt'],
+        options: {
+          ignores: 'test/fixtures/dontlint_gruntfile.txt'
+        }
+      },
+      // this rule should ignore node_modules and dontlint.txt using .jshintignore.
+      // only nodemodule.js should be linted
+      ignoresSupportIgnoreFile: {
+        src: ['test/fixtures/dontlint.txt', 'node_modules/**', 'test/fixtures/nodemodule.js']
       },
       options: {
         jshintrc: '.jshintrc',

tasks/lib/jshint.js

@@ -152,7 +152,6 @@ exports.init = function(grunt) {
       delete options.extract;
     }
     var reporterOutputDir;
-    
     // Get reporter output directory for relative paths in reporters
     if (options.hasOwnProperty('reporterOutput')) {
       reporterOutputDir = path.dirname(options.reporterOutput);

test/fixtures/dontlint.txt renamed to test/fixtures/dontlint_gruntfile.txt

@@ -116,6 +116,17 @@ exports.jshint = {
       test.done();
     });
   },
+  jshintignoreInGruntfile: function(test) {
+    test.expect(1);
+    var files = [path.join(fixtures, 'dontlint_gruntfile.txt')];
+    var options = {
+      ignores: 'test/fixtures/dontlint_gruntfile.txt'
+    };
+    jshint.lint(files, options, function(results, data) {
+      test.equal(data.length, 0, 'Should not have linted a file listed in the Gruntfile ignores setting');
+      test.done();
+    });
+  },
   jshintignore: function(test) {
     test.expect(1);
     var files = [path.join(fixtures, 'dontlint.txt')];