@vitest/coverage-v8 3.2.0-beta.1 → 3.2.0-beta.3

package/dist/browser.js

@@ -1,5 +1,5 @@
 import { cdp } from '@vitest/browser/context';
-import { l as loadProvider } from './load-provider-pEnu7lco.js';
+import { l as loadProvider } from './load-provider-CdgAx3rL.js';
 
 const session = cdp();
 let enabled = false;
@@ -18,6 +18,7 @@ const mod = {
 	async takeCoverage() {
 		const coverage = await session.send("Profiler.takePreciseCoverage");
 		const result = [];
+		// Reduce amount of data sent over rpc by doing some early result filtering
 		for (const entry of coverage.result) {
 			if (filterResult(entry)) {
 				result.push({
@@ -28,7 +29,9 @@ const mod = {
 		}
 		return { result };
 	},
-	stopCoverage() {},
+	stopCoverage() {
+		// Browser mode should not stop coverage as same V8 instance is shared between tests
+	},
 	async getProvider() {
 		return loadProvider();
 	}
@@ -49,9 +52,6 @@ function filterResult(coverage) {
 	if (coverage.url === window.location.href) {
 		return false;
 	}
-	if (coverage.url.includes("?browserv=") || coverage.url.includes("&browserv=")) {
-		return false;
-	}
 	return true;
 }
 

package/dist/index.js

@@ -1,7 +1,7 @@
 import inspector from 'node:inspector';
 import { fileURLToPath } from 'node:url';
 import { provider } from 'std-env';
-import { l as loadProvider } from './load-provider-pEnu7lco.js';
+import { l as loadProvider } from './load-provider-CdgAx3rL.js';
 
 const session = new inspector.Session();
 let enabled = false;

package/dist/{load-provider-pEnu7lco.js → load-provider-CdgAx3rL.js}

@@ -1,3 +1,4 @@
+// to not bundle the provider
 const name = "./provider.js";
 async function loadProvider() {
 	const { V8CoverageProvider } = await import(

package/dist/provider.js

@@ -2541,8 +2541,9 @@ new Set([
 	"wasi"
 ]);
 
-var version = "3.2.0-beta.1";
+var version = "3.2.0-beta.3";
 
+// Note that this needs to match the line ending as well
 const VITE_EXPORTS_LINE_PATTERN = /Object\.defineProperty\(__vite_ssr_exports__.*\n/g;
 const DECORATOR_METADATA_PATTERN = /_ts_metadata\("design:paramtypes", \[[^\]]*\]\),*/g;
 const FILE_PROTOCOL = "file://";
@@ -2572,6 +2573,7 @@ class V8CoverageProvider extends BaseCoverageProvider {
 		await this.readCoverageFiles({
 			onFileRead(coverage) {
 				merged = mergeProcessCovs([merged, coverage]);
+				// mergeProcessCovs sometimes loses startOffset, e.g. in vue
 				merged.result.forEach((result) => {
 					if (!result.startOffset) {
 						const original = coverage.result.find((r) => r.url === result.url);
@@ -2581,12 +2583,16 @@ class V8CoverageProvider extends BaseCoverageProvider {
 			},
 			onFinished: async (project, transformMode) => {
 				const converted = await this.convertCoverage(merged, project, transformMode);
+				// Source maps can change based on projectName and transform mode.
+				// Coverage transform re-uses source maps so we need to separate transforms from each other.
 				const transformedCoverage = await transformCoverage(converted);
 				coverageMap.merge(transformedCoverage);
 				merged = { result: [] };
 			},
 			onDebug: debug
 		});
+		// Include untested files when all tests were run (not a single file re-run)
+		// or if previous results are preserved by "cleanOnRerun: false"
 		if (this.options.all && (allTestsRun || !this.options.cleanOnRerun)) {
 			const coveredFiles = coverageMap.files();
 			const untestedCoverage = await this.getUntestedFiles(coveredFiles);
@@ -2613,6 +2619,7 @@ class V8CoverageProvider extends BaseCoverageProvider {
 			this.ctx.logger.log(c.blue(" % ") + c.dim("Coverage report from ") + c.yellow(this.name));
 		}
 		for (const reporter of this.options.reporter) {
+			// Type assertion required for custom reporters
 			reports.create(reporter[0], {
 				skipFull: this.options.skipFull,
 				projectRoot: this.ctx.config.root,
@@ -2689,9 +2696,11 @@ class V8CoverageProvider extends BaseCoverageProvider {
 				ignoreClassMethods: this.options.ignoreClassMethods,
 				wrapperLength,
 				ignoreNode: (node, type) => {
+					// SSR transformed imports
 					if (type === "statement" && node.type === "AwaitExpression" && node.argument.type === "CallExpression" && node.argument.callee.type === "Identifier" && node.argument.callee.name === "__vite_ssr_import__") {
 						return true;
 					}
+					// SSR transformed exports
 					if (type === "statement" && node.type === "ExpressionStatement" && node.expression.type === "AssignmentExpression" && node.expression.left.type === "MemberExpression" && node.expression.left.object.type === "Identifier" && node.expression.left.object.name === "__vite_ssr_exports__") {
 						return true;
 					}
@@ -2718,10 +2727,13 @@ class V8CoverageProvider extends BaseCoverageProvider {
 		const sourcesContent = map?.sourcesContent || [];
 		if (!sourcesContent[0]) {
 			sourcesContent[0] = await promises.readFile(filePath, "utf-8").catch(() => {
+				// If file does not exist construct a dummy source for it.
+				// These can be files that were generated dynamically during the test run and were removed after it.
 				const length = findLongestFunctionLength(functions);
 				return "/".repeat(length);
 			});
 		}
+		// These can be uncovered files included by "all: true" or files that are loaded outside vite-node
 		if (!map) {
 			return {
 				source: code || sourcesContent[0],
@@ -2766,6 +2778,8 @@ class V8CoverageProvider extends BaseCoverageProvider {
 			if (transformMode === "browser") {
 				if (result.url.startsWith("/@fs")) {
 					result.url = `${FILE_PROTOCOL}${removeStartsWith(result.url, "/@fs")}`;
+				} else if (result.url.startsWith(project.config.root)) {
+					result.url = `${FILE_PROTOCOL}${result.url}`;
 				} else {
 					result.url = `${FILE_PROTOCOL}${project.config.root}${result.url}`;
 				}
@@ -2821,6 +2835,7 @@ function excludeGeneratedCode(source, map) {
 	trimmed.replaceAll(VITE_EXPORTS_LINE_PATTERN, "\n");
 	trimmed.replaceAll(DECORATOR_METADATA_PATTERN, (match) => "\n".repeat(match.split("\n").length - 1));
 	const trimmedMap = trimmed.generateMap({ hires: "boundary" });
+	// A merged source map where the first one excludes generated parts
 	const combinedMap = remapping([{
 		...trimmedMap,
 		version: 3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@vitest/coverage-v8",
   "type": "module",
-  "version": "3.2.0-beta.1",
+  "version": "3.2.0-beta.3",
   "description": "V8 coverage provider for Vitest",
   "author": "Anthony Fu <anthonyfu117@hotmail.com>",
   "license": "MIT",
@@ -41,8 +41,8 @@
     "dist"
   ],
   "peerDependencies": {
-    "@vitest/browser": "3.2.0-beta.1",
-    "vitest": "3.2.0-beta.1"
+    "@vitest/browser": "3.2.0-beta.3",
+    "vitest": "3.2.0-beta.3"
   },
   "peerDependenciesMeta": {
     "@vitest/browser": {
@@ -53,7 +53,7 @@
     "@ampproject/remapping": "^2.3.0",
     "@bcoe/v8-coverage": "^1.0.2",
     "ast-v8-to-istanbul": "^0.3.1",
-    "debug": "^4.4.0",
+    "debug": "^4.4.1",
     "istanbul-lib-coverage": "^3.2.2",
     "istanbul-lib-report": "^3.0.1",
     "istanbul-lib-source-maps": "^5.0.6",
@@ -73,9 +73,9 @@
     "@types/test-exclude": "^6.0.2",
     "pathe": "^2.0.3",
     "v8-to-istanbul": "^9.3.0",
-    "@vitest/browser": "3.2.0-beta.1",
-    "vitest": "3.2.0-beta.1",
-    "vite-node": "3.2.0-beta.1"
+    "@vitest/browser": "3.2.0-beta.3",
+    "vitest": "3.2.0-beta.3",
+    "vite-node": "3.2.0-beta.3"
   },
   "scripts": {
     "build": "rimraf dist && rollup -c",