Make withResetModules "not async"

packages/jest-runtime/src/__tests__/runtime_require_module_or_mock.test.js

@@ -201,9 +201,9 @@ describe('withResetModules', () => {
   it('resets all modules after the block', async () =>
     createRuntime(__filename, {
       moduleNameMapper,
-    }).then(async runtime => {
+    }).then(runtime => {
       let exports;
-      await runtime.withResetModules(() => {
+      runtime.withResetModules(() => {
         exports = runtime.requireModuleOrMock(
           runtime.__mockRootPath,
           'ModuleWithState',
@@ -223,9 +223,9 @@ describe('withResetModules', () => {
   it('can call resetModules within a withResetModules block', async () =>
     createRuntime(__filename, {
       moduleNameMapper,
-    }).then(async runtime => {
+    }).then(runtime => {
       let exports;
-      await runtime.withResetModules(() => {
+      runtime.withResetModules(() => {
         exports = runtime.requireModuleOrMock(
           runtime.__mockRootPath,
           'ModuleWithState',

packages/jest-runtime/src/index.js

@@ -428,9 +428,9 @@ class Runtime {
     }
   }
 
-  async withResetModules(fn: () => Promise<*> | void) {
+  withResetModules(fn: () => void) {
     this._sandboxRegistries.unshift(Object.create(null));
-    await fn();
+    fn();
     this._sandboxRegistries.shift();
   }
 
@@ -892,8 +892,8 @@ class Runtime {
       this.resetModules();
       return jestObject;
     };
-    const withResetModules = async (fn: () => Promise<*>) => {
-      await this.withResetModules(fn);
+    const withResetModules = (fn: () => void) => {
+      this.withResetModules(fn);
       return jestObject;
     };
     const fn = this._moduleMocker.fn.bind(this._moduleMocker);

types/Jest.js

@@ -49,5 +49,5 @@ export type Jest = {|
   unmock(moduleName: string): Jest,
   useFakeTimers(): Jest,
   useRealTimers(): Jest,
-  withResetModules(fn: () => Promise<*>): Promise<Jest>,
+  withResetModules(fn: () => void): Jest,
 |};