Adding test matrix entry using core-js/modules/es.error.cause

Author: Lucas-C

.github/workflows/ci.yml

@@ -18,6 +18,7 @@ jobs:
         - io.js 3.x
         - Node.js 4.x
         - Node.js 5.x
+        - Node.js 5.x with core-js polyfill
         - Node.js 6.x
         - Node.js 7.x
         - Node.js 8.x
@@ -63,6 +64,12 @@ jobs:
           node-version: "5.12"
           npm-i: browserify@15 [email protected] [email protected]
 
+        - name: Node.js 5.x with core-js polyfill
+          node-version: "5.12"
+          npm-i: browserify@15 [email protected] [email protected]
+          env:
+            ENABLE_COREJS_ERROR_POLYFILL: true
+
         - name: Node.js 6.x
           node-version: "6.17"
           npm-i: browserify@15 [email protected] [email protected]
@@ -158,6 +165,7 @@ jobs:
 
     - name: Run tests
       shell: bash
+      env: ${{ matrix.env }}
       run: |
         if npm -ps ls nyc | grep -q nyc; then
           npm run test-ci

index.js

@@ -265,25 +265,22 @@ function log (message, site) {
  */
 
 function callSiteLocation (callSite) {
-  var functionName, site
   if (callSite) {
-    functionName = callSite.getFunctionName()
     var file = callSite.getFileName() || '<anonymous>'
     var line = callSite.getLineNumber()
     var colm = callSite.getColumnNumber()
     if (callSite.isEval()) {
       file = callSite.getEvalOrigin() + ', ' + file
     }
-    site = [file, line, colm]
+    var site = [file, line, colm]
+    site.callSite = callSite
+    site.name = callSite.getFunctionName()
   } else {
-    // eslint-disable-next-line no-param-reassign
-    callSite = {}
-    callSite.getThis = function () { return null }
-    functionName = '<unknown function>'
-    site = ['<unknown file>', '<unknown line>', '<unknown column>']
+    var site = ['<unknown file>', '<unknown line>', '<unknown column>']
+    site.callSite = {}
+    site.callSite.getThis = function () { return null }
+    site.name = '<unknown function>'
   }
-  site.callSite = callSite
-  site.name = functionName
   return site
 }
 

package.json

@@ -11,17 +11,18 @@
   "repository": "dougwilson/nodejs-depd",
   "browser": "lib/browser/index.js",
   "devDependencies": {
-    "benchmark": "2.1.4",
     "beautify-benchmark": "0.2.4",
+    "benchmark": "2.1.4",
+    "core-js": "^3.21.1",
     "eslint": "7.21.0",
     "eslint-config-standard": "14.1.1",
     "eslint-plugin-import": "2.22.1",
     "eslint-plugin-markdown": "1.0.2",
     "eslint-plugin-node": "9.2.0",
     "eslint-plugin-promise": "4.2.1",
     "eslint-plugin-standard": "4.1.0",
-    "nyc": "15.1.0",
     "mocha": "8.2.1",
+    "nyc": "15.1.0",
     "safe-buffer": "5.1.2"
   },
   "files": [

test/test.js

@@ -10,6 +10,10 @@ var script = path.join(__dirname, 'fixtures', 'script.js')
 var spawn = require('child_process').spawn
 var strictlib = libs.strict
 
+if (process.env.ENABLE_COREJS_ERROR_POLYFILL) { // cf. PR #48
+  require('core-js/modules/es.error.cause')
+}
+
 describe('depd(namespace)', function () {
   it('creates deprecated function', function () {
     assert.strictEqual(typeof depd('test'), 'function')