fix(runner): ensure sequential suite overrides sequence.concurrent

packages/runner/src/suite.ts

@@ -520,16 +520,17 @@ function createSuite() {
       optionsOrFactory,
     )
 
+    const isConcurrentSpecified = options.concurrent || this.concurrent || options.sequential === false
+    const isSequentialSpecified = options.sequential || this.sequential || options.concurrent === false
+
     // inherit options from current suite
     if (currentSuite?.options) {
       options = { ...currentSuite.options, ...options }
     }
 
     // inherit concurrent / sequential from suite
-    const isConcurrent
-      = options.concurrent || (this.concurrent && !this.sequential)
-    const isSequential
-      = options.sequential || (this.sequential && !this.concurrent)
+    const isConcurrent = isConcurrentSpecified || (options.concurrent && !isSequentialSpecified)
+    const isSequential = isSequentialSpecified || (options.sequential && !isConcurrentSpecified)
     options.concurrent = isConcurrent && !isSequential
     options.sequential = isSequential && !isConcurrent
 

test/core/test/sequential-sequence-concurrent.test.ts

@@ -10,26 +10,28 @@ const delay = (timeout: number) => new Promise(resolve => setTimeout(resolve, ti
 
 let count = 0
 
-describe.sequential('running sequential suite when sequence.concurrent is true', () => {
-  test('first test completes first', async ({ task }) => {
+describe.concurrent('concurrent parent suite', () => {
+  describe.sequential('running sequential suite when sequence.concurrent is true', () => {
+    test('first test completes first', async ({task}) => {
+      await delay(50)
+      expect(task.concurrent).toBeFalsy()
+      expect(++count).toBe(1)
+    })
+
+    test('second test completes second', ({task}) => {
+      expect(task.concurrent).toBeFalsy()
+      expect(++count).toBe(2)
+    })
+  })
+
+  test.sequential('third test completes third', async ({task}) => {
     await delay(50)
     expect(task.concurrent).toBeFalsy()
-    expect(++count).toBe(1)
+    expect(++count).toBe(3)
   })
 
-  test('second test completes second', ({ task }) => {
+  test.sequential('fourth test completes fourth', ({task}) => {
     expect(task.concurrent).toBeFalsy()
-    expect(++count).toBe(2)
+    expect(++count).toBe(4)
   })
-})
-
-test.sequential('third test completes third', async ({ task }) => {
-  await delay(50)
-  expect(task.concurrent).toBeFalsy()
-  expect(++count).toBe(3)
-})
-
-test.sequential('fourth test completes fourth', ({ task }) => {
-  expect(task.concurrent).toBeFalsy()
-  expect(++count).toBe(4)
-})
+});