mocha 9.1.1 → 9.2.0

package/lib/mocha.js

@@ -13,9 +13,7 @@ var growl = require('./nodejs/growl');
 var utils = require('./utils');
 var mocharc = require('./mocharc.json');
 var Suite = require('./suite');
-var esmUtils = utils.supportsEsModules(true)
-  ? require('./esm-utils')
-  : undefined;
+var esmUtils = require('./nodejs/esm-utils');
 var createStatsCollector = require('./stats-collector');
 const {
   warn,
@@ -25,11 +23,8 @@ const {
   createMochaInstanceAlreadyRunningError,
   createUnsupportedError
 } = require('./errors');
-const {
-  EVENT_FILE_PRE_REQUIRE,
-  EVENT_FILE_POST_REQUIRE,
-  EVENT_FILE_REQUIRE
-} = Suite.constants;
+const {EVENT_FILE_PRE_REQUIRE, EVENT_FILE_POST_REQUIRE, EVENT_FILE_REQUIRE} =
+  Suite.constants;
 var debug = require('debug')('mocha:mocha');
 
 exports = module.exports = Mocha;
@@ -96,46 +91,46 @@ exports.Hook = require('./hook');
 exports.Test = require('./test');
 
 let currentContext;
-exports.afterEach = function(...args) {
+exports.afterEach = function (...args) {
   return (currentContext.afterEach || currentContext.teardown).apply(
     this,
     args
   );
 };
-exports.after = function(...args) {
+exports.after = function (...args) {
   return (currentContext.after || currentContext.suiteTeardown).apply(
     this,
     args
   );
 };
-exports.beforeEach = function(...args) {
+exports.beforeEach = function (...args) {
   return (currentContext.beforeEach || currentContext.setup).apply(this, args);
 };
-exports.before = function(...args) {
+exports.before = function (...args) {
   return (currentContext.before || currentContext.suiteSetup).apply(this, args);
 };
-exports.describe = function(...args) {
+exports.describe = function (...args) {
   return (currentContext.describe || currentContext.suite).apply(this, args);
 };
-exports.describe.only = function(...args) {
+exports.describe.only = function (...args) {
   return (currentContext.describe || currentContext.suite).only.apply(
     this,
     args
   );
 };
-exports.describe.skip = function(...args) {
+exports.describe.skip = function (...args) {
   return (currentContext.describe || currentContext.suite).skip.apply(
     this,
     args
   );
 };
-exports.it = function(...args) {
+exports.it = function (...args) {
   return (currentContext.it || currentContext.test).apply(this, args);
 };
-exports.it.only = function(...args) {
+exports.it.only = function (...args) {
   return (currentContext.it || currentContext.test).only.apply(this, args);
 };
-exports.it.skip = function(...args) {
+exports.it.skip = function (...args) {
   return (currentContext.it || currentContext.test).skip.apply(this, args);
 };
 exports.xdescribe = exports.describe.skip;
@@ -146,7 +141,7 @@ exports.suiteTeardown = exports.after;
 exports.suite = exports.describe;
 exports.teardown = exports.afterEach;
 exports.test = exports.it;
-exports.run = function(...args) {
+exports.run = function (...args) {
   return currentContext.run.apply(this, args);
 };
 
@@ -231,7 +226,7 @@ function Mocha(options = {}) {
     'growl',
     'inlineDiffs',
     'invert'
-  ].forEach(function(opt) {
+  ].forEach(function (opt) {
     if (options[opt]) {
       this[opt]();
     }
@@ -289,7 +284,7 @@ function Mocha(options = {}) {
  * @returns {Mocha} this
  * @chainable
  */
-Mocha.prototype.bail = function(bail) {
+Mocha.prototype.bail = function (bail) {
   this.suite.bail(bail !== false);
   return this;
 };
@@ -307,7 +302,7 @@ Mocha.prototype.bail = function(bail) {
  * @returns {Mocha} this
  * @chainable
  */
-Mocha.prototype.addFile = function(file) {
+Mocha.prototype.addFile = function (file) {
   this.files.push(file);
   return this;
 };
@@ -328,7 +323,7 @@ Mocha.prototype.addFile = function(file) {
  * // Use XUnit reporter and direct its output to file
  * mocha.reporter('xunit', { output: '/path/to/testspec.xunit.xml' });
  */
-Mocha.prototype.reporter = function(reporterName, reporterOptions) {
+Mocha.prototype.reporter = function (reporterName, reporterOptions) {
   if (typeof reporterName === 'function') {
     this._reporter = reporterName;
   } else {
@@ -384,7 +379,7 @@ Mocha.prototype.reporter = function(reporterName, reporterOptions) {
  * @chainable
  * @throws {Error} if requested interface cannot be loaded
  */
-Mocha.prototype.ui = function(ui) {
+Mocha.prototype.ui = function (ui) {
   var bindInterface;
   if (typeof ui === 'function') {
     bindInterface = ui;
@@ -401,7 +396,7 @@ Mocha.prototype.ui = function(ui) {
   }
   bindInterface(this.suite);
 
-  this.suite.on(EVENT_FILE_PRE_REQUIRE, function(context) {
+  this.suite.on(EVENT_FILE_PRE_REQUIRE, function (context) {
     currentContext = context;
   });
 
@@ -423,10 +418,10 @@ Mocha.prototype.ui = function(ui) {
  * @see {@link Mocha#loadFilesAsync}
  * @param {Function} [fn] - Callback invoked upon completion.
  */
-Mocha.prototype.loadFiles = function(fn) {
+Mocha.prototype.loadFiles = function (fn) {
   var self = this;
   var suite = this.suite;
-  this.files.forEach(function(file) {
+  this.files.forEach(function (file) {
     file = path.resolve(file);
     suite.emit(EVENT_FILE_PRE_REQUIRE, global, file, self);
     suite.emit(EVENT_FILE_REQUIRE, require(file), file, self);
@@ -455,23 +450,17 @@ Mocha.prototype.loadFiles = function(fn) {
  *   .then(() => mocha.run(failures => process.exitCode = failures ? 1 : 0))
  *   .catch(() => process.exitCode = 1);
  */
-Mocha.prototype.loadFilesAsync = function() {
+Mocha.prototype.loadFilesAsync = function () {
   var self = this;
   var suite = this.suite;
   this.lazyLoadFiles(true);
 
-  if (!esmUtils) {
-    return new Promise(function(resolve) {
-      self.loadFiles(resolve);
-    });
-  }
-
   return esmUtils.loadFilesAsync(
     this.files,
-    function(file) {
+    function (file) {
       suite.emit(EVENT_FILE_PRE_REQUIRE, global, file, self);
     },
-    function(file, resultModule) {
+    function (file, resultModule) {
       suite.emit(EVENT_FILE_REQUIRE, resultModule, file, self);
       suite.emit(EVENT_FILE_POST_REQUIRE, global, file, self);
     }
@@ -486,7 +475,7 @@ Mocha.prototype.loadFilesAsync = function() {
  * @see {@link Mocha#unloadFiles}
  * @param {string} file - Pathname of file to be unloaded.
  */
-Mocha.unloadFile = function(file) {
+Mocha.unloadFile = function (file) {
   if (utils.isBrowser()) {
     throw createUnsupportedError(
       'unloadFile() is only suported in a Node.js environment'
@@ -510,7 +499,7 @@ Mocha.unloadFile = function(file) {
  * @returns {Mocha} this
  * @chainable
  */
-Mocha.prototype.unloadFiles = function() {
+Mocha.prototype.unloadFiles = function () {
   if (this._state === mochaStates.DISPOSED) {
     throw createMochaInstanceAlreadyDisposedError(
       'Mocha instance is already disposed, it cannot be used again.',
@@ -519,7 +508,7 @@ Mocha.prototype.unloadFiles = function() {
     );
   }
 
-  this.files.forEach(function(file) {
+  this.files.forEach(function (file) {
     Mocha.unloadFile(file);
   });
   this._state = mochaStates.INIT;
@@ -539,7 +528,7 @@ Mocha.prototype.unloadFiles = function() {
  * // Select tests whose full title begins with `"foo"` followed by a period
  * mocha.fgrep('foo.');
  */
-Mocha.prototype.fgrep = function(str) {
+Mocha.prototype.fgrep = function (str) {
   if (!str) {
     return this;
   }
@@ -580,7 +569,7 @@ Mocha.prototype.fgrep = function(str) {
  * // Given embedded test `it('only-this-test')`...
  * mocha.grep('/^only-this-test$/');    // NO! Use `.only()` to do this!
  */
-Mocha.prototype.grep = function(re) {
+Mocha.prototype.grep = function (re) {
   if (utils.isString(re)) {
     // extract args if it's regex-like, i.e: [string, pattern, flag]
     var arg = re.match(/^\/(.*)\/([gimy]{0,4})$|.*/);
@@ -603,7 +592,7 @@ Mocha.prototype.grep = function(re) {
  * // Select tests whose full title does *not* contain `"match"`, ignoring case
  * mocha.grep(/match/i).invert();
  */
-Mocha.prototype.invert = function() {
+Mocha.prototype.invert = function () {
   this.options.invert = true;
   return this;
 };
@@ -617,7 +606,7 @@ Mocha.prototype.invert = function() {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.checkLeaks = function(checkLeaks) {
+Mocha.prototype.checkLeaks = function (checkLeaks) {
   this.options.checkLeaks = checkLeaks !== false;
   return this;
 };
@@ -632,7 +621,7 @@ Mocha.prototype.checkLeaks = function(checkLeaks) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.cleanReferencesAfterRun = function(cleanReferencesAfterRun) {
+Mocha.prototype.cleanReferencesAfterRun = function (cleanReferencesAfterRun) {
   this._cleanReferencesAfterRun = cleanReferencesAfterRun !== false;
   return this;
 };
@@ -642,7 +631,7 @@ Mocha.prototype.cleanReferencesAfterRun = function(cleanReferencesAfterRun) {
  * It also removes function references to tests functions and hooks, so variables trapped in closures can be cleaned by the garbage collector.
  * @public
  */
-Mocha.prototype.dispose = function() {
+Mocha.prototype.dispose = function () {
   if (this._state === mochaStates.RUNNING) {
     throw createMochaInstanceAlreadyRunningError(
       'Cannot dispose while the mocha instance is still running tests.'
@@ -663,7 +652,7 @@ Mocha.prototype.dispose = function() {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.fullTrace = function(fullTrace) {
+Mocha.prototype.fullTrace = function (fullTrace) {
   this.options.fullTrace = fullTrace !== false;
   return this;
 };
@@ -676,7 +665,7 @@ Mocha.prototype.fullTrace = function(fullTrace) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.growl = function() {
+Mocha.prototype.growl = function () {
   this.options.growl = this.isGrowlCapable();
   if (!this.options.growl) {
     var detail = utils.isBrowser()
@@ -725,11 +714,11 @@ Mocha.prototype._growl = growl.notify;
  * // Specify variables to be expected in global scope
  * mocha.global(['jQuery', 'MyLib']);
  */
-Mocha.prototype.global = function(global) {
+Mocha.prototype.global = function (global) {
   this.options.global = (this.options.global || [])
     .concat(global)
     .filter(Boolean)
-    .filter(function(elt, idx, arr) {
+    .filter(function (elt, idx, arr) {
       return arr.indexOf(elt) === idx;
     });
   return this;
@@ -746,7 +735,7 @@ Mocha.prototype.globals = Mocha.prototype.global;
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.color = function(color) {
+Mocha.prototype.color = function (color) {
   this.options.color = color !== false;
   return this;
 };
@@ -761,7 +750,7 @@ Mocha.prototype.color = function(color) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.inlineDiffs = function(inlineDiffs) {
+Mocha.prototype.inlineDiffs = function (inlineDiffs) {
   this.options.inlineDiffs = inlineDiffs !== false;
   return this;
 };
@@ -775,7 +764,7 @@ Mocha.prototype.inlineDiffs = function(inlineDiffs) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.diff = function(diff) {
+Mocha.prototype.diff = function (diff) {
   this.options.diff = diff !== false;
   return this;
 };
@@ -803,7 +792,7 @@ Mocha.prototype.diff = function(diff) {
  * // Same as above but using string argument
  * mocha.timeout('1s');
  */
-Mocha.prototype.timeout = function(msecs) {
+Mocha.prototype.timeout = function (msecs) {
   this.suite.timeout(msecs);
   return this;
 };
@@ -822,7 +811,7 @@ Mocha.prototype.timeout = function(msecs) {
  * // Allow any failed test to retry one more time
  * mocha.retries(1);
  */
-Mocha.prototype.retries = function(retry) {
+Mocha.prototype.retries = function (retry) {
   this.suite.retries(retry);
   return this;
 };
@@ -844,7 +833,7 @@ Mocha.prototype.retries = function(retry) {
  * // Same as above but using string argument
  * mocha.slow('0.5s');
  */
-Mocha.prototype.slow = function(msecs) {
+Mocha.prototype.slow = function (msecs) {
   this.suite.slow(msecs);
   return this;
 };
@@ -858,7 +847,7 @@ Mocha.prototype.slow = function(msecs) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.asyncOnly = function(asyncOnly) {
+Mocha.prototype.asyncOnly = function (asyncOnly) {
   this.options.asyncOnly = asyncOnly !== false;
   return this;
 };
@@ -870,7 +859,7 @@ Mocha.prototype.asyncOnly = function(asyncOnly) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.noHighlighting = function() {
+Mocha.prototype.noHighlighting = function () {
   this.options.noHighlighting = true;
   return this;
 };
@@ -884,7 +873,7 @@ Mocha.prototype.noHighlighting = function() {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.allowUncaught = function(allowUncaught) {
+Mocha.prototype.allowUncaught = function (allowUncaught) {
   this.options.allowUncaught = allowUncaught !== false;
   return this;
 };
@@ -915,7 +904,7 @@ Mocha.prototype.delay = function delay() {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.dryRun = function(dryRun) {
+Mocha.prototype.dryRun = function (dryRun) {
   this.options.dryRun = dryRun !== false;
   return this;
 };
@@ -929,7 +918,7 @@ Mocha.prototype.dryRun = function(dryRun) {
  * @return {Mocha} this
  * @chainable
  */
-Mocha.prototype.failZero = function(failZero) {
+Mocha.prototype.failZero = function (failZero) {
   this.options.failZero = failZero !== false;
   return this;
 };
@@ -943,7 +932,7 @@ Mocha.prototype.failZero = function(failZero) {
  * @returns {Mocha} this
  * @chainable
  */
-Mocha.prototype.forbidOnly = function(forbidOnly) {
+Mocha.prototype.forbidOnly = function (forbidOnly) {
   this.options.forbidOnly = forbidOnly !== false;
   return this;
 };
@@ -957,7 +946,7 @@ Mocha.prototype.forbidOnly = function(forbidOnly) {
  * @returns {Mocha} this
  * @chainable
  */
-Mocha.prototype.forbidPending = function(forbidPending) {
+Mocha.prototype.forbidPending = function (forbidPending) {
   this.options.forbidPending = forbidPending !== false;
   return this;
 };
@@ -966,7 +955,7 @@ Mocha.prototype.forbidPending = function(forbidPending) {
  * Throws an error if mocha is in the wrong state to be able to transition to a "running" state.
  * @private
  */
-Mocha.prototype._guardRunningStateTransition = function() {
+Mocha.prototype._guardRunningStateTransition = function () {
   if (this._state === mochaStates.RUNNING) {
     throw createMochaInstanceAlreadyRunningError(
       'Mocha instance is currently running tests, cannot start a next test run until this one is done',
@@ -1025,7 +1014,7 @@ Object.defineProperty(Mocha.prototype, 'version', {
  * // exit with non-zero status if there were test failures
  * mocha.run(failures => process.exitCode = failures ? 1 : 0);
  */
-Mocha.prototype.run = function(fn) {
+Mocha.prototype.run = function (fn) {
   this._guardRunningStateTransition();
   this._state = mochaStates.RUNNING;
   if (this._previousRunner) {
@@ -1328,9 +1317,10 @@ Mocha.prototype.hasGlobalSetupFixtures = function hasGlobalSetupFixtures() {
  * @public
  * @returns {boolean}
  */
-Mocha.prototype.hasGlobalTeardownFixtures = function hasGlobalTeardownFixtures() {
-  return Boolean(this.options.globalTeardown.length);
-};
+Mocha.prototype.hasGlobalTeardownFixtures =
+  function hasGlobalTeardownFixtures() {
+    return Boolean(this.options.globalTeardown.length);
+  };
 
 /**
  * An alternative way to define root hooks that works with parallel runs.

package/lib/nodejs/buffered-worker-pool.js

@@ -75,7 +75,23 @@ class BufferedWorkerPool {
       process.execArgv.join(' ')
     );
 
-    this.options = {...WORKER_POOL_DEFAULT_OPTS, opts, maxWorkers};
+    let counter = 0;
+    const onCreateWorker = ({forkOpts}) => {
+      return {
+        forkOpts: {
+          ...forkOpts,
+          // adds an incremental id to all workers, which can be useful to allocate resources for each process
+          env: {...process.env, MOCHA_WORKER_ID: counter++}
+        }
+      };
+    };
+
+    this.options = {
+      ...WORKER_POOL_DEFAULT_OPTS,
+      ...opts,
+      maxWorkers,
+      onCreateWorker
+    };
     this._pool = workerpool.pool(WORKER_PATH, this.options);
   }
 

package/lib/{esm-utils.js → nodejs/esm-utils.js}

@@ -53,15 +53,30 @@ exports.requireOrImport = hasStableEsmImplementation
           err.code === 'ERR_UNSUPPORTED_DIR_IMPORT'
         ) {
           try {
+            // Importing a file usually works, but the resolution of `import` is the ESM
+            // resolution algorithm, and not the CJS resolution algorithm. So in this case
+            // if we fail, we may have failed because we tried the ESM resolution and failed
+            // So we try to `require` it
             return require(file);
           } catch (requireErr) {
-            if (requireErr.code === 'ERR_REQUIRE_ESM') {
-              // This happens when the test file is a JS file, but via type:module is actually ESM,
+            if (
+              requireErr.code === 'ERR_REQUIRE_ESM' ||
+              (requireErr instanceof SyntaxError &&
+                requireErr
+                  .toString()
+                  .includes('Cannot use import statement outside a module'))
+            ) {
+              // ERR_REQUIRE_ESM happens when the test file is a JS file, but via type:module is actually ESM,
               // AND has an import to a file that doesn't exist.
-              // This throws an `ERR_MODULE_NOT_FOUND` // error above,
+              // This throws an `ERR_MODULE_NOT_FOUND` error above,
               // and when we try to `require` it here, it throws an `ERR_REQUIRE_ESM`.
               // What we want to do is throw the original error (the `ERR_MODULE_NOT_FOUND`),
               // and not the `ERR_REQUIRE_ESM` error, which is a red herring.
+              //
+              // SyntaxError happens when in an edge case: when we're using an ESM loader that loads
+              // a `test.ts` file (i.e. unrecognized extension), and that file includes an unknown
+              // import (which thows an ERR_MODULE_NOT_FOUND). require-ing it will throw the
+              // syntax error, because we cannot require a file that has import-s.
               throw err;
             } else {
               throw requireErr;

package/lib/reporters/base.js

@@ -107,7 +107,7 @@ exports.symbols = {
  * @param {string} str
  * @return {string}
  */
-var color = (exports.color = function(type, str) {
+var color = (exports.color = function (type, str) {
   if (!exports.useColors) {
     return String(str);
   }
@@ -135,23 +135,23 @@ if (isatty) {
  */
 
 exports.cursor = {
-  hide: function() {
+  hide: function () {
     isatty && process.stdout.write('\u001b[?25l');
   },
 
-  show: function() {
+  show: function () {
     isatty && process.stdout.write('\u001b[?25h');
   },
 
-  deleteLine: function() {
+  deleteLine: function () {
     isatty && process.stdout.write('\u001b[2K');
   },
 
-  beginningOfLine: function() {
+  beginningOfLine: function () {
     isatty && process.stdout.write('\u001b[0G');
   },
 
-  CR: function() {
+  CR: function () {
     if (isatty) {
       exports.cursor.deleteLine();
       exports.cursor.beginningOfLine();
@@ -161,7 +161,7 @@ exports.cursor = {
   }
 };
 
-var showDiff = (exports.showDiff = function(err) {
+var showDiff = (exports.showDiff = function (err) {
   return (
     err &&
     err.showDiff !== false &&
@@ -188,7 +188,7 @@ function stringifyDiffObjs(err) {
  * @param {string} expected
  * @return {string} Diff
  */
-var generateDiff = (exports.generateDiff = function(actual, expected) {
+var generateDiff = (exports.generateDiff = function (actual, expected) {
   try {
     const diffSize = 2048;
     if (actual.length > diffSize) {
@@ -220,10 +220,10 @@ var generateDiff = (exports.generateDiff = function(actual, expected) {
  * @param {Object[]} failures - Each is Test instance with corresponding
  *     Error property
  */
-exports.list = function(failures) {
+exports.list = function (failures) {
   var multipleErr, multipleTest;
   Base.consoleLog();
-  failures.forEach(function(test, i) {
+  failures.forEach(function (test, i) {
     // format
     var fmt =
       color('error title', '  %s) %s:\n') +
@@ -282,7 +282,7 @@ exports.list = function(failures) {
 
     // indented test title
     var testTitle = '';
-    test.titlePath().forEach(function(str, index) {
+    test.titlePath().forEach(function (str, index) {
       if (index !== 0) {
         testTitle += '\n     ';
       }
@@ -318,7 +318,7 @@ function Base(runner, options) {
   this.runner = runner;
   this.stats = runner.stats; // assigned so Reporters keep a closer reference
 
-  runner.on(EVENT_TEST_PASS, function(test) {
+  runner.on(EVENT_TEST_PASS, function (test) {
     if (test.duration > test.slow()) {
       test.speed = 'slow';
     } else if (test.duration > test.slow() / 2) {
@@ -328,7 +328,7 @@ function Base(runner, options) {
     }
   });
 
-  runner.on(EVENT_TEST_FAIL, function(test, err) {
+  runner.on(EVENT_TEST_FAIL, function (test, err) {
     if (showDiff(err)) {
       stringifyDiffObjs(err);
     }
@@ -348,7 +348,7 @@ function Base(runner, options) {
  * @public
  * @memberof Mocha.reporters
  */
-Base.prototype.epilogue = function() {
+Base.prototype.epilogue = function () {
   var stats = this.stats;
   var fmt;
 
@@ -411,7 +411,7 @@ function inlineDiff(actual, expected) {
   if (lines.length > 4) {
     var width = String(lines.length).length;
     msg = lines
-      .map(function(str, i) {
+      .map(function (str, i) {
         return pad(++i, width) + ' |' + ' ' + str;
       })
       .join('\n');
@@ -468,10 +468,7 @@ function unifiedDiff(actual, expected) {
     ' ' +
     colorLines('diff removed', '- actual') +
     '\n\n' +
-    lines
-      .map(cleanUp)
-      .filter(notBlank)
-      .join('\n')
+    lines.map(cleanUp).filter(notBlank).join('\n')
   );
 }
 
@@ -486,7 +483,7 @@ function unifiedDiff(actual, expected) {
 function errorDiff(actual, expected) {
   return diff
     .diffWordsWithSpace(actual, expected)
-    .map(function(str) {
+    .map(function (str) {
       if (str.added) {
         return colorLines('diff added inline', str.value);
       }
@@ -509,7 +506,7 @@ function errorDiff(actual, expected) {
 function colorLines(name, str) {
   return str
     .split('\n')
-    .map(function(str) {
+    .map(function (str) {
       return color(name, str);
     })
     .join('\n');

package/lib/reporters/doc.js

@@ -39,7 +39,7 @@ function Doc(runner, options) {
     return Array(indents).join('  ');
   }
 
-  runner.on(EVENT_SUITE_BEGIN, function(suite) {
+  runner.on(EVENT_SUITE_BEGIN, function (suite) {
     if (suite.root) {
       return;
     }
@@ -50,7 +50,7 @@ function Doc(runner, options) {
     Base.consoleLog('%s<dl>', indent());
   });
 
-  runner.on(EVENT_SUITE_END, function(suite) {
+  runner.on(EVENT_SUITE_END, function (suite) {
     if (suite.root) {
       return;
     }
@@ -60,14 +60,14 @@ function Doc(runner, options) {
     --indents;
   });
 
-  runner.on(EVENT_TEST_PASS, function(test) {
+  runner.on(EVENT_TEST_PASS, function (test) {
     Base.consoleLog('%s  <dt>%s</dt>', indent(), utils.escape(test.title));
     Base.consoleLog('%s  <dt>%s</dt>', indent(), utils.escape(test.file));
     var code = utils.escape(utils.clean(test.body));
     Base.consoleLog('%s  <dd><pre><code>%s</code></pre></dd>', indent(), code);
   });
 
-  runner.on(EVENT_TEST_FAIL, function(test, err) {
+  runner.on(EVENT_TEST_FAIL, function (test, err) {
     Base.consoleLog(
       '%s  <dt class="error">%s</dt>',
       indent(),