refactor(lib/utils.js): Replace hasMatchingExtname implementation

Swapped out the `RegExp.test` implementation for `Array.some` one, providing ~3.5x speedup.
mochajs · Feb 13, 2019 · 6f96594 · 6f96594
1 parent b82d65d
commit 6f96594
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -528,8 +528,10 @@ exports.canonicalize = function canonicalize(value, stack, typeHint) {
  * hasMatchingExtname('foo.html', ['js', 'css']); // => false
  */
 function hasMatchingExtname(pathname, exts) {
-  var re = new RegExp('\\.(?:' + exts.join('|') + ')$');
-  return re.test(path.extname(pathname));
+  var suffix = path.extname(pathname).slice(1);
+  return exts.some(function(element) {
+    return suffix === element;
+  });
 }
 
 /**