fix --reporter-option to allow comma-separated options; closes #3706

Signed-off-by: Christopher Hiller <boneskull@boneskull.com>
mochajs · Feb 10, 2019 · 4a0d9df · 4a0d9df
1 parent 186ca36
commit 4a0d9df
Show file tree

Hide file tree

Showing 2 changed files with 68 additions and 0 deletions.
diff --git a/test/integration/fixtures/options/reporter-with-options.fixture.js b/test/integration/fixtures/options/reporter-with-options.fixture.js
@@ -0,0 +1,7 @@
+'use strict';
+
+function ReporterWithOptions(runner, options) {
+  console.log(JSON.stringify(options.reporterOption));
+}
+
+module.exports = ReporterWithOptions;
diff --git a/test/integration/options/reporter-option.spec.js b/test/integration/options/reporter-option.spec.js
@@ -1,6 +1,7 @@
 'use strict';
 
 var runMocha = require('../helpers').runMocha;
+var path = require('path');
 
 describe('--reporter-option', function() {
   describe('when given options w/ invalid format', function() {
@@ -21,5 +22,65 @@ describe('--reporter-option', function() {
         'pipe'
       );
     });
+
+    it('should allow comma-separated values', function(done) {
+      runMocha(
+        'passing.fixture.js',
+        [
+          '--reporter',
+          path.join(
+            __dirname,
+            '..',
+            'fixtures',
+            'options',
+            'reporter-with-options.fixture.js'
+          ),
+          '--reporter-option',
+          'foo=bar,baz=quux'
+        ],
+        function(err, res) {
+          if (err) {
+            return done(err);
+          }
+          expect(res, 'to have passed').and(
+            'to contain output',
+            /{"foo":"bar","baz":"quux"}/
+          );
+          done();
+        },
+        'pipe'
+      );
+    });
+
+    it('should allow repeated options', function(done) {
+      runMocha(
+        'passing.fixture.js',
+        [
+          '--reporter',
+          path.join(
+            __dirname,
+            '..',
+            'fixtures',
+            'options',
+            'reporter-with-options.fixture.js'
+          ),
+          '--reporter-option',
+          'foo=bar',
+          '--reporter-option',
+          'baz=quux'
+        ],
+        function(err, res) {
+          if (err) {
+            return done(err);
+          }
+          expect(res, 'to have passed').and(
+            'to contain output',
+            /{"foo":"bar","baz":"quux"}/
+          );
+          done();
+        },
+        'pipe'
+      );
+    });
   });
 });