mocha 2.2.5 → 2.3.3

package/lib/utils.js

@@ -1,14 +1,17 @@
+/* eslint-env browser */
+
 /**
  * Module dependencies.
  */
 
-var fs = require('fs')
-  , path = require('path')
-  , basename = path.basename
-  , exists = fs.existsSync || path.existsSync
-  , glob = require('glob')
-  , join = path.join
-  , debug = require('debug')('mocha:watch');
+var basename = require('path').basename;
+var debug = require('debug')('mocha:watch');
+var exists = require('fs').existsSync || require('path').existsSync;
+var glob = require('glob');
+var join = require('path').join;
+var readdirSync = require('fs').readdirSync;
+var statSync = require('fs').statSync;
+var watchFile = require('fs').watchFile;
 
 /**
  * Ignored directories.
@@ -16,15 +19,16 @@ var fs = require('fs')
 
 var ignore = ['node_modules', '.git'];
 
+exports.inherits = require('util').inherits;
+
 /**
  * Escape special characters in the given string of html.
  *
- * @param  {String} html
- * @return {String}
  * @api private
+ * @param  {string} html
+ * @return {string}
  */
-
-exports.escape = function(html){
+exports.escape = function(html) {
   return String(html)
     .replace(/&/g, '&')
     .replace(/"/g, '"')
@@ -35,57 +39,59 @@ exports.escape = function(html){
 /**
  * Array#forEach (<=IE8)
  *
- * @param {Array} array
+ * @api private
+ * @param {Array} arr
  * @param {Function} fn
  * @param {Object} scope
- * @api private
  */
-
-exports.forEach = function(arr, fn, scope){
-  for (var i = 0, l = arr.length; i < l; i++)
+exports.forEach = function(arr, fn, scope) {
+  for (var i = 0, l = arr.length; i < l; i++) {
     fn.call(scope, arr[i], i);
+  }
 };
 
 /**
- * Test if the given obj is type of string
+ * Test if the given obj is type of string.
  *
+ * @api private
  * @param {Object} obj
- * @returns Boolean
+ * @return {boolean}
  */
-
 exports.isString = function(obj) {
-  return 'string' === typeof obj;
+  return typeof obj === 'string';
 };
 
 /**
  * Array#map (<=IE8)
  *
- * @param {Array} array
+ * @api private
+ * @param {Array} arr
  * @param {Function} fn
  * @param {Object} scope
- * @api private
+ * @return {Array}
  */
-
-exports.map = function(arr, fn, scope){
+exports.map = function(arr, fn, scope) {
   var result = [];
-  for (var i = 0, l = arr.length; i < l; i++)
+  for (var i = 0, l = arr.length; i < l; i++) {
     result.push(fn.call(scope, arr[i], i, arr));
+  }
   return result;
 };
 
 /**
  * Array#indexOf (<=IE8)
  *
- * @parma {Array} arr
- * @param {Object} obj to find index of
- * @param {Number} start
  * @api private
+ * @param {Array} arr
+ * @param {Object} obj to find index of
+ * @param {number} start
+ * @return {number}
  */
-
-exports.indexOf = function(arr, obj, start){
+exports.indexOf = function(arr, obj, start) {
   for (var i = start || 0, l = arr.length; i < l; i++) {
-    if (arr[i] === obj)
+    if (arr[i] === obj) {
       return i;
+    }
   }
   return -1;
 };
@@ -93,13 +99,13 @@ exports.indexOf = function(arr, obj, start){
 /**
  * Array#reduce (<=IE8)
  *
- * @param {Array} array
- * @param {Function} fn
- * @param {Object} initial value
  * @api private
+ * @param {Array} arr
+ * @param {Function} fn
+ * @param {Object} val Initial value.
+ * @return {*}
  */
-
-exports.reduce = function(arr, fn, val){
+exports.reduce = function(arr, fn, val) {
   var rval = val;
 
   for (var i = 0, l = arr.length; i < l; i++) {
@@ -112,17 +118,19 @@ exports.reduce = function(arr, fn, val){
 /**
  * Array#filter (<=IE8)
  *
- * @param {Array} array
- * @param {Function} fn
  * @api private
+ * @param {Array} arr
+ * @param {Function} fn
+ * @return {Array}
  */
-
-exports.filter = function(arr, fn){
+exports.filter = function(arr, fn) {
   var ret = [];
 
   for (var i = 0, l = arr.length; i < l; i++) {
     var val = arr[i];
-    if (fn(val, i, arr)) ret.push(val);
+    if (fn(val, i, arr)) {
+      ret.push(val);
+    }
   }
 
   return ret;
@@ -131,14 +139,13 @@ exports.filter = function(arr, fn){
 /**
  * Object.keys (<=IE8)
  *
+ * @api private
  * @param {Object} obj
  * @return {Array} keys
- * @api private
  */
-
-exports.keys = Object.keys || function(obj) {
-  var keys = []
-    , has = Object.prototype.hasOwnProperty; // for `window` on <=IE8
+exports.keys = typeof Object.keys === 'function' ? Object.keys : function(obj) {
+  var keys = [];
+  var has = Object.prototype.hasOwnProperty; // for `window` on <=IE8
 
   for (var key in obj) {
     if (has.call(obj, key)) {
@@ -153,17 +160,18 @@ exports.keys = Object.keys || function(obj) {
  * Watch the given `files` for changes
  * and invoke `fn(file)` on modification.
  *
+ * @api private
  * @param {Array} files
  * @param {Function} fn
- * @api private
  */
-
-exports.watch = function(files, fn){
+exports.watch = function(files, fn) {
   var options = { interval: 100 };
-  files.forEach(function(file){
+  files.forEach(function(file) {
     debug('file %s', file);
-    fs.watchFile(file, options, function(curr, prev){
-      if (prev.mtime < curr.mtime) fn(file);
+    watchFile(file, options, function(curr, prev) {
+      if (prev.mtime < curr.mtime) {
+        fn(file);
+      }
     });
   });
 };
@@ -171,51 +179,56 @@ exports.watch = function(files, fn){
 /**
  * Array.isArray (<=IE8)
  *
+ * @api private
  * @param {Object} obj
  * @return {Boolean}
- * @api private
  */
-var isArray = Array.isArray || function (obj) {
-  return '[object Array]' == {}.toString.call(obj);
+var isArray = typeof Array.isArray === 'function' ? Array.isArray : function(obj) {
+  return Object.prototype.toString.call(obj) === '[object Array]';
 };
 
 /**
- * @description
- * Buffer.prototype.toJSON polyfill
+ * Buffer.prototype.toJSON polyfill.
+ *
  * @type {Function}
  */
-if(typeof Buffer !== 'undefined' && Buffer.prototype) {
-  Buffer.prototype.toJSON = Buffer.prototype.toJSON || function () {
+if (typeof Buffer !== 'undefined' && Buffer.prototype) {
+  Buffer.prototype.toJSON = Buffer.prototype.toJSON || function() {
     return Array.prototype.slice.call(this, 0);
   };
 }
 
 /**
  * Ignored files.
+ *
+ * @api private
+ * @param {string} path
+ * @return {boolean}
  */
-
-function ignored(path){
+function ignored(path) {
   return !~ignore.indexOf(path);
 }
 
 /**
  * Lookup files in the given `dir`.
  *
- * @return {Array}
  * @api private
+ * @param {string} dir
+ * @param {string[]} [ext=['.js']]
+ * @param {Array} [ret=[]]
+ * @return {Array}
  */
-
-exports.files = function(dir, ext, ret){
+exports.files = function(dir, ext, ret) {
   ret = ret || [];
   ext = ext || ['js'];
 
   var re = new RegExp('\\.(' + ext.join('|') + ')$');
 
-  fs.readdirSync(dir)
+  readdirSync(dir)
     .filter(ignored)
-    .forEach(function(path){
+    .forEach(function(path) {
       path = join(dir, path);
-      if (fs.statSync(path).isDirectory()) {
+      if (statSync(path).isDirectory()) {
         exports.files(path, ext, ret);
       } else if (path.match(re)) {
         ret.push(path);
@@ -228,12 +241,11 @@ exports.files = function(dir, ext, ret){
 /**
  * Compute a slug from the given `str`.
  *
- * @param {String} str
- * @return {String}
  * @api private
+ * @param {string} str
+ * @return {string}
  */
-
-exports.slug = function(str){
+exports.slug = function(str) {
   return str
     .toLowerCase()
     .replace(/ +/g, '-')
@@ -241,19 +253,20 @@ exports.slug = function(str){
 };
 
 /**
- * Strip the function definition from `str`,
- * and re-indent for pre whitespace.
+ * Strip the function definition from `str`, and re-indent for pre whitespace.
+ *
+ * @param {string} str
+ * @return {string}
  */
-
 exports.clean = function(str) {
   str = str
-    .replace(/\r\n?|[\n\u2028\u2029]/g, "\n").replace(/^\uFEFF/, '')
+    .replace(/\r\n?|[\n\u2028\u2029]/g, '\n').replace(/^\uFEFF/, '')
     .replace(/^function *\(.*\)\s*{|\(.*\) *=> *{?/, '')
     .replace(/\s+\}$/, '');
 
-  var spaces = str.match(/^\n?( *)/)[1].length
-    , tabs = str.match(/^\n?(\t*)/)[1].length
-    , re = new RegExp('^\n?' + (tabs ? '\t' : ' ') + '{' + (tabs ? tabs : spaces) + '}', 'gm');
+  var spaces = str.match(/^\n?( *)/)[1].length;
+  var tabs = str.match(/^\n?(\t*)/)[1].length;
+  var re = new RegExp('^\n?' + (tabs ? '\t' : ' ') + '{' + (tabs ? tabs : spaces) + '}', 'gm');
 
   str = str.replace(re, '');
 
@@ -263,28 +276,26 @@ exports.clean = function(str) {
 /**
  * Trim the given `str`.
  *
- * @param {String} str
- * @return {String}
  * @api private
+ * @param {string} str
+ * @return {string}
  */
-
-exports.trim = function(str){
+exports.trim = function(str) {
   return str.replace(/^\s+|\s+$/g, '');
 };
 
 /**
  * Parse the given `qs`.
  *
- * @param {String} qs
- * @return {Object}
  * @api private
+ * @param {string} qs
+ * @return {Object}
  */
-
-exports.parseQuery = function(qs){
-  return exports.reduce(qs.replace('?', '').split('&'), function(obj, pair){
-    var i = pair.indexOf('=')
-      , key = pair.slice(0, i)
-      , val = pair.slice(++i);
+exports.parseQuery = function(qs) {
+  return exports.reduce(qs.replace('?', '').split('&'), function(obj, pair) {
+    var i = pair.indexOf('=');
+    var key = pair.slice(0, i);
+    var val = pair.slice(++i);
 
     obj[key] = decodeURIComponent(val);
     return obj;
@@ -294,11 +305,10 @@ exports.parseQuery = function(qs){
 /**
  * Highlight the given string of `js`.
  *
- * @param {String} js
- * @return {String}
  * @api private
+ * @param {string} js
+ * @return {string}
  */
-
 function highlight(js) {
   return js
     .replace(/</g, '&lt;')
@@ -308,16 +318,15 @@ function highlight(js) {
     .replace(/(\d+\.\d+)/gm, '<span class="number">$1</span>')
     .replace(/(\d+)/gm, '<span class="number">$1</span>')
     .replace(/\bnew[ \t]+(\w+)/gm, '<span class="keyword">new</span> <span class="init">$1</span>')
-    .replace(/\b(function|new|throw|return|var|if|else)\b/gm, '<span class="keyword">$1</span>')
+    .replace(/\b(function|new|throw|return|var|if|else)\b/gm, '<span class="keyword">$1</span>');
 }
 
 /**
  * Highlight the contents of tag `name`.
  *
- * @param {String} name
  * @api private
+ * @param {string} name
  */
-
 exports.highlightTags = function(name) {
   var code = document.getElementById('mocha').getElementsByTagName(name);
   for (var i = 0, len = code.length; i < len; ++i) {
@@ -326,22 +335,23 @@ exports.highlightTags = function(name) {
 };
 
 /**
- * If a value could have properties, and has none, this function is called, which returns
- * a string representation of the empty value.
+ * If a value could have properties, and has none, this function is called,
+ * which returns a string representation of the empty value.
  *
  * Functions w/ no properties return `'[Function]'`
  * Arrays w/ length === 0 return `'[]'`
  * Objects w/ no properties return `'{}'`
  * All else: return result of `value.toString()`
  *
- * @param {*} value Value to inspect
+ * @api private
+ * @param {*} value The value to inspect.
  * @param {string} [type] The type of the value, if known.
  * @returns {string}
  */
-var emptyRepresentation = function emptyRepresentation(value, type) {
+function emptyRepresentation(value, type) {
   type = type || exports.type(value);
 
-  switch(type) {
+  switch (type) {
     case 'function':
       return '[Function]';
     case 'object':
@@ -351,11 +361,16 @@ var emptyRepresentation = function emptyRepresentation(value, type) {
     default:
       return value.toString();
   }
-};
+}
 
 /**
- * Takes some variable and asks `{}.toString()` what it thinks it is.
- * @param {*} value Anything
+ * Takes some variable and asks `Object.prototype.toString()` what it thinks it
+ * is.
+ *
+ * @api private
+ * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/toString
+ * @param {*} value The value to test.
+ * @returns {string}
  * @example
  * type({}) // 'object'
  * type([]) // 'array'
@@ -367,12 +382,13 @@ var emptyRepresentation = function emptyRepresentation(value, type) {
  * type(/foo/) // 'regexp'
  * type('type') // 'string'
  * type(global) // 'global'
- * @api private
- * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/toString
- * @returns {string}
  */
 exports.type = function type(value) {
-  if (typeof Buffer !== 'undefined' && Buffer.isBuffer(value)) {
+  if (value === undefined) {
+    return 'undefined';
+  } else if (value === null) {
+    return 'null';
+  } else if (typeof Buffer !== 'undefined' && Buffer.isBuffer(value)) {
     return 'buffer';
   }
   return Object.prototype.toString.call(value)
@@ -381,8 +397,8 @@ exports.type = function type(value) {
 };
 
 /**
- * @summary Stringify `value`.
- * @description Different behavior depending on type of value.
+ * Stringify `value`. Different behavior depending on type of value:
+ *
  * - If `value` is undefined or null, return `'[undefined]'` or `'[null]'`, respectively.
  * - If `value` is not an object, function or array, return result of `value.toString()` wrapped in double-quotes.
  * - If `value` is an *empty* object, function, or array, return result of function
@@ -390,17 +406,16 @@ exports.type = function type(value) {
  * - If `value` has properties, call {@link exports.canonicalize} on it, then return result of
  *   JSON.stringify().
  *
+ * @api private
  * @see exports.type
  * @param {*} value
  * @return {string}
- * @api private
  */
-
 exports.stringify = function(value) {
   var type = exports.type(value);
 
   if (!~exports.indexOf(['object', 'array', 'function'], type)) {
-    if(type != 'buffer') {
+    if (type !== 'buffer') {
       return jsonStringify(value);
     }
     var json = value.toJSON();
@@ -419,23 +434,29 @@ exports.stringify = function(value) {
 };
 
 /**
- * @description
  * like JSON.stringify but more sense.
+ *
+ * @api private
  * @param {Object}  object
- * @param {Number=} spaces
+ * @param {number=} spaces
  * @param {number=} depth
  * @returns {*}
- * @private
  */
 function jsonStringify(object, spaces, depth) {
-  if(typeof spaces == 'undefined') return _stringify(object);  // primitive types
+  if (typeof spaces === 'undefined') {
+    // primitive types
+    return _stringify(object);
+  }
 
   depth = depth || 1;
-  var space = spaces * depth
-    , str = isArray(object) ? '[' : '{'
-    , end = isArray(object) ? ']' : '}'
-    , length = object.length || exports.keys(object).length
-    , repeat = function(s, n) { return new Array(n).join(s); }; // `.repeat()` polyfill
+  var space = spaces * depth;
+  var str = isArray(object) ? '[' : '{';
+  var end = isArray(object) ? ']' : '}';
+  var length = object.length || exports.keys(object).length;
+  // `.repeat()` polyfill
+  function repeat(s, n) {
+    return new Array(n).join(s);
+  }
 
   function _stringify(val) {
     switch (exports.type(val)) {
@@ -450,7 +471,7 @@ function jsonStringify(object, spaces, depth) {
       case 'boolean':
       case 'regexp':
       case 'number':
-        val = val === 0 && (1/val) === -Infinity // `-0`
+        val = val === 0 && (1 / val) === -Infinity // `-0`
           ? '-0'
           : val.toString();
         break;
@@ -467,40 +488,44 @@ function jsonStringify(object, spaces, depth) {
         val = '[Buffer: ' + jsonStringify(json, 2, depth + 1) + ']';
         break;
       default:
-        val = (val == '[Function]' || val == '[Circular]')
+        val = (val === '[Function]' || val === '[Circular]')
           ? val
-          : JSON.stringify(val); //string
+          : JSON.stringify(val); // string
     }
     return val;
   }
 
-  for(var i in object) {
-    if(!object.hasOwnProperty(i)) continue;        // not my business
+  for (var i in object) {
+    if (!object.hasOwnProperty(i)) {
+      continue; // not my business
+    }
     --length;
     str += '\n ' + repeat(' ', space)
       + (isArray(object) ? '' : '"' + i + '": ') // key
-      +  _stringify(object[i])                   // value
+      + _stringify(object[i])                     // value
       + (length ? ',' : '');                     // comma
   }
 
-  return str + (str.length != 1                    // [], {}
-    ? '\n' + repeat(' ', --space) + end
-    : end);
+  return str
+    // [], {}
+    + (str.length !== 1 ? '\n' + repeat(' ', --space) + end : end);
 }
 
 /**
- * Return if obj is a Buffer
- * @param {Object} arg
- * @return {Boolean}
+ * Test if a value is a buffer.
+ *
  * @api private
+ * @param {*} value The value to test.
+ * @return {boolean} True if `value` is a buffer, otherwise false
  */
-exports.isBuffer = function (arg) {
-  return typeof Buffer !== 'undefined' && Buffer.isBuffer(arg);
+exports.isBuffer = function(value) {
+  return typeof Buffer !== 'undefined' && Buffer.isBuffer(value);
 };
 
 /**
- * @summary Return a new Thing that has the keys in sorted order.  Recursive.
- * @description If the Thing...
+ * Return a new Thing that has the keys in sorted order. Recursive.
+ *
+ * If the Thing...
  * - has already been seen, return string `'[Circular]'`
  * - is `undefined`, return string `'[undefined]'`
  * - is `null`, return value `null`
@@ -509,22 +534,23 @@ exports.isBuffer = function (arg) {
  * - is a non-empty `Array`, `Object`, or `Function`, return the result of calling this function again.
  * - is an empty `Array`, `Object`, or `Function`, return the result of calling `emptyRepresentation()`
  *
+ * @api private
+ * @see {@link exports.stringify}
  * @param {*} value Thing to inspect.  May or may not have properties.
  * @param {Array} [stack=[]] Stack of seen values
  * @return {(Object|Array|Function|string|undefined)}
- * @see {@link exports.stringify}
- * @api private
  */
-
 exports.canonicalize = function(value, stack) {
-  var canonicalizedObj,
-    type = exports.type(value),
-    prop,
-    withStack = function withStack(value, fn) {
-      stack.push(value);
-      fn();
-      stack.pop();
-    };
+  var canonicalizedObj;
+  /* eslint-disable no-unused-vars */
+  var prop;
+  /* eslint-enable no-unused-vars */
+  var type = exports.type(value);
+  function withStack(value, fn) {
+    stack.push(value);
+    fn();
+    stack.pop();
+  }
 
   stack = stack || [];
 
@@ -532,24 +558,26 @@ exports.canonicalize = function(value, stack) {
     return '[Circular]';
   }
 
-  switch(type) {
+  switch (type) {
     case 'undefined':
     case 'buffer':
     case 'null':
       canonicalizedObj = value;
       break;
     case 'array':
-      withStack(value, function () {
-        canonicalizedObj = exports.map(value, function (item) {
+      withStack(value, function() {
+        canonicalizedObj = exports.map(value, function(item) {
           return exports.canonicalize(item, stack);
         });
       });
       break;
     case 'function':
+      /* eslint-disable guard-for-in */
       for (prop in value) {
         canonicalizedObj = {};
         break;
       }
+      /* eslint-enable guard-for-in */
       if (!canonicalizedObj) {
         canonicalizedObj = emptyRepresentation(value, type);
         break;
@@ -557,8 +585,8 @@ exports.canonicalize = function(value, stack) {
     /* falls through */
     case 'object':
       canonicalizedObj = canonicalizedObj || {};
-      withStack(value, function () {
-        exports.forEach(exports.keys(value).sort(), function (key) {
+      withStack(value, function() {
+        exports.forEach(exports.keys(value).sort(), function(key) {
           canonicalizedObj[key] = exports.canonicalize(value[key], stack);
         });
       });
@@ -578,6 +606,12 @@ exports.canonicalize = function(value, stack) {
 
 /**
  * Lookup file names at the given `path`.
+ *
+ * @api public
+ * @param {string} path Base path to start searching from.
+ * @param {string[]} extensions File extensions to look for.
+ * @param {boolean} recursive Whether or not to recurse into subdirectories.
+ * @return {string[]} An array of paths.
  */
 exports.lookupFiles = function lookupFiles(path, extensions, recursive) {
   var files = [];
@@ -588,34 +622,40 @@ exports.lookupFiles = function lookupFiles(path, extensions, recursive) {
       path += '.js';
     } else {
       files = glob.sync(path);
-      if (!files.length) throw new Error("cannot resolve path (or pattern) '" + path + "'");
+      if (!files.length) {
+        throw new Error("cannot resolve path (or pattern) '" + path + "'");
+      }
       return files;
     }
   }
 
   try {
-    var stat = fs.statSync(path);
-    if (stat.isFile()) return path;
-  }
-  catch (ignored) {
+    var stat = statSync(path);
+    if (stat.isFile()) {
+      return path;
+    }
+  } catch (err) {
+    // ignore error
     return;
   }
 
-  fs.readdirSync(path).forEach(function(file) {
+  readdirSync(path).forEach(function(file) {
     file = join(path, file);
     try {
-      var stat = fs.statSync(file);
+      var stat = statSync(file);
       if (stat.isDirectory()) {
         if (recursive) {
           files = files.concat(lookupFiles(file, extensions, recursive));
         }
         return;
       }
+    } catch (err) {
+      // ignore error
+      return;
     }
-    catch (ignored) {
+    if (!stat.isFile() || !re.test(file) || basename(file)[0] === '.') {
       return;
     }
-    if (!stat.isFile() || !re.test(file) || basename(file)[0] === '.') return;
     files.push(file);
   });
 
@@ -643,62 +683,50 @@ exports.getError = function(err) {
   return err || exports.undefinedError();
 };
 
-
 /**
  * @summary
  * This Filter based on `mocha-clean` module.(see: `github.com/rstacruz/mocha-clean`)
  * @description
  * When invoking this function you get a filter function that get the Error.stack as an input,
  * and return a prettify output.
- * (i.e: strip Mocha, node_modules, bower and componentJS from stack trace).
+ * (i.e: strip Mocha and internal node functions from stack trace).
  * @returns {Function}
  */
-
 exports.stackTraceFilter = function() {
-  var slash = '/'
-    , is = typeof document === 'undefined'
-      ? { node: true }
-      : { browser: true }
-    , cwd = is.node
+  // TODO: Replace with `process.browser`
+  var slash = '/';
+  var is = typeof document === 'undefined' ? { node: true } : { browser: true };
+  var cwd = is.node
       ? process.cwd() + slash
-      : location.href.replace(/\/[^\/]*$/, '/');
+      : (typeof location === 'undefined' ? window.location : location).href.replace(/\/[^\/]*$/, '/');
 
-  function isNodeModule (line) {
-    return (~line.indexOf('node_modules'));
+  function isMochaInternal(line) {
+    return (~line.indexOf('node_modules' + slash + 'mocha' + slash))
+      || (~line.indexOf('components' + slash + 'mochajs' + slash))
+      || (~line.indexOf('components' + slash + 'mocha' + slash))
+      || (~line.indexOf(slash + 'mocha.js'));
   }
 
-  function isMochaInternal (line) {
-    return (~line.indexOf('node_modules' + slash + 'mocha'))  ||
-      (~line.indexOf('components' + slash + 'mochajs'))       ||
-      (~line.indexOf('components' + slash + 'mocha'));
-  }
-
-  // node_modules, bower, componentJS
-  function isBrowserModule(line) {
-    return (~line.indexOf('node_modules')) ||
-      (~line.indexOf('components'));
-  }
-
-  function isNodeInternal (line) {
-    return (~line.indexOf('(timers.js:')) ||
-      (~line.indexOf('(events.js:'))      ||
-      (~line.indexOf('(node.js:'))        ||
-      (~line.indexOf('(module.js:'))      ||
-      (~line.indexOf('GeneratorFunctionPrototype.next (native)')) ||
-      false
+  function isNodeInternal(line) {
+    return (~line.indexOf('(timers.js:'))
+      || (~line.indexOf('(events.js:'))
+      || (~line.indexOf('(node.js:'))
+      || (~line.indexOf('(module.js:'))
+      || (~line.indexOf('GeneratorFunctionPrototype.next (native)'))
+      || false;
   }
 
   return function(stack) {
     stack = stack.split('\n');
 
     stack = exports.reduce(stack, function(list, line) {
-      if (is.node && (isNodeModule(line) ||
-        isMochaInternal(line) ||
-        isNodeInternal(line)))
+      if (isMochaInternal(line)) {
         return list;
+      }
 
-      if (is.browser && (isBrowserModule(line)))
+      if (is.node && isNodeInternal(line)) {
         return list;
+      }
 
       // Clean up cwd(absolute)
       list.push(line.replace(cwd, ''));
@@ -706,5 +734,5 @@ exports.stackTraceFilter = function() {
     }, []);
 
     return stack.join('\n');
-  }
-};
+  };
+};