ci: use workspaces to run node tests

modernweb-dev · Jan 3, 2024 · 90e4472 · 90e4472
1 parent c9168e5
commit 90e4472
Show file tree

Hide file tree

Showing 24 changed files with 26 additions and 28 deletions.
diff --git a/package.json b/package.json
@@ -33,7 +33,7 @@
     "start:build": "node packages/dev-server/dist/bin.js --root-dir _site --open",
     "test": "npm run test:node && npm run test:browser && node scripts/workspaces-scripts-bin.mjs test:ci",
     "test:browser": "npm run test:browser --workspaces --if-present",
-    "test:node": "mocha \"packages/!(*test-runner-selenium|*test-runner-webdriver|*test-runner-module-mocking)/test/**/*.test.{ts,js,mjs,cjs}\" && TS_NODE_PROJECT='./tsconfig.node-16-base.json' mocha \"packages/test-runner-module-mocking/test/**/*.test.{ts,js,mjs,cjs}\" --experimental-loader=ts-node/esm",
+    "test:node": "npm run test:node --workspaces --if-present",
     "types": "wireit",
     "update": "npm run update:mjs-dts-entrypoints && npm run update:tsconfigs",
     "update-dependency": "node scripts/update-dependency.js",

diff --git a/packages/browser-logs/package.json b/packages/browser-logs/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
     "test:watch": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/config-loader/package.json b/packages/config-loader/package.json
@@ -19,7 +19,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.js --reporter dot",
+    "test:node": "mocha test/**/*.test.js --reporter dot",
     "test:ci": "npm run test",
     "test:watch": "mocha test/**/*.test.js --watch --watch-files .,src,test --reporter dot"
   },

diff --git a/packages/dev-server-core/package.json b/packages/dev-server-core/package.json
@@ -39,7 +39,7 @@
     "start:event-stream": "node demo/event-stream/start-server.js",
     "start:http2": "node demo/http2/start-server.js",
     "start:import-asset": "node demo/import-asset/start-server.js",
-    "test": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
+    "test": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --exit",
     "test:watch": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/dev-server-esbuild/package.json b/packages/dev-server-esbuild/package.json
@@ -28,7 +28,7 @@
     "build": "tsc",
     "start:demo:jsx": "es-dev-server --config demo/jsx/server.config.js",
     "start:demo:ts": "es-dev-server --config demo/ts/server.config.js",
-    "test": "mocha \"test/**/*.test.ts\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.ts\" --require ts-node/register",
     "test:watch": "mocha \"test/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/dev-server-hmr/package.json b/packages/dev-server-hmr/package.json
@@ -28,7 +28,7 @@
     "build": "tsc",
     "start:lit-html": "wds --config demo/lit-html/server.config.mjs",
     "start:vanilla": "wds --config demo/vanilla/server.config.mjs",
-    "test": "mocha \"test/**/*.test.ts\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.ts\" --require ts-node/register",
     "test:watch": "mocha \"test/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/dev-server-legacy/package.json b/packages/dev-server-legacy/package.json
@@ -27,7 +27,7 @@
   "scripts": {
     "build": "tsc",
     "start": "wds --open --config demo/server.config.mjs",
-    "test": "mocha \"test/**/*.test.ts\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.ts\" --require ts-node/register",
     "test:watch": "mocha \"test/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/dev-server-rollup/package.json b/packages/dev-server-rollup/package.json
@@ -25,7 +25,7 @@
     "node": ">=18.0.0"
   },
   "scripts": {
-    "test": "mocha \"test/node/**/*.test.ts\" --require ts-node/register",
+    "test:node": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --exit",
     "test:watch": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/dev-server-storybook/package.json b/packages/dev-server-storybook/package.json
@@ -31,9 +31,7 @@
     "build:wc": "node dist/build/cli.js -c demo/wc/.storybook",
     "start:build": "wds --root-dir storybook-static --open",
     "start:preact": "wds --config demo/preact/web-dev-server.config.mjs",
-    "start:wc": "wds --config demo/wc/web-dev-server.config.mjs",
-    "test": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --reporter dot",
-    "test:watch": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test --reporter dot"
+    "start:wc": "wds --config demo/wc/web-dev-server.config.mjs"
   },
   "files": [
     "*.d.ts",

diff --git a/packages/dev-server/package.json b/packages/dev-server/package.json
@@ -37,7 +37,7 @@
     "start:plugin-serve": "node dist/bin.js --config demo/plugin-serve/config.mjs --open",
     "start:static": "node dist/bin.js --config demo/static/config.mjs --open demo/static/",
     "start:syntax": "node dist/bin.js --config demo/syntax/config.mjs --open demo/syntax/",
-    "test": "mocha \"test/**/*.test.mjs\" --reporter dot",
+    "test:node": "mocha \"test/**/*.test.mjs\" --reporter dot",
     "test:watch": "mocha \"test/**/*.test.mjs\" --watch --watch-files src,test --reporter dot"
   },
   "files": [

diff --git a/packages/parse5-utils/package.json b/packages/parse5-utils/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
     "test:watch": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --watch"
   },
   "files": [

diff --git a/packages/polyfills-loader/package.json b/packages/polyfills-loader/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
+    "test:node": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register",
     "test:update-snapshots": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --update-snapshots",
     "test:watch": "mocha \"test/**/*.test.{ts,js,mjs,cjs}\" --require ts-node/register --watch --watch-files src,test"
   },

