[New] Add --ignore-pattern flag

This PR introduces the --ignore-pattern flag as a shorthand for ignoring test files.

The flag may be used together with --ignore; the input will be concatenated in that case. I figured this behavior would make the most sense, since users may want to ignore everything in .gitignore and some other files on top.

Fixes tape-testing#586

.eslintrc

@@ -10,6 +10,7 @@
         "allowReserved": false,
     },
     "rules": {
+        "array-bracket-newline": "off",
         "array-bracket-spacing": "off",
         "complexity": "off",
         "func-style": ["error", "declaration"],

bin/tape

@@ -7,9 +7,9 @@ var objectKeys = require('object-keys');
 
 var opts = parseOpts(process.argv.slice(2), {
 	alias: { r: 'require', i: 'ignore' },
-	string: ['require', 'ignore'],
+	string: ['require', 'ignore', 'ignore-pattern'],
 	boolean: ['only'],
-	default: { r: [], i: null, only: null }
+	default: { r: [], i: null, 'ignore-pattern': null, only: null }
 });
 
 if (typeof opts.only === 'boolean') {
@@ -36,14 +36,22 @@ var resolvePath = require('path').resolve;
 var requireResolve = require.resolve;
 
 var matcher;
+var ignoreStr = '';
 if (typeof opts.ignore === 'string') {
 	var readFileSync = require('fs').readFileSync;
 	try {
-		var ignoreStr = readFileSync(resolvePath(cwd, opts.ignore || '.gitignore'), 'utf-8');
+		ignoreStr = readFileSync(resolvePath(cwd, opts.ignore || '.gitignore'), 'utf-8');
 	} catch (e) {
 		console.error(e.message);
 		process.exit(2);
 	}
+}
+
+if (typeof opts['ignore-pattern'] === 'string') {
+	ignoreStr += '\n' + opts['ignore-pattern'];
+}
+
+if (ignoreStr) {
 	var ignore = require('dotignore');
 	matcher = ignore.createMatcher(ignoreStr);
 }

test/ignore-pattern.js

@@ -0,0 +1,34 @@
+'use strict';
+
+var tap = require('tap');
+var path = require('path');
+var execFile = require('child_process').execFile;
+
+var tapeBin = path.join(process.cwd(), 'bin/tape');
+
+tap.test('should allow ignore file together with --ignorePattern', function (tt) {
+	tt.plan(1);
+	var proc = execFile(tapeBin, ['--ignore', '.ignore', '--ignorePattern', 'fake_other_ignored_dir', '**/*.js'], { cwd: path.join(__dirname, 'ignorePattern') });
+
+	proc.on('exit', function (code) {
+		tt.equals(code, 0);
+	});
+});
+
+tap.test('should allow --ignorePattern without ignore file', function (tt) {
+	tt.plan(1);
+	var proc = execFile(tapeBin, ['--ignorePattern', 'fake_*', '**/*.js'], { cwd: path.join(__dirname, 'ignorePattern') });
+
+	proc.on('exit', function (code) {
+		tt.equals(code, 0);
+	});
+});
+
+tap.test('should fail if not ignoring', function (tt) {
+	tt.plan(1);
+	var proc = execFile(tapeBin, ['**/*.js'], { cwd: path.join(__dirname, 'ignorePattern') });
+
+	proc.on('exit', function (code) {
+		tt.equals(code, 1);
+	});
+});

test/ignorePattern/.ignore

@@ -0,0 +1 @@
+fake_node_modules

test/ignorePattern/fake_other_ignored_dir/stub1.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../');
+
+tape.test(function (t) {
+	t.plan(1);
+	t.fail('Should not print');
+});

test/ignorePattern/fake_other_ignored_dir/stub2.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../');
+
+tape.test(function (t) {
+	t.fail('Should not print');
+	t.end();
+});

test/ignorePattern/test.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../');
+
+tape.test(function (t) {
+	t.pass('Should print');
+	t.end();
+});

test/ignorePattern/test/stub1.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../');
+
+tape.test(function (t) {
+	t.plan(1);
+	t.pass('test/stub1');
+});

test/ignorePattern/test/stub2.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../');
+
+tape.test(function (t) {
+	t.pass('test/stub2');
+	t.end();
+});

test/ignorePattern/test/sub/sub.stub1.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../../');
+
+tape.test(function (t) {
+	t.plan(1);
+	t.pass('test/sub/stub1');
+});

test/ignorePattern/test/sub/sub.stub2.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../../../');
+
+tape.test(function (t) {
+	t.pass('test/sub/stub2');
+	t.end();
+});

test/ignorePattern/test2.js

@@ -0,0 +1,8 @@
+'use strict';
+
+var tape = require('../../');
+
+tape.test(function (t) {
+	t.pass('Should print');
+	t.end();
+});