Clear timers when disposing an environment. (#1242)

Author: cpojer

integration_tests/__tests__/set-immediate-test.js

@@ -0,0 +1,23 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the BSD-style license found in the
+ * LICENSE file in the root directory of this source tree. An additional grant
+ * of patent rights can be found in the PATENTS file in the same directory.
+ *
+ * @emails oncall+jsinfra
+ */
+'use strict';
+
+const runJest = require('../runJest');
+
+test('setImmediate', () => {
+  const result = runJest('set_immediate', ['--verbose']);
+  const stdout = result.stdout.toString();
+  const stderr = result.stderr.toString();
+
+  expect(stdout).toMatch('setImmediate test');
+  expect(stderr).toBe('');
+
+  expect(result.status).toBe(0);
+});

integration_tests/set_immediate/__tests__/set-immediate-test.js

@@ -0,0 +1,17 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the BSD-style license found in the
+ * LICENSE file in the root directory of this source tree. An additional grant
+ * of patent rights can be found in the PATENTS file in the same directory.
+ */
+
+'use strict';
+
+test('setImmediate test', () => {
+  expect(true).toBe(true);
+
+  setImmediate(() => {
+    throw new Error('Scheduled Error');
+  });
+});

integration_tests/set_immediate/package.json

@@ -0,0 +1 @@
+{}

packages/jest-environment-jsdom/src/index.js

@@ -34,6 +34,9 @@ class JSDOMEnvironment {
   }
 
   dispose(): void {
+    if (this.fakeTimers) {
+      this.fakeTimers.clearAllTimers();
+    }
     if (this.global) {
       this.global.close();
     }

packages/jest-environment-node/src/index.js

@@ -39,6 +39,9 @@ class NodeEnvironment {
   }
 
   dispose() {
+    if (this.fakeTimers) {
+      this.fakeTimers.clearAllTimers();
+    }
     this.global = null;
     this.fakeTimers = null;
   }