diff --git a/packages/rollup-plugin-copy/package.json b/packages/rollup-plugin-copy/package.json
@@ -29,7 +29,7 @@
     "node": ">=18.0.0"
   },
   "scripts": {
-    "test": "mocha test/**/*.test.js --reporter dot",
+    "test:node": "mocha test/**/*.test.js --reporter dot",
     "test:watch": "mocha test/**/*.test.js --watch --watch-files src,test --reporter dot"
   },
   "files": [

diff --git a/packages/rollup-plugin-html/package.json b/packages/rollup-plugin-html/package.json
@@ -28,7 +28,7 @@
     "demo:mpa": "rm -rf demo/dist && rollup -c demo/mpa/rollup.config.js --watch & npm run serve-demo",
     "demo:spa": "rm -rf demo/dist && rollup -c demo/spa/rollup.config.js --watch & npm run serve-demo",
     "serve-demo": "node ../dev-server/dist/bin.js --watch --root-dir demo/dist --app-index index.html --compatibility none --open",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/rollup-plugin-polyfills-loader/package.json b/packages/rollup-plugin-polyfills-loader/package.json
@@ -25,7 +25,7 @@
     "node": ">=18.0.0"
   },
   "scripts": {
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:update-snapshots": "mocha test/**/*.test.ts --require ts-node/register --update-snapshots",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },

diff --git a/packages/test-runner-chrome/package.json b/packages/test-runner-chrome/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-commands/package.json b/packages/test-runner-commands/package.json
@@ -30,7 +30,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test --watch-ignore **/*.snap.js"
   },
   "files": [

diff --git a/packages/test-runner-core/package.json b/packages/test-runner-core/package.json
@@ -33,7 +33,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-core/test/src/runner/TestRunner.test.ts b/packages/test-runner-core/test/src/runner/TestRunner.test.ts
@@ -252,8 +252,8 @@ describe('TestRunner', function () {
       const normalize = (x: string): string => x.replace(/\//g, path.sep);
       const { runner } = await createTestRunner({
         files: [
-          'packages/test-runner-core/test/fixtures/**/*.test.js',
-          '!packages/test-runner-core/test/fixtures/group-c/*',
+          'test/fixtures/**/*.test.js',
+          '!test/fixtures/group-c/*',
         ].map(normalize),
       });
 

diff --git a/packages/test-runner-module-mocking/package.json b/packages/test-runner-module-mocking/package.json
@@ -24,7 +24,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --loader=ts-node/esm",
+    "test:node": "mocha test/**/*.test.ts --loader=ts-node/esm",
     "test:watch": "mocha test/**/*.test.ts --loader ts-node/esm --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-playwright/package.json b/packages/test-runner-playwright/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-puppeteer/package.json b/packages/test-runner-puppeteer/package.json
@@ -26,7 +26,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-visual-regression/package.json b/packages/test-runner-visual-regression/package.json
@@ -30,7 +30,7 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "mocha test/**/*.test.ts --require ts-node/register",
+    "test:node": "mocha test/**/*.test.ts --require ts-node/register",
     "test:watch": "mocha test/**/*.test.ts --require ts-node/register --watch --watch-files src,test"
   },
   "files": [

diff --git a/packages/test-runner-visual-regression/test/visualRegressionPlugin.test.ts b/packages/test-runner-visual-regression/test/visualRegressionPlugin.test.ts
@@ -28,7 +28,7 @@ describe('visualRegressionPlugin', function test() {
           },
         },
         visualRegressionPlugin({
-          baseDir: 'packages/test-runner-visual-regression/screenshots',
+          baseDir: 'screenshots',
           update: process.argv.includes('--update-visual-diffs'),
         }),
       ],
@@ -54,7 +54,7 @@ describe('visualRegressionPlugin', function test() {
             },
           },
           visualRegressionPlugin({
-            baseDir: 'packages/test-runner-visual-regression/screenshots',
+            baseDir: 'screenshots',
             update: process.argv.includes('--update-visual-diffs'),
           }),
         ],