test: Test browser and more Node.js versions (#16)

Author: timfish

.github/workflows/CI.yml

@@ -7,18 +7,66 @@ on:
 
 jobs:
   build:
-    name: Build & Test
+    name: Build
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
+          cache: yarn
           node-version-file: 'package.json'
       - name: Install
         run: yarn install
       - name: Lint
         run: yarn lint
       - name: Build
         run: yarn build
-      - name: Test
-        run: yarn test
+      - name: Upload Artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: ${{ github.sha }}
+          path: ./packages/**/dist/**/*
+
+  test_node:
+    name: Test Node.js v${{ matrix.version }}
+    runs-on: ubuntu-latest
+    needs: build
+    strategy:
+      fail-fast: false
+      matrix:
+        version: [18, 20, 22]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          cache: yarn
+          node-version: ${{ matrix.version }}
+      - name: Install
+        run: yarn install
+      - name: Download Artifacts
+        uses: actions/download-artifact@v4
+        with:
+          path: ./packages
+          merge-multiple: true
+      - name: Run Unit Tests
+        run: yarn test:node
+
+  test_browser:
+    name: Test Browser
+    runs-on: macos-latest
+    needs: build
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          cache: yarn
+          node-version-file: 'package.json'
+      - name: Install
+        run: yarn install
+      - name: Download Artifacts
+        uses: actions/download-artifact@v4
+        with:
+          path: ./packages
+          merge-multiple: true
+      - name: Run Unit Tests
+        run: yarn playwright install && yarn test:browser

.gitignore

@@ -2,3 +2,4 @@ node_modules/
 dist/
 .DS_Store
 .parcel-cache/
+test-results/

package.json

@@ -5,7 +5,9 @@
   "private": true,
   "scripts": {
     "build": "rollup --config rollup.config.mjs",
-    "test": "vitest run ./test/src/**/*.test.ts",
+    "test": "yarn test:node && yarn test:browser",
+    "test:node": "cd test/node && yarn test",
+    "test:browser": "cd test/browser && yarn test",
     "lint": "biome check",
     "fix": "biome check --write"
   },
@@ -15,18 +17,17 @@
     "@rollup/plugin-typescript": "^12.1.0",
     "@rspack/cli": "^1.0.8",
     "@rspack/core": "^1.0.8",
-    "@types/node": "^20.0.0",
+    "@types/node": "^18.0.0",
     "esbuild": "^0.24.0",
     "parcel": "^2.12.0",
     "rolldown": "^0.13.2",
     "rollup": "^4.22.5",
     "typescript": "^5.6.2",
     "vite": "^5.4.8",
-    "vitest": "^2.1.1",
     "webpack": "^5.95.0",
     "webpack-cli": "^5.1.4"
   },
-  "workspaces": ["packages/*", "test"],
+  "workspaces": ["packages/*", "test/*"],
   "volta": {
     "node": "20.18.0",
     "yarn": "1.22.22"

packages/cli/package.json

@@ -6,7 +6,7 @@
     "debugids": "./cli.js"
   },
   "engines": {
-    "node": ">=14"
+    "node": ">=18"
   },
   "dependencies": {
     "@debugids/common": "0.0.1"

test/browser/package.json

@@ -0,0 +1,19 @@
+{
+  "name": "@debugids/integration-tests-browser",
+  "version": "0.0.1",
+  "private": true,
+  "scripts": {
+    "test": "yarn build && playwright test",
+    "build": "vite build --target esnext",
+    "server": "http-server ./dist --port 8080"
+  },
+  "dependencies": {
+    "@sentry/browser": "^8.33.1",
+    "@debugids/browser": "0.0.1",
+    "@debugids/cli": "0.0.1"
+  },
+  "devDependencies": {
+    "@playwright/test": "^1.48.0",
+    "http-server": "^14.1.1"
+  }
+}

test/browser/playwright.config.ts

@@ -0,0 +1,35 @@
+import { defineConfig, devices } from '@playwright/test';
+
+export default defineConfig({
+  testDir: './tests',
+  fullyParallel: true,
+  forbidOnly: !!process.env.CI,
+  retries: process.env.CI ? 2 : 0,
+  workers: process.env.CI ? 1 : undefined,
+  reporter: process.env.CI ? 'github' : 'list',
+  use: {
+    baseURL: 'http://127.0.0.1:8080',
+    trace: 'on-first-retry',
+  },
+  projects: [
+    {
+      name: 'chromium',
+      use: { ...devices['Desktop Chrome'] },
+    },
+
+    {
+      name: 'firefox',
+      use: { ...devices['Desktop Firefox'] },
+    },
+
+    {
+      name: 'webkit',
+      use: { ...devices['Desktop Safari'] },
+    },
+  ],
+  webServer: {
+    command: 'yarn server',
+    url: 'http://127.0.0.1:8080',
+    reuseExistingServer: false,
+  },
+});

test/browser/src/first.js

@@ -0,0 +1,4 @@
+export async function doWork() {
+  const { doWork } = await import('./second.js');
+  await doWork();
+}

test/src/vite/no-sourcemaps/src/index.html renamed to test/browser/src/index.html

@@ -0,0 +1,3 @@
+import('./first.js').then(async ({ doWork }) => {
+  await doWork();
+});

test/browser/src/main.js

@@ -0,0 +1,4 @@
+export async function doWork() {
+  const { doWork } = await import('./third.js');
+  await doWork();
+}