mocha 9.1.2 → 9.2.1

package/lib/reporters/xunit.js

@@ -74,19 +74,19 @@ function XUnit(runner, options) {
   // fall back to the default suite name
   suiteName = suiteName || DEFAULT_SUITE_NAME;
 
-  runner.on(EVENT_TEST_PENDING, function(test) {
+  runner.on(EVENT_TEST_PENDING, function (test) {
     tests.push(test);
   });
 
-  runner.on(EVENT_TEST_PASS, function(test) {
+  runner.on(EVENT_TEST_PASS, function (test) {
     tests.push(test);
   });
 
-  runner.on(EVENT_TEST_FAIL, function(test) {
+  runner.on(EVENT_TEST_FAIL, function (test) {
     tests.push(test);
   });
 
-  runner.once(EVENT_RUN_END, function() {
+  runner.once(EVENT_RUN_END, function () {
     self.write(
       tag(
         'testsuite',
@@ -103,7 +103,7 @@ function XUnit(runner, options) {
       )
     );
 
-    tests.forEach(function(t) {
+    tests.forEach(function (t) {
       self.test(t);
     });
 
@@ -122,9 +122,9 @@ inherits(XUnit, Base);
  * @param failures
  * @param {Function} fn
  */
-XUnit.prototype.done = function(failures, fn) {
+XUnit.prototype.done = function (failures, fn) {
   if (this.fileStream) {
-    this.fileStream.end(function() {
+    this.fileStream.end(function () {
       fn(failures);
     });
   } else {
@@ -137,7 +137,7 @@ XUnit.prototype.done = function(failures, fn) {
  *
  * @param {string} line
  */
-XUnit.prototype.write = function(line) {
+XUnit.prototype.write = function (line) {
   if (this.fileStream) {
     this.fileStream.write(line + '\n');
   } else if (typeof process === 'object' && process.stdout) {
@@ -152,7 +152,7 @@ XUnit.prototype.write = function(line) {
  *
  * @param {Test} test
  */
-XUnit.prototype.test = function(test) {
+XUnit.prototype.test = function (test) {
   Base.useColors = false;
 
   var attrs = {

package/lib/runnable.js

@@ -57,7 +57,7 @@ utils.inherits(Runnable, EventEmitter);
 /**
  * Resets the state initially or for a next run.
  */
-Runnable.prototype.reset = function() {
+Runnable.prototype.reset = function () {
   this.timedOut = false;
   this._currentRetry = 0;
   this.pending = false;
@@ -86,7 +86,7 @@ Runnable.prototype.reset = function() {
  * @returns {Runnable} this
  * @chainable
  */
-Runnable.prototype.timeout = function(ms) {
+Runnable.prototype.timeout = function (ms) {
   if (!arguments.length) {
     return this._timeout;
   }
@@ -120,7 +120,7 @@ Runnable.prototype.timeout = function(ms) {
  * @param {number|string} ms
  * @return {Runnable|number} ms or Runnable instance.
  */
-Runnable.prototype.slow = function(ms) {
+Runnable.prototype.slow = function (ms) {
   if (!arguments.length || typeof ms === 'undefined') {
     return this._slow;
   }
@@ -138,7 +138,7 @@ Runnable.prototype.slow = function(ms) {
  * @memberof Mocha.Runnable
  * @public
  */
-Runnable.prototype.skip = function() {
+Runnable.prototype.skip = function () {
   this.pending = true;
   throw new Pending('sync skip; aborting execution');
 };
@@ -148,7 +148,7 @@ Runnable.prototype.skip = function() {
  *
  * @private
  */
-Runnable.prototype.isPending = function() {
+Runnable.prototype.isPending = function () {
   return this.pending || (this.parent && this.parent.isPending());
 };
 
@@ -157,7 +157,7 @@ Runnable.prototype.isPending = function() {
  * @return {boolean}
  * @private
  */
-Runnable.prototype.isFailed = function() {
+Runnable.prototype.isFailed = function () {
   return !this.isPending() && this.state === constants.STATE_FAILED;
 };
 
@@ -166,7 +166,7 @@ Runnable.prototype.isFailed = function() {
  * @return {boolean}
  * @private
  */
-Runnable.prototype.isPassed = function() {
+Runnable.prototype.isPassed = function () {
   return !this.isPending() && this.state === constants.STATE_PASSED;
 };
 
@@ -175,7 +175,7 @@ Runnable.prototype.isPassed = function() {
  *
  * @private
  */
-Runnable.prototype.retries = function(n) {
+Runnable.prototype.retries = function (n) {
   if (!arguments.length) {
     return this._retries;
   }
@@ -187,7 +187,7 @@ Runnable.prototype.retries = function(n) {
  *
  * @private
  */
-Runnable.prototype.currentRetry = function(n) {
+Runnable.prototype.currentRetry = function (n) {
   if (!arguments.length) {
     return this._currentRetry;
   }
@@ -202,7 +202,7 @@ Runnable.prototype.currentRetry = function(n) {
  * @public
  * @return {string}
  */
-Runnable.prototype.fullTitle = function() {
+Runnable.prototype.fullTitle = function () {
   return this.titlePath().join(' ');
 };
 
@@ -213,7 +213,7 @@ Runnable.prototype.fullTitle = function() {
  * @public
  * @return {string}
  */
-Runnable.prototype.titlePath = function() {
+Runnable.prototype.titlePath = function () {
   return this.parent.titlePath().concat([this.title]);
 };
 
@@ -222,7 +222,7 @@ Runnable.prototype.titlePath = function() {
  *
  * @private
  */
-Runnable.prototype.clearTimeout = function() {
+Runnable.prototype.clearTimeout = function () {
   clearTimeout(this.timer);
 };
 
@@ -231,7 +231,7 @@ Runnable.prototype.clearTimeout = function() {
  *
  * @private
  */
-Runnable.prototype.resetTimeout = function() {
+Runnable.prototype.resetTimeout = function () {
   var self = this;
   var ms = this.timeout();
 
@@ -239,7 +239,7 @@ Runnable.prototype.resetTimeout = function() {
     return;
   }
   this.clearTimeout();
-  this.timer = setTimeout(function() {
+  this.timer = setTimeout(function () {
     if (self.timeout() === 0) {
       return;
     }
@@ -254,7 +254,7 @@ Runnable.prototype.resetTimeout = function() {
  * @private
  * @param {string[]} globals
  */
-Runnable.prototype.globals = function(globals) {
+Runnable.prototype.globals = function (globals) {
   if (!arguments.length) {
     return this._allowedGlobals;
   }
@@ -267,7 +267,7 @@ Runnable.prototype.globals = function(globals) {
  * @param {Function} fn
  * @private
  */
-Runnable.prototype.run = function(fn) {
+Runnable.prototype.run = function (fn) {
   var self = this;
   var start = new Date();
   var ctx = this.ctx;
@@ -367,13 +367,13 @@ Runnable.prototype.run = function(fn) {
     if (result && typeof result.then === 'function') {
       self.resetTimeout();
       result.then(
-        function() {
+        function () {
           done();
           // Return null so libraries like bluebird do not warn about
           // subsequently constructed Promises.
           return null;
         },
-        function(reason) {
+        function (reason) {
           done(reason || new Error('Promise rejected with no or falsy reason'));
         }
       );
@@ -391,7 +391,7 @@ Runnable.prototype.run = function(fn) {
   }
 
   function callFnAsync(fn) {
-    var result = fn.call(ctx, function(err) {
+    var result = fn.call(ctx, function (err) {
       if (err instanceof Error || toString.call(err) === '[object Error]') {
         return done(err);
       }
@@ -423,7 +423,7 @@ Runnable.prototype.run = function(fn) {
  * @returns {Error} a "timeout" error
  * @private
  */
-Runnable.prototype._timeoutError = function(ms) {
+Runnable.prototype._timeoutError = function (ms) {
   let msg = `Timeout of ${ms}ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.`;
   if (this.file) {
     msg += ' (' + this.file + ')';
@@ -463,7 +463,7 @@ var constants = utils.defineConstants(
  * @returns {*|Error} `value`, otherwise an `Error`
  * @private
  */
-Runnable.toValueOrError = function(value) {
+Runnable.toValueOrError = function (value) {
   return (
     value ||
     createInvalidExceptionError(

package/lib/runner.js

@@ -139,7 +139,7 @@ class Runner extends EventEmitter {
    * @param {boolean} [opts.cleanReferencesAfterRun] - Whether to clean references to test fns and hooks when a suite is done.
    * @param {boolean} [opts.delay] - Whether to delay execution of root suite until ready.
    * @param {boolean} [opts.dryRun] - Whether to report tests without running them.
-   * @param {boolean} [options.failZero] - Whether to fail test run if zero tests encountered.
+   * @param {boolean} [opts.failZero] - Whether to fail test run if zero tests encountered.
    */
   constructor(suite, opts) {
     super();
@@ -168,7 +168,7 @@ class Runner extends EventEmitter {
      * @type {Map<EventEmitter,Map<string,Set<EventListener>>>}
      */
     this._eventListeners = new Map();
-    this.on(constants.EVENT_TEST_END, function(test) {
+    this.on(constants.EVENT_TEST_END, function (test) {
       if (test.type === 'test' && test.retriedTest() && test.parent) {
         var idx =
           test.parent.tests && test.parent.tests.indexOf(test.retriedTest());
@@ -176,7 +176,7 @@ class Runner extends EventEmitter {
       }
       self.checkGlobals(test);
     });
-    this.on(constants.EVENT_HOOK_END, function(hook) {
+    this.on(constants.EVENT_HOOK_END, function (hook) {
       self.checkGlobals(hook);
     });
     this._defaultGrep = /.*/;
@@ -225,7 +225,7 @@ Runner.immediately = global.setImmediate || process.nextTick;
  * @param {string} fn - Listener function
  * @private
  */
-Runner.prototype._addEventListener = function(target, eventName, listener) {
+Runner.prototype._addEventListener = function (target, eventName, listener) {
   debug(
     '_addEventListener(): adding for event %s; %d current listeners',
     eventName,
@@ -235,10 +235,7 @@ Runner.prototype._addEventListener = function(target, eventName, listener) {
   if (
     this._eventListeners.has(target) &&
     this._eventListeners.get(target).has(eventName) &&
-    this._eventListeners
-      .get(target)
-      .get(eventName)
-      .has(listener)
+    this._eventListeners.get(target).get(eventName).has(listener)
   ) {
     debug(
       'warning: tried to attach duplicate event listener for %s',
@@ -265,7 +262,7 @@ Runner.prototype._addEventListener = function(target, eventName, listener) {
  * @param {function} listener - Listener function
  * @private
  */
-Runner.prototype._removeEventListener = function(target, eventName, listener) {
+Runner.prototype._removeEventListener = function (target, eventName, listener) {
   target.removeListener(eventName, listener);
 
   if (this._eventListeners.has(target)) {
@@ -289,7 +286,7 @@ Runner.prototype._removeEventListener = function(target, eventName, listener) {
  * Removes all event handlers set during a run on this instance.
  * Remark: this does _not_ clean/dispose the tests or suites themselves.
  */
-Runner.prototype.dispose = function() {
+Runner.prototype.dispose = function () {
   this.removeAllListeners();
   this._eventListeners.forEach((targetListeners, target) => {
     targetListeners.forEach((targetEventListeners, eventName) => {
@@ -311,7 +308,7 @@ Runner.prototype.dispose = function() {
  * @param {boolean} invert
  * @return {Runner} Runner instance.
  */
-Runner.prototype.grep = function(re, invert) {
+Runner.prototype.grep = function (re, invert) {
   debug('grep(): setting to %s', re);
   this._grep = re;
   this._invert = invert;
@@ -328,11 +325,11 @@ Runner.prototype.grep = function(re, invert) {
  * @param {Suite} suite
  * @return {number}
  */
-Runner.prototype.grepTotal = function(suite) {
+Runner.prototype.grepTotal = function (suite) {
   var self = this;
   var total = 0;
 
-  suite.eachTest(function(test) {
+  suite.eachTest(function (test) {
     var match = self._grep.test(test.fullTitle());
     if (self._invert) {
       match = !match;
@@ -351,7 +348,7 @@ Runner.prototype.grepTotal = function(suite) {
  * @return {Array}
  * @private
  */
-Runner.prototype.globalProps = function() {
+Runner.prototype.globalProps = function () {
   var props = Object.keys(global);
 
   // non-enumerables
@@ -373,7 +370,7 @@ Runner.prototype.globalProps = function() {
  * @param {Array} arr
  * @return {Runner} Runner instance.
  */
-Runner.prototype.globals = function(arr) {
+Runner.prototype.globals = function (arr) {
   if (!arguments.length) {
     return this._globals;
   }
@@ -387,7 +384,7 @@ Runner.prototype.globals = function(arr) {
  *
  * @private
  */
-Runner.prototype.checkGlobals = function(test) {
+Runner.prototype.checkGlobals = function (test) {
   if (!this.checkLeaks) {
     return;
   }
@@ -435,7 +432,7 @@ Runner.prototype.checkGlobals = function(test) {
  * @param {Error} err
  * @param {boolean} [force=false] - Whether to fail a pending test.
  */
-Runner.prototype.fail = function(test, err, force) {
+Runner.prototype.fail = function (test, err, force) {
   force = force === true;
   if (test.isPending() && !force) {
     return;
@@ -476,7 +473,7 @@ Runner.prototype.fail = function(test, err, force) {
  * @param {Function} fn
  */
 
-Runner.prototype.hook = function(name, fn) {
+Runner.prototype.hook = function (name, fn) {
   if (this._opts.dryRun) return fn();
 
   var suite = this.suite;
@@ -505,7 +502,7 @@ Runner.prototype.hook = function(name, fn) {
     self.emit(constants.EVENT_HOOK_BEGIN, hook);
 
     if (!hook.listeners('error').length) {
-      self._addEventListener(hook, 'error', function(err) {
+      self._addEventListener(hook, 'error', function (err) {
         self.fail(hook, err);
       });
     }
@@ -530,10 +527,10 @@ Runner.prototype.hook = function(name, fn) {
           hook.pending = false; // activates hook for next test
           return fn(new Error('abort hookDown'));
         } else if (name === HOOK_TYPE_BEFORE_ALL) {
-          suite.tests.forEach(function(test) {
+          suite.tests.forEach(function (test) {
             test.pending = true;
           });
-          suite.suites.forEach(function(suite) {
+          suite.suites.forEach(function (suite) {
             suite.pending = true;
           });
           hooks = [];
@@ -570,7 +567,7 @@ Runner.prototype.hook = function(name, fn) {
     }
   }
 
-  Runner.immediately(function() {
+  Runner.immediately(function () {
     next(0);
   });
 };
@@ -584,7 +581,7 @@ Runner.prototype.hook = function(name, fn) {
  * @param {Array} suites
  * @param {Function} fn
  */
-Runner.prototype.hooks = function(name, suites, fn) {
+Runner.prototype.hooks = function (name, suites, fn) {
   var self = this;
   var orig = this.suite;
 
@@ -596,7 +593,7 @@ Runner.prototype.hooks = function(name, suites, fn) {
       return fn();
     }
 
-    self.hook(name, function(err) {
+    self.hook(name, function (err) {
       if (err) {
         var errSuite = self.suite;
         self.suite = orig;
@@ -617,7 +614,7 @@ Runner.prototype.hooks = function(name, suites, fn) {
  * @param {Function} fn
  * @private
  */
-Runner.prototype.hookUp = function(name, fn) {
+Runner.prototype.hookUp = function (name, fn) {
   var suites = [this.suite].concat(this.parents()).reverse();
   this.hooks(name, suites, fn);
 };
@@ -629,7 +626,7 @@ Runner.prototype.hookUp = function(name, fn) {
  * @param {Function} fn
  * @private
  */
-Runner.prototype.hookDown = function(name, fn) {
+Runner.prototype.hookDown = function (name, fn) {
   var suites = [this.suite].concat(this.parents());
   this.hooks(name, suites, fn);
 };
@@ -641,7 +638,7 @@ Runner.prototype.hookDown = function(name, fn) {
  * @return {Array}
  * @private
  */
-Runner.prototype.parents = function() {
+Runner.prototype.parents = function () {
   var suite = this.suite;
   var suites = [];
   while (suite.parent) {
@@ -657,7 +654,7 @@ Runner.prototype.parents = function() {
  * @param {Function} fn
  * @private
  */
-Runner.prototype.runTest = function(fn) {
+Runner.prototype.runTest = function (fn) {
   if (this._opts.dryRun) return fn();
 
   var self = this;
@@ -670,7 +667,7 @@ Runner.prototype.runTest = function(fn) {
   if (this.asyncOnly) {
     test.asyncOnly = true;
   }
-  this._addEventListener(test, 'error', function(err) {
+  this._addEventListener(test, 'error', function (err) {
     self.fail(test, err);
   });
   if (this.allowUncaught) {
@@ -691,7 +688,7 @@ Runner.prototype.runTest = function(fn) {
  * @param {Suite} suite
  * @param {Function} fn
  */
-Runner.prototype.runTests = function(suite, fn) {
+Runner.prototype.runTests = function (suite, fn) {
   var self = this;
   var tests = suite.tests.slice();
   var test;
@@ -705,7 +702,7 @@ Runner.prototype.runTests = function(suite, fn) {
     self.suite = after ? errSuite.parent : errSuite;
 
     if (self.suite) {
-      self.hookUp(HOOK_TYPE_AFTER_EACH, function(err2, errSuite2) {
+      self.hookUp(HOOK_TYPE_AFTER_EACH, function (err2, errSuite2) {
         self.suite = orig;
         // some hooks may fail even now
         if (err2) {
@@ -779,7 +776,7 @@ Runner.prototype.runTests = function(suite, fn) {
 
     // execute test and hook(s)
     self.emit(constants.EVENT_TEST_BEGIN, (self.test = test));
-    self.hookDown(HOOK_TYPE_BEFORE_EACH, function(err, errSuite) {
+    self.hookDown(HOOK_TYPE_BEFORE_EACH, function (err, errSuite) {
       // conditional skip within beforeEach
       if (test.isPending()) {
         if (self.forbidPending) {
@@ -792,7 +789,7 @@ Runner.prototype.runTests = function(suite, fn) {
         // skip inner afterEach hooks below errSuite level
         var origSuite = self.suite;
         self.suite = errSuite || self.suite;
-        return self.hookUp(HOOK_TYPE_AFTER_EACH, function(e, eSuite) {
+        return self.hookUp(HOOK_TYPE_AFTER_EACH, function (e, eSuite) {
           self.suite = origSuite;
           next(e, eSuite);
         });
@@ -801,7 +798,7 @@ Runner.prototype.runTests = function(suite, fn) {
         return hookErr(err, errSuite, false);
       }
       self.currentRunnable = self.test;
-      self.runTest(function(err) {
+      self.runTest(function (err) {
         test = self.test;
         // conditional skip within it
         if (test.pending) {
@@ -852,7 +849,7 @@ Runner.prototype.runTests = function(suite, fn) {
  * @param {Suite} suite
  * @param {Function} fn
  */
-Runner.prototype.runSuite = function(suite, fn) {
+Runner.prototype.runSuite = function (suite, fn) {
   var i = 0;
   var self = this;
   var total = this.grepTotal(suite);
@@ -892,7 +889,7 @@ Runner.prototype.runSuite = function(suite, fn) {
     // huge recursive loop and thus a maximum call stack error.
     // See comment in `this.runTests()` for more information.
     if (self._grep !== self._defaultGrep) {
-      Runner.immediately(function() {
+      Runner.immediately(function () {
         self.runSuite(curr, next);
       });
     } else {
@@ -907,7 +904,7 @@ Runner.prototype.runSuite = function(suite, fn) {
     // remove reference to test
     delete self.test;
 
-    self.hook(HOOK_TYPE_AFTER_ALL, function() {
+    self.hook(HOOK_TYPE_AFTER_ALL, function () {
       self.emit(constants.EVENT_SUITE_END, suite);
       fn(errSuite);
     });
@@ -915,7 +912,7 @@ Runner.prototype.runSuite = function(suite, fn) {
 
   this.nextSuite = next;
 
-  this.hook(HOOK_TYPE_BEFORE_ALL, function(err) {
+  this.hook(HOOK_TYPE_BEFORE_ALL, function (err) {
     if (err) {
       return done();
     }
@@ -939,7 +936,7 @@ Runner.prototype.runSuite = function(suite, fn) {
  * @param {Error} err - Some uncaught error
  * @private
  */
-Runner.prototype._uncaught = function(err) {
+Runner.prototype._uncaught = function (err) {
   // this is defensive to prevent future developers from mis-calling this function.
   // it's more likely that it'd be called with the incorrect context--say, the global
   // `process` object--than it would to be called with a context that is not a "subclass"
@@ -1037,12 +1034,12 @@ Runner.prototype._uncaught = function(err) {
  * @param {{files: string[], options: Options}} [opts] - For subclasses
  * @returns {Runner} Runner instance.
  */
-Runner.prototype.run = function(fn, opts = {}) {
+Runner.prototype.run = function (fn, opts = {}) {
   var rootSuite = this.suite;
   var options = opts.options || {};
 
   debug('run(): got options: %O', options);
-  fn = fn || function() {};
+  fn = fn || function () {};
 
   const end = () => {
     if (!this.total && this._opts.failZero) this.failures = 1;
@@ -1083,7 +1080,7 @@ Runner.prototype.run = function(fn, opts = {}) {
   }
 
   // callback
-  this.on(constants.EVENT_RUN_END, function() {
+  this.on(constants.EVENT_RUN_END, function () {
     this.state = constants.STATE_STOPPED;
     debug('run(): emitted %s', constants.EVENT_RUN_END);
     fn(this.failures);
@@ -1130,7 +1127,7 @@ Runner.prototype.run = function(fn, opts = {}) {
  *   }
  * }
  */
-Runner.prototype.linkPartialObjects = function(value) {
+Runner.prototype.linkPartialObjects = function (value) {
   return this;
 };
 
@@ -1155,7 +1152,7 @@ Runner.prototype.runAsync = async function runAsync(opts = {}) {
  * @public
  * @return {Runner} Runner instance.
  */
-Runner.prototype.abort = function() {
+Runner.prototype.abort = function () {
   debug('abort(): aborting');
   this._abort = true;
 
@@ -1183,7 +1180,7 @@ Runner.prototype.isParallelMode = function isParallelMode() {
  * @chainable
  * @abstract
  */
-Runner.prototype.workerReporter = function() {
+Runner.prototype.workerReporter = function () {
   throw createUnsupportedError('workerReporter() not supported in serial mode');
 };
 
@@ -1196,7 +1193,7 @@ Runner.prototype.workerReporter = function() {
  * @return {Array}
  */
 function filterLeaks(ok, globals) {
-  return globals.filter(function(key) {
+  return globals.filter(function (key) {
     // Firefox and Chrome exposes iframes as index inside the window object
     if (/^\d+/.test(key)) {
       return false;
@@ -1220,7 +1217,7 @@ function filterLeaks(ok, globals) {
       return false;
     }
 
-    var matched = ok.filter(function(ok) {
+    var matched = ok.filter(function (ok) {
       if (~ok.indexOf('*')) {
         return key.indexOf(ok.split('*')[0]) === 0;
       }

package/lib/stats-collector.js

@@ -56,25 +56,25 @@ function createStatsCollector(runner) {
 
   runner.stats = stats;
 
-  runner.once(EVENT_RUN_BEGIN, function() {
+  runner.once(EVENT_RUN_BEGIN, function () {
     stats.start = new Date();
   });
-  runner.on(EVENT_SUITE_BEGIN, function(suite) {
+  runner.on(EVENT_SUITE_BEGIN, function (suite) {
     suite.root || stats.suites++;
   });
-  runner.on(EVENT_TEST_PASS, function() {
+  runner.on(EVENT_TEST_PASS, function () {
     stats.passes++;
   });
-  runner.on(EVENT_TEST_FAIL, function() {
+  runner.on(EVENT_TEST_FAIL, function () {
     stats.failures++;
   });
-  runner.on(EVENT_TEST_PENDING, function() {
+  runner.on(EVENT_TEST_PENDING, function () {
     stats.pending++;
   });
-  runner.on(EVENT_TEST_END, function() {
+  runner.on(EVENT_TEST_END, function () {
     stats.tests++;
   });
-  runner.once(EVENT_RUN_END, function() {
+  runner.once(EVENT_RUN_END, function () {
     stats.end = new Date();
     stats.duration = stats.end - stats.start;
   });