mocha 9.1.2 → 9.2.1

package/lib/suite.js

@@ -10,7 +10,6 @@ var {
   assignNewMochaID,
   clamp,
   constants: utilsConstants,
-  createMap,
   defineConstants,
   getMochaID,
   inherits,
@@ -36,7 +35,7 @@ exports = module.exports = Suite;
  * @param {string} title - Title
  * @return {Suite}
  */
-Suite.create = function(parent, title) {
+Suite.create = function (parent, title) {
   var suite = new Suite(title, parent.ctx);
   suite.parent = parent;
   title = suite.fullTitle();
@@ -92,16 +91,6 @@ function Suite(title, parentContext, isRoot) {
   });
 
   this.reset();
-
-  this.on('newListener', function(event) {
-    if (deprecatedEvents[event]) {
-      errors.deprecate(
-        'Event "' +
-          event +
-          '" is deprecated.  Please let the Mocha team know about your use case: https://git.io/v6Lwm'
-      );
-    }
-  });
 }
 
 /**
@@ -112,7 +101,7 @@ inherits(Suite, EventEmitter);
 /**
  * Resets the state initially or for a next run.
  */
-Suite.prototype.reset = function() {
+Suite.prototype.reset = function () {
   this.delayed = false;
   function doReset(thingToReset) {
     thingToReset.reset();
@@ -131,7 +120,7 @@ Suite.prototype.reset = function() {
  * @private
  * @return {Suite}
  */
-Suite.prototype.clone = function() {
+Suite.prototype.clone = function () {
   var suite = new Suite(this.title);
   debug('clone');
   suite.ctx = this.ctx;
@@ -151,7 +140,7 @@ Suite.prototype.clone = function() {
  * @param {number|string} ms
  * @return {Suite|number} for chaining
  */
-Suite.prototype.timeout = function(ms) {
+Suite.prototype.timeout = function (ms) {
   if (!arguments.length) {
     return this._timeout;
   }
@@ -176,7 +165,7 @@ Suite.prototype.timeout = function(ms) {
  * @param {number|string} n
  * @return {Suite|number} for chaining
  */
-Suite.prototype.retries = function(n) {
+Suite.prototype.retries = function (n) {
   if (!arguments.length) {
     return this._retries;
   }
@@ -192,7 +181,7 @@ Suite.prototype.retries = function(n) {
  * @param {number|string} ms
  * @return {Suite|number} for chaining
  */
-Suite.prototype.slow = function(ms) {
+Suite.prototype.slow = function (ms) {
   if (!arguments.length) {
     return this._slow;
   }
@@ -211,7 +200,7 @@ Suite.prototype.slow = function(ms) {
  * @param {boolean} bail
  * @return {Suite|number} for chaining
  */
-Suite.prototype.bail = function(bail) {
+Suite.prototype.bail = function (bail) {
   if (!arguments.length) {
     return this._bail;
   }
@@ -225,7 +214,7 @@ Suite.prototype.bail = function(bail) {
  *
  * @private
  */
-Suite.prototype.isPending = function() {
+Suite.prototype.isPending = function () {
   return this.pending || (this.parent && this.parent.isPending());
 };
 
@@ -236,7 +225,7 @@ Suite.prototype.isPending = function() {
  * @param {Function} fn - Hook callback
  * @returns {Hook} A new hook
  */
-Suite.prototype._createHook = function(title, fn) {
+Suite.prototype._createHook = function (title, fn) {
   var hook = new Hook(title, fn);
   hook.parent = this;
   hook.timeout(this.timeout());
@@ -255,7 +244,7 @@ Suite.prototype._createHook = function(title, fn) {
  * @param {Function} fn
  * @return {Suite} for chaining
  */
-Suite.prototype.beforeAll = function(title, fn) {
+Suite.prototype.beforeAll = function (title, fn) {
   if (this.isPending()) {
     return this;
   }
@@ -279,7 +268,7 @@ Suite.prototype.beforeAll = function(title, fn) {
  * @param {Function} fn
  * @return {Suite} for chaining
  */
-Suite.prototype.afterAll = function(title, fn) {
+Suite.prototype.afterAll = function (title, fn) {
   if (this.isPending()) {
     return this;
   }
@@ -303,7 +292,7 @@ Suite.prototype.afterAll = function(title, fn) {
  * @param {Function} fn
  * @return {Suite} for chaining
  */
-Suite.prototype.beforeEach = function(title, fn) {
+Suite.prototype.beforeEach = function (title, fn) {
   if (this.isPending()) {
     return this;
   }
@@ -327,7 +316,7 @@ Suite.prototype.beforeEach = function(title, fn) {
  * @param {Function} fn
  * @return {Suite} for chaining
  */
-Suite.prototype.afterEach = function(title, fn) {
+Suite.prototype.afterEach = function (title, fn) {
   if (this.isPending()) {
     return this;
   }
@@ -350,7 +339,7 @@ Suite.prototype.afterEach = function(title, fn) {
  * @param {Suite} suite
  * @return {Suite} for chaining
  */
-Suite.prototype.addSuite = function(suite) {
+Suite.prototype.addSuite = function (suite) {
   suite.parent = this;
   suite.root = false;
   suite.timeout(this.timeout());
@@ -369,7 +358,7 @@ Suite.prototype.addSuite = function(suite) {
  * @param {Test} test
  * @return {Suite} for chaining
  */
-Suite.prototype.addTest = function(test) {
+Suite.prototype.addTest = function (test) {
   test.parent = this;
   test.timeout(this.timeout());
   test.retries(this.retries());
@@ -388,7 +377,7 @@ Suite.prototype.addTest = function(test) {
  * @public
  * @return {string}
  */
-Suite.prototype.fullTitle = function() {
+Suite.prototype.fullTitle = function () {
   return this.titlePath().join(' ');
 };
 
@@ -400,7 +389,7 @@ Suite.prototype.fullTitle = function() {
  * @public
  * @return {string}
  */
-Suite.prototype.titlePath = function() {
+Suite.prototype.titlePath = function () {
   var result = [];
   if (this.parent) {
     result = result.concat(this.parent.titlePath());
@@ -418,9 +407,9 @@ Suite.prototype.titlePath = function() {
  * @public
  * @return {number}
  */
-Suite.prototype.total = function() {
+Suite.prototype.total = function () {
   return (
-    this.suites.reduce(function(sum, suite) {
+    this.suites.reduce(function (sum, suite) {
       return sum + suite.total();
     }, 0) + this.tests.length
   );
@@ -434,9 +423,9 @@ Suite.prototype.total = function() {
  * @param {Function} fn
  * @return {Suite}
  */
-Suite.prototype.eachTest = function(fn) {
+Suite.prototype.eachTest = function (fn) {
   this.tests.forEach(fn);
-  this.suites.forEach(function(suite) {
+  this.suites.forEach(function (suite) {
     suite.eachTest(fn);
   });
   return this;
@@ -462,7 +451,7 @@ Suite.prototype.hasOnly = function hasOnly() {
   return (
     this._onlyTests.length > 0 ||
     this._onlySuites.length > 0 ||
-    this.suites.some(function(suite) {
+    this.suites.some(function (suite) {
       return suite.hasOnly();
     })
   );
@@ -482,7 +471,7 @@ Suite.prototype.filterOnly = function filterOnly() {
   } else {
     // Otherwise, do not run any of the tests in this suite.
     this.tests = [];
-    this._onlySuites.forEach(function(onlySuite) {
+    this._onlySuites.forEach(function (onlySuite) {
       // If there are other `only` tests/suites nested in the current `only` suite, then filter that `only` suite.
       // Otherwise, all of the tests on this `only` suite should be run, so don't filter it.
       if (onlySuite.hasOnly()) {
@@ -491,7 +480,7 @@ Suite.prototype.filterOnly = function filterOnly() {
     });
     // Run the `only` suites, as well as any other suites that have `only` tests/suites as descendants.
     var onlySuites = this._onlySuites;
-    this.suites = this.suites.filter(function(childSuite) {
+    this.suites = this.suites.filter(function (childSuite) {
       return onlySuites.indexOf(childSuite) !== -1 || childSuite.filterOnly();
     });
   }
@@ -505,7 +494,7 @@ Suite.prototype.filterOnly = function filterOnly() {
  * @private
  * @param {Suite} suite
  */
-Suite.prototype.appendOnlySuite = function(suite) {
+Suite.prototype.appendOnlySuite = function (suite) {
   this._onlySuites.push(suite);
 };
 
@@ -514,7 +503,7 @@ Suite.prototype.appendOnlySuite = function(suite) {
  *
  * @private
  */
-Suite.prototype.markOnly = function() {
+Suite.prototype.markOnly = function () {
   this.parent && this.parent.appendOnlySuite(this);
 };
 
@@ -524,7 +513,7 @@ Suite.prototype.markOnly = function() {
  * @private
  * @param {Test} test
  */
-Suite.prototype.appendOnlyTest = function(test) {
+Suite.prototype.appendOnlyTest = function (test) {
   this._onlyTests.push(test);
 };
 
@@ -539,8 +528,8 @@ Suite.prototype.getHooks = function getHooks(name) {
 /**
  * cleans all references from this suite and all child suites.
  */
-Suite.prototype.dispose = function() {
-  this.suites.forEach(function(suite) {
+Suite.prototype.dispose = function () {
+  this.suites.forEach(function (suite) {
     suite.dispose();
   });
   this.cleanReferences();
@@ -614,7 +603,7 @@ var constants = defineConstants(
    */
   {
     /**
-     * Event emitted after a test file has been loaded Not emitted in browser.
+     * Event emitted after a test file has been loaded. Not emitted in browser.
      */
     EVENT_FILE_POST_REQUIRE: 'post-require',
     /**
@@ -626,70 +615,52 @@ var constants = defineConstants(
      */
     EVENT_FILE_REQUIRE: 'require',
     /**
-     * Event emitted when `global.run()` is called (use with `delay` option)
+     * Event emitted when `global.run()` is called (use with `delay` option).
      */
     EVENT_ROOT_SUITE_RUN: 'run',
 
     /**
-     * Namespace for collection of a `Suite`'s "after all" hooks
+     * Namespace for collection of a `Suite`'s "after all" hooks.
      */
     HOOK_TYPE_AFTER_ALL: 'afterAll',
     /**
-     * Namespace for collection of a `Suite`'s "after each" hooks
+     * Namespace for collection of a `Suite`'s "after each" hooks.
      */
     HOOK_TYPE_AFTER_EACH: 'afterEach',
     /**
-     * Namespace for collection of a `Suite`'s "before all" hooks
+     * Namespace for collection of a `Suite`'s "before all" hooks.
      */
     HOOK_TYPE_BEFORE_ALL: 'beforeAll',
     /**
-     * Namespace for collection of a `Suite`'s "before all" hooks
+     * Namespace for collection of a `Suite`'s "before each" hooks.
      */
     HOOK_TYPE_BEFORE_EACH: 'beforeEach',
 
-    // the following events are all deprecated
-
     /**
-     * Emitted after an "after all" `Hook` has been added to a `Suite`. Deprecated
+     * Emitted after a child `Suite` has been added to a `Suite`.
+     */
+    EVENT_SUITE_ADD_SUITE: 'suite',
+    /**
+     * Emitted after an "after all" `Hook` has been added to a `Suite`.
      */
     EVENT_SUITE_ADD_HOOK_AFTER_ALL: 'afterAll',
     /**
-     * Emitted after an "after each" `Hook` has been added to a `Suite` Deprecated
+     * Emitted after an "after each" `Hook` has been added to a `Suite`.
      */
     EVENT_SUITE_ADD_HOOK_AFTER_EACH: 'afterEach',
     /**
-     * Emitted after an "before all" `Hook` has been added to a `Suite` Deprecated
+     * Emitted after an "before all" `Hook` has been added to a `Suite`.
      */
     EVENT_SUITE_ADD_HOOK_BEFORE_ALL: 'beforeAll',
     /**
-     * Emitted after an "before each" `Hook` has been added to a `Suite` Deprecated
+     * Emitted after an "before each" `Hook` has been added to a `Suite`.
      */
     EVENT_SUITE_ADD_HOOK_BEFORE_EACH: 'beforeEach',
     /**
-     * Emitted after a child `Suite` has been added to a `Suite`. Deprecated
-     */
-    EVENT_SUITE_ADD_SUITE: 'suite',
-    /**
-     * Emitted after a `Test` has been added to a `Suite`. Deprecated
+     * Emitted after a `Test` has been added to a `Suite`.
      */
     EVENT_SUITE_ADD_TEST: 'test'
   }
 );
 
-/**
- * @summary There are no known use cases for these events.
- * @desc This is a `Set`-like object having all keys being the constant's string value and the value being `true`.
- * @todo Remove eventually
- * @type {Object<string,boolean>}
- * @ignore
- */
-var deprecatedEvents = Object.keys(constants)
-  .filter(function(constant) {
-    return constant.substring(0, 15) === 'EVENT_SUITE_ADD';
-  })
-  .reduce(function(acc, constant) {
-    acc[constants[constant]] = true;
-    return acc;
-  }, createMap());
-
 Suite.constants = constants;

package/lib/test.js

@@ -41,7 +41,7 @@ utils.inherits(Test, Runnable);
 /**
  * Resets the state initially or for a next run.
  */
-Test.prototype.reset = function() {
+Test.prototype.reset = function () {
   Runnable.prototype.reset.call(this);
   this.pending = !this.fn;
   delete this.state;
@@ -52,7 +52,7 @@ Test.prototype.reset = function() {
  *
  * @private
  */
-Test.prototype.retriedTest = function(n) {
+Test.prototype.retriedTest = function (n) {
   if (!arguments.length) {
     return this._retriedTest;
   }
@@ -64,11 +64,11 @@ Test.prototype.retriedTest = function(n) {
  *
  * @private
  */
-Test.prototype.markOnly = function() {
+Test.prototype.markOnly = function () {
   this.parent.appendOnlyTest(this);
 };
 
-Test.prototype.clone = function() {
+Test.prototype.clone = function () {
   var test = new Test(this.title, this.fn);
   test.timeout(this.timeout());
   test.slow(this.slow());

package/lib/utils.js

@@ -34,7 +34,7 @@ exports.inherits = util.inherits;
  * @param  {string} html
  * @return {string}
  */
-exports.escape = function(html) {
+exports.escape = function (html) {
   return he.encode(String(html), {useNamedReferences: false});
 };
 
@@ -45,7 +45,7 @@ exports.escape = function(html) {
  * @param {Object} obj
  * @return {boolean}
  */
-exports.isString = function(obj) {
+exports.isString = function (obj) {
   return typeof obj === 'string';
 };
 
@@ -56,7 +56,7 @@ exports.isString = function(obj) {
  * @param {string} str
  * @return {string}
  */
-exports.slug = function(str) {
+exports.slug = function (str) {
   return str
     .toLowerCase()
     .replace(/\s+/g, '-')
@@ -70,7 +70,7 @@ exports.slug = function(str) {
  * @param {string} str
  * @return {string}
  */
-exports.clean = function(str) {
+exports.clean = function (str) {
   str = str
     .replace(/\r\n?|[\n\u2028\u2029]/g, '\n')
     .replace(/^\uFEFF/, '')
@@ -212,7 +212,7 @@ exports.type = function type(value) {
  * @param {*} value
  * @return {string}
  */
-exports.stringify = function(value) {
+exports.stringify = function (value) {
   var typeHint = canonicalType(value);
 
   if (!~['object', 'array', 'function'].indexOf(typeHint)) {
@@ -228,7 +228,7 @@ exports.stringify = function(value) {
     // IE7/IE8 has a bizarre String constructor; needs to be coerced
     // into an array and back to obj.
     if (typeHint === 'string' && typeof value === 'object') {
-      value = value.split('').reduce(function(acc, char, idx) {
+      value = value.split('').reduce(function (acc, char, idx) {
         acc[idx] = char;
         return acc;
       }, {});
@@ -383,8 +383,8 @@ exports.canonicalize = function canonicalize(value, stack, typeHint) {
       canonicalizedObj = value;
       break;
     case 'array':
-      withStack(value, function() {
-        canonicalizedObj = value.map(function(item) {
+      withStack(value, function () {
+        canonicalizedObj = value.map(function (item) {
           return exports.canonicalize(item, stack);
         });
       });
@@ -403,10 +403,10 @@ exports.canonicalize = function canonicalize(value, stack, typeHint) {
     /* falls through */
     case 'object':
       canonicalizedObj = canonicalizedObj || {};
-      withStack(value, function() {
+      withStack(value, function () {
         Object.keys(value)
           .sort()
-          .forEach(function(key) {
+          .forEach(function (key) {
             canonicalizedObj[key] = exports.canonicalize(value[key], stack);
           });
       });
@@ -434,7 +434,7 @@ exports.canonicalize = function canonicalize(value, stack, typeHint) {
  * (i.e: strip Mocha and internal node functions from stack trace).
  * @returns {Function}
  */
-exports.stackTraceFilter = function() {
+exports.stackTraceFilter = function () {
   // TODO: Replace with `process.browser`
   var is = typeof document === 'undefined' ? {node: true} : {browser: true};
   var slash = path.sep;
@@ -442,9 +442,8 @@ exports.stackTraceFilter = function() {
   if (is.node) {
     cwd = exports.cwd() + slash;
   } else {
-    cwd = (typeof location === 'undefined'
-      ? window.location
-      : location
+    cwd = (
+      typeof location === 'undefined' ? window.location : location
     ).href.replace(/\/[^/]*$/, '/');
     slash = '/';
   }
@@ -468,10 +467,10 @@ exports.stackTraceFilter = function() {
     );
   }
 
-  return function(stack) {
+  return function (stack) {
     stack = stack.split('\n');
 
-    stack = stack.reduce(function(list, line) {
+    stack = stack.reduce(function (list, line) {
       if (isMochaInternal(line)) {
         return list;
       }
@@ -522,7 +521,7 @@ exports.clamp = function clamp(value, range) {
  * It's a noop.
  * @public
  */
-exports.noop = function() {};
+exports.noop = function () {};
 
 /**
  * Creates a map-like object.
@@ -539,7 +538,7 @@ exports.noop = function() {};
  * @param {...*} [obj] - Arguments to `Object.assign()`.
  * @returns {Object} An object with no prototype, having `...obj` properties
  */
-exports.createMap = function(obj) {
+exports.createMap = function (obj) {
   return Object.assign.apply(
     null,
     [Object.create(null)].concat(Array.prototype.slice.call(arguments))
@@ -558,7 +557,7 @@ exports.createMap = function(obj) {
  * @returns {Object} A frozen object with no prototype, having `...obj` properties
  * @throws {TypeError} if argument is not a non-empty object.
  */
-exports.defineConstants = function(obj) {
+exports.defineConstants = function (obj) {
   if (canonicalType(obj) !== 'object' || !Object.keys(obj).length) {
     throw new TypeError('Invalid argument; expected a non-empty object');
   }