mocha 10.8.1 → 11.0.0-beta

package/lib/cli/lookup-files.js

@@ -87,10 +87,15 @@ module.exports = function lookupFiles(
       debug('looking for files using glob pattern: %s', pattern);
     }
     files.push(
-      ...glob.sync(pattern, {
-        nodir: true,
-        windowsPathsNoEscape: true
-      })
+      ...glob
+        .sync(pattern, {
+          nodir: true,
+          windowsPathsNoEscape: true
+        })
+        // glob@8 and earlier sorted results in en; glob@9 depends on OS sorting.
+        // This preserves the older glob behavior.
+        // https://github.com/mochajs/mocha/pull/5250/files#r1840469747
+        .sort((a, b) => a.localeCompare(b, 'en'))
     );
     if (!files.length) {
       throw createNoFilesMatchPatternError(

package/lib/interfaces/common.js

@@ -57,7 +57,7 @@ module.exports = function (suites, context, mocha) {
      * @param {Function} fn
      */
     before: function (name, fn) {
-      suites[0].beforeAll(name, fn);
+      return suites[0].beforeAll(name, fn);
     },
 
     /**
@@ -67,7 +67,7 @@ module.exports = function (suites, context, mocha) {
      * @param {Function} fn
      */
     after: function (name, fn) {
-      suites[0].afterAll(name, fn);
+      return suites[0].afterAll(name, fn);
     },
 
     /**
@@ -77,7 +77,7 @@ module.exports = function (suites, context, mocha) {
      * @param {Function} fn
      */
     beforeEach: function (name, fn) {
-      suites[0].beforeEach(name, fn);
+      return suites[0].beforeEach(name, fn);
     },
 
     /**
@@ -87,7 +87,7 @@ module.exports = function (suites, context, mocha) {
      * @param {Function} fn
      */
     afterEach: function (name, fn) {
-      suites[0].afterEach(name, fn);
+      return suites[0].afterEach(name, fn);
     },
 
     suite: {

package/lib/nodejs/parallel-buffered-runner.js

@@ -374,16 +374,16 @@ class ParallelBufferedRunner extends Runner {
    * // this reporter needs proper object references when run in parallel mode
    * class MyReporter() {
    *   constructor(runner) {
-   *     this.runner.linkPartialObjects(true)
+   *     runner.linkPartialObjects(true)
    *       .on(EVENT_SUITE_BEGIN, suite => {
-             // this Suite may be the same object...
-  *       })
-  *       .on(EVENT_TEST_BEGIN, test => {
-  *         // ...as the `test.parent` property
-  *       });
-  *   }
-  * }
-  */
+   *         // this Suite may be the same object...
+   *       })
+   *       .on(EVENT_TEST_BEGIN, test => {
+   *         // ...as the `test.parent` property
+   *       });
+   *   }
+   * }
+   */
   linkPartialObjects(value) {
     this._linkPartialObjects = Boolean(value);
     return super.linkPartialObjects(value);

package/lib/nodejs/serializer.js

@@ -262,9 +262,15 @@ class SerializableEvent {
     breakCircularDeps(result.error);
 
     const pairs = Object.keys(result).map(key => [result, key]);
-
+    const seenPairs = new Set();
     let pair;
+
     while ((pair = pairs.shift())) {
+      if (seenPairs.has(pair[1])) {
+        continue;
+      }
+
+      seenPairs.add(pair[1]);
       SerializableEvent._serialize(pairs, ...pair);
     }
 

package/lib/reporters/html.js

@@ -285,9 +285,9 @@ function HTML(runner, options) {
 function makeUrl(s) {
   var search = window.location.search;
 
-  // Remove previous grep query parameter if present
+  // Remove previous {grep, fgrep, invert} query parameters if present
   if (search) {
-    search = search.replace(/[?&]grep=[^&\s]*/g, '').replace(/^&/, '?');
+    search = search.replace(/[?&](?:f?grep|invert)=[^&\s]*/g, '').replace(/^&/, '?');
   }
 
   return (

package/lib/runner.js

@@ -1116,11 +1116,11 @@ Runner.prototype.run = function (fn, opts = {}) {
  * @public
  * @example
  * // this reporter needs proper object references when run in parallel mode
- * class MyReporter() {
+ * class MyReporter {
  *   constructor(runner) {
- *     this.runner.linkPartialObjects(true)
+ *     runner.linkPartialObjects(true)
  *       .on(EVENT_SUITE_BEGIN, suite => {
-           // this Suite may be the same object...
+ *         // this Suite may be the same object...
  *       })
  *       .on(EVENT_TEST_BEGIN, test => {
  *         // ...as the `test.parent` property

package/lib/suite.js

@@ -257,7 +257,7 @@ Suite.prototype.beforeAll = function (title, fn) {
   var hook = this._createHook(title, fn);
   this._beforeAll.push(hook);
   this.emit(constants.EVENT_SUITE_ADD_HOOK_BEFORE_ALL, hook);
-  return this;
+  return hook;
 };
 
 /**
@@ -281,7 +281,7 @@ Suite.prototype.afterAll = function (title, fn) {
   var hook = this._createHook(title, fn);
   this._afterAll.push(hook);
   this.emit(constants.EVENT_SUITE_ADD_HOOK_AFTER_ALL, hook);
-  return this;
+  return hook;
 };
 
 /**
@@ -305,7 +305,7 @@ Suite.prototype.beforeEach = function (title, fn) {
   var hook = this._createHook(title, fn);
   this._beforeEach.push(hook);
   this.emit(constants.EVENT_SUITE_ADD_HOOK_BEFORE_EACH, hook);
-  return this;
+  return hook;
 };
 
 /**
@@ -329,7 +329,7 @@ Suite.prototype.afterEach = function (title, fn) {
   var hook = this._createHook(title, fn);
   this._afterEach.push(hook);
   this.emit(constants.EVENT_SUITE_ADD_HOOK_AFTER_EACH, hook);
-  return this;
+  return hook;
 };
 
 /**

package/lib/utils.js

@@ -675,7 +675,9 @@ exports.breakCircularDeps = inputObj => {
 
     seen.add(obj);
     for (const k in obj) {
-      if (Object.prototype.hasOwnProperty.call(obj, k)) {
+      const descriptor = Object.getOwnPropertyDescriptor(obj, k);
+
+      if (descriptor && descriptor.writable) {
         obj[k] = _breakCircularDeps(obj[k], k);
       }
     }

package/mocha.js

@@ -1,4 +1,4 @@
-// mocha@10.8.1 in javascript ES2018
+// mocha@11.0.0-beta in javascript ES2018
 (function (global, factory) {
   typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
   typeof define === 'function' && define.amd ? define(factory) :
@@ -11638,7 +11638,9 @@
 
       seen.add(obj);
       for (const k in obj) {
-        if (Object.prototype.hasOwnProperty.call(obj, k)) {
+        const descriptor = Object.getOwnPropertyDescriptor(obj, k);
+
+        if (descriptor && descriptor.writable) {
           obj[k] = _breakCircularDeps(obj[k]);
         }
       }
@@ -13769,7 +13771,7 @@
     var hook = this._createHook(title, fn);
     this._beforeAll.push(hook);
     this.emit(constants.EVENT_SUITE_ADD_HOOK_BEFORE_ALL, hook);
-    return this;
+    return hook;
   };
 
   /**
@@ -13793,7 +13795,7 @@
     var hook = this._createHook(title, fn);
     this._afterAll.push(hook);
     this.emit(constants.EVENT_SUITE_ADD_HOOK_AFTER_ALL, hook);
-    return this;
+    return hook;
   };
 
   /**
@@ -13817,7 +13819,7 @@
     var hook = this._createHook(title, fn);
     this._beforeEach.push(hook);
     this.emit(constants.EVENT_SUITE_ADD_HOOK_BEFORE_EACH, hook);
-    return this;
+    return hook;
   };
 
   /**
@@ -13841,7 +13843,7 @@
     var hook = this._createHook(title, fn);
     this._afterEach.push(hook);
     this.emit(constants.EVENT_SUITE_ADD_HOOK_AFTER_EACH, hook);
-    return this;
+    return hook;
   };
 
   /**
@@ -15294,11 +15296,11 @@
    * @public
    * @example
    * // this reporter needs proper object references when run in parallel mode
-   * class MyReporter() {
+   * class MyReporter {
    *   constructor(runner) {
-   *     this.runner.linkPartialObjects(true)
+   *     runner.linkPartialObjects(true)
    *       .on(EVENT_SUITE_BEGIN, suite => {
-             // this Suite may be the same object...
+   *         // this Suite may be the same object...
    *       })
    *       .on(EVENT_TEST_BEGIN, test => {
    *         // ...as the `test.parent` property
@@ -16981,9 +16983,9 @@
   function makeUrl(s) {
     var search = window.location.search;
 
-    // Remove previous grep query parameter if present
+    // Remove previous {grep, fgrep, invert} query parameters if present
     if (search) {
-      search = search.replace(/[?&]grep=[^&\s]*/g, '').replace(/^&/, '?');
+      search = search.replace(/[?&](?:f?grep|invert)=[^&\s]*/g, '').replace(/^&/, '?');
     }
 
     return (
@@ -18591,7 +18593,7 @@
        * @param {Function} fn
        */
       before: function (name, fn) {
-        suites[0].beforeAll(name, fn);
+        return suites[0].beforeAll(name, fn);
       },
 
       /**
@@ -18601,7 +18603,7 @@
        * @param {Function} fn
        */
       after: function (name, fn) {
-        suites[0].afterAll(name, fn);
+        return suites[0].afterAll(name, fn);
       },
 
       /**
@@ -18611,7 +18613,7 @@
        * @param {Function} fn
        */
       beforeEach: function (name, fn) {
-        suites[0].beforeEach(name, fn);
+        return suites[0].beforeEach(name, fn);
       },
 
       /**
@@ -18621,7 +18623,7 @@
        * @param {Function} fn
        */
       afterEach: function (name, fn) {
-        suites[0].afterEach(name, fn);
+        return suites[0].afterEach(name, fn);
       },
 
       suite: {
@@ -19199,7 +19201,7 @@
   };
 
   var name = "mocha";
-  var version = "10.8.1";
+  var version = "11.0.0-beta";
   var homepage = "https://mochajs.org/";
   var notifyLogo = "https://ibin.co/4QuRuGjXvl36.png";
   var require$$17 = {