Merge pull request karma-runner#2951 from johnjbarton/fix-2942

fix(common): Proxy function toString does not contain Proxy.
briandipalma · May 2, 2018 · a6577d9 · a6577d9
2 parents bc1453e + 4fb3484
commit a6577d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/common/stringify.js b/common/stringify.js
@@ -25,8 +25,8 @@ var stringify = function stringify (obj, depth) {
         return obj.toString().replace(/\{[\s\S]*\}/, '{ ... }')
       } catch (err) {
         if (err instanceof TypeError) {
-          // Proxy(function abc(...) { ... })
-          return 'Proxy(function ' + (obj.name || '') + '(...) { ... })'
+          // Support older browsers
+          return 'function ' + (obj.name || '') + '() { ... }'
         } else {
           throw err
         }

diff --git a/test/client/stringify.spec.js b/test/client/stringify.spec.js
@@ -40,11 +40,8 @@ describe('stringify', function () {
   //   http://caniuse.com/#feat=proxy
   if (window.Proxy) {
     it('should serialize proxied functions', function () {
-      var abcProxy = new Proxy(function abc (a, b, c) { return 'whatever' }, {})
       var defProxy = new Proxy(function (d, e, f) { return 'whatever' }, {})
-
-      assert.deepEqual(stringify(abcProxy), 'Proxy(function abc(...) { ... })')
-      assert.deepEqual(stringify(defProxy), 'Proxy(function (...) { ... })')
+      assert.deepEqual(stringify(defProxy), 'function () { ... }')
     })
   }