more tests and fixes

Author: bartveneman

src/lib/decuplicate.ts

@@ -1,4 +1,22 @@
 import type { Coverage, Range } from './parse-coverage.js'
+
+function dedupe_list(ranges: Range[]): Set<Range> {
+	let new_ranges: Set<Range> = new Set()
+
+	outer: for (let range of ranges) {
+		for (let processed_range of new_ranges) {
+			if (range.start <= processed_range.start && range.end >= processed_range.end) {
+				new_ranges.delete(processed_range)
+				new_ranges.add(range)
+				continue outer
+			}
+		}
+		new_ranges.add(range)
+	}
+
+	return new_ranges
+}
+
 /**
  * @description
  * prerequisites
@@ -7,7 +25,7 @@ import type { Coverage, Range } from './parse-coverage.js'
  * - only bytes of deduplicated stylesheets are counted
  */
 export function deduplicate_entries(entries: Coverage[]): Coverage[] {
-	let checked_stylesheets = new Map<string, { url: string; ranges: Range[] }>()
+	let checked_stylesheets = new Map<string, { url: string; ranges: Set<Range> }>()
 
 	for (let entry of entries) {
 		let text = entry.text
@@ -18,23 +36,30 @@ export function deduplicate_entries(entries: Coverage[]): Coverage[] {
 			// If not, add them
 			for (let range of entry.ranges) {
 				let found = false
+
 				for (let checked_range of ranges) {
+					// find exact range
 					if (checked_range.start === range.start && checked_range.end === range.end) {
 						found = true
 						break
 					}
 				}
+
 				if (!found) {
-					ranges.push(range)
+					ranges.add(range)
 				}
 			}
 		} else {
 			checked_stylesheets.set(text, {
 				url: entry.url,
-				ranges: entry.ranges,
+				ranges: dedupe_list(entry.ranges),
 			})
 		}
 	}
 
-	return Array.from(checked_stylesheets, ([text, { url, ranges }]) => ({ text, url, ranges: ranges.sort((a, b) => a.start - b.start) }))
+	return Array.from(checked_stylesheets, ([text, { url, ranges }]) => ({
+		text,
+		url,
+		ranges: Array.from(ranges).sort((a, b) => a.start - b.start),
+	}))
 }

src/lib/test/kitchen-sink.test.ts

@@ -1,7 +1,9 @@
 import { test, expect } from '@playwright/test'
-import { calculate_coverage, type Coverage } from '../index.js'
+import { calculate_coverage, type Coverage, type CoverageResult } from '../index.js'
 import { generate_coverage } from './generate-coverage.js'
 import { format } from '@projectwallace/format-css'
+import * as fs from 'node:fs/promises'
+import * as path from 'node:path'
 
 test('project wallace Container component', async () => {
 	// Coverage:
@@ -220,3 +222,32 @@ test.describe('@rules', () => {
 		])
 	})
 })
+
+test.describe('Wallace mega soverage suite', () => {
+	let coverage: Coverage[]
+
+	test.beforeAll(async () => {
+		let file_contents = await fs.readFile(path.resolve('./src/lib/test/wallace.json'), 'utf-8')
+		coverage = JSON.parse(file_contents)
+	})
+
+	test('CopyButton has full coverage', async () => {
+		let data = coverage.find(({ url }) => url.includes('CopyButton')) as Coverage
+		let result = await calculate_coverage([data])
+		expect.soft(result.line_coverage_ratio).toBe(1)
+		expect.soft(result.total_lines).toBe(17)
+	})
+
+	test('Meter has full coverage', async () => {
+		let data = coverage.find(({ url }) => url.includes('Meter')) as Coverage
+		let result = await calculate_coverage([data])
+		expect.soft(result.line_coverage_ratio).not.toBe(1)
+		expect.soft(result.total_lines).toBe(35)
+
+		let sheet = result.coverage_per_stylesheet.at(0)!
+		expect.soft(sheet.chunks.map(({ is_covered, start_line, end_line }) => ({ is_covered, start_line, end_line }))).toEqual([
+			{ is_covered: true, start_line: 1, end_line: 22 },
+			{ is_covered: false, start_line: 23, end_line: 35 },
+		])
+	})
+})