vitest-dev / vitest

Next generation testing framework powered by Vite.
https://vitest.dev
MIT License
13.15k stars 1.19k forks source link

No indicator for skipped test cases on 2.2.0 reporter #6964

Closed hi-ogawa closed 3 days ago

hi-ogawa commented 3 days ago

Describe the bug

 DEV  v2.1.5 /home/projects/vitest-dev-vitest-fuycoy
      UI started at http://localhost:51204/__vitest__/

 ✓ test/basic.test.ts (3)
   ✓ Math.sqrt()
   ↓ Squared [skipped] 
   ✓ JSON
 DEV  v2.2.0-beta.2 /home/projects/vitest-dev-vitest-fuycoy
      UI started at http://localhost:51204/__vitest__/

 ✓ test/basic.test.ts (3 tests | 1 skipped) 2ms
   ✓ Math.sqrt()
   ✓ Squared
   ✓ JSON

Reproduction

https://stackblitz.com/edit/vitest-dev-vitest-nfatqs?file=test%2Fbasic.test.ts

System Info

System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.20.3 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.6 - /usr/local/bin/pnpm
  npmPackages:
    @vitest/ui: 2.2.0-beta.2 => 2.2.0-beta.2 
    vite: latest => 5.4.11 
    vitest: 2.2.0-beta.2 => 2.2.0-beta.2

Used Package Manager

npm

Validations

AriPerkkio commented 3 days ago

Does latest main fix this? https://pkg.pr.new/vitest@main

hi-ogawa commented 3 days ago

Ah, I didn't see the commit there. I was testing on top of main but just one commit before that PR.

It's looking good after rebase :+1:

 DEV  v2.2.0-beta.2 /home/hiroshi/code/others/vitest/test/attest/fixtures

⏳ Waiting for TypeScript to check your project (this may take a while)...
 ✓ test/snapshot.test.ts (4 tests | 3 skipped) 13ms
   ✓ inline snapshot
   ↓ file snapshot
   ↓ type error
   ↓ completions