Close #260 PR: Ignore possible test files under node_modules folder. F…

…ixes #226.. Fixes #226
avajs · Nov 24, 2015 · a4f8e24 · a4f8e24
1 parent ffb750b
commit a4f8e24
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 0 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
 node_modules
+!test/fixture/node_modules
 .nyc_output
 coverage
diff --git a/cli.js b/cli.js
@@ -207,6 +207,8 @@ function handlePaths(files) {
 		];
 	}
 
+	files.push('!**/node_modules/**');
+
 	// convert pinkie-promise to Bluebird promise
 	files = Promise.resolve(globby(files));
 

diff --git a/test/cli.js b/test/cli.js
@@ -245,3 +245,11 @@ test('test file that immediately exits with 0 exit code ', function (t) {
 		t.end();
 	});
 });
+
+test('test file in node_modules is ignored', function (t) {
+	execCli('fixture/node_modules/test.js', function (err, stdout, stderr) {
+		t.ok(err);
+		t.true(/Couldn't find any files to test/.test(stderr));
+		t.end();
+	});
+});
diff --git a/test/fixture/node_modules/test.js b/test/fixture/node_modules/test.js