feat(@schematics/angular): add refactor-fake-async migration

[yjaaidi]

PR Checklist

Please check to confirm your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

There is no migration from fakeAsync to Vitest fake timer APIs.

Issue Number: N/A

What is the new behavior?

The current migration transforms the following test:

it("should work", fakeAsync(() => {
  let value = 0;

  setTimeout(() => value = 42, 100);

  tick(100);
  flush();

  expect(value).toBe(42);
}));

into:

import { onTestFinished, vi } from 'vitest';

it("should work", async () => {
  vi.useFakeTimers();
  onTestFinished(() => {
    vi.useRealTimers();
  });  

  let value = 0;

  setTimeout(() => value = 42, 100);

  await vi.advanceTimersByTimeAsync(100);
  await vi.runAllTimersAsync();

  expect(value).toBe(42);
});

Does this PR introduce a breaking change?

• Yes
• No

Open Questions

• This migration could be part of the jasmine-vitest migration. Should I moved it there and add an opt-out?
• flushMicrotasks can only be partially reproduced (i.e. explicit queueMicrotask calls), transforming it have high chances of producing a broken test. Should we just skip transforming tests that use it?
• Should we generate code that measures time "manually" when the return value of flush is used?
• Should we generate an "ad-hoc" function in the test file that reproduces flush's maxTurns option?
• What should we do about processNewMacroTasksSynchronously option?
• This only migrates flush and tick if used inside fakeAsync but it's probably better to just replace in the whole file as users might create some wrappers. What do you think?

[gemini-code-assist]

Code Review

This pull request introduces a new experimental schematic, refactor-fake-async, to migrate Angular's fakeAsync tests to use Vitest's fake timer APIs. The changes include the schematic implementation, transformers for fakeAsync, tick, and flush, and corresponding tests. Additionally, the findTestFiles utility function has been refactored into a shared location.

My review focuses on improving the robustness of the new transformers. I've suggested changes to ensure that the necessary vitest imports are always added when tick() or flush() are transformed, which will prevent potential issues with broken code after migration.