Fix require option for multiple entries (#173)

sindresorhus · Apr 27, 2017 · 55004ca · 55004ca · leosco · Jan 17, 2018
1 parent e878086
commit 55004ca
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -8,6 +8,11 @@ const npmRunPath = require('npm-run-path');
 
 const HUNDRED_MEGABYTES = 1000 * 1000 * 100;
 
+// Mocha options that can be specified multiple times
+const MULTIPLE_OPTS = [
+	'require'
+];
+
 module.exports = opts => {
 	opts = Object.assign({
 		colors: true,
@@ -18,7 +23,7 @@ module.exports = opts => {
 	for (const key of Object.keys(opts)) {
 		const val = opts[key];
 
-		if (Array.isArray(val)) {
+		if (MULTIPLE_OPTS.indexOf(key) > 0 && Array.isArray(val)) {
 			opts[key] = val.join(',');
 		}
 	}

diff --git a/test/fixtures/fixture-require1.js b/test/fixtures/fixture-require1.js
@@ -0,0 +1 @@
+'use strict';
diff --git a/test/fixtures/fixture-require2.js b/test/fixtures/fixture-require2.js
@@ -0,0 +1 @@
+'use strict';
diff --git a/test/test.js b/test/test.js
@@ -61,4 +61,18 @@ describe('mocha()', () => {
 		stream.write(fixture('fixture-pass.js'));
 		stream.end();
 	});
+
+	it('should require two files', done => {
+		const stream = mocha({require: [
+			'test/fixtures/fixture-require1.js',
+			'test/fixtures/fixture-require2.js'
+		]});
+
+		stream.once('_result', result => {
+			assert(/1 passing/.test(result.stdout));
+			done();
+		});
+		stream.write(fixture('fixture-pass.js'));
+		stream.end();
+	});
 });