Merge pull request #4 from Matt-Williams1/adjust-test-tooling-for-dom-tests

feat(test): add browser vitest config

Author: Matt-Williams1

CONTRIBUTING.md

@@ -260,6 +260,14 @@ Unit tests use [vitest](https://vitest.dev)
 pnpm test.unit
 ```
 
+For tests that rely on browser APIs, run the DOM-focused build:
+
+```shell
+pnpm test.unit.browser
+```
+
+This preset uses [`jsdom`](https://github.com/jsdom/jsdom) under the hood, so ensure it is installed.
+
 ### E2E Tests Only
 
 E2E tests use [Playwright](https://playwright.dev/).

package.json

@@ -256,6 +256,7 @@
     "test.rust": "make test",
     "test.rust.update": "make test-update",
     "test.unit": "vitest packages",
+    "test.unit.browser": "vitest -c vitest.browser.config.ts packages",
     "test.unit.debug": "vitest --inspect-brk packages",
     "test.vite": "playwright test starters/e2e/qwikcity --browser=chromium --config starters/playwright.config.ts",
     "tsc.check": "tsc --noEmit",

vitest.browser.config.ts

@@ -0,0 +1,18 @@
+import { mergeConfig, defineConfig } from 'vitest/config';
+import baseConfig from './vitest.config';
+
+// Browser-targeted Vitest config for tests requiring DOM APIs
+export default mergeConfig(
+  baseConfig,
+  defineConfig({
+    esbuild: {
+      platform: 'browser',
+    },
+    resolve: {
+      conditions: ['browser'],
+    },
+    test: {
+      environment: 'jsdom',
+    },
+  })
+);

vitest.workspace.js

@@ -1,4 +1,4 @@
 import { defineWorkspace } from 'vitest/config';
 
 // needed by the vscode vitest integration but it also speeds up vitest cli
-export default defineWorkspace(['./vitest.config.ts']);
+export default defineWorkspace(['./vitest.config.ts', './vitest.browser.config.ts']);