Snapshot generated by toMatchSnapshot("...") are considered obsolete for skipped tests

[hi-ogawa]

Describe the bug

Related: #7106

import { expect, test } from 'vitest';

test('x test', () => {
  expect(0).toMatchSnapshot();
  expect(0).toMatchSnapshot();
});

// when test is skipped, `toMatchSnapshot('xxx')` are considered obsolete,
// which can happen e.g. by having `test.skip` here or `test.only` above, or test name filter etc...
test.skip('y test', () => {
  expect(0).toMatchSnapshot('foo');
  expect(0).toMatchSnapshot('bar'); 
});

We have an "obsolete protection" for auto-incremented snapshot name (x test 1 and x test 2 in this case), but this probably isn't handling for toMatchSnapshot("...") API (y test > foo and y test > bar in this case).

Reproduction

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

System Info

(stackblitz)

  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: latest => 2.1.8 
    vite: latest => 5.4.11 
    vitest: latest => 2.1.8

Used Package Manager

npm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.