fix: correctly inherit test options on extended tests (#8618)

Author: sheremet-va

packages/runner/src/suite.ts

@@ -245,7 +245,7 @@ function parseArguments<T extends (...args: any[]) => any>(
   optionsOrFn: T | object | undefined,
   timeoutOrTest: T | number | undefined,
 ) {
-  if (typeof timeoutOrTest === 'object') {
+  if (timeoutOrTest != null && typeof timeoutOrTest === 'object') {
     throw new TypeError(`Siganture "test(name, fn, { ... })" was deprecated in Vitest 3 and removed in Vitest 4. Please, provide options as a second argument instead.`)
   }
 
@@ -790,9 +790,7 @@ export function createTaskCollector(
           scopedFixtures,
         )
       }
-      const { handler, options } = parseArguments(optionsOrFn, optionsOrTest)
-      const timeout = options.timeout ?? collector.options?.timeout ?? runner?.config.testTimeout
-      originalWrapper.call(context, formatName(name), handler, timeout)
+      originalWrapper.call(context, formatName(name), optionsOrFn, optionsOrTest)
     }, _context)
   }
 

test/core/test/retry.test.ts

@@ -64,3 +64,16 @@ describe.each([
     expect(flag3).toBe(false)
   })
 })
+
+describe('extended tests keep the options', () => {
+  const test$ = it.extend({
+    extended: true,
+  })
+
+  test$('inherits all options', { retry: 2, repeats: 1, timeout: 3456 }, ({ task, extended }) => {
+    expect(task.retry).toBe(2)
+    expect(task.repeats).toBe(1)
+    expect(task.timeout).toBe(3456)
+    expect(extended).toBe(true)
+  })
+})