test: Change coverage config to be vite compatible

This inclusion stanza excludes node_modules as well as the test files. Also, update launch.json to allow debuggin in vscode and fix a broken html comment. Merge branch 'main' into melink14/vitejs
melink14 · Dec 18, 2024 · bad5ade · bad5ade
2 parents a443d76 + 2560055
commit bad5ade
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 197 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -8,7 +8,9 @@
       "address": "localhost",
       "port": 9333,
       "sourceMaps": true,
-      "trace": true
+      "trace": true,
+      // Needed for vite based project.
+      "webRoot": "${workspaceFolder}/extension"
     }
   ]
 }
diff --git a/package-lock.json b/package-lock.json
diff --git a/web-test-runner.config.js b/web-test-runner.config.js
@@ -127,7 +127,9 @@ if (isDocker) {
 const defaultConfig = {
   rootDir: 'extension',
   coverageConfig: {
-    exclude: ['**/*_test.ts*'],
+    // Including this excludes tests and random node_module files from the report.
+    // Excluding doesn't appear to do anything with vite.
+    include: ['**/extension/*.ts'],
   },
 
   browsers: [
@@ -155,7 +157,7 @@ const defaultConfig = {
   testRunnerHtml: (testFramework) =>
     `<html>
       <body>
-        <-- vite doesn't add global by default so add it here. -->
+        <!-- vite doesn't add global by default so add it here. -->
         <script>
           window.global ||= window;
         </script>