fix: runWithRealTimers to be compatible with new version of jest (#649)

Closes #612

Author: marcosvega91

src/__tests__/helpers.js

@@ -1,4 +1,13 @@
-import {getDocument, getWindowFromNode, checkContainerType} from '../helpers'
+import {
+  getDocument,
+  getWindowFromNode,
+  checkContainerType,
+  runWithRealTimers,
+} from '../helpers'
+
+const globalObj = typeof window === 'undefined' ? global : window
+
+afterEach(() => jest.useRealTimers())
 
 test('returns global document if exists', () => {
   expect(getDocument()).toBe(document)
@@ -43,3 +52,19 @@ describe('query container validation throws when validation fails', () => {
     )
   })
 })
+
+test('should always use realTimers before using callback', () => {
+  const originalSetTimeout = globalObj.setTimeout
+
+  jest.useFakeTimers('legacy')
+  runWithRealTimers(() => {
+    expect(originalSetTimeout).toEqual(globalObj.setTimeout)
+  })
+
+  jest.useRealTimers()
+
+  jest.useFakeTimers('modern')
+  runWithRealTimers(() => {
+    expect(originalSetTimeout).toEqual(globalObj.setTimeout)
+  })
+})

src/helpers.js

@@ -4,7 +4,8 @@ const globalObj = typeof window === 'undefined' ? global : window
 function runWithRealTimers(callback) {
   const usingJestFakeTimers =
     globalObj.setTimeout &&
-    globalObj.setTimeout._isMockFunction &&
+    (globalObj.setTimeout._isMockFunction ||
+      typeof globalObj.setTimeout.clock !== 'undefined') &&
     typeof jest !== 'undefined'
 
   if (usingJestFakeTimers) {