treat modern timers as default, but be explicit in tests

Author: thymikee

src/__tests__/auto-cleanup.js

@@ -33,13 +33,13 @@ test('component is automatically umounted after first test ends', () => {
   expect(isMounted).toEqual(false);
 });
 
-test('does not time out with fake timers', () => {
-  jest.useFakeTimers();
+test('does not time out with legacy fake timers', () => {
+  jest.useFakeTimers('legacy');
   render(<Test />);
   expect(isMounted).toEqual(true);
 });
 
-test('does not time out with modern fake timers', () => {
+test('does not time out with fake timers', () => {
   jest.useFakeTimers('modern');
   render(<Test />);
   expect(isMounted).toEqual(true);

src/__tests__/waitFor.test.js

@@ -79,8 +79,8 @@ test('waits for element with custom interval', async () => {
   expect(mockFn).toHaveBeenCalledTimes(3);
 });
 
-test('works with fake timers', async () => {
-  jest.useFakeTimers();
+test('works with legacy fake timers', async () => {
+  jest.useFakeTimers('legacy');
 
   const mockFn = jest.fn(() => {
     throw Error('test');
@@ -96,7 +96,7 @@ test('works with fake timers', async () => {
   expect(mockFn).toHaveBeenCalledTimes(3);
 });
 
-test('works with modern fake timers', async () => {
+test('works with fake timers', async () => {
   jest.useFakeTimers('modern');
 
   const mockFn = jest.fn(() => {

src/__tests__/waitForElementToBeRemoved.test.js

@@ -130,8 +130,8 @@ test('waits with custom interval', async () => {
   expect(mockFn).toHaveBeenCalledTimes(4);
 });
 
-test('works with fake timers', async () => {
-  jest.useFakeTimers();
+test('works with legacy fake timers', async () => {
+  jest.useFakeTimers('legacy');
 
   const mockFn = jest.fn(() => <View />);
 
@@ -144,7 +144,7 @@ test('works with fake timers', async () => {
   expect(mockFn).toHaveBeenCalledTimes(4);
 });
 
-test('works with modern fake timers', async () => {
+test('works with fake timers', async () => {
   jest.useFakeTimers('modern');
 
   const mockFn = jest.fn(() => <View />);