Description

• related
  • fix(runner): long synchronous tasks does not time out (fix #2920) #6944
  • fix: don't await an empty timeout after every test #7281

Reproduction

import { beforeAll, test } from 'vitest';

beforeAll(() => {
  const start = Date.now();
  while (Date.now() < start + 200) {}
}, 100);

test('ok', () => {});

• timeout on v3.0.1 (expected) https://stackblitz.com/edit/vitest-dev-vitest-gntxziwv?file=test%2Frepro.test.ts
• no timeout on v3.0.2 https://stackblitz.com/edit/vitest-dev-vitest-man5rryk?file=test%2Frepro.test.ts

#7281 is a great fix, but it was missing handling the same timeout for non promise result, which only happens on sync hooks since test functions are always wrapped with withAwaitAsyncAssertions.

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

• It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
• Ideally, include a test that fails without this PR but passes with it.
• Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

• Run the tests with pnpm test:ci.

Documentation

• If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

• Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.