test: use runAllTimers when running fakeTimers

Author: WillianAgostini

e2e/__tests__/__snapshots__/testRetries.test.ts.snap

@@ -37,3 +37,41 @@ exports[`Test Retries logs error(s) before retry 1`] = `
 PASS __tests__/logErrorsBeforeRetries.test.js
   ✓ retryTimes set"
 `;
+
+exports[`Test Retries logs error(s) before retry 2`] = `
+"LOGGING RETRY ERRORS  retryTimes set with fake timers
+ RETRY 1 
+
+    expect(received).toBeFalsy()
+
+    Received: true
+
+      18 |     );
+      19 |   } else {
+    > 20 |     expect(true).toBeFalsy();
+         |                  ^
+      21 |     jest.runAllTimers();
+      22 |   }
+      23 | });
+
+      at Object.toBeFalsy (__tests__/logErrorsBeforeRetriesFakeTimers.test.js:20:18)
+
+ RETRY 2 
+
+    expect(received).toBeFalsy()
+
+    Received: true
+
+      18 |     );
+      19 |   } else {
+    > 20 |     expect(true).toBeFalsy();
+         |                  ^
+      21 |     jest.runAllTimers();
+      22 |   }
+      23 | });
+
+      at Object.toBeFalsy (__tests__/logErrorsBeforeRetriesFakeTimers.test.js:20:18)
+
+PASS __tests__/logErrorsBeforeRetriesFakeTimers.test.js
+  ✓ retryTimes set with fake timers"
+`;

e2e/__tests__/testRetries.test.ts

@@ -42,6 +42,14 @@ describe('Test Retries', () => {
     expect(extractSummary(result.stderr).rest).toMatchSnapshot();
   });
 
+  it('logs error(s) before retry', () => {
+    const result = runJest('test-retries', ['logErrorsBeforeRetriesFakeTimers.test.js']);
+    expect(result.exitCode).toBe(0);
+    expect(result.failed).toBe(false);
+    expect(result.stderr).toContain(logErrorsBeforeRetryErrorMessage);
+    expect(extractSummary(result.stderr).rest).toMatchSnapshot();
+  });
+
   it('reporter shows more than 1 invocation if test is retried', () => {
     let jsonResult;
 

e2e/test-retries/__tests__/logErrorsBeforeRetries.test.js

@@ -17,18 +17,3 @@ it('retryTimes set', () => {
     expect(true).toBeFalsy();
   }
 });
-
-let iFakeTimers = 0;
-const startTimeInSecondsFakeTimers = new Date().getTime();
-jest.retryTimes(3, {logErrorsBeforeRetry: true, waitBeforeRetry: 100});
-it('retryTimes set with fake timers', () => {
-  jest.useFakeTimers();
-  iFakeTimers++;
-  if (iFakeTimers === 3) {
-    expect(new Date().getTime() - startTimeInSecondsFakeTimers).toBeGreaterThan(
-      200,
-    );
-  } else {
-    expect(true).toBeFalsy();
-  }
-});

e2e/test-retries/__tests__/logErrorsBeforeRetriesFakeTimers.test.js

@@ -0,0 +1,23 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+'use strict';
+
+let i = 0;
+const startTimeInSeconds = new Date().getTime();
+jest.retryTimes(3, {logErrorsBeforeRetry: true, waitBeforeRetry: 100});
+it('retryTimes set with fake timers', () => {
+  jest.useFakeTimers();
+  i++;
+  if (i === 3) {
+    expect(new Date().getTime() - startTimeInSeconds).toBeGreaterThan(
+      200,
+    );
+  } else {
+    expect(true).toBeFalsy();
+    jest.runAllTimers();
+  }
+});