Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No indicator for skipped test cases on 2.2.0 reporter #6964

Closed
6 tasks done
hi-ogawa opened this issue Nov 26, 2024 · 2 comments
Closed
6 tasks done

No indicator for skipped test cases on 2.2.0 reporter #6964

hi-ogawa opened this issue Nov 26, 2024 · 2 comments

Comments

@hi-ogawa
Copy link
Contributor

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
Copy link
Member

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

@hi-ogawa
Copy link
Contributor Author

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 👍

 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants