[DO NOT MERGE] WIP: run tests concurrently; see #2839

.eleventy.js

@@ -31,6 +31,8 @@ module.exports = function(eleventyConfig) {
 
   markdown.use(require('markdown-it-prism'));
 
+  markdown.use(require('markdown-it-emoji'));
+
   eleventyConfig.setLibrary('md', markdown);
 
   return {

.eslintrc.yml

@@ -16,17 +16,22 @@ rules:
     - safe
 overrides:
   - files:
-      - scripts/**/*.js
-      - package-scripts.js
-      - karma.conf.js
-      - .wallaby.js
-      - .eleventy.js
-      - bin/*
-      - lib/cli/**/*.js
-      - test/node-unit/**/*.js
-      - test/integration/options/watch.spec.js
-      - test/integration/helpers.js
-      - lib/growl.js
+      - 'scripts/**/*.js'
+      - 'package-scripts.js'
+      - 'karma.conf.js'
+      - '.wallaby.js'
+      - '.eleventy.js'
+      - 'bin/*'
+      - 'lib/cli/**/*.js'
+      - 'test/node-unit/**/*.js'
+      - 'test/integration/options/watch.spec.js'
+      - 'test/integration/helpers.js'
+      - 'lib/growl.js'
+      - 'lib/buffered-runner.js'
+      - 'lib/worker.js'
+      - 'lib/reporters/buffered.js'
+      - 'lib/serializer.js'
+      - 'test/reporters/buffered.spec.js'
     parserOptions:
       ecmaVersion: 2018
     env:

.markdownlint.json

@@ -6,5 +6,6 @@
   "first-header-h1": false,
   "first-line-h1": false,
   "commands-show-output": false,
-  "single-h1": false
+  "single-h1": false,
+  "fenced-code-language": false
 }

.mocharc.yml

@@ -1,7 +1,12 @@
-require: test/setup
-ui: bdd
+require: 'test/setup'
+ui: 'bdd'
 global:
-  - okGlobalA,okGlobalB
-  - okGlobalC
-  - callback*
-timeout: 300
+  - 'okGlobalA,okGlobalB'
+  - 'okGlobalC'
+  - 'callback*'
+timeout: 1000
+parallel: true
+watch-ignore:
+  - '.*'
+  - 'docs/_dist/**'
+  - 'node_modules'

.nycrc

@@ -4,7 +4,16 @@
     "text-summary"
   ],
   "exclude": [
-    "lib/browser",
-    "test/**/*"
+    "coverage/**",
+    "packages/*/test{,s}/**",
+    "**/*.d.ts",
+    "test{,s}/**",
+    "test{,-*}.{js,cjs,mjs,ts}",
+    "**/*{.,-}test.{js,cjs,mjs,ts}",
+    "**/__tests__/**",
+    "**/{karma,rollup,webpack}.config.js",
+    "**/{babel.config,.eslintrc,.mocharc}.{js,cjs}",
+    "lib/browser/**",
+    "package-scripts.js"
   ]
 }

.travis.yml

@@ -38,15 +38,22 @@ jobs:
   include:
     - script: COVERAGE=1 npm start test.node
       after_success: npm start coveralls
+      name: 'Latest Node.js (with coverage)'
+
+    - script: MOCHA_PARALLEL=0 npm start test.node.unit
+      name: 'Latest Node.js (in serial mode)'
 
     - &node
       script: npm start test.node
       node_js: '12'
+      name: 'Node.js v12'
 
     - <<: *node
       node_js: '10'
+      name: 'Node.js v10'
 
     - script: npm start test.bundle test.browser
+      name: 'Browser'
       node_js: 12
       install: npm ci # we need the native modules here
       addons:
@@ -59,14 +66,15 @@ jobs:
 
     - stage: lint
       script: npm start lint
+      name: 'JS & Markdown'
 
     # smoke tests use default npm.
     - &smoke
       stage: smoke
       env: null
       before_install: true
       install: npm install --production
-
+      name: 'Latest Node.js'
       script: ./bin/mocha --no-config --reporter spec test/sanity/sanity.spec.js
       cache:
         directories:
@@ -75,19 +83,23 @@ jobs:
 
     - <<: *smoke
       node_js: '12'
+      name: 'Node.js v12'
 
     - <<: *smoke
       node_js: '10'
+      name: 'Node.js v10'
 
     - stage: precache
       script: true
+      name: 'Prime cache'
 
 notifications:
   email: false
