Multiple async describe causes some test to not be run.

[MonsieurMan]

Describe the bug

The bug is quite strange, when using multiple nested async describe, some test can be skipped for no apparent reason.

As pointed out in the repro comments, commenting some describes make the previously skipped test to run.

There is no part of the form to do it, so, thank you guys a lot, that is amazing work you are doing here! 🚀

Reproduction

https://stackblitz.com/edit/vitest-dev-vitest-trh7ej?file=test/suite.test.ts

I isolated a minimal repro with only three describe and an async function using a timeout.

System Info

See provided 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.

[Demivan]

Is this supported by Jest? I only see one closed merge request in Jest saying that they will not support it.

You can use async beforeAll and beforeEach in describe and don't mark describe as async.

[sheremet-va]

Is this supported by Jest? I only see one closed merge request in Jest saying that they will not support it.

You can use async beforeAll and beforeEach in describe and don't mark describe as async.

Agree, describe should not be async.

[MonsieurMan]

@Demivan, it makes sense. In this case, shouldn't the type SuiteFactory here :

vitest/packages/vitest/src/types/tasks.ts

Line 81 in 735e507

export type SuiteFactory = (test: (name: string, fn: TestFunction) => void) => Awaitable<void>

be changed to not accept asynchronous functions?

[MonsieurMan]

To sum up for everyone. @antfu, if I understand #750 correctly, async describe will be supported? 🚀

[sheremet-va]

To sum up for everyone. @antfu, if I understand #750 correctly, async describe will be supported? 🚀

Yes, async describe is supported in Vitest, unlike Jest.