@k03mad/actual-versions 1.1.0 → 1.1.2

package/app/cli.js

@@ -2,7 +2,9 @@
 
 import {log} from '@k03mad/simple-log';
 import chalk from 'chalk';
-import {table} from 'table';
+import {getBorderCharacters, table} from 'table';
+
+import config from '../config.js';
 
 import * as api from './api.js';
 
@@ -22,4 +24,8 @@ const output = await Promise.all(
     }),
 );
 
-log(table(output));
+const formattedTable = table(output, {
+    border: getBorderCharacters(config.table.border),
+});
+
+log(formattedTable);

package/config.js

@@ -0,0 +1,9 @@
+export default {
+    version: {
+        re: String.raw`\d[\d.]+$`,
+    },
+    table: {
+        // https://www.npmjs.com/package/table#getbordercharacters
+        border: 'honeywell',
+    },
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@k03mad/actual-versions",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Get tool actual version string",
   "maintainers": [
     "Kirill Molchanov <k03.mad@gmail.com"
@@ -19,7 +19,7 @@
     "node": ">=20"
   },
   "dependencies": {
-    "@k03mad/request": "5.8.1",
+    "@k03mad/request": "6.0.3",
     "@k03mad/simple-log": "2.2.1",
     "chalk": "5.3.0",
     "table": "6.8.2"
@@ -27,7 +27,7 @@
   "devDependencies": {
     "@k03mad/eslint-config": "22.5.0",
     "eslint": "8.57.0",
-    "husky": "9.1.1",
+    "husky": "9.1.4",
     "npm-run-all": "4.1.5",
     "strip-ansi": "7.1.0"
   },

package/tests/api.js

@@ -2,24 +2,25 @@ import assert from 'node:assert/strict';
 import {describe, it} from 'node:test';
 
 import * as api from '../app/api.js';
+import config from '../config.js';
 
-const TESTS = {
-    getAriaVersion: /^\d\.\d+\.\d$/,
-    getChromeVersion: /^\d{3}(?:\.\d+){3}$/,
-    getCurlVersion: /^\d\.\d\.\d$/,
-};
+const API_FUNCTIONS = [
+    'getAriaVersion',
+    'getChromeVersion',
+    'getCurlVersion',
+];
 
-describe('api', () => {
-    const steps = Object.entries(TESTS);
+const versionRe = new RegExp(`^${config.version.re}`);
 
-    it('coverage', () => {
-        assert.equal(steps.length, Object.keys(api).length);
+describe('api', () => {
+    it('check tests count', () => {
+        assert.equal(API_FUNCTIONS.length, Object.keys(api).length);
     });
 
-    Object.entries(TESTS).forEach(([key, value]) => {
-        it(key, async () => {
-            const version = await api[key]();
-            assert.match(version, value);
+    API_FUNCTIONS.forEach(fn => {
+        it(`check fn output: ${fn}()`, async () => {
+            const version = await api[fn]();
+            assert.match(version, versionRe);
         });
     });
 });

package/tests/cli.js

@@ -4,17 +4,23 @@ import {before, describe, it} from 'node:test';
 import {promisify} from 'node:util';
 
 import stripAnsi from 'strip-ansi';
+import {getBorderCharacters} from 'table';
+
+import config from '../config.js';
 
 const cliFile = process.env.npm_package_bin_acver;
 const exec = promisify(cp.exec);
 
-const TESTS = [
-    /^aria\s+\d\.\d+\.\d$/,
-    /^chrome\s+\d{3}(?:\.\d+){3}$/,
-    /^curl\s+\d\.\d\.\d$/,
+const tableBorderChars = Object.values(getBorderCharacters(config.table.border));
+const tableBorderCharsRe = new RegExp(tableBorderChars.join('|'), 'g');
+
+const STDOUT_TOOLS = [
+    'aria',
+    'chrome',
+    'curl',
 ];
 
-const TABLE_CHARS_RE = /[─│┼═║╔╗╚╝╟╢╤╧]+/g;
+const versionRe = tool => new RegExp(`^${tool}\\s+${config.version.re}`);
 
 describe('cli', () => {
     let versions;
@@ -25,17 +31,19 @@ describe('cli', () => {
         versions = stdout
             .split('\n')
             .map(elem => stripAnsi(elem)
-                .replaceAll(TABLE_CHARS_RE, '')
+                .replaceAll(tableBorderCharsRe, '')
                 .trim(),
             )
             .filter(Boolean);
     });
 
-    it('coverage', () => {
-        assert.equal(TESTS.length, versions.length);
+    it('check tests count', () => {
+        assert.equal(STDOUT_TOOLS.length, versions.length);
     });
 
-    TESTS.forEach((re, i) => {
-        it(String(re), () => assert.match(versions[i], re));
+    STDOUT_TOOLS.forEach((tool, i) => {
+        it(`check stdout output: #${i + 1} tool: ${tool}`, () => {
+            assert.match(versions[i], versionRe(tool));
+        });
     });
 });