Fix: --no-ignore should not un-ignore default ignores (fixes #5547)

Author: alberto

lib/ignored-paths.js

@@ -114,10 +114,9 @@ function IgnoredPaths(options) {
         addPattern(this.ig.default, [".*", "!../"]);
     }
 
-    if (options.ignore !== false) {
-
-        addPattern(this.ig.default, this.defaultPatterns);
+    addPattern(this.ig.default, this.defaultPatterns);
 
+    if (options.ignore !== false) {
         var ignorePath;
 
         if (options.ignorePattern) {

tests/lib/ignored-paths.js

@@ -367,10 +367,10 @@ describe("IgnoredPaths", function() {
             assert.isTrue(ignoredPaths.contains(getFixturePath("node_modules/package/file.js")));
         });
 
-        it("should not apply defaultPatterns if ignore option is is false", function() {
+        it("should still apply defaultPatterns if ignore option is is false", function() {
             var ignoredPaths = new IgnoredPaths({ ignore: false, cwd: getFixturePath() });
-            assert.isFalse(ignoredPaths.contains(getFixturePath("bower_components/package/file.js")));
-            assert.isFalse(ignoredPaths.contains(getFixturePath("node_modules/package/file.js")));
+            assert.isTrue(ignoredPaths.contains(getFixturePath("bower_components/package/file.js")));
+            assert.isTrue(ignoredPaths.contains(getFixturePath("node_modules/package/file.js")));
         });
 
         it("should not ignore files in defaultPatterns within a subdirectory", function() {