Rebuild with additional Fix #22 for IE11

t2ym · Jan 9, 2017 · 40e1a00 · 40e1a00
1 parent 131bb7b
commit 40e1a00
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 5 deletions.
diff --git a/Suite.es5.js b/Suite.es5.js
@@ -113,7 +113,10 @@ Copyright (c) 2016, Tetsuya Mori <t2y3141592@gmail.com>. All rights reserved.
       key: '_checkIdentifier',
       value: function _checkIdentifier(name) {
         try {
-          new Function('return function ' + name + ' () {}')();
+          var f = new Function('return function ' + name + ' () {}')();
+          if (f.name !== name || !f.name.match(/^[$\w]*$/)) {
+            throw new Error(name + ' is defined as ' + f.name);
+          }
         } catch (e) {
           throw new Error(this.constructor.name + '.' + this.scope + ':_checkIdentifier ' + name + ' is not a valid identifier ' + e.message);
         }

diff --git a/Suite.es6.js b/Suite.es6.js
@@ -61,7 +61,10 @@ class Suite {
   }
   _checkIdentifier(name) {
     try {
-      (new Function('return function ' + name + ' () {}'))();
+      let f = (new Function('return function ' + name + ' () {}'))();
+      if (f.name !== name || !f.name.match(/^[$\w]*$/)) {
+        throw new Error(name + ' is defined as ' + f.name);
+      }
     }
     catch (e) {
       throw new Error(this.constructor.name + '.' + this.scope + ':_checkIdentifier ' + name + ' is not a valid identifier ' + e.message);

diff --git a/Suite.js b/Suite.js
@@ -91,7 +91,10 @@ class Suite {
   }
   _checkIdentifier(name) {
     try {
-      (new Function('return function ' + name + ' () {}'))();
+      let f = (new Function('return function ' + name + ' () {}'))();
+      if (f.name !== name || !f.name.match(/^[$\w]*$/)) {
+        throw new Error(name + ' is defined as ' + f.name);
+      }
     }
     catch (e) {
       throw new Error(this.constructor.name + '.' + this.scope + ':_checkIdentifier ' + name + ' is not a valid identifier ' + e.message);