using legacy fake timers (#23110)

gracepark · web-flow · commit ed2495ed9778 · 2021-11-29T18:40:31.000Z
diff --git a/tests/browser/browser.js b/tests/browser/browser.js
@@ -2,7 +2,7 @@ import { jest } from '@jest/globals'
 import { latest } from '../../lib/enterprise-server-releases.js'
 import languages from '../../lib/languages.js'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 /* global page, browser */
 describe('homepage', () => {
diff --git a/tests/rendering/events.js b/tests/rendering/events.js
@@ -4,7 +4,7 @@ import cheerio from 'cheerio'
 import createApp from '../../lib/app.js'
 import { jest } from '@jest/globals'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('POST /events', () => {
   jest.setTimeout(60 * 1000)
diff --git a/tests/rendering/head.js b/tests/rendering/head.js
@@ -2,7 +2,7 @@ import { getDOM } from '../helpers/supertest.js'
 import languages from '../../lib/languages.js'
 import { jest } from '@jest/globals'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('<head>', () => {
   jest.setTimeout(5 * 60 * 1000)
diff --git a/tests/rendering/server.js b/tests/rendering/server.js
@@ -12,7 +12,7 @@ const activeProducts = Object.values(productMap).filter(
   (product) => !product.wip && !product.hidden
 )
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('server', () => {
   jest.setTimeout(60 * 1000)
diff --git a/tests/routing/deprecated-enterprise-versions.js b/tests/routing/deprecated-enterprise-versions.js
@@ -4,7 +4,7 @@ import { get, getDOM } from '../helpers/supertest.js'
 import supertest from 'supertest'
 import { jest } from '@jest/globals'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 const app = createApp()
 
diff --git a/tests/routing/developer-site-redirects.js b/tests/routing/developer-site-redirects.js
@@ -10,7 +10,7 @@ const developerRedirectFixtures = readJsonFile('./tests/fixtures/developer-redir
 
 const MAX_CONCURRENT_REQUESTS = 50
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('developer redirects', () => {
   jest.setTimeout(4 * 60 * 1000)
diff --git a/tests/routing/release-notes.js b/tests/routing/release-notes.js
@@ -1,7 +1,7 @@
 import { get, getDOM } from '../helpers/supertest.js'
 import { jest } from '@jest/globals'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('release notes', () => {
   jest.setTimeout(60 * 1000)
diff --git a/tests/routing/top-developer-site-path-redirects.js b/tests/routing/top-developer-site-path-redirects.js
@@ -4,7 +4,7 @@ import { jest } from '@jest/globals'
 
 const topOldDeveloperSitePaths = readJsonFile('tests/fixtures/top-old-developer-site-paths.json')
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('developer.github.com redirects', () => {
   jest.setTimeout(30 * 60 * 1000)
diff --git a/tests/unit/early-access.js b/tests/unit/early-access.js
@@ -5,7 +5,7 @@ import { testViaActionsOnly } from '../helpers/conditional-runs.js'
 import { getDOM } from '../helpers/supertest.js'
 import got from 'got'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('cloning early-access', () => {
   testViaActionsOnly('the content directory exists', async () => {
diff --git a/tests/unit/products.js b/tests/unit/products.js
@@ -5,7 +5,7 @@ import schema from '../helpers/schemas/products-schema.js'
 import { getDOM, getJSON } from '../helpers/supertest.js'
 import nonEnterpriseDefaultVersion from '../../lib/non-enterprise-default-version.js'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('products module', () => {
   test('is an object with product ids as keys', () => {
diff --git a/tests/unit/versions.js b/tests/unit/versions.js
@@ -6,7 +6,7 @@ import schema from '../helpers/schemas/versions-schema.js'
 import { getJSON } from '../helpers/supertest.js'
 import nonEnterpriseDefaultVersion from '../../lib/non-enterprise-default-version.js'
 
-jest.useFakeTimers()
+jest.useFakeTimers('legacy')
 
 describe('versions module', () => {
   test('is an object with versions as keys', () => {

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ const activeProducts = Object.values(productMap).filter(`
`12`	`12`	`(product) => !product.wip && !product.hidden`
`13`	`13`	`)`
`14`	`14`
`15`		`-jest.useFakeTimers()`
	`15`	`+jest.useFakeTimers('legacy')`
`16`	`16`
`17`	`17`	`describe('server', () => {`
`18`	`18`	`jest.setTimeout(60 * 1000)`