fix: [#2022] Make DetachedWindowAPI abort() test deterministic

[TrevorBurnham]

Fixes #2022

Problem

The test DetachedWindowAPI > abort() > Cancels all ongoing asynchrounous tasks was flaky in CI, timing out with a 500ms limit. The test used setTimeout(..., 10) to check if waitUntilComplete() promises had resolved, which is unreliable in CI environments under load.

// Before (flaky)
setTimeout(() => {
    expect(isFirstWhenAsyncCompleteCalled).toBe(true);
    expect(isSecondWhenAsyncCompleteCalled).toBe(true);
    resolve(null);
}, 10);

Solution

Replaced the timing-based assertion with direct promise awaiting:

// After (deterministic)
await Promise.all([firstWaitPromise, secondWaitPromise]);

Changes:

• Directly await the waitUntilComplete() promises using Promise.all()
• Remove the manual Promise wrapper and setTimeout-based assertion
• Keep the synchronous tasksDone check immediately after abort()