testem 3.17.0 → 3.19.0

package/assets/phantom.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var system = require('system');
 var page = require('webpage').create();

package/lib/api.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const log = require('npmlog');
 const Config = require('./config');

package/lib/app.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const EventEmitter = require('events').EventEmitter;
 const Bluebird = require('bluebird');

package/lib/browser_launcher.js

@@ -6,7 +6,7 @@ browser_launcher.js
 This file more or less figures out how to launch any browser on any platform.
 
 */
-'use strict';
+
 
 var Bluebird = require('bluebird');
 

package/lib/browser_tap_consumer.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const TapConsumer = require('./tap_consumer');
 

package/lib/config.js

@@ -7,14 +7,14 @@ This object returns all config info for the app. It handles reading the `testem.
 or `testem.json` config file.
 
 */
-'use strict';
+
 
 const os = require('os');
 const fs = require('fs');
 const yaml = require('js-yaml');
 const log = require('npmlog');
 const path = require('path');
-const glob = require('glob');
+const glob = require('glob').glob;
 const url = require('url');
 const querystring = require('querystring');
 const Bluebird = require('bluebird');
@@ -28,7 +28,8 @@ const fileExists = require('./utils/fileutils').fileExists;
 const _ = require('lodash');
 
 const knownBrowsers = require('./utils/known-browsers');
-const globAsync = Bluebird.promisify(glob);
+const { convertToPosix } = require('./utils/posix');
+const globAsync = glob;
 
 class Config {
   constructor(appMode, progOptions, config) {
@@ -409,10 +410,12 @@ class Config {
         return allThatIWant.concat({ src: pattern, attrs: attrs });
       }
 
-      return globAsync(this.resolvePath(pattern), { ignore: dontWant }).then(files => allThatIWant.concat(files.map(f => {
-        f = this.reverseResolvePath(f);
-        return { src: f, attrs: attrs };
-      })));
+      return globAsync(convertToPosix(this.resolvePath(pattern)), { ignore: dontWant.map(convertToPosix) })
+        .then(files => files.sort())
+        .then(files => allThatIWant.concat(files.map(f => {
+          f = this.reverseResolvePath(f);
+          return { src: f, attrs: attrs };
+        })));
     }, [])
       .then(result => _.uniqBy(result, 'src'))
       .asCallback(callback);

package/lib/file_watcher.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const fireworm = require('fireworm');
 const EventEmitter = require('events').EventEmitter;

package/lib/launcher-factory.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Launcher = require('./launcher');
 const _ = require('lodash');

package/lib/launcher.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const log = require('npmlog');
 const tmp = require('tmp');

package/lib/process-ctl.js

@@ -1,7 +1,7 @@
-'use strict';
+
 
 const log = require('npmlog');
-const execa = require('execa');
+const execa = require('execa').execa;
 const Bluebird = require('bluebird');
 const util = require('util');
 const EventEmitter = require('events').EventEmitter;
@@ -11,6 +11,7 @@ const _ = require('lodash');
 const envWithLocalPath = require('./utils/env-with-local-path');
 const fileutils = require('./utils/fileutils');
 const Process = require('./utils/process');
+const isWin = require('./utils/is-win')();
 
 const fileExists = fileutils.fileExists;
 const executableExists = fileutils.executableExists;
@@ -37,7 +38,20 @@ module.exports = class ProcessCtl extends EventEmitter {
 
   _spawn(exe, args, options) {
     log.info('spawning: ' + exe + ' - ' + util.inspect(args));
-    let p  = new Process(this.name, { killTimeout: this.killTimeout }, execa(exe, args, options));
+
+    // promise should always succeed, Process will uses .on('error') to handle errors
+    let process = execa(exe, args, Object.assign({ reject: false }, options));
+
+    process.then(result => {
+      if (result.failed && result.exitCode !== 0 &&
+        result.shortMessage.includes(isWin ? 'is not recognized' : 'ENOENT'))
+      {
+        this.emit('processError', new Error(result.shortMessage), result.stdout, result.stderr);
+      }
+      return Promise.resolve(result);
+    });
+
+    let p  = new Process(this.name, { killTimeout: this.killTimeout }, process);
     this.emit('processStarted', p);
     return Bluebird.resolve(p);
   }

package/lib/reporters/dev/constants.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 exports.TabWidth = 15;     // column width of each tab
 exports.TabStartLine = 4;  // row from the top to start drawing tabs

package/lib/reporters/dev/error_messages_panel.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const ScrollableTextPanel = require('./scrollable_text_panel');
 const Chars = require('../../utils/chars');

package/lib/reporters/dev/index.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 /*
 

package/lib/reporters/dev/patchcharm.js

@@ -7,7 +7,7 @@ A patch to charm to add text scrolling capability
 and also a fix to `erase()`.
 
 */
-'use strict';
+
 
 const charm = require('charm');
 const Charm = charm.Charm;

package/lib/reporters/dev/runner.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Backbone = require('backbone');
 const TestResults = require('./test_results');

package/lib/reporters/dev/runner_tabs.js

@@ -7,7 +7,7 @@ Implementation of the tabbed UI. Each tab contains its own log panel.
 When the tab is not selected, it hides the associated log panel.
 
 */
-'use strict';
+
 
 const SplitLogPanel = require('./split_log_panel');
 const View = require('./view');

package/lib/reporters/dev/screen.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 let charm;
 

package/lib/reporters/dev/scrollable_text_panel.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var View = require('./view');
 var splitLines = require('../../utils/strutils').splitLines;

package/lib/reporters/dev/split_log_panel.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var View = require('./view');
 var ScrollableTextPanel = require('./scrollable_text_panel');

package/lib/reporters/dev/test_results.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var Backbone = require('backbone');
 

package/lib/reporters/dev/view.js

@@ -5,7 +5,7 @@ view.js
 
 Base-class for our view-models.
 */
-'use strict';
+
 
 var Backbone = require('backbone');
 

package/lib/reporters/dot_reporter.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const indent = require('../utils/strutils').indent;
 const printf = require('printf');

package/lib/reporters/index.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 module.exports = {
   tap: require('./tap_reporter'),

package/lib/reporters/tap_reporter.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const displayutils = require('../utils/displayutils');
 

package/lib/reporters/teamcity_reporter.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 function namify(prefix, result) {
   let line = (prefix ? (prefix + ' - ') : '') +

package/lib/reporters/xunit_reporter.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var XmlDom = require('@xmldom/xmldom');
 var indent = require('../utils/strutils').indent;

package/lib/runners/browser_test_runner.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const log = require('npmlog');
 const BrowserTapConsumer = require('../browser_tap_consumer');

package/lib/runners/hook_runner.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Bluebird = require('bluebird');
 const log = require('npmlog');

package/lib/runners/process_test_runner.js

@@ -1,4 +1,4 @@
-'use strict';
+
 var Bluebird = require('bluebird');
 
 var toResult = require('./to-result');
@@ -21,6 +21,7 @@ class ProcessTestRunner {
         this.process = testProcess;
         this.process.once('processExit', this.onProcessExit.bind(this));
         this.process.once('processError', this.onProcessError.bind(this));
+        return Promise.resolve(testProcess);
       }).catch(reject);
     }).asCallback(onFinish);
   }

package/lib/runners/tap_process_test_runner.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 var TapConsumer = require('../tap_consumer');
 var log = require('npmlog');

package/lib/runners/to-result.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const log = require('npmlog');
 

package/lib/server/index.js

@@ -7,7 +7,7 @@ Testem's server. Serves up the HTML, JS, and CSS required for
 running the tests in a browser.
 
 */
-'use strict';
+
 
 var express = require('express');
 var socketIO = require('socket.io');

package/lib/tap_consumer.js

@@ -1,7 +1,7 @@
-'use strict';
+
 
 const yaml = require('js-yaml');
-const extend = require('util')._extend;
+const extend = Object.assign;
 const EventEmitter = require('events').EventEmitter;
 const TapParser = require('tap-parser');
 const log = require('npmlog');

package/lib/utils/add-to-PATH.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const _ = require('lodash');
 const isWin = require('./is-win')();

package/lib/utils/browser-args.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const capitalize = require('./capitalize');
 const log = require('npmlog');

package/lib/utils/capitalize.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 module.exports = str => {
   if (typeof str !== 'string') { return; }

package/lib/utils/chars.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const isWin = require('./is-win')();
 

package/lib/utils/clean_exit.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 module.exports = function cleanExit(code) {
   // Workaround for this node core bug <https://github.com/joyent/node/issues/3584>

package/lib/utils/current-time.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 function currentTimeAsLocaleTimeString() {
   let dateTime = new Date();

package/lib/utils/displayutils.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 // Method to format test results.
 const strutils = require('./strutils');

package/lib/utils/env-with-local-path.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const path = require('path');
 const fs = require('fs');

package/lib/utils/fileutils.js

@@ -1,7 +1,7 @@
-'use strict';
+
 
 const fs = require('fs');
-const execa = require('execa');
+const execa = require('execa').execa;
 const Bluebird = require('bluebird');
 const log = require('npmlog');
 
@@ -15,9 +15,14 @@ exports.fileExists = function fileExists(path) {
 
 exports.executableExists = function executableExists(exe, options) {
   let cmd = isWin ? 'where' : 'which';
-  let test = execa(cmd, [exe], options);
-
-  test.on('error', error => log.error('Error spawning "' + cmd + exe + '"', error));
+  let test = execa(cmd, [exe], Object.assign({ reject: false }, options));
 
-  return test.then(result => result.code === 0, () => false);
+  return test.then(result => {
+    if (result.exitCode === 0) {
+      return true;
+    } else if (!result.exitCode) {
+      log.error('Error spawning "' + cmd + ' ' + exe + '"', result);
+    }
+    return false;
+  });
 };

package/lib/utils/is-node-lt.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 module.exports = version => {
   let nodeVer = process.version.substr(1).split('.').map(num => parseInt(num, 10));

package/lib/utils/is-win.js

@@ -1,5 +1,3 @@
-'use strict';
-
 function test(platform) {
   return /^win/.test(platform);
 }

package/lib/utils/isa.js

@@ -11,7 +11,7 @@ Examples:
   isa("abc", String) // true
 
 */
-'use strict';
+
 
 const toString = Object.prototype.toString;
 

package/lib/utils/known-browsers.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const browserArgs = require('./browser-args');
 const path = require('path');
@@ -84,6 +84,7 @@ function firefoxPaths() {
   return [
     'C:\\Program Files\\Mozilla Firefox\\firefox.exe',
     'C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe',
+    'C:\\Users\\' + process.env.USERNAME + '\\AppData\\Local\\Mozilla Firefox\\firefox.exe',
     process.env.HOME + '/Applications/Firefox.app/Contents/MacOS/firefox',
     '/Applications/Firefox.app/Contents/MacOS/firefox'
   ];
@@ -144,7 +145,7 @@ module.exports = function knownBrowsers(platform, config) {
       possibleExe: 'firefox',
 
       args(config, url) {
-        return ['-profile', '--headless', this.browserTmpDir(), url];
+        return ['-profile', '-headless', this.browserTmpDir(), url];
       },
       setup(config, done) {
         firefoxSetup(this.browserTmpDir(), config, done);

package/lib/utils/log-entry.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 /**
  * A class to a single log entry.

package/lib/utils/posix.js

@@ -0,0 +1,13 @@
+const isWin = require('./is-win');
+
+module.exports = {
+  isPosix(path) {
+    return path.includes('/') && !path.includes('\\');
+  },
+  convertToPosix(path) {
+    if (isWin()) {
+      return path.replace(/\\/g, '/');
+    }
+    return path;
+  }
+};

package/lib/utils/process.js

@@ -1,6 +1,4 @@
-'use strict';
-
-const execa = require('execa');
+const execa = require('execa').execa;
 const log = require('npmlog');
 const Bluebird = require('bluebird');
 const EventEmitter = require('events').EventEmitter;
@@ -29,6 +27,15 @@ module.exports = class Process extends EventEmitter {
 
     this.process.on('close', this.onClose.bind(this));
     this.process.on('error', this.onError.bind(this));
+
+    this.process.then(result => {
+      if (result.failed && result.exitCode !== 0 &&
+        result.shortMessage.includes(isWin ? 'is not recognized' : 'ENOENT'))
+      {
+        this.onError(new Error(result.shortMessage), this.stdout, this.stderr);
+      }
+      return Promise.resolve(result);
+    });
   }
 
   onKillTimeout() {
@@ -48,7 +55,7 @@ module.exports = class Process extends EventEmitter {
 
   onError(error) {
     log.warn(this.name + ' errored', error);
-    this.process = null;
+    // this.process = null;
     this.exitCode = 1;
     this.emit('processError', error, this.stdout, this.stderr);
   }
@@ -132,14 +139,18 @@ function kill(p, sig) {
       args.push('/f');
     }
 
-    execa(command, args).then(result => {
+    const killProcess = execa(command, args, { reject: false });
+
+    killProcess.on('error', err => {
+      log.error(err);
+    });
+
+    killProcess.then(result => {
       // Processes without windows can't be killed without /F, detect and force
       // kill them directly
       if (result.stderr.indexOf('can only be terminated forcefully') !== -1) {
         kill(p, 'SIGKILL');
       }
-    }).catch(err => {
-      log.error(err);
     });
   } else {
     p.kill(sig);

package/lib/utils/report-file.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const fs = require('fs');
 const path = require('path');

package/lib/utils/reporter.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Bluebird = require('bluebird');
 const log = require('npmlog');

package/lib/utils/run-timeout.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Bluebird = require('bluebird');
 const EventEmitter = require('events').EventEmitter;

package/lib/utils/signal-listeners.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 const Bluebird = require('bluebird');
 const EventEmitter = require('events').EventEmitter;

package/lib/utils/strutils.js

@@ -1,4 +1,4 @@
-'use strict';
+
 
 // String padding function adapted from <http://jsfromhell.com/string/pad>
 exports.pad = function pad(str, l, s, t) {

package/package.json

@@ -7,7 +7,7 @@
     "unittest",
     "browser"
   ],
-  "version": "3.17.0",
+  "version": "3.19.0",
   "author": "Toby Ho <airportyh@gmail.com>",
   "repository": {
     "type": "git",
@@ -28,10 +28,10 @@
     "commander": "^2.6.0",
     "compression": "^1.7.4",
     "consolidate": "^0.16.0",
-    "execa": "^1.0.0",
+    "execa": "^9.6.1",
     "express": "^4.10.7",
     "fireworm": "^0.7.2",
-    "glob": "^7.0.4",
+    "glob": "^13.0.6",
     "http-proxy": "^1.13.1",
     "js-yaml": "^3.2.5",
     "lodash": "^4.17.21",
@@ -40,12 +40,12 @@
     "node-notifier": "^10.0.0",
     "npmlog": "^6.0.0",
     "printf": "^0.6.1",
-    "rimraf": "^3.0.2",
+    "rimraf": "^6.1.3",
     "socket.io": "^4.5.4",
     "spawn-args": "^0.2.0",
     "styled_string": "0.0.1",
     "tap-parser": "^7.0.0",
-    "tmp": "0.0.33"
+    "tmp": "^0.2.5"
   },
   "files": [
     "lib",
@@ -67,20 +67,20 @@
   "devDependencies": {
     "bluebird-retry": "^0.11.0",
     "browserify": "^17.0.0",
-    "chai": "^4.0.2",
+    "chai": "^6.2.2",
     "chai-files": "^1.2.0",
     "chai-shallow-deep-equal": "^1.4.4",
     "cheerio": "1.0.0-rc.12",
-    "dirty-chai": "^2.0.0",
-    "eslint": "^8.12.0",
-    "eslint-plugin-chai-expect": "^3.0.0",
-    "eslint-plugin-mocha": "^10.0.3",
-    "mocha": "^9.2.0",
+    "dirty-chai": "^3.0.0",
+    "eslint": "^9.39.4",
+    "eslint-plugin-chai-expect": "^3.1.0",
+    "eslint-plugin-mocha": "^11.2.0",
+    "mocha": "^11.7.5",
     "request": "^2.51.0",
     "saucie": "^3.3.3",
     "shelljs": "^0.8.2",
     "sinon": "10.0.0",
-    "sinon-chai": "^3.2.0",
+    "sinon-chai": "^4.0.1",
     "socket.io-client": "^4.1.2",
     "tape": "^5.2.2"
   },

package/testem.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-'use strict';
+
 
 const program = require('commander');
 let progOptions = program;