Package maintenance

.github/workflows/test.yml

@@ -13,9 +13,11 @@ jobs:
     strategy:
       matrix:
         node:
-          - 12
           - 14
           - 16
+          - 18
+          - 19
+          - 20
         os:
           - ubuntu-latest
           - windows-latest

lib/report.js

@@ -2,6 +2,7 @@
 
 const { iter, capitalize } = require('@metarhia/common');
 const yaml = require('yaml');
+// eslint-disable-next-line import/no-unresolved
 const yamlCustomTags = require('tap-yaml/lib/types/index.js');
 
 // TODO: replace tap-yaml and tap-mocha-reporter - due to the fact that
@@ -56,18 +57,18 @@
 
   // Record test
   //   test <Test>
   record() {
     throw new Error('Not implemented');
   }
 
   // Fail test with error
   //   test <Test>
   //   error <Error>
   error() {
     throw new Error('Not implemented');
   }
 
   finish() {
     throw new Error('Not implemented');
   }
 

package.json

@@ -48,33 +48,33 @@
     "fix": "eslint --fix . && prettier --write \"**/*.js\" \"**/*.json\" \"**/*.md\" \"**/.*rc\" \"**/*.yml\" \"**/*.yaml\" \"**/*.ts\""
   },
   "engines": {
-    "node": ">=12.0.0"
+    "node": ">=14.0.0"
   },
   "dependencies": {
-    "@metarhia/common": "^2.2.0",
+    "@metarhia/common": "^2.2.2",
     "tap-mocha-reporter": "^5.0.3",
-    "tap-yaml": "^1.0.0",
+    "tap-yaml": "^1.0.2",
     "yaml": "^1.10.2",
-    "yargs": "^15.4.1"
+    "yargs": "^16.2.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "^16.2.3",
-    "@commitlint/config-conventional": "^16.2.1",
+    "@commitlint/cli": "^17.6.6",
+    "@commitlint/config-conventional": "^16.2.4",
     "@metarhia/doc": "^0.7.0",
-    "@stryker-mutator/core": "^5.6.1",
-    "@types/node": "^14.18.12",
-    "@typescript-eslint/eslint-plugin": "^5.15.0",
-    "@typescript-eslint/parser": "^5.15.0",
-    "eslint": "^8.11.0",
-    "eslint-config-metarhia": "^7.0.1",
-    "eslint-config-prettier": "^8.5.0",
-    "eslint-plugin-import": "^2.25.4",
-    "eslint-plugin-prettier": "^4.0.0",
-    "husky": "^7.0.4",
-    "prettier": "^2.6.0",
-    "remark-cli": "^10.0.1",
+    "@stryker-mutator/core": "^6.4.2",
+    "@types/node": "^14.18.53",
+    "@typescript-eslint/eslint-plugin": "^6.1.0",
+    "@typescript-eslint/parser": "^6.1.0",
+    "eslint": "^8.45.0",
+    "eslint-config-metarhia": "^8.2.0",
+    "eslint-config-prettier": "^8.8.0",
+    "eslint-plugin-import": "^2.27.5",
+    "eslint-plugin-prettier": "^5.0.0",
+    "husky": "^8.0.3",
+    "prettier": "^3.0.0",
+    "remark-cli": "^11.0.0",
     "remark-preset-lint-metarhia": "^2.0.1",
-    "remark-validate-links": "^11.0.2",
-    "typescript": "^4.6.2"
+    "remark-validate-links": "^12.1.1",
+    "typescript": "^5.1.6"
   }
 }

test/fixtures/mock-reporter.js

@@ -5,11 +5,8 @@
 } = require('../..');
 
 module.exports = class MockReporter extends Reporter {
   record() {}
-
   error() {}
-
   finish() {}
-
   log() {}
 };

test/no-reporter-test.js

@@ -17,7 +17,7 @@ if (process.argv[2] === 'child-success') {
   test.end();
   metatests.runner.instance.on('finish', hasFailures => {
     // this test must succeed if test has 'error' - the above end-after-end
-    process.exit(!hasFailures);
+    process.exit(hasFailures ? 0 : 1);
   });
 } else {
   test.plan(2);

test/unit/compare.js

@@ -176,10 +176,12 @@ assert(
   equal(
     (a, b) => {
       a += 3;
+      // eslint-disable-next-line no-invalid-this
       return this.c + a + b;
     },
     (a, b) => {
       a += 3;
+      // eslint-disable-next-line no-invalid-this
       return this.c + a + b;
     }
   )
@@ -308,10 +310,12 @@ assert(
   strictEqual(
     (a, b) => {
       a += 3;
+      // eslint-disable-next-line no-invalid-this
       return this.c + a + b;
     },
     (a, b) => {
       a += 3;
+      // eslint-disable-next-line no-invalid-this
       return this.c + a + b;
     }
   )

test/unit/declarative.run-with-method.js

@@ -4,6 +4,7 @@ const assert = require('assert');
 const { DeclarativeTest } = require('../..');
 
 class CustomClass {
+  // eslint-disable-next-line class-methods-use-this
   bufferify(string) {
     return Buffer.from(string);
   }

test/unit/imperative.dependent-and-parallel.js

@@ -25,7 +25,12 @@ assert.throws(
   )
 );
 
-assert.doesNotThrow(() => {
-  const test = new ImperativeTest('', null, { parallelSubtests: true });
-  test.end();
-}, checkError(new Error('parallelSubtests and dependentSubtests are contradictory')));
+assert.doesNotThrow(
+  () => {
+    const test = new ImperativeTest('', null, { parallelSubtests: true });
+    test.end();
+  },
+  checkError(
+    new Error('parallelSubtests and dependentSubtests are contradictory')
+  )
+);

types/lib/imperative-test.d.ts

@@ -13,7 +13,7 @@ export interface ImperativeTestResult {
 
 export interface ImperativeTestOptions<
   C extends TestContext = {},
-  R extends Runner = Runner
+  R extends Runner = Runner,
 > extends TestOptions<C, R> {
   run: boolean;
   async: boolean;
@@ -240,7 +240,7 @@ export function test<C extends TestContext = {}, R extends Runner = Runner>(
 // Create an asynchronous test
 export function testAsync<
   C extends TestContext = {},
-  R extends Runner = Runner
+  R extends Runner = Runner,
 >(
   caption: string,
   func: (test: ImperativeTest) => any | Promise<any>,

types/lib/runner.d.ts

@@ -7,7 +7,7 @@ interface RunnerOptions {
 }
 
 export class Runner<
-  O extends RunnerOptions = RunnerOptions
+  O extends RunnerOptions = RunnerOptions,
 > extends events.EventEmitter {
   public options: O;
   public reporter: Reporter<ReporterOptions>;

types/lib/test.d.ts

@@ -19,7 +19,7 @@ interface TestMetadata {
 export class Test<
   R extends TestResult = TestResult,
   O extends TestOptions = TestOptions,
-  M extends TestMetadata = TestMetadata
+  M extends TestMetadata = TestMetadata,
 > extends events.EventEmitter {
   public caption: string;
   public results: R[];