feat: add test seed to banner (#2877)

btkostner · web-flow · commit bdb3956960c5 · 2023-02-21T19:56:04.000+01:00
diff --git a/packages/runner/src/types/runner.ts b/packages/runner/src/types/runner.ts
@@ -9,7 +9,7 @@ export interface VitestRunnerConfig {
   allowOnly?: boolean
   sequence: {
     shuffle?: boolean
-    seed?: number
+    seed: number
     hooks: SequenceHooks
   }
   maxConcurrency: number
diff --git a/packages/vitest/src/node/config.ts b/packages/vitest/src/node/config.ts
@@ -231,6 +231,8 @@ export function resolveConfig(
       : BaseSequencer
   }
   resolved.sequence.hooks ??= 'parallel'
+  if (resolved.sequence.sequencer === RandomSequencer)
+    resolved.sequence.seed ??= Date.now()
 
   resolved.typecheck = {
     ...configDefaults.typecheck,
diff --git a/packages/vitest/src/node/logger.ts b/packages/vitest/src/node/logger.ts
@@ -4,6 +4,7 @@ import { version } from '../../../../package.json'
 import type { ErrorWithDiff } from '../types'
 import type { TypeCheckError } from '../typecheck/typechecker'
 import { divider } from './reporters/renderers/utils'
+import { RandomSequencer } from './sequencers/RandomSequencer'
 import type { Vitest } from './core'
 import { printError } from './error'
 
@@ -105,6 +106,9 @@ export class Logger {
 
     this.log(`${c.inverse(c.bold(mode))} ${versionTest} ${c.gray(this.ctx.config.root)}`)
 
+    if (this.ctx.config.sequence.sequencer === RandomSequencer)
+      this.log(c.gray(`      Running tests with seed "${this.ctx.config.sequence.seed}"`))
+
     if (this.ctx.config.browser)
       this.log(c.dim(c.green(`      Browser runner started at http://${this.ctx.config.api?.host || 'localhost'}:${c.bold(`${this.ctx.server.config.server.port}`)}`)))
     else if (this.ctx.config.ui)
diff --git a/packages/vitest/src/node/sequencers/RandomSequencer.ts b/packages/vitest/src/node/sequencers/RandomSequencer.ts
@@ -5,8 +5,6 @@ export class RandomSequencer extends BaseSequencer {
   public async sort(files: string[]) {
     const { sequence } = this.ctx.config
 
-    const seed = sequence?.seed ?? Date.now()
-
-    return shuffle(files, seed)
+    return shuffle(files, sequence.seed)
   }
 }
diff --git a/packages/vitest/src/types/config.ts b/packages/vitest/src/types/config.ts
@@ -636,7 +636,7 @@ export interface ResolvedConfig extends Omit<Required<UserConfig>, 'config' | 'f
     sequencer: TestSequencerConstructor
     hooks: SequenceHooks
     shuffle?: boolean
-    seed?: number
+    seed: number
   }
 
   typecheck: TypecheckConfig

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ export interface VitestRunnerConfig {`
`9`	`9`	`allowOnly?: boolean`
`10`	`10`	`sequence: {`
`11`	`11`	`shuffle?: boolean`
`12`		`- seed?: number`
	`12`	`+ seed: number`
`13`	`13`	`hooks: SequenceHooks`
`14`	`14`	`}`
`15`	`15`	`maxConcurrency: number`
Original file line number	Diff line number	Diff line change
`@@ -231,6 +231,8 @@ export function resolveConfig(`
`231`	`231`	`: BaseSequencer`
`232`	`232`	`}`
`233`	`233`	`resolved.sequence.hooks ??= 'parallel'`
	`234`	`+ if (resolved.sequence.sequencer === RandomSequencer)`
	`235`	`+ resolved.sequence.seed ??= Date.now()`
`234`	`236`
`235`	`237`	`resolved.typecheck = {`
`236`	`238`	`...configDefaults.typecheck,`
Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,6 @@ export class RandomSequencer extends BaseSequencer {`
`5`	`5`	`public async sort(files: string[]) {`
`6`	`6`	`const { sequence } = this.ctx.config`
`7`	`7`
`8`		`- const seed = sequence?.seed ?? Date.now()`
`9`		`-`
`10`		`- return shuffle(files, seed)`
	`8`	`+ return shuffle(files, sequence.seed)`
`11`	`9`	`}`
`12`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -636,7 +636,7 @@ export interface ResolvedConfig extends Omit<Required<UserConfig>, 'config' \| 'f`
`636`	`636`	`sequencer: TestSequencerConstructor`
`637`	`637`	`hooks: SequenceHooks`
`638`	`638`	`shuffle?: boolean`
`639`		`- seed?: number`
	`639`	`+ seed: number`
`640`	`640`	`}`
`641`	`641`
`642`	`642`	`typecheck: TypecheckConfig`