Convert project to an ES module (#28)

* Replace simple-assert with a throw

* Convert project to ES module

* Simplify karma config

* Remove unused dependencies

* Set `engines` key to node >= 12

* Remove `.travis.yml`

* Add back dependencies that are actually needed

* Update eslint and friends

* Remove obsolete build npm script

Co-authored-by: Keith Cirkel <keithamus@users.noreply.github.com>

Author: koddsson

.travis.yml

@@ -1,11 +1,3 @@
-'use strict';
-
-/* !
- * Chai - getFuncName utility
- * Copyright(c) 2012-2016 Jake Luer <jake@alogicalparadox.com>
- * MIT Licensed
- */
-
 /**
  * ### .getFuncName(constructorFn)
  *
@@ -19,26 +11,28 @@
  * @api public
  */
 
-var toString = Function.prototype.toString;
-var functionNameMatch = /\s*function(?:\s|\s*\/\*[^(?:*\/)]+\*\/\s*)*([^\s\(\/]+)/;
+const { toString } = Function.prototype;
+const functionNameMatch = /\s*function(?:\s|\s*\/\*[^(?:*/)]+\*\/\s*)*([^\s(/]+)/;
 function getFuncName(aFunc) {
   if (typeof aFunc !== 'function') {
     return null;
   }
 
-  var name = '';
+  let name = '';
   if (typeof Function.prototype.name === 'undefined' && typeof aFunc.name === 'undefined') {
     // Here we run a polyfill if Function does not support the `name` property and if aFunc.name is not defined
-    var match = toString.call(aFunc).match(functionNameMatch);
+    // eslint-disable-next-line prefer-reflect
+    const match = toString.call(aFunc).match(functionNameMatch);
     if (match) {
-      name = match[1];
+      [ name ] = match;
     }
   } else {
     // If we've got a `name` property we just use it
+    // eslint-disable-next-line prefer-destructuring
     name = aFunc.name;
   }
 
   return name;
 }
 
-module.exports = getFuncName;
+export { getFuncName };

index.js

@@ -0,0 +1,15 @@
+module.exports = function configureKarma(config) {
+  config.set({
+    basePath: '.',
+    browsers: [ 'ChromeHeadless' ],
+    frameworks: [ 'mocha' ],
+    files: [
+      { pattern: './index.js', type: 'module' },
+      { pattern: './test/*.js', type: 'module' },
+    ],
+    reporters: [ 'progress' ],
+    colors: true,
+    autoWatch: false,
+    singleRun: true,
+  });
+};

karma.conf.cjs

@@ -15,22 +15,20 @@
     "Aleksey Shvayka (https://github.com/shvaikalesh)"
   ],
   "files": [
-    "index.js",
-    "get-func-name.js"
+    "index.js"
   ],
+  "type": "module",
   "main": "./index.js",
   "repository": {
     "type": "git",
     "url": "git+ssh://git@github.com/chaijs/get-func-name.git"
   },
   "scripts": {
-    "build": "browserify --bare $npm_package_main --standalone getFuncName -o get-func-name.js",
     "lint": "eslint --ignore-path .gitignore .",
-    "prepublish": "npm run build",
     "semantic-release": "semantic-release pre && npm publish && semantic-release post",
     "pretest": "npm run lint",
     "test": "npm run test:node && npm run test:browser && npm run upload-coverage",
-    "test:browser": "karma start --singleRun=true",
+    "test:browser": "karma start karma.conf.cjs --singleRun=true",
     "test:node": "istanbul cover _mocha",
     "upload-coverage": "lcov-result-merger 'coverage/**/lcov.info' | coveralls; exit 0"
   },
@@ -41,7 +39,7 @@
   },
   "eslintConfig": {
     "extends": [
-      "strict/es5"
+      "strict/es6"
     ],
     "env": {
       "es6": true
@@ -51,33 +49,27 @@
     },
     "rules": {
       "complexity": 0,
-      "max-statements": 0
+      "max-statements": 0,
+      "prefer-arrow-callback": "off"
     }
   },
   "dependencies": {},
   "devDependencies": {
-    "browserify": "^13.0.0",
-    "browserify-istanbul": "^2.0.0",
-    "coveralls": "2.11.14",
-    "eslint": "^2.4.0",
-    "eslint-config-strict": "^9.1.0",
+    "coveralls": "^3.1.1",
+    "eslint": "^7.32.0",
+    "eslint-config-strict": "^14.0.1",
     "eslint-plugin-filenames": "^1.1.0",
-    "ghooks": "^1.0.1",
-    "istanbul": "^0.4.2",
-    "karma": "^1.3.0",
-    "karma-browserify": "^5.0.2",
-    "karma-coverage": "^1.1.1",
-    "karma-mocha": "^1.2.0",
-    "karma-phantomjs-launcher": "^1.0.0",
-    "karma-sauce-launcher": "^1.0.0",
-    "lcov-result-merger": "^1.0.2",
-    "mocha": "^3.1.2",
-    "phantomjs-prebuilt": "^2.1.5",
-    "semantic-release": "^4.3.5",
-    "travis-after-all": "^1.4.4",
+    "ghooks": "^2.0.4",
+    "istanbul": "^0.4.5",
+    "karma": "^6.3.4",
+    "karma-chrome-launcher": "^3.1.0",
+    "karma-mocha": "^2.0.1",
+    "lcov-result-merger": "^3.1.0",
+    "mocha": "^9.1.2",
+    "semantic-release": "^18.0.0",
     "validate-commit-msg": "^2.3.1"
   },
   "engines": {
-    "node": "*"
+    "node": ">= 12"
   }
 }

karma.conf.js

@@ -1,5 +1,4 @@
-'use strict';
-var getFuncName = require('..');
+import { getFuncName } from '../index.js';
 function assert(expr, msg) {
   if (!expr) {
     throw new Error(msg || 'Assertion Failed');
@@ -24,7 +23,7 @@ describe('getFuncName', function () {
   });
 
   it('should return empty string for anonymous functions', function () {
-    var anonymousFunc = (function () {
+    const anonymousFunc = (function () {
       return function () { // eslint-disable-line func-style
         return 2;
       };
@@ -54,7 +53,7 @@ describe('getFuncName', function () {
 
   it('should return `null` when passed a Symbol as argument', function () {
     if (typeof Symbol !== 'undefined') {
-      assert(getFuncName(Symbol()) === null);
+      assert(getFuncName(Symbol('symbol')) === null);
     }
   });