don't use Symbol for our promise

SimenB · Oct 20, 2018 · 48da087 · 48da087
1 parent 493f7ee
commit 48da087
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/packages/jest-util/src/install_common_globals.js b/packages/jest-util/src/install_common_globals.js
@@ -19,7 +19,12 @@ export default function(globalObject: Global, globals: ConfigGlobals) {
   globalObject.process = createProcessObject();
 
   // Keep a reference to "Promise", since "jasmine_light.js" needs it.
-  globalObject[globalObject.Symbol.for('jest-native-promise')] = Promise;
+  Object.defineProperty(globalObject, 'jest-promise-stay-away', {
+    configurable: false,
+    enumerable: false,
+    value: Promise,
+    writable: false,
+  });
 
   // Forward some APIs.
   DTRACE.forEach(dtrace => {

diff --git a/scripts/babel-plugin-jest-native-promise.js b/scripts/babel-plugin-jest-native-promise.js
@@ -10,8 +10,7 @@
 // This plugin exists to make sure that we use a `Promise` that has not been messed with by user code.
 // Might consider extending this to other globals as well in the future
 
-const jestPromise =
-  "(global[Symbol.for('jest-native-promise')] || global.Promise)";
+const jestPromise = "(global['jest-promise-stay-away'] || global.Promise)";
 
 module.exports = () => ({
   name: 'jest-native-promise',