qunitx-cli 0.5.6 → 0.5.7

package/lib/commands/run/tests-in-browser.js

@@ -63,7 +63,7 @@ export default async function runTestsInBrowser(
 
   // In group mode the COUNTER is shared across all groups and managed by run.js.
   if (!config._groupMode) {
-    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0 };
+    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0, errorCount: 0 };
   }
   config.lastRanTestFiles = targetTestFilesToFilter || allTestFilePaths;
 

package/lib/commands/run.js

@@ -71,7 +71,7 @@ export default async function run(config) {
     const groups = splitIntoGroups(allFiles, groupCount);
 
     // Shared COUNTER so TAP test numbers are globally sequential across all groups.
-    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0 };
+    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0, errorCount: 0 };
     config.lastRanTestFiles = allFiles;
 
     const groupConfigs = groups.map((groupFiles, i) => ({

package/lib/setup/file-watcher.js

@@ -65,15 +65,19 @@ export default function setupFileWatchers(testFileLookupPaths, config, onEventFu
   };
 }
 
-function mutateFSTree(fsTree, event, path) {
+/**
+ * Mutates `fsTree` in place based on a chokidar file-system event.
+ * @returns {void}
+ */
+export function mutateFSTree(fsTree, event, path) {
   if (event === 'add') {
     fsTree[path] = null;
   } else if (event === 'unlink') {
     delete fsTree[path];
   } else if (event === 'unlinkDir') {
-    const targetPaths = Object.keys(config.fsTree).filter((treePath) => treePath.startsWith(path));
-
-    targetPaths.forEach((path) => delete config.fsTree[path]);
+    for (const treePath of Object.keys(fsTree)) {
+      if (treePath.startsWith(path)) delete fsTree[treePath];
+    }
   }
 }
 

package/lib/tap/display-test-result.js

@@ -26,7 +26,7 @@ export default function TAPDisplayTestResult(COUNTER, details) {
     );
     details.assertions.reduce((errorCount, assertion, index) => {
       if (!assertion.passed && assertion.todo === false) {
-        COUNTER.errorCount++;
+        COUNTER.errorCount = (COUNTER.errorCount ?? 0) + 1;
         const stack = assertion.stack?.match(/\(.+\)/g);
 
         console.log('  ---');

package/lib/utils/parse-cli-flags.js

@@ -13,7 +13,7 @@ export default function parseCliFlags(projectRoot) {
       } else if (arg.startsWith('--failfast') || arg.startsWith('--failFast')) {
         return Object.assign(result, { failFast: parseBoolean(arg.split('=')[1]) });
       } else if (arg.startsWith('--timeout')) {
-        return Object.assign(result, { timeout: arg.split('=')[1] || 10000 });
+        return Object.assign(result, { timeout: Number(arg.split('=')[1]) || 10000 });
       } else if (arg.startsWith('--output')) {
         return Object.assign(result, { output: arg.split('=')[1] });
       } else if (arg.endsWith('.html')) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qunitx-cli",
   "type": "module",
-  "version": "0.5.6",
+  "version": "0.5.7",
   "description": "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   "main": "cli.js",
   "author": "Izel Nakri",

package/scripts/lint-docs.js

@@ -1,40 +0,0 @@
-#!/usr/bin/env node
-// TODO: REMOVE THIS FILE once deno fixes the missing-return-type regression.
-//
-// WHY THIS FILE EXISTS:
-// `deno doc --lint` has a regression in deno 2.7.x where JSDoc `@returns` tags
-// are silently ignored for the `missing-return-type` check in JavaScript files.
-// The check requires TypeScript-style return type annotations (`: ReturnType`)
-// which are not valid syntax in `.js` files.
-//
-// All 22 `missing-return-type` errors are false positives caused by this bug.
-// Every function has a correct `@returns` JSDoc tag — deno just doesn't read it.
-//
-// FIX OPTIONS (when ready):
-//   1. Wait for deno to fix the regression (check deno changelog).
-//   2. Convert lib/*.js files to lib/*.ts and add TS return type annotations.
-//
-// This script runs `deno doc --lint` and fails only on `missing-jsdoc` errors
-// (i.e., the real quality check: "is every exported symbol documented?").
-import { spawn } from 'node:child_process';
-
-const proc = spawn('deno', ['doc', '--lint', 'lib/', 'cli.js'], { encoding: 'utf8' });
-let output = '';
-proc.stdout.on('data', (chunk) => (output += chunk));
-proc.stderr.on('data', (chunk) => (output += chunk));
-proc.on('close', () => {
-  // Strip ANSI escape codes so we can match on plain text
-  const plain = output.replace(/\x1b\[[0-9;]*m/g, '');
-
-  // Split into per-error blocks (each block starts with "error[")
-  const blocks = plain.split(/(?=^error\[)/m);
-  const relevant = blocks.filter((b) => !b.startsWith('error[missing-return-type]'));
-  const result = relevant.join('').trim();
-
-  if (result.includes('error[')) {
-    process.stderr.write(result + '\n');
-    process.exit(1);
-  } else if (result) {
-    process.stdout.write(result + '\n');
-  }
-});