feat: rerun should undo

packages/vitest/src/node/reporters/default.ts

@@ -7,10 +7,18 @@ import { createListRenderer } from './renderers/listRenderer'
 export class DefaultReporter extends BaseReporter {
   renderer?: ReturnType<typeof createListRenderer>
   rendererOptions: ListRendererOptions = {} as any
+  private renderSucceedDefault? = false
+
+  constructor() {
+    super()
+    this.renderSucceedDefault = this.rendererOptions.renderSucceed
+  }
 
   onPathsCollected(paths: string[] = []) {
-    if (paths.length === 1)
-      this.rendererOptions.renderSucceed = true
+    if (this.isTTY) {
+      if (!this.renderSucceedDefault)
+        this.rendererOptions.renderSucceed = paths.length <= 1
+    }
   }
 
   async onTestRemoved(trigger?: string) {

test/reporters/fixtures/default/MockReporter.ts

@@ -0,0 +1,5 @@
+import { DefaultReporter } from '../../../../packages/vitest/src/node/reporters/default'
+
+export default class MockDefaultReporter extends DefaultReporter {
+  isTTY = true
+}

test/reporters/fixtures/default/vitest.config.ts

@@ -0,0 +1,10 @@
+import { defineConfig } from 'vitest/config'
+
+process.stdin.isTTY = true
+process.stdin.setRawMode = () => process.stdin
+
+export default defineConfig({
+  test: {
+    reporters: './MockReporter',
+  },
+})

test/reporters/tests/default.test.ts

@@ -1,22 +1,16 @@
-import { resolve } from 'pathe'
+import path from 'pathe'
 import { describe, expect, test } from 'vitest'
-import { runVitest } from '../../test-utils'
-import { DefaultReporter } from '../../../packages/vitest/src/node/reporters/default'
+import { runVitestCli } from '../../test-utils'
 
-class MockDefaultReporter extends DefaultReporter {
-  isTTY = true
-}
-
-const root = resolve(__dirname, '../fixtures')
-async function run(fileFilter: string[]) {
-  return await runVitest({
-    deps: {
-      inline: ['std-env'],
-    },
-    root,
-    reporters: new MockDefaultReporter(),
-    config: false,
-  }, fileFilter)
+const resolve = (id = '') => path.resolve(__dirname, '../fixtures/default', id)
+async function run(fileFilter: string[], watch = false, ...args: string[]) {
+  return runVitestCli(
+    ...fileFilter,
+    '--root',
+    resolve(),
+    watch ? '--watch' : '--run',
+    ...args,
+  )
 }
 
 describe('default reporter', async () => {
@@ -25,13 +19,37 @@ describe('default reporter', async () => {
     expect(stdout).contain('✓ b2 test')
     expect(stdout).not.contain('✓ nested b1 test')
     expect(stdout).contain('× b failed test')
-  }, 120000)
+  })
 
   test('show full test suite when only one file', async () => {
     const { stdout } = await run(['a.test.ts'])
     expect(stdout).contain('✓ a1 test')
     expect(stdout).contain('✓ nested a3 test')
     expect(stdout).contain('× a failed test')
     expect(stdout).contain('nested a failed 1 test')
-  }, 120000)
-})
+  })
+
+  test('rerun should undo', async () => {
+    const vitest = await run([], true, '-t', 'passed')
+    await vitest.waitForStdout('help')
+
+    // one file
+    vitest.write('p')
+    await vitest.waitForStdout('Input filename pattern')
+    vitest.write('a\n')
+    await vitest.waitForStdout('Waiting for file changes')
+    expect(vitest.stdout).contain('✓ a1 test')
+    expect(vitest.stdout).contain('✓ nested a3 test')
+
+    // rerun and two files
+    vitest.write('p')
+    await vitest.waitForStdout('Input filename pattern')
+    vitest.write('b\n')
+    await vitest.waitForStdout('Waiting for file changes')
+    expect(vitest.stdout).toContain('RERUN')
+    expect(vitest.stdout).toContain('b1.test.ts')
+    expect(vitest.stdout).toContain('b2.test.ts')
+    expect(vitest.stdout).not.toContain('nested b1 test')
+    expect(vitest.stdout).not.toContain('b1 test')
+  })
+}, 120000)