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,11 @@ 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] core-js@3
+          ENABLE_COREJS_ERROR_POLYFILL: true
+
         - name: Node.js 6.x
           node-version: "6.17"
           npm-i: browserify@15 [email protected] [email protected]
@@ -158,6 +164,8 @@ jobs:
 
     - name: Run tests
       shell: bash
+      env:
+        ENABLE_COREJS_ERROR_POLYFILL: ${{ matrix.ENABLE_COREJS_ERROR_POLYFILL }}
       run: |
         if npm -ps ls nyc | grep -q nyc; then
           npm run test-ci

index.js

@@ -265,25 +265,23 @@ function log (message, site) {
  */
 
 function callSiteLocation (callSite) {
-  var functionName, site
+  var 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]
+    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>']
+    site.callSite = {}
+    site.callSite.getThis = function () { return null }
+    site.name = '<unknown function>'
   }
-  site.callSite = callSite
-  site.name = functionName
   return site
 }
 

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')