fix(vm-threads): tests not cancelled on key press, cancelled tests shown twice

[AriPerkkio]

Description

• Fixes bug where test run is not cancelled when one of the registered keys is pressed. This is only present on pool: 'vmThreads'.
• Fixes bug where cancelled tests were shown twice by reporters

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

• It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
• Ideally, include a test that fails without this PR but passes with it.
• Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

• Run the tests with pnpm test:ci.

Documentation

• If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

• Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

[netlify]

✅ Deploy Preview for fastidious-cascaron-4ded94 canceled.

Name	Link
🔨 Latest commit	4298d79
🔍 Latest deploy log	https://app.netlify.com/sites/fastidious-cascaron-4ded94/deploys/6583270f2298f500088c7011

[AriPerkkio]

There's still one more bug to fix. Once tests have been interrupted and are re-run, the "skip status" of the tests is not reseted. Earlier this line has done it:

vitest/packages/vitest/src/node/pools/vm-threads.ts

Line 91 in 9497f48

ctx.state.clearFiles(project, files)

Skip status is set here:

vitest/packages/vitest/src/node/state.ts

Lines 153 to 168 in 9497f48

	cancelFiles(files: string[], root: string, projectName: string) {
	this.collectFiles(files.map(filepath => ({
	filepath,
	name: relative(root, filepath),
	id: filepath,
	mode: 'skip',
	type: 'suite',
	result: {
	state: 'skip',
	},
	meta: {},
	// Cancelled files have not yet collected tests
	tasks: [],
	projectName,
	})))
	}

This bug is present on all pools.

[AriPerkkio]

This needs to match with the collected files:

vitest/packages/runner/src/collect.ts

Line 28 in 9497f48

projectName: config.name,

This feels a bit hacky. The WorkspaceProject.getSerializableConfig() always returns undefined for .name instead of '' when there's no name set for the project (no workspaces used).

The state.clearFiles uses the projectName when comparing files. When they don't match, there can be duplicate entries where one is from collection and one from cancelling:

'/x/vitest-example-project/vitest/test/sum.2.test.js' => [
  {
    filepath: '/x/vitest-example-project/vitest/test/sum.2.test.js',
    id: '/x/vitest-example-project/vitest/test/sum.2.test.js',
    mode: 'skip',   // <-- From test cancel
    name: 'test/sum.2.test.js',
    projectName: '',
    type: 'suite',
    ...
  },
  {
    filepath: '/x/vitest-example-project/vitest/test/sum.2.test.js',
    mode: 'run',     // <-- From test collect
    name: 'test/sum.2.test.js',
    type: 'suite',
    ...
  }
],

These show up in reports like below:

test-cancel.mp4