Bug: Polyfill is always enabled.

rousan#2
garry-kueez · Jan 26, 2020 · ea915d1 · ea915d1
1 parent 0d33ba8
commit ea915d1
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 5 deletions.
diff --git a/dist/symbol-es6.min.js b/dist/symbol-es6.min.js
diff --git a/dist/symbol-es6.min.js.map b/dist/symbol-es6.min.js.map
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "symbol-es6",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "ES6 Symbol polyfill in pure ES5",
   "main": "symbol-es6.js",
   "scripts": {

diff --git a/symbol-es6.js b/symbol-es6.js
@@ -38,6 +38,8 @@
 
     var ES6 = typeof global.ES6 === "object" ? global.ES6 : (global.ES6 = {});
 
+    var isTestMode = global.symbolES6TestingMode || false;
+
     var isArray = Array.isArray;
 
     var objectToString = Object.prototype.toString;
@@ -73,7 +75,19 @@
     };
 
     var isES6Running = function() {
-        return false; /* Now 'false' for testing purpose */
+        if (isTestMode)
+            return false;
+
+        var isSymbolSupported = false;
+        if (typeof Symbol === "function") {
+            try {
+                Symbol();
+                isSymbolSupported = true;
+            } catch (ignore) {}
+        } else {
+            isSymbolSupported = false;
+        }
+        return isSymbolSupported; /* Now 'false' for testing purpose */
     };
 
     var isObject = function (value) {

diff --git a/test/index.html b/test/index.html
@@ -3,6 +3,7 @@
 <head>
     <meta charset="UTF-8">
     <title>Title</title>
+    <script>window.symbolES6TestingMode = true;</script>
     <script src="../symbol-es6.js"></script>
 </head>
 <body>

diff --git a/test/test.js b/test/test.js
@@ -1,5 +1,5 @@
 "use strict";
-
+global.symbolES6TestingMode = true
 var ES6 = require("../symbol-es6");
 
 console.log(Symbol("bar") === Symbol("bar")); //false