qunitx-cli 0.11.0 → 0.15.0

package/README.md

@@ -188,6 +188,13 @@ All CLI flags can also be set in `package.json` under the `qunitx` key, so you d
 
 CLI flags always override `package.json` values when both are present.
 
+### Environment variables
+
+| Variable         | Description                                                                                                   |
+|------------------|---------------------------------------------------------------------------------------------------------------|
+| `CHROME_BIN`     | Path to the Chrome/Chromium executable. Required on systems where Chrome is not on `PATH` (e.g. many CI environments). Set automatically when using `browser-actions/setup-chrome` in GitHub Actions. |
+| `QUNITX_BROWSER` | Browser engine to use (`chromium`, `firefox`, `webkit`). Equivalent to `--browser` on the CLI. Useful in CI matrix jobs. |
+
 If you do not provide any HTML template, qunitx falls back to its built-in `test/tests.html` boilerplate internally, so `qunitx init` is optional.
 
 You can also pass a custom HTML file on the CLI:

package/dist/cli.js

@@ -682,17 +682,38 @@ function setupWebServer(config, cachedContent) {
     socket.on("message", function message(data) {
       const { event, details, abort } = JSON.parse(data);
       if (event === "wsOpen") {
+        config._phase = "loading";
         config._onWsOpen?.();
       } else if (event === "connection") {
+        config._phase = "running";
         if (!config._groupMode) console.log("TAP version 13");
+        if (config.debug && config._groupMode) {
+          const allFiles = Object.keys(config.fsTree);
+          const relFiles = allFiles.map((f) => f.replace(`${config.projectRoot}/`, ""));
+          const shown = relFiles.slice(0, 2);
+          const rest = relFiles.length - shown.length;
+          const fileList = rest > 0 ? `${shown.join("  ")}  +${rest} more` : shown.join("  ");
+          console.log("#", blue(`\u2500\u2500 ${fileList} \u2500\u2500`));
+        }
         config._resetTestTimeout?.();
       } else if (event === "testEnd" && !abort) {
         if (details.status === "failed") {
           config.lastFailedTestFiles = config.lastRanTestFiles;
         }
+        if (config.debug && details.runtime > config.timeout * 0.8) {
+          console.log(
+            `# SLOW (${details.runtime.toFixed(0)}ms / ${config.timeout}ms timeout): ${details.fullName.join(" | ")}`
+          );
+        }
         config._resetTestTimeout?.();
         TAPDisplayTestResult(config.COUNTER, details);
       } else if (event === "done") {
+        config._phase = "done";
+        if (config.debug && config._groupMode) {
+          console.log(
+            `# group done: ${details.passed} passed, ${details.failed} failed (${details.runtime}ms)`
+          );
+        }
         if (typeof config._testRunDone === "function") {
           config._testRunDone();
           config._testRunDone = null;
@@ -966,6 +987,7 @@ var init_web_server = __esm({
     init_find_internal_assets_from_html();
     init_html_content_marker();
     init_display_test_result();
+    init_color();
     init_path_exists();
     init_http();
     fsPromise = fs7.promises;
@@ -1122,7 +1144,7 @@ async function runUserModule(modulePath, params, scriptPosition) {
     console.log("#", red(`QUnitX ${scriptPosition} script failed:`));
     console.trace(error);
     console.error(error);
-    return process.exit(1);
+    process.stdout.write("", () => process.exit(1));
   }
 }
 var init_run_user_module = __esm({
@@ -1470,6 +1492,7 @@ function setupFileWatchers(testFileLookupPaths, config, onEventFunc, onFinishFun
 function handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFinishFunc) {
   const isFileEvent = extensions.some((ext) => filePath.endsWith(`.${ext}`));
   if (!isFileEvent && event !== "unlinkDir") return;
+  if (event === "change" && config._building && config._justAddedFiles?.has(filePath)) return;
   mutateFSTree(config.fsTree, event, filePath);
   console.log(
     "#",
@@ -1482,6 +1505,7 @@ function handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFi
   );
   if (!config._building) {
     config._building = true;
+    config._justAddedFiles = event === "add" ? /* @__PURE__ */ new Set([filePath]) : /* @__PURE__ */ new Set();
     const result = onEventFunc(event, filePath);
     if (!(result instanceof Promise)) {
       config._building = false;
@@ -1500,6 +1524,7 @@ function handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFi
       }
     });
   } else {
+    if (event === "add") config._justAddedFiles?.add(filePath);
     config._pendingBuildTrigger = () => handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFinishFunc);
   }
 }
@@ -1594,7 +1619,7 @@ function setupKeyboardEvents(config, cachedContent, connections) {
   });
 }
 function abortBrowserQUnit(_config, connections) {
-  connections.server.publish("abort", "abort");
+  connections.server.publish("abort");
 }
 var init_keyboard_events = __esm({
   "lib/setup/keyboard-events.ts"() {
@@ -1671,11 +1696,21 @@ async function run(config) {
           if (["change", "unlink", "unlinkDir"].includes(event)) {
             if (event === "change" && !(file in config.fsTree)) return;
             cachedContent.allTestCode = null;
+            if (config.debug) {
+              console.log(
+                `# Rerun triggered: ${event} \u2192 ${file.replace(`${config.projectRoot}/`, "")}`
+              );
+            }
             return await runTestsInBrowser(config, cachedContent, connections);
           }
+          if (config.debug) {
+            console.log(
+              `# Rerun triggered: ${event} \u2192 ${file.replace(`${config.projectRoot}/`, "")}`
+            );
+          }
           await runTestsInBrowser(config, cachedContent, connections, [file]);
         },
-        (_path, _event) => connections.server.publish("refresh", "refresh")
+        (_path, _event) => connections.server.publish("refresh")
       );
       await watcherReady;
     }
@@ -1691,10 +1726,14 @@ async function run(config) {
       fsTree: Object.fromEntries(groupFiles.map((f) => [f, config.fsTree[f]])),
       // Single group keeps the root output dir for backward-compatible file paths.
       output: groupCount === 1 ? config.output : `${config.output}/group-${i}`,
-      _groupMode: true
+      _groupMode: true,
+      _phase: "bundling"
     }));
     const groupCachedContents = groups.map(() => ({ ...cachedContent }));
     console.log("TAP version 13");
+    console.log(
+      `# Running ${allFiles.length} test file${allFiles.length === 1 ? "" : "s"} across ${groupCount} group${groupCount === 1 ? "" : "s"}`
+    );
     const [browser] = await Promise.all([
       launchBrowser(config),
       Promise.all(
@@ -1716,14 +1755,22 @@ async function run(config) {
     const groupResults = await Promise.allSettled(
       groupConfigs.map((groupConfig, i) => {
         const groupTimeout = new Promise((_, reject) => {
-          const t = setTimeout(
-            () => reject(new Error(`Group ${i} timed out after ${GROUP_TIMEOUT_MS}ms`)),
-            GROUP_TIMEOUT_MS
-          );
+          const t = setTimeout(() => {
+            const files = Object.keys(groupConfig.fsTree).map(
+              (f) => f.replace(`${groupConfig.projectRoot}/`, "")
+            );
+            reject(
+              new Error(
+                `Group ${i} timed out after ${GROUP_TIMEOUT_MS / 1e3}s in phase '${groupConfig._phase ?? "unknown"}'
+  Files: ${files.join(", ")}`
+              )
+            );
+          }, GROUP_TIMEOUT_MS);
           t.unref();
         });
         return Promise.race([
           (async () => {
+            groupConfig._phase = "connecting";
             const connections = await setupBrowser(groupConfig, groupCachedContents[i], browser);
             groupConfig.expressApp = connections.server;
             if (config.before) {
@@ -1881,7 +1928,7 @@ init_color();
 var package_default = {
   name: "qunitx-cli",
   type: "module",
-  version: "0.11.0",
+  version: "0.15.0",
   description: "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   author: "Izel Nakri",
   license: "MIT",
@@ -1911,8 +1958,10 @@ var package_default = {
     "changelog:preview": "git-cliff",
     "changelog:update": "git-cliff --output CHANGELOG.md",
     postinstall: "PLAYWRIGHT_SKIP_DOWNLOAD=true deno install --allow-scripts=npm:playwright-core || true",
-    test: `node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require("os").availableParallelism()') test/**/*-test.ts`,
-    "test:browser": `node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require("os").availableParallelism()') test/flags/*-test.ts test/inputs/*-test.ts`,
+    test: "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test test/**/*-test.ts",
+    "test:debug": "node --experimental-strip-types test/setup.ts && QUNITX_DEBUG=1 node --experimental-strip-types --test test/**/*-test.ts",
+    dev: "node --experimental-strip-types test/setup.ts && node --experimental-strip-types --test --watch test/**/*-test.ts",
+    "test:browser": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test test/flags/*-test.ts test/inputs/*-test.ts",
     "test:release": "bash scripts/test-release.sh",
     "test:sanity-first": "./cli.ts test/helpers/failing-tests.js test/helpers/failing-tests.ts",
     "test:sanity-second": "./cli.ts test/helpers/passing-tests.js test/helpers/passing-tests.ts"

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qunitx-cli",
   "type": "module",
-  "version": "0.11.0",
+  "version": "0.15.0",
   "description": "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   "author": "Izel Nakri",
   "license": "MIT",
@@ -31,8 +31,10 @@
     "changelog:preview": "git-cliff",
     "changelog:update": "git-cliff --output CHANGELOG.md",
     "postinstall": "PLAYWRIGHT_SKIP_DOWNLOAD=true deno install --allow-scripts=npm:playwright-core || true",
-    "test": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require(\"os\").availableParallelism()') test/**/*-test.ts",
-    "test:browser": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require(\"os\").availableParallelism()') test/flags/*-test.ts test/inputs/*-test.ts",
+    "test": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test test/**/*-test.ts",
+    "test:debug": "node --experimental-strip-types test/setup.ts && QUNITX_DEBUG=1 node --experimental-strip-types --test test/**/*-test.ts",
+    "dev": "node --experimental-strip-types test/setup.ts && node --experimental-strip-types --test --watch test/**/*-test.ts",
+    "test:browser": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test test/flags/*-test.ts test/inputs/*-test.ts",
     "test:release": "bash scripts/test-release.sh",
     "test:sanity-first": "./cli.ts test/helpers/failing-tests.js test/helpers/failing-tests.ts",
     "test:sanity-second": "./cli.ts test/helpers/passing-tests.js test/helpers/passing-tests.ts"