qunitx-cli 0.5.1 → 0.5.3

package/README.md

@@ -1,6 +1,7 @@
 # qunitx-cli
 
 [![CI](https://github.com/izelnakri/qunitx-cli/actions/workflows/ci.yml/badge.svg)](https://github.com/izelnakri/qunitx-cli/actions/workflows/ci.yml)
+[![codecov](https://codecov.io/gh/izelnakri/qunitx-cli/branch/main/graph/badge.svg)](https://codecov.io/gh/izelnakri/qunitx-cli)
 [![npm](https://img.shields.io/npm/v/qunitx-cli)](https://www.npmjs.com/package/qunitx-cli)
 [![npm downloads](https://img.shields.io/npm/dm/qunitx-cli)](https://www.npmjs.com/package/qunitx-cli)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue)](LICENSE)
@@ -8,7 +9,7 @@
 Browser-based test runner for [QUnitX](https://github.com/izelnakri/qunitx) — bundles your JS/TS tests
 with esbuild, runs them in headless Chrome, and streams TAP output to the terminal.
 
-![qunitx-cli demo](demo/demo.gif)
+![qunitx-cli demo](docs/demo.gif)
 
 ## Features
 
@@ -154,7 +155,7 @@ Options:
 npm install
 make check                 # lint + test (run before every commit)
 make test                  # run tests only
-make demo                  # regenerate demo output
+make demo                  # regenerate docs/demo.gif
 make release LEVEL=patch   # bump version, update changelog, tag, push
 ```
 

package/cli.js

@@ -19,7 +19,7 @@ process.title = 'qunitx';
     return await initializeProject();
   }
 
-  let config = await setupConfig();
+  const config = await setupConfig();
 
   return await run(config);
 })();

package/deno.json

@@ -0,0 +1,7 @@
+{
+  "lint": {
+    "rules": {
+      "exclude": ["no-process-global", "no-node-globals", "no-window"]
+    }
+  }
+}

package/lib/boilerplates/test.js

@@ -1,6 +1,6 @@
 import { module, test } from 'qunitx';
 
-module('{{moduleName}}', function (hooks) {
+module('{{moduleName}}', function (_hooks) {
   test('assert true works', function (assert) {
     assert.expect(3);
     assert.ok(true);
@@ -11,11 +11,11 @@ module('{{moduleName}}', function (hooks) {
   test('async test finishes', async function (assert) {
     assert.expect(3);
 
-    let wait = () =>
-      new Promise((resolve, reject) => {
+    const wait = () =>
+      new Promise((resolve) => {
         setTimeout(() => resolve(true), 50);
       });
-    let result = await wait();
+    const result = await wait();
 
     assert.ok(true);
     assert.equal(true, result);

package/lib/commands/generate.js

@@ -8,9 +8,9 @@ import pathExists from '../utils/path-exists.js';
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
 export default async function () {
-  let projectRoot = await findProjectRoot();
-  let moduleName = process.argv[3]; // TODO: classify this maybe in future
-  let path =
+  const projectRoot = await findProjectRoot();
+  const moduleName = process.argv[3]; // TODO: classify this maybe in future
+  const path =
     process.argv[3].endsWith('.js') || process.argv[3].endsWith('.ts')
       ? `${projectRoot}/${process.argv[3]}`
       : `${projectRoot}/${process.argv[3]}.js`;
@@ -19,8 +19,8 @@ export default async function () {
     return console.log(`${path} already exists!`);
   }
 
-  let testJSContent = await fs.readFile(`${__dirname}/../boilerplates/test.js`);
-  let targetFolderPaths = path.split('/');
+  const testJSContent = await fs.readFile(`${__dirname}/../boilerplates/test.js`);
+  const targetFolderPaths = path.split('/');
 
   targetFolderPaths.pop();
 

package/lib/commands/init.js

@@ -8,9 +8,9 @@ import defaultProjectConfigValues from '../boilerplates/default-project-config-v
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
 export default async function () {
-  let projectRoot = await findProjectRoot();
-  let oldPackageJSON = JSON.parse(await fs.readFile(`${projectRoot}/package.json`));
-  let htmlPaths = process.argv.slice(2).reduce(
+  const projectRoot = await findProjectRoot();
+  const oldPackageJSON = JSON.parse(await fs.readFile(`${projectRoot}/package.json`));
+  const htmlPaths = process.argv.slice(2).reduce(
     (result, arg) => {
       if (arg.endsWith('.html')) {
         result.push(arg);
@@ -20,7 +20,7 @@ export default async function () {
     },
     oldPackageJSON.qunitx && oldPackageJSON.qunitx.htmlPaths ? oldPackageJSON.qunitx.htmlPaths : [],
   );
-  let newQunitxConfig = Object.assign(
+  const newQunitxConfig = Object.assign(
     defaultProjectConfigValues,
     htmlPaths.length > 0 ? { htmlPaths } : { htmlPaths: ['test/tests.html'] },
     oldPackageJSON.qunitx,
@@ -34,20 +34,20 @@ export default async function () {
 }
 
 async function writeTestsHTML(projectRoot, newQunitxConfig, oldPackageJSON) {
-  let testHTMLTemplateBuffer = await fs.readFile(`${__dirname}/../boilerplates/setup/tests.hbs`);
+  const testHTMLTemplateBuffer = await fs.readFile(`${__dirname}/../boilerplates/setup/tests.hbs`);
 
   return await Promise.all(
     newQunitxConfig.htmlPaths.map(async (htmlPath) => {
-      let targetPath = `${projectRoot}/${htmlPath}`;
+      const targetPath = `${projectRoot}/${htmlPath}`;
       if (await pathExists(targetPath)) {
         return console.log(`${htmlPath} already exists`);
       } else {
-        let targetDirectory = path.dirname(targetPath);
-        let targetOutputPath = path.relative(
+        const targetDirectory = path.dirname(targetPath);
+        const _targetOutputPath = path.relative(
           targetDirectory,
           `${projectRoot}/${newQunitxConfig.output}/tests.js`,
         );
-        let testHTMLTemplate = testHTMLTemplateBuffer
+        const testHTMLTemplate = testHTMLTemplateBuffer
           .toString()
           .replace('{{applicationName}}', oldPackageJSON.name);
 
@@ -61,15 +61,15 @@ async function writeTestsHTML(projectRoot, newQunitxConfig, oldPackageJSON) {
 }
 
 async function rewritePackageJSON(projectRoot, newQunitxConfig, oldPackageJSON) {
-  let newPackageJSON = Object.assign(oldPackageJSON, { qunitx: newQunitxConfig });
+  const newPackageJSON = Object.assign(oldPackageJSON, { qunitx: newQunitxConfig });
 
   await fs.writeFile(`${projectRoot}/package.json`, JSON.stringify(newPackageJSON, null, 2));
 }
 
 async function writeTSConfigIfNeeded(projectRoot) {
-  let targetPath = `${projectRoot}/tsconfig.json`;
+  const targetPath = `${projectRoot}/tsconfig.json`;
   if (!(await pathExists(targetPath))) {
-    let tsConfigTemplateBuffer = await fs.readFile(
+    const tsConfigTemplateBuffer = await fs.readFile(
       `${__dirname}/../boilerplates/setup/tsconfig.json`,
     );
 

package/lib/commands/run.js

@@ -146,7 +146,7 @@ export default async function (config) {
 async function buildCachedContent(config, htmlPaths) {
   const htmlBuffers = await Promise.all(config.htmlPaths.map((htmlPath) => fs.readFile(htmlPath)));
   const cachedContent = htmlPaths.reduce(
-    (result, htmlPath, index) => {
+    (result, _htmlPath, index) => {
       const filePath = config.htmlPaths[index];
       const html = htmlBuffers[index].toString();
 
@@ -208,7 +208,7 @@ function splitIntoGroups(files, groupCount) {
   return groups.filter((g) => g.length > 0);
 }
 
-function logWatcherAndKeyboardShortcutInfo(config, server) {
+function logWatcherAndKeyboardShortcutInfo(config, _server) {
   console.log(
     '#',
     kleur.blue(`Watching files... You can browse the tests on http://localhost:${config.port} ...`),

package/lib/servers/http.js

@@ -15,14 +15,13 @@ export const MIME_TYPES = {
 
 export default class HTTPServer {
   static serve(config = { port: 1234 }, handler) {
-    let onListen = config.onListen || ((server) => {});
-    let onError = config.onError || ((error) => {});
+    const onListen = config.onListen || ((_server) => {});
+    const onError = config.onError || ((_error) => {});
 
     return new Promise((resolve, reject) => {
-      let server = http.createServer((req, res) => {
+      const server = http.createServer((req, res) => {
         return handler(req, res);
       });
-      server = server;
       server
         .on('error', (error) => {
           onError(error);
@@ -221,7 +220,7 @@ export default class HTTPServer {
     return true;
   }
 
-  buildRegexPattern(path, paramNames) {
+  buildRegexPattern(path, _paramNames) {
     let regexPattern = path.replace(/:[^/]+/g, '([^/]+)');
     regexPattern = regexPattern.replace(/\//g, '\\/');
 
@@ -235,7 +234,7 @@ export default class HTTPServer {
     return paramMatches ? paramMatches.map((match) => match.slice(1)) : [];
   }
 
-  extractParams(route, url) {
+  extractParams(route, _url) {
     const { paramNames, paramValues } = route;
     const params = {};
 

package/lib/setup/browser.js

@@ -13,7 +13,7 @@ export default async function setupBrowser(
   cachedContent,
   existingBrowser = null,
 ) {
-  let [server, browser] = await Promise.all([
+  const [server, browser] = await Promise.all([
     setupWebServer(config, cachedContent),
     existingBrowser
       ? Promise.resolve(existingBrowser)
@@ -29,7 +29,7 @@ export default async function setupBrowser(
           headless: true,
         }),
   ]);
-  let [page, _] = await Promise.all([browser.newPage(), bindServerToPort(server, config)]);
+  const [page] = await Promise.all([browser.newPage(), bindServerToPort(server, config)]);
 
   page.on('console', async (msg) => {
     if (config.debug) {
@@ -39,20 +39,12 @@ export default async function setupBrowser(
     }
   });
   page.on('error', (msg) => {
-    try {
-      throw msg;
-    } catch (e) {
-      console.error(e, e.stack);
-      console.log(e, e.stack);
-    }
+    console.error(msg, msg.stack);
+    console.log(msg, msg.stack);
   });
-  page.on('pageerror', async (error) => {
-    try {
-      throw error;
-    } catch (e) {
-      console.log(e.toString());
-      console.error(e.toString());
-    }
+  page.on('pageerror', (error) => {
+    console.log(error.toString());
+    console.error(error.toString());
   });
 
   return { server, browser, page };

package/lib/setup/config.js

@@ -6,11 +6,11 @@ import setupTestFilePaths from './test-file-paths.js';
 import parseCliFlags from '../utils/parse-cli-flags.js';
 
 export default async function setupConfig() {
-  let projectRoot = await findProjectRoot();
-  let cliConfigFlags = parseCliFlags(projectRoot);
-  let projectPackageJSON = await readConfigFromPackageJSON(projectRoot);
-  let inputs = cliConfigFlags.inputs.concat(readInputsFromPackageJSON(projectPackageJSON));
-  let config = {
+  const projectRoot = await findProjectRoot();
+  const cliConfigFlags = parseCliFlags(projectRoot);
+  const projectPackageJSON = await readConfigFromPackageJSON(projectRoot);
+  const inputs = cliConfigFlags.inputs.concat(readInputsFromPackageJSON(projectPackageJSON));
+  const config = {
     ...defaultProjectConfigValues,
     htmlPaths: [],
     ...projectPackageJSON.qunitx,
@@ -28,7 +28,7 @@ export default async function setupConfig() {
 }
 
 async function readConfigFromPackageJSON(projectRoot) {
-  let packageJSON = await fs.readFile(`${projectRoot}/package.json`);
+  const packageJSON = await fs.readFile(`${projectRoot}/package.json`);
 
   return JSON.parse(packageJSON.toString());
 }
@@ -38,7 +38,7 @@ function normalizeHTMLPaths(projectRoot, htmlPaths) {
 }
 
 function readInputsFromPackageJSON(packageJSON) {
-  let qunitx = packageJSON.qunitx;
+  const qunitx = packageJSON.qunitx;
 
   return qunitx && qunitx.inputs ? qunitx.inputs : [];
 }

package/lib/setup/file-watcher.js

@@ -1,14 +1,9 @@
 import chokidar from 'chokidar';
 import kleur from 'kleur';
 
-export default async function setupFileWatchers(
-  testFileLookupPaths,
-  config,
-  onEventFunc,
-  onFinishFunc,
-) {
-  let extensions = ['js', 'ts'];
-  let fileWatchers = testFileLookupPaths.reduce((watcher, watchPath) => {
+export default function setupFileWatchers(testFileLookupPaths, config, onEventFunc, onFinishFunc) {
+  const extensions = ['js', 'ts'];
+  const fileWatchers = testFileLookupPaths.reduce((watcher, watchPath) => {
     return Object.assign(watcher, {
       [watchPath]: chokidar.watch(watchPath, { ignoreInitial: true }).on('all', (event, path) => {
         if (extensions.some((extension) => path.endsWith(extension))) {
@@ -31,7 +26,7 @@ export default async function setupFileWatchers(
           if (!global.chokidarBuild) {
             global.chokidarBuild = true;
 
-            let result = extensions.some((extension) => path.endsWith(extension))
+            const result = extensions.some((extension) => path.endsWith(extension))
               ? onEventFunc(event, path)
               : null;
 
@@ -72,7 +67,7 @@ function mutateFSTree(fsTree, event, path) {
   } else if (event === 'unlink') {
     delete fsTree[path];
   } else if (event === 'unlinkDir') {
-    let targetPaths = Object.keys(config.fsTree).filter((treePath) => treePath.startsWith(path));
+    const targetPaths = Object.keys(config.fsTree).filter((treePath) => treePath.startsWith(path));
 
     targetPaths.forEach((path) => delete config.fsTree[path]);
   }

package/lib/setup/fs-tree.js

@@ -2,19 +2,19 @@ import fs from 'node:fs/promises';
 import picomatch from 'picomatch';
 import recursiveLookup from 'recursive-lookup';
 
-export default async function buildFSTree(fileAbsolutePaths, config = {}) {
-  let targetExtensions = ['js', 'ts'];
-  let fsTree = {};
+export default async function buildFSTree(fileAbsolutePaths, _config = {}) {
+  const targetExtensions = ['js', 'ts'];
+  const fsTree = {};
 
   await Promise.all(
     fileAbsolutePaths.map(async (fileAbsolutePath) => {
-      let glob = picomatch.scan(fileAbsolutePath);
+      const glob = picomatch.scan(fileAbsolutePath);
 
       // TODO: maybe allow absolute path references
 
       try {
         if (glob.isGlob) {
-          let fileNames = await recursiveLookup(glob.base, (path) => {
+          const fileNames = await recursiveLookup(glob.base, (path) => {
             return targetExtensions.some((extension) => path.endsWith(extension));
           });
 
@@ -24,12 +24,12 @@ export default async function buildFSTree(fileAbsolutePaths, config = {}) {
             }
           });
         } else {
-          let entry = await fs.stat(fileAbsolutePath);
+          const entry = await fs.stat(fileAbsolutePath);
 
           if (entry.isFile()) {
             fsTree[fileAbsolutePath] = null;
           } else if (entry.isDirectory()) {
-            let fileNames = await recursiveLookup(glob.base, (path) => {
+            const fileNames = await recursiveLookup(glob.base, (path) => {
               return targetExtensions.some((extension) => path.endsWith(extension));
             });
 

package/lib/setup/keyboard-events.js

@@ -27,11 +27,6 @@ export default function setupKeyboardEvents(config, cachedContent, connections)
   });
 }
 
-function abortBrowserQUnit(config, connections) {
+function abortBrowserQUnit(_config, connections) {
   connections.server.publish('abort', 'abort');
 }
-
-function abortNodejsQUnit(config) {
-  window.QUnit.config.queue.length = 0;
-  config.aborted = true;
-}

package/lib/setup/test-file-paths.js

@@ -1,10 +1,10 @@
 import picomatch from 'picomatch';
 
-export default function setupTestFilePaths(projectRoot, inputs) {
+export default function setupTestFilePaths(_projectRoot, inputs) {
   // NOTE: very complex algorithm, order is very important
-  let [folders, filesWithGlob, filesWithoutGlob] = inputs.reduce(
+  const [folders, filesWithGlob, filesWithoutGlob] = inputs.reduce(
     (result, input) => {
-      let isGlob = picomatch.scan(input).isGlob;
+      const isGlob = picomatch.scan(input).isGlob;
 
       if (!pathIsFile(input)) {
         result[0].push({
@@ -25,7 +25,7 @@ export default function setupTestFilePaths(projectRoot, inputs) {
     [[], [], []],
   );
 
-  let result = folders.reduce((folderResult, folder) => {
+  const result = folders.reduce((folderResult, folder) => {
     if (!pathIsIncludedInPaths(folders, folder)) {
       folderResult.push(folder);
     }
@@ -48,7 +48,7 @@ export default function setupTestFilePaths(projectRoot, inputs) {
 }
 
 function pathIsFile(path) {
-  let inputs = path.split('/');
+  const inputs = path.split('/');
 
   return inputs[inputs.length - 1].includes('.');
 }
@@ -59,7 +59,7 @@ function pathIsIncludedInPaths(paths, targetPath) {
       return false;
     }
 
-    let globFormat = buildGlobFormat(path);
+    const globFormat = buildGlobFormat(path);
 
     return picomatch.isMatch(targetPath.input, globFormat, { bash: true });
   });

package/lib/setup/web-server.js

@@ -7,7 +7,7 @@ import HTTPServer, { MIME_TYPES } from '../servers/http.js';
 
 const fsPromise = fs.promises;
 
-export default async function setupWebServer(
+export default function setupWebServer(
   config = {
     port: 1234,
     debug: false,
@@ -16,8 +16,8 @@ export default async function setupWebServer(
   },
   cachedContent,
 ) {
-  let STATIC_FILES_PATH = path.join(config.projectRoot, config.output);
-  let server = new HTTPServer();
+  const STATIC_FILES_PATH = path.join(config.projectRoot, config.output);
+  const server = new HTTPServer();
 
   server.wss.on('connection', function connection(socket) {
     socket.on('message', function message(data) {
@@ -42,9 +42,9 @@ export default async function setupWebServer(
     });
   });
 
-  server.get('/', async (req, res) => {
-    let TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
-    let htmlContent = escapeAndInjectTestsToHTML(
+  server.get('/', async (_req, res) => {
+    const TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
+    const htmlContent = escapeAndInjectTestsToHTML(
       replaceAssetPaths(
         cachedContent.mainHTML.html,
         cachedContent.mainHTML.filePath,
@@ -63,9 +63,9 @@ export default async function setupWebServer(
     );
   });
 
-  server.get('/qunitx.html', async (req, res) => {
-    let TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
-    let htmlContent = escapeAndInjectTestsToHTML(
+  server.get('/qunitx.html', async (_req, res) => {
+    const TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
+    const htmlContent = escapeAndInjectTestsToHTML(
       replaceAssetPaths(
         cachedContent.mainHTML.html,
         cachedContent.mainHTML.filePath,
@@ -85,10 +85,11 @@ export default async function setupWebServer(
   });
 
   server.get('/*', async (req, res) => {
-    let possibleDynamicHTML = cachedContent.dynamicContentHTMLs[`${config.projectRoot}${req.path}`];
+    const possibleDynamicHTML =
+      cachedContent.dynamicContentHTMLs[`${config.projectRoot}${req.path}`];
     if (possibleDynamicHTML) {
-      let TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
-      let htmlContent = escapeAndInjectTestsToHTML(
+      const TEST_RUNTIME_TO_INJECT = testRuntimeToInject(config.port, config);
+      const htmlContent = escapeAndInjectTestsToHTML(
         possibleDynamicHTML,
         TEST_RUNTIME_TO_INJECT,
         cachedContent.allTestCode,
@@ -103,12 +104,12 @@ export default async function setupWebServer(
       );
     }
 
-    let url = req.url;
-    let requestStartedAt = new Date();
-    let filePath = (
+    const url = req.url;
+    const requestStartedAt = new Date();
+    const filePath = (
       url.endsWith('/') ? [STATIC_FILES_PATH, url, 'index.html'] : [STATIC_FILES_PATH, url]
     ).join('');
-    let statusCode = (await pathExists(filePath)) ? 200 : 404;
+    const statusCode = (await pathExists(filePath)) ? 200 : 404;
 
     res.writeHead(statusCode, {
       'Content-Type': req.headers.accept?.includes('text/html')
@@ -129,11 +130,11 @@ export default async function setupWebServer(
 }
 
 function replaceAssetPaths(html, htmlPath, projectRoot) {
-  let assetPaths = findInternalAssetsFromHTML(html);
-  let htmlDirectory = htmlPath.split('/').slice(0, -1).join('/');
+  const assetPaths = findInternalAssetsFromHTML(html);
+  const htmlDirectory = htmlPath.split('/').slice(0, -1).join('/');
 
   return assetPaths.reduce((result, assetPath) => {
-    let normalizedFullAbsolutePath = path.normalize(`${htmlDirectory}/${assetPath}`);
+    const normalizedFullAbsolutePath = path.normalize(`${htmlDirectory}/${assetPath}`);
 
     return result.replace(assetPath, normalizedFullAbsolutePath.replace(projectRoot, '.'));
   }, html);

package/lib/setup/write-output-static-files.js

@@ -1,8 +1,8 @@
 import fs from 'node:fs/promises';
 
 export default async function writeOutputStaticFiles({ projectRoot, output }, cachedContent) {
-  let staticHTMLPromises = Object.keys(cachedContent.staticHTMLs).map(async (staticHTMLKey) => {
-    let htmlRelativePath = staticHTMLKey.replace(`${projectRoot}/`, '');
+  const staticHTMLPromises = Object.keys(cachedContent.staticHTMLs).map(async (staticHTMLKey) => {
+    const htmlRelativePath = staticHTMLKey.replace(`${projectRoot}/`, '');
 
     await ensureFolderExists(`${projectRoot}/${output}/${htmlRelativePath}`);
     await fs.writeFile(
@@ -10,8 +10,8 @@ export default async function writeOutputStaticFiles({ projectRoot, output }, ca
       cachedContent.staticHTMLs[staticHTMLKey],
     );
   });
-  let assetPromises = Array.from(cachedContent.assets).map(async (assetAbsolutePath) => {
-    let assetRelativePath = assetAbsolutePath.replace(`${projectRoot}/`, '');
+  const assetPromises = Array.from(cachedContent.assets).map(async (assetAbsolutePath) => {
+    const assetRelativePath = assetAbsolutePath.replace(`${projectRoot}/`, '');
 
     await ensureFolderExists(`${projectRoot}/${output}/${assetRelativePath}`);
     await fs.copyFile(assetAbsolutePath, `${projectRoot}/${output}/${assetRelativePath}`);

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

@@ -22,7 +22,7 @@ export default function (COUNTER, details) {
     details.assertions.reduce((errorCount, assertion, index) => {
       if (!assertion.passed && assertion.todo === false) {
         COUNTER.errorCount++;
-        let stack = assertion.stack?.match(/\(.+\)/g);
+        const stack = assertion.stack?.match(/\(.+\)/g);
 
         console.log('  ---');
         console.log(
@@ -59,7 +59,7 @@ export default function (COUNTER, details) {
 
 function getCircularReplacer() {
   const ancestors = [];
-  return function (key, value) {
+  return function (_key, value) {
     if (typeof value !== 'object' || value === null) {
       return value;
     }

package/lib/utils/find-chrome.js

@@ -2,8 +2,8 @@ import { exec } from 'node:child_process';
 
 const CANDIDATES = ['google-chrome-stable', 'google-chrome', 'chromium', 'chromium-browser'];
 
-export default async function findChrome() {
-  if (process.env.CHROME_BIN) return process.env.CHROME_BIN;
+export default function findChrome() {
+  if (process.env.CHROME_BIN) return Promise.resolve(process.env.CHROME_BIN);
 
   return Promise.any(
     CANDIDATES.map(

package/lib/utils/find-project-root.js

@@ -3,13 +3,13 @@ import searchInParentDirectories from './search-in-parent-directories.js';
 
 export default async function () {
   try {
-    let absolutePath = await searchInParentDirectories('.', 'package.json');
+    const absolutePath = await searchInParentDirectories('.', 'package.json');
     if (!absolutePath.includes('package.json')) {
       throw new Error('package.json mising');
     }
 
     return absolutePath.replace('/package.json', '');
-  } catch (error) {
+  } catch (_error) {
     console.log('couldnt find projects package.json, did you run $ npm init ??');
     process.exit(1);
   }

package/lib/utils/listen-to-keyboard-key.js

@@ -1,8 +1,8 @@
 import process from 'node:process';
 
-let stdin = process.stdin;
-let targetInputs = {};
-let inputs = [];
+const stdin = process.stdin;
+const targetInputs = {};
+const inputs = [];
 let listenerAdded = false;
 
 export default function listenToKeyboardKey(
@@ -22,10 +22,10 @@ export default function listenToKeyboardKey(
       inputs.shift();
       inputs.push(key);
 
-      let inputString = inputs.join('');
-      let targetListener = targetInputs[inputString.toUpperCase()];
-      if (targetListener && targetListenerConformsToCase(targetListener, inputString)) {
-        targetListener.closure(inputString);
+      const currentInput = inputs.join('');
+      const targetListener = targetInputs[currentInput.toUpperCase()];
+      if (targetListener && targetListenerConformsToCase(targetListener, currentInput)) {
+        targetListener.closure(currentInput);
         inputs.fill(undefined);
       }
     });

package/lib/utils/resolve-port-number-for.js

@@ -6,15 +6,13 @@ export default async function resolvePortNumberFor(portNumber) {
   return await resolvePortNumberFor(portNumber + 1);
 }
 
-function portIsAvailable(portNumber) {
-  return new Promise(async (resolve) => {
-    const net = await import('net');
+async function portIsAvailable(portNumber) {
+  const net = await import('net');
+  return new Promise((resolve) => {
     const server = net.createServer();
 
-    server.once('error', function (err) {
-      if (err.code === 'EADDRINUSE') {
-        resolve(false);
-      }
+    server.once('error', function (_err) {
+      resolve(false);
     });
 
     server.once('listening', function () {

package/lib/utils/run-user-module.js

@@ -2,7 +2,7 @@ import kleur from 'kleur';
 
 export default async function runUserModule(modulePath, params, scriptPosition) {
   try {
-    let func = await import(modulePath);
+    const func = await import(modulePath);
     if (func) {
       func.default
         ? await func.default(params)

package/lib/utils/search-in-parent-directories.js

@@ -1,16 +1,16 @@
 import pathExists from './path-exists.js';
 
 async function searchInParentDirectories(directory, targetEntry) {
-  directory = directory === '.' ? process.cwd() : directory;
+  const resolvedDirectory = directory === '.' ? process.cwd() : directory;
 
-  if (await pathExists(`${directory}/${targetEntry}`)) {
-    return `${directory}/${targetEntry}`;
-  } else if (directory === '') {
+  if (await pathExists(`${resolvedDirectory}/${targetEntry}`)) {
+    return `${resolvedDirectory}/${targetEntry}`;
+  } else if (resolvedDirectory === '') {
     return;
   }
 
   return await searchInParentDirectories(
-    directory.slice(0, directory.lastIndexOf('/')),
+    resolvedDirectory.slice(0, resolvedDirectory.lastIndexOf('/')),
     targetEntry,
   );
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qunitx-cli",
   "type": "module",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   "main": "cli.js",
   "author": "Izel Nakri",
@@ -16,14 +16,15 @@
   ],
   "scripts": {
     "bin": "chmod +x cli.js && ./cli.js",
-    "lint": "prettier --check \"lib/**/*.js\" \"test/**/*.js\" \"*.js\" \"package.json\" \".github/**/*.yml\"",
-    "lint:fix": "prettier --write \"lib/**/*.js\" \"test/**/*.js\" \"*.js\" \"package.json\" \".github/**/*.yml\"",
+    "format": "prettier --check \"lib/**/*.js\" \"test/**/*.js\" \"*.js\" \"package.json\" \".github/**/*.yml\"",
+    "format:fix": "prettier --write \"lib/**/*.js\" \"test/**/*.js\" \"*.js\" \"package.json\" \".github/**/*.yml\"",
+    "lint": "deno lint lib/ cli.js",
     "build": "node build.js",
     "changelog:unreleased": "git-cliff --unreleased --strip all",
     "changelog:preview": "git-cliff",
     "changelog:update": "git-cliff --output CHANGELOG.md",
     "prepack": "npm run build",
-    "test": "node test/setup.js && node --test test/**/*-test.js",
+    "test": "node test/setup.js && FORCE_COLOR=0 node --test test/**/*-test.js",
     "test:sanity-first": "./cli.js test/helpers/failing-tests.js test/helpers/failing-tests.ts",
     "test:sanity-second": "./cli.js test/helpers/passing-tests.js test/helpers/passing-tests.ts"
   },
@@ -53,7 +54,7 @@
     "express": "^5.2.1",
     "prettier": "^3.8.1",
     "qunit": "^2.25.0",
-    "qunitx": "^0.9.3"
+    "qunitx": "^1.0.0"
   },
   "volta": {
     "node": "24.14.0"