Revert "don't use Symbol for our promise"

This reverts commit 48da087.
jestjs · Oct 20, 2018 · bdc4883 · bdc4883
1 parent 5f8754c
commit bdc4883
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/packages/jest-util/src/install_common_globals.js b/packages/jest-util/src/install_common_globals.js
@@ -19,12 +19,7 @@ export default function(globalObject: Global, globals: ConfigGlobals) {
   globalObject.process = createProcessObject();
 
   // Keep a reference to "Promise", since "jasmine_light.js" needs it.
-  Object.defineProperty(globalObject, 'jest-promise-stay-away', {
-    configurable: false,
-    enumerable: false,
-    value: Promise,
-    writable: false,
-  });
+  globalObject[globalObject.Symbol.for('jest-native-promise')] = Promise;
 
   // 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,7 +10,8 @@
 // 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['jest-promise-stay-away'] || global.Promise)";
+const jestPromise =
+  "(global[Symbol.for('jest-native-promise')] || global.Promise)";
 
 module.exports = () => ({
   name: 'jest-native-promise',