feat(coverage): support ignore start/end ignore hints

Author: AriPerkkio

packages/coverage-v8/package.json

@@ -56,7 +56,7 @@
   "dependencies": {
     "@bcoe/v8-coverage": "^1.0.2",
     "@vitest/utils": "workspace:*",
-    "ast-v8-to-istanbul": "^0.3.8",
+    "ast-v8-to-istanbul": "https://pkg.pr.new/AriPerkkio/ast-v8-to-istanbul@109",
     "istanbul-lib-coverage": "catalog:",
     "istanbul-lib-report": "catalog:",
     "istanbul-lib-source-maps": "catalog:",

pnpm-lock.yaml

@@ -28,6 +28,7 @@ overrides:
   '@vitest/browser-webdriverio': 'workspace:*'
   '@vitest/ui': 'workspace:*'
   acorn: 8.11.3
+  istanbul-lib-source-maps: 'file:../istanbuljs/packages/istanbul-lib-source-maps'
   mlly: ^1.8.0
   rollup: $rollup
   vite: $vite

pnpm-workspace.yaml

@@ -8,6 +8,16 @@ export function second() {
   return "Second"
 }
 
+/* istanbul ignore start */
+export function third() {
+  return "Third"
+}
+
+export function fourth() {
+  return "fourh"
+}
+/* istanbul ignore end */
+
 // Covered line
 second()
 
@@ -16,3 +26,5 @@ second()
 
 /* istanbul ignore next -- @preserve, Uncovered line istanbul */
 second()
+
+fourth()

test/coverage-test/fixtures/src/ignore-hints.ts

@@ -16,15 +16,30 @@ test('ignore hints work', async () => {
   const fileCoverage = coverageMap.fileCoverageFor('<process-cwd>/fixtures/src/ignore-hints.ts')
   const lines = fileCoverage.getLineCoverage()
 
+  // Covered
   expect(lines[8]).toBeGreaterThanOrEqual(1)
-  expect(lines[12]).toBeGreaterThanOrEqual(1)
+  expect(lines[22]).toBeGreaterThanOrEqual(1)
+  expect(lines[30]).toBeGreaterThanOrEqual(1)
 
+  // Ignored start+end lines
+  expect(lines[11]).toBeUndefined()
+  expect(lines[12]).toBeUndefined()
+  expect(lines[13]).toBeUndefined()
+  expect(lines[14]).toBeUndefined()
+  expect(lines[15]).toBeUndefined()
+  expect(lines[16]).toBeUndefined()
+  expect(lines[17]).toBeUndefined()
+  expect(lines[18]).toBeUndefined()
+  expect(lines[19]).toBeUndefined()
+
+  // Ignore istanbul
+  expect(lines[28]).toBeUndefined()
+
+  // Line 25 = Ignore v8
   if (isV8Provider()) {
-    expect(lines[15]).toBeUndefined()
-    expect(lines[18]).toBeUndefined()
+    expect(lines[25]).toBeUndefined()
   }
   else {
-    expect(lines[15]).toBeGreaterThanOrEqual(1)
-    expect(lines[18]).toBeUndefined()
+    expect(lines[25]).toBeGreaterThanOrEqual(1)
   }
 })