-  urls:
-    # for gitter mochajs/mocha
-    - secure: fUrHenYJs+pTuLtgBRoYyrlyfVekxaIGmLWq7bhUUqBj/7p5eCkQFn13LlPht0/4WWZOiPBcdTN7tKnz3Ho7ATUJhAchvOWDUgL5gtTvOzeCHbPuCvHz/VLK6hMoPdbLA45M864NDLotfHvyh62WgQaVw9iPc80eb+umaDPrYiU=
-    # for gitter mochajs/contributors
-    - secure: rGMGYWBaZgEa9i997jJHKzjI8WxECqLi6BqsMhvstDq9EeTeXkZFVfz4r6G3Xugsk3tFwb/pDpiYo1OK36kA5arUJTCia51u4Wn+c7lHKcpef/vXztoyucvw6/jXdVm/FQz1jztYYbqdyAOWC2BV8gYvg5F8TpK05UGCe5R0bRA=
-  on_success: change
-  on_failure: always
+  webhooks:
+    urls:
+      # for gitter mochajs/mocha
+      - secure: fUrHenYJs+pTuLtgBRoYyrlyfVekxaIGmLWq7bhUUqBj/7p5eCkQFn13LlPht0/4WWZOiPBcdTN7tKnz3Ho7ATUJhAchvOWDUgL5gtTvOzeCHbPuCvHz/VLK6hMoPdbLA45M864NDLotfHvyh62WgQaVw9iPc80eb+umaDPrYiU=
+      # for gitter mochajs/contributors
+      - secure: rGMGYWBaZgEa9i997jJHKzjI8WxECqLi6BqsMhvstDq9EeTeXkZFVfz4r6G3Xugsk3tFwb/pDpiYo1OK36kA5arUJTCia51u4Wn+c7lHKcpef/vXztoyucvw6/jXdVm/FQz1jztYYbqdyAOWC2BV8gYvg5F8TpK05UGCe5R0bRA=
+    on_success: change
+    on_failure: always

.wallaby.js

@@ -36,7 +36,7 @@ module.exports = () => {
       // running mocha instance is not the same as mocha under test,
       // running mocha is the project's source code mocha, mocha under test is instrumented version of the source code
       const runningMocha = wallaby.testFramework;
-      runningMocha.timeout(200);
+      runningMocha.timeout(1000);
       // to expose it/describe etc. on the mocha under test
       const MochaUnderTest = require('./');
       const mochaUnderTest = new MochaUnderTest();
@@ -47,17 +47,17 @@ module.exports = () => {
         mochaUnderTest
       );
       // to make test/node-unit/color.spec.js pass, we need to run mocha in the project's folder context
-      const childProcess = require('child_process');
-      const execFile = childProcess.execFile;
-      childProcess.execFile = function() {
-        let opts = arguments[2];
-        if (typeof opts === 'function') {
-          opts = {};
-          Array.prototype.splice.call(arguments, 2, 0, opts);
-        }
-        opts.cwd = wallaby.localProjectDir;
-        return execFile.apply(this, arguments);
-      };
+      // const childProcess = require('child_process');
+      // const execFile = childProcess.execFile;
+      // childProcess.execFile = function() {
+      //   let opts = arguments[2];
+      //   if (typeof opts === 'function') {
+      //     opts = {};
+      //     Array.prototype.splice.call(arguments, 2, 0, opts);
+      //   }
+      //   opts.cwd = wallaby.localProjectDir;
+      //   return execFile.apply(this, arguments);
+      // };
       require('./test/setup');
     },
     debug: true

bin/mocha

@@ -108,7 +108,9 @@ if (Object.keys(nodeArgs).length) {
     unparse(mochaArgs, {alias: aliases})
   );
 
-  debug(`exec ${process.execPath} w/ args:`, args);
+  debug(
+    `forking child process via command: ${process.execPath} ${args.join(' ')}`
+  );
 
   const proc = spawn(process.execPath, args, {
     stdio: 'inherit'
@@ -126,9 +128,25 @@ if (Object.keys(nodeArgs).length) {
 
   // terminate children.
   process.on('SIGINT', () => {
-    proc.kill('SIGINT'); // calls runner.abort()
-    proc.kill('SIGTERM'); // if that didn't work, we're probably in an infinite loop, so make it die.
+    // XXX: a previous comment said this would abort the runner, but I can't see that it does
+    // anything with the default runner.
+    debug('main process caught SIGINT');
+    proc.kill('SIGINT');
+    // if running in parallel mode, we will have a proper SIGINT handler, so the below won't
+    // be needed.
+    if (!args.parallel || args.jobs < 2) {
+      // win32 does not support SIGTERM, so use next best thing.
+      if (require('os').platform() === 'win32') {
+        proc.kill('SIGKILL');
+      } else {
+        // using SIGKILL won't cleanly close the output streams, which can result
+        // in cut-off text or a befouled terminal.
+        debug('sending SIGTERM to child process');
+        proc.kill('SIGTERM');
+      }
+    }
   });
 } else {
+  debug('running Mocha in-process');
   require('../lib/cli/cli').main(unparse(mochaArgs, {alias: aliases}));
 }

docs/css/style.css

@@ -171,6 +171,13 @@ ul {
   padding: 0 15px;
 }
 
+ul.single-column,
+ul.single-column > li > ul {
+  column-count: 1;
+  margin-top: 0;
+  padding-right: 0;
+}
+
 ul li {
   border-bottom: 1px solid #eee;
   break-inside: avoid;
@@ -179,6 +186,17 @@ ul li {
   padding: 5px 0;
 }
 
+ul.single-column li,
+ul.single-column li > ul > li {
+  list-style: disc;
+  padding: 0;
+  border-bottom: 0;
+}
+
+ul.single-column li > ul > li {
+  list-style: circle;
+}
+
 code {
   font: 14px monaco, monospace;
   line-height: 1.8;