fix: log seed when only sequence.shuffle.tests is enabled (#9576)

kaigritun · Kai Gritun · sheremet-va · web-flow · commit 8182b77ad787 · 2026-02-08T12:36:11.000+01:00
Co-authored-by: Kai Gritun &lt;kai@kaigritun.com&gt;
Co-authored-by: Vladimir &lt;sleuths.slews0s@icloud.com&gt;
diff --git a/packages/vitest/src/node/config/resolveConfig.ts b/packages/vitest/src/node/config/resolveConfig.ts
@@ -742,7 +742,8 @@ export function resolveConfig(
   }
   resolved.sequence.groupOrder ??= 0
   resolved.sequence.hooks ??= 'stack'
-  if (resolved.sequence.sequencer === RandomSequencer) {
+  // Set seed if either files or tests are shuffled
+  if (resolved.sequence.sequencer === RandomSequencer || resolved.sequence.shuffle) {
     resolved.sequence.seed ??= Date.now()
   }
 
diff --git a/packages/vitest/src/node/logger.ts b/packages/vitest/src/node/logger.ts
@@ -232,7 +232,8 @@ export class Logger {
 
     this.log(withLabel(color, mode, `v${this.ctx.version} `) + c.gray(this.ctx.config.root))
 
-    if (this.ctx.config.sequence.sequencer === RandomSequencer) {
+    // Log seed if either files (RandomSequencer) or tests are shuffled
+    if (this.ctx.config.sequence.sequencer === RandomSequencer || this.ctx.config.sequence.shuffle) {
       this.log(PAD + c.gray(`Running tests with seed "${this.ctx.config.sequence.seed}"`))
     }
 
diff --git a/test/cli/test/config/sequence-shuffle.test.ts b/test/cli/test/config/sequence-shuffle.test.ts
@@ -115,3 +115,50 @@ test('shuffle with a known seed', async () => {
     }
   `)
 })
+
+test('should log seed when only shuffle.tests is enabled', async () => {
+  const { stdout } = await runInlineTests({
+    'basic.test.js': /* js */ `
+      import { test } from 'vitest'
+      test('example', () => {})
+    `,
+  }, {
+    sequence: {
+      seed: 12345,
+      shuffle: { files: false, tests: true },
+    },
+  })
+
+  expect(stdout).toContain('Running tests with seed "12345"')
+})
+
+test('should log seed when shuffle is true', async () => {
+  const { stdout } = await runInlineTests({
+    'basic.test.js': /* js */ `
+      import { test } from 'vitest'
+      test('example', () => {})
+    `,
+  }, {
+    sequence: {
+      seed: 67890,
+      shuffle: true,
+    },
+  })
+
+  expect(stdout).toContain('Running tests with seed "67890"')
+})
+
+test('should not log seed when shuffle is disabled', async () => {
+  const { stdout } = await runInlineTests({
+    'basic.test.js': /* js */ `
+      import { test } from 'vitest'
+      test('example', () => {})
+    `,
+  }, {
+    sequence: {
+      shuffle: false,
+    },
+  })
+
+  expect(stdout).not.toContain('Running tests with seed')
+})

Original file line number	Diff line number	Diff line change
`@@ -742,7 +742,8 @@ export function resolveConfig(`
`742`	`742`	`}`
`743`	`743`	`resolved.sequence.groupOrder ??= 0`
`744`	`744`	`resolved.sequence.hooks ??= 'stack'`
`745`		`- if (resolved.sequence.sequencer === RandomSequencer) {`
	`745`	`+ // Set seed if either files or tests are shuffled`
	`746`	`+ if (resolved.sequence.sequencer === RandomSequencer \|\| resolved.sequence.shuffle) {`
`746`	`747`	`resolved.sequence.seed ??= Date.now()`
`747`	`748`	`}`
`748`	`749`
Original file line number	Diff line number	Diff line change
`@@ -232,7 +232,8 @@ export class Logger {`
`232`	`232`
`233`	`233`	this.log(withLabel(color, mode, `v${this.ctx.version} `) + c.gray(this.ctx.config.root))
`234`	`234`
`235`		`- if (this.ctx.config.sequence.sequencer === RandomSequencer) {`
	`235`	`+ // Log seed if either files (RandomSequencer) or tests are shuffled`
	`236`	`+ if (this.ctx.config.sequence.sequencer === RandomSequencer \|\| this.ctx.config.sequence.shuffle) {`
`236`	`237`	this.log(PAD + c.gray(`Running tests with seed "${this.ctx.config.sequence.seed}"`))
`237`	`238`	`}`
`238`	`239`