refactor(coverage): remove unnecessary __vite_ssr_exports__ manipulation

[hi-ogawa]

Description

• Related Missing function coverage (v8) when transform prepends code right before the function #7130

Vite SSR doesn't create a mapping for Object.defineProperty(__vite_ssr_exports__, ...) (as they are simply injected by MagicString.appendLeft), so I'm not sure what this replacement on v8 coverage is for.

It looks like test/coverage-test is passing locally. Let me check others on CI.

Sadly, this doesn't seem to entirely help #7130. There's still some odd changes (namely export getter get appears in function coverage) in #7096 after applying the same patch.

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 vitest-dev ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	cd186af
🔍 Latest deploy log	https://app.netlify.com/sites/vitest-dev/deploys/676ca2d8bf75700008330cc9
😎 Deploy Preview	https://deploy-preview-7132--vitest-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[hi-ogawa]

/ecosystem-ci run

[vitest-ecosystem-ci]

📝 Ran ecosystem CI: Open

suite	result
aria-live-capture	✅ success
nuxt	❌ failure
nuxt-test-utils	❌ failure
elk	✅ success
effect	❌ failure
lerna-lite	✅ success
zustand	✅ success
vue	✅ success
vite	✅ success
vitest-vscode	❌ failure
vitest-sonar-reporter	❌ failure
vitest-github-actions-reporter	❌ failure
vitest-browser-simple	✅ success
vitest-browser-examples	❌ failure
vitest-coverage-large	✅ success
vitest-reporters-large	✅ success
vitest-benchmark-large	✅ success

[AriPerkkio]

We should add test case that validates function counts. We used to have one but looks like it got removed during some test setup refactorings.

So maybe something like:

vitest/test/coverage-test/coverage-report-tests/generic.report.test.ts

Lines 132 to 141 in 7fd341c

	test('function count is correct', async () => {
	const coverageJson = await readCoverageJson()
	const coverageMap = libCoverage.createCoverageMap(coverageJson as any)
	const fileCoverage = coverageMap.fileCoverageFor('<process-cwd>/src/function-count.ts')
	const { functions } = fileCoverage.toSummary()
	expect(functions.total).toBe(5)
	expect(functions.covered).toBe(3)
	})

vitest/test/coverage-test/src/function-count.ts

Lines 1 to 36 in 7fd341c

	/*
	* This file should have:
	* - 5 functions in total
	* - 3 covered functions
	* - 2 uncovered functions
	*/
	/* eslint-disable unused-imports/no-unused-vars */
	// This function is covered
	function first() {
	return 1
	}
	first()
	// This function is covered
	export function second() {
	fifth()
	return 2
	}
	// This function is NOT covered
	export function third() {
	throw new Error('Do not call this function')
	}
	// This function is NOT covered
	function fourth() {
	return 4
	}
	// This function is covered
	function fifth() {
	return 5
	}

[hi-ogawa]

We should add test case that validates function counts.

It looks like there are several tests related to function count like this snapshot

vitest/test/coverage-test/test/__snapshots__/results-v8.snapshot.json

Lines 108 to 110 in f9a6284

	"fnMap": {
	"0": {
	"name": "isEven",

In #7096, I'm seeing get gets included in the snapshot, so there's something wrong, but in this PR, that's not happening.