Fake timers behave unexpectedly with `Promise.race()`.

🐛 Bug Report

With jest.useFakeTimers (either ‘modern’ or ‘legacy’), a Promise.race between a short sleep Promise and a long sleep Promise…resolves to the long one’s resolution, if the short one uses .then.

An identical Promise.race using real timers behaves as expected.

To Reproduce

For example, with modern fake timers (but also observed with the legacy fake timers):

describe('modern fake timers', () => {
  jest.useFakeTimers('modern');

  test('Promise.race without "then"', async () => {
    const shortPromise = new Promise(resolve => setTimeout(() => resolve('short'), 1000));
    const longPromise = new Promise(resolve => setTimeout(() => resolve('long'), 2000));
    
    const racedPromise = Promise.race([shortPromise, longPromise]);

    jest.runAllTimers();
    const result = await racedPromise;

    expect(result).toBe('short');
  });

  test('Promise.race with "then"', async () => {
    const shortPromise = new Promise(resolve => setTimeout(resolve, 1000)).then(() => 'short');
    const longPromise = new Promise(resolve => setTimeout(() => resolve('long'), 2000));
    
    const racedPromise = Promise.race([shortPromise, longPromise]);

    jest.runAllTimers();
    const result = await racedPromise;
    
    // Surprise! It's 'long'.
    expect(result).toBe('short');
  });
});

The second test, ‘Promise.race with “then”’, fails.

Expected behavior

I would expect the second test, ‘Promise.race with “then”’, to succeed; the shorter sleep should “win” the race, same as the version without “then”. An identical test succeeds if you do jest.useRealTimers().

Link to repl or repo (highly encouraged)

Here, we have with-“then” and without-“then” tests for each of the following:

• Modern fake timers (surprising behavior observed)
• Legacy fake timers (surprising behavior observed)
• Real timers (behaves as expected)

https://github.com/chrisbobbe/jest-modern-timers-race-then

envinfo

  System:
    OS: macOS 10.15.5
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
  Binaries:
    Node: 10.20.1 - ~/.nvm/versions/node/v10.20.1/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.4 - ~/.nvm/versions/node/v10.20.1/bin/npm
  npmPackages:
    jest: ^26.1.0 => 26.1.0 

[MaxMotovilov]

Running into similar issues with multiple timers and promises. Wouldn’t it help to have an asynchronous version of jest.runAllTimers() – i.e. one that would setImmediate() between timer firings? This assumes promise implementation uses an equivalent of nextTick() to schedule the callback execution. Basically we need to be sure that all promise chains resolvable from a given timer would fully resolve themselves before the logically next (i.e. set to a later time) timer would fire.

By the looks of it, got to be a one-liner – @sinonjs/faketimers runAllAsync() is already available but not used by Jest.

https://github.com/facebook/jest/blob/412954fd260f17976a450bd3b252a5d7e2488a64/packages/jest-fake-timers/src/modernFakeTimers.ts#L50

Can either add runAllTimersAsync() or a boolean async parameter to the existing call… For the time being, perhaps

jest._clock.runAllAsync()

could be used as a kludge.

[github-actions[bot]]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.