When sequence.concurrent is true, tests in a sequential suite run concurrently

[dsyddall]

Describe the bug

The documentation for describe.sequential states that tests should run sequentially inside a sequential suite when Vitest is run with the --sequence.concurrent option enabled, however tests actually run concurrently.

Reproduction

import { describe, expect, test } from 'vitest';

const delay = (timeout: number) =>
  new Promise((resolve) => setTimeout(resolve, timeout));

let count = 0;

describe.sequential('sequential suite', () => {
  test('test 1', async ({ task }) => {
    await delay(2000);
    expect(task.concurrent).toBeFalsy();
    expect(++count).toBe(1);
  });

  test('test 2', ({ task }) => {
    expect(task.concurrent).toBeFalsy();
    expect(++count).toBe(2);
  });
});

https://stackblitz.com/edit/vitest-dev-vitest-mpeirq?file=sequential.test.ts

Expected behaviour

When running the test suite above with vitest --sequence.concurrent, the tests run sequentially and pass.

Actual behaviour

Tests run concurrently and fail.

Reproduction

https://stackblitz.com/edit/vitest-dev-vitest-mpeirq?file=sequential.test.ts

System Info

System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.18.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.4.2 - /usr/local/bin/npm
    pnpm: 8.9.2 - /usr/local/bin/pnpm
  npmPackages:
    @vitest/ui: 1.0.0-beta.4 => 1.0.0-beta.4 
    vite: latest => 4.5.0 
    vitest: 1.0.0-beta.4 => 1.0.0-beta.4

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.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• vitest-dev-vitest-mpeirq (Open reproduction)

[dsyddall]

Fixed by #4482