fake timers do not mock AbortSignal.timeout or timers/promises

[everett1992]

Describe the bug

mock timers do not mock node's timers/promises or AbortSignal.timeout

There is an open issue in sinon fake-timers to implement these.
sinonjs/fake-timers#418

Reproduction

https://stackblitz.com/edit/vitest-dev-vitest-5rmugo?file=test/basic.test.ts

describe('with fake timers', () => {
  beforeAll(() => {
    vi.useFakeTimers();
  });

  test('signal should abort after timeout', () => {
    const signal = AbortSignal.timeout(100);
    expect(signal.aborted).toBe(false);
    
    vi.advanceTimersByTime(200);
    
    // This will fail, aborted is still false.
    expect(signal.aborted).toBe(true);
  });
});

System Info

stackblitz

Used Package Manager

npm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[zbyte64]

Related: sinonjs/fake-timers#418

[LumaKernel]

Could mocking module "timers/promises" and "node:timers/promises" builtin solve the problem? Or waiting sinonJS' decision?

[everett1992]

I think we're waiting on sinon. In the mean time I've been mocking timers/promises with an implementation that delegates to global setTimeout so it's correctly mocked by useFakeTimers. I'm not sure how accurate my mock is so I won't bother sharing it here.

[sheremet-va]

I am closing the issue here since Vitest cannot do anything to resolve this unless @sinonjs/fake-timers implement the correct API.