closes #4142

In #4004 there was a change to use the documented `reporterOption` in favour of the setting that had always been used in practice called `reporterOptions`. This broke a lot of configurations that used the `reporterOptions`, which was the only way it every worked AFAIK.

This changes the documentation to specify `reporterOptions` instead and ensure that any one that has switched to `reporterOption` after upgrade, still works.

lib/mocha.js

@@ -100,7 +100,10 @@ function Mocha(options) {
   this.grep(options.grep)
     .fgrep(options.fgrep)
     .ui(options.ui)
-    .reporter(options.reporter, options.reporterOption)
+    .reporter(
+      options.reporter,
+      options.reporterOption || options.reporterOptions // reporterOptions was previously the only way to specify options to reporter
+    )
     .slow(options.slow)
     .global(options.global);
 
@@ -232,6 +235,7 @@ Mocha.prototype.reporter = function(reporter, reporterOptions) {
   this.options.reporterOption = reporterOptions;
   // alias option name is used in public reporters xunit/tap/progress
   this.options.reporterOptions = reporterOptions;
+
   return this;
 };
 

test/integration/fixtures/options/reporter-with-options.fixture.js

@@ -1,7 +1,7 @@
 'use strict';
 
 function ReporterWithOptions(runner, options) {
-  console.log(JSON.stringify(options.reporterOption));
+  console.log(JSON.stringify(options.reporterOptions));
 }
 
 module.exports = ReporterWithOptions;

test/unit/mocha.spec.js

@@ -446,6 +446,36 @@ describe('Mocha', function() {
       var mocha = new Mocha(opts);
       expect(mocha.reporter(), 'to be', mocha);
     });
+
+    it('should keep reporterOption on options', function() {
+      var mocha = new Mocha({
+        reporter: 'spec',
+        reporterOption: {
+          foo: 'bar'
+        }
+      });
+      expect(mocha.options.reporterOption, 'to have property', 'foo', 'bar');
+    });
+
+    it('should alias reporterOption to legacy reporterOptions', function() {
+      var mocha = new Mocha({
+        reporter: 'spec',
+        reporterOption: {
+          foo: 'bar'
+        }
+      });
+      expect(mocha.options.reporterOptions, 'to have property', 'foo', 'bar');
+    });
+
+    it('should support legacy reporterOptions', function() {
+      var mocha = new Mocha({
+        reporter: 'spec',
+        reporterOptions: {
+          foo: 'bar'
+        }
+      });
+      expect(mocha.options.reporterOption, 'to have property', 'foo', 'bar');
+    });
   });
 
   describe('#run(fn)', function() {