fix(snapshot)!: fail test with obsolete snapshot on CI

packages/runner/src/run.ts

@@ -562,9 +562,9 @@ export async function runSuite(suite: Suite, runner: VitestRunner): Promise<void
 
     suite.result.duration = now() - start
 
-    updateTask('suite-finished', suite, runner)
-
     await runner.onAfterRunSuite?.(suite)
+
+    updateTask('suite-finished', suite, runner)
   }
 }
 

packages/vitest/src/runtime/runners/test.ts

@@ -62,6 +62,14 @@ export class VitestTestRunner implements VitestRunner {
       }
 
       const result = await this.snapshotClient.finish(suite.file.filepath)
+      if (
+        this.workerState.config.snapshotOptions.updateSnapshot === 'none'
+        && result.unchecked
+      ) {
+        const error = new Error(`Obsolete snapshots found: ${result.uncheckedKeys.join(', ')}`)
+        suite.result!.errors ??= []
+        suite.result!.errors.push(error)
+      }
       await rpc().snapshotSaved(result)
     }
 

test/snapshots/test/fixtures/obsolete/src/__snapshots__/test1.test.ts.snap

@@ -0,0 +1,5 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`bar 1`] = `"bar"`;
+
+exports[`foo 1`] = `"foo"`;

test/snapshots/test/fixtures/obsolete/src/__snapshots__/test2.test.ts.snap

@@ -0,0 +1,5 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`bar 1`] = `"bar"`;
+
+exports[`foo 1`] = `"foo"`;

test/snapshots/test/fixtures/obsolete/src/test1.test.ts

@@ -0,0 +1,10 @@
+import { expect, it } from 'vitest'
+
+it('foo', () => {
+  if (process.env.TEST_OBSOLETE) return
+  expect("foo").toMatchSnapshot();
+})
+
+it('bar', () => {
+  expect("bar").toMatchSnapshot();
+})

test/snapshots/test/fixtures/obsolete/src/test2.test.ts

@@ -0,0 +1,9 @@
+import { expect, it } from 'vitest'
+
+it('foo', () => {
+  expect("foo").toMatchSnapshot();
+})
+
+it('bar', () => {
+  expect("bar").toMatchSnapshot();
+})

test/snapshots/test/fixtures/obsolete/vitest.config.ts

@@ -0,0 +1,3 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({})

test/snapshots/test/obsolete.test.ts

@@ -0,0 +1,32 @@
+import fs from 'node:fs'
+import path from 'node:path'
+import { expect, test } from 'vitest'
+import { runVitestCli } from '../../test-utils'
+
+test('obsolete snapshot fails CI', async () => {
+  // cleanup snapshot
+  const root = path.join(import.meta.dirname, 'fixtures/obsolete')
+  fs.rmSync(path.join(root, 'src/__snapshots__'), { recursive: true, force: true })
+
+  // initial run to write snapshot
+  let vitest = await runVitestCli('--root', root, '--update')
+  expect(vitest.stdout).toContain('Snapshots  4 written')
+  expect(vitest.stderr).toBe('')
+
+  // test fails with obsolete snapshots
+  // (use cli to test `updateSnapshot: 'none'`)
+  vitest = await runVitestCli(
+    {
+      nodeOptions: {
+        env: {
+          CI: 'true',
+          TEST_OBSOLETE: 'true',
+        },
+      },
+    },
+    '--root',
+    root,
+  )
+  expect(vitest.stdout).toContain('1 obsolete')
+  expect(vitest.stderr).toContain(`Error: Obsolete snapshots found: foo 1`)
+})