native esm autotest runtime

.github/workflows/node.js.yml

@@ -14,7 +14,7 @@ jobs:
   build:
     strategy:
       matrix:
-        node-version: [16, 18, 22]
+        node-version: ['22', '24']
         os: [ubuntu-latest, macos-latest]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 

.nvmrc

@@ -1 +1 @@
-14
+20.19

core/api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/api",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -17,12 +17,12 @@
   "author": "RingCentral",
   "license": "MIT",
   "dependencies": {
-    "@testring/async-breakpoints": "0.8.11",
-    "@testring/http-api": "0.8.11",
-    "@testring/logger": "0.8.11",
-    "@testring/transport": "0.8.11",
-    "@testring/types": "0.8.11",
-    "@testring/utils": "0.8.11",
-    "@testring/web-application": "0.8.11"
+    "@testring/async-breakpoints": "0.8.13",
+    "@testring/http-api": "0.8.13",
+    "@testring/logger": "0.8.13",
+    "@testring/transport": "0.8.13",
+    "@testring/types": "0.8.13",
+    "@testring/utils": "0.8.13",
+    "@testring/web-application": "0.8.13"
   }
 }

core/async-assert/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/async-assert",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "description": "an async wrapper around chai",
   "keywords": [
     "assert"
@@ -26,7 +26,7 @@
     "url": "https://github.com/ringcentral/testring/issues"
   },
   "dependencies": {
-    "@testring/types": "0.8.11",
+    "@testring/types": "0.8.13",
     "chai": "4.3.10"
   }
 }

core/async-breakpoints/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/async-breakpoints",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {

core/child-process/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/child-process",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -12,8 +12,8 @@
     "access": "public"
   },
   "dependencies": {
-    "@testring/types": "0.8.11",
-    "@testring/utils": "0.8.11"
+    "@testring/types": "0.8.13",
+    "@testring/utils": "0.8.13"
   },
   "scripts": {
     "build": "tsc -p tsconfig.build.json"

core/cli-config/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/cli-config",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -17,9 +17,9 @@
   "author": "RingCentral",
   "license": "MIT",
   "dependencies": {
-    "@testring/logger": "0.8.11",
-    "@testring/types": "0.8.11",
-    "@testring/utils": "0.8.11",
+    "@testring/logger": "0.8.13",
+    "@testring/types": "0.8.13",
+    "@testring/utils": "0.8.13",
     "@types/deepmerge": "2.2.3",
     "@types/yargs": "17.0.33",
     "deepmerge": "4.3.1",

core/cli-config/src/index.ts

@@ -4,6 +4,7 @@ import {getArguments} from './arguments-parser';
 import {getFileConfig} from './config-file-reader';
 import {defaultConfiguration} from './default-config';
 import {mergeConfigs} from './merge-configs';
+import {validateRestartWorker} from './validate-restart-worker';
 
 const isDebugging = () => !!inspector.url();
 
@@ -26,13 +27,17 @@ async function getConfig(argv: Array<string> = []): Promise<IConfig> {
         temporaryConfig,
     );
 
-    return mergeConfigs(
+    const config = mergeConfigs(
         defaultConfiguration,
         envConfig || {},
         fileConfig || {},
         args || {},
         debugProperty,
     );
+
+    validateRestartWorker(config);
+
+    return config;
 }
 
 export {defaultConfiguration, getConfig};

core/cli-config/src/validate-restart-worker.ts

@@ -0,0 +1,25 @@
+import {IConfig} from '@testring/types';
+
+export function validateRestartWorker(config: IConfig): void {
+    const {restartWorker} = config;
+
+    if (
+        typeof restartWorker === 'boolean' ||
+        restartWorker === 'always'
+    ) {
+        return;
+    }
+
+    if (
+        typeof restartWorker === 'number' &&
+        Number.isInteger(restartWorker) &&
+        restartWorker >= 0
+    ) {
+        return;
+    }
+
+    throw new Error(
+        `Invalid "restartWorker" config value: ${JSON.stringify(restartWorker)}. ` +
+            `Expected a boolean, a non-negative integer, or the string "always".`,
+    );
+}

core/cli-config/test/get-config.spec.ts

@@ -134,4 +134,25 @@ describe('Get config', () => {
 
         chai.expect(config).to.have.property('workerLimit', override);
     });
+
+    it('should accept a numeric --restart-worker argument', async () => {
+        const config = await getConfig(['--restart-worker=3']);
+
+        chai.expect(config).to.have.property('restartWorker', 3);
+    });
+
+    it('should fail fast on an invalid --restart-worker argument', async () => {
+        let caughtError: Error | null = null;
+
+        try {
+            await getConfig(['--restart-worker=-2']);
+        } catch (error) {
+            caughtError = error as Error;
+        }
+
+        chai.expect(caughtError).to.be.instanceOf(Error);
+        chai.expect((caughtError as Error).message).to.match(
+            /Invalid "restartWorker"/,
+        );
+    });
 });

core/cli-config/test/validate-restart-worker.spec.ts

@@ -0,0 +1,63 @@
+/// <reference types="mocha" />
+
+import * as chai from 'chai';
+import {IConfig} from '@testring/types';
+import {validateRestartWorker} from '../src/validate-restart-worker';
+import {defaultConfiguration} from '../src/default-config';
+
+const withRestartWorker = (restartWorker: IConfig['restartWorker']): IConfig => ({
+    ...defaultConfiguration,
+    restartWorker,
+});
+
+describe('validateRestartWorker', () => {
+    it('should accept false (default)', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(false)),
+        ).to.not.throw();
+    });
+
+    it('should accept true', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(true)),
+        ).to.not.throw();
+    });
+
+    it("should accept 'always'", () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker('always')),
+        ).to.not.throw();
+    });
+
+    it('should accept 0', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(0)),
+        ).to.not.throw();
+    });
+
+    it('should accept a positive integer', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(5)),
+        ).to.not.throw();
+    });
+
+    it('should reject a negative number', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(-1)),
+        ).to.throw(/Invalid "restartWorker"/);
+    });
+
+    it('should reject a non-integer number', () => {
+        chai.expect(() =>
+            validateRestartWorker(withRestartWorker(1.5)),
+        ).to.throw(/Invalid "restartWorker"/);
+    });
+
+    it('should reject an arbitrary string', () => {
+        chai.expect(() =>
+            validateRestartWorker(
+                withRestartWorker('sometimes' as unknown as IConfig['restartWorker']),
+            ),
+        ).to.throw(/Invalid "restartWorker"/);
+    });
+});

core/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/cli",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -17,18 +17,18 @@
   "author": "RingCentral",
   "license": "MIT",
   "dependencies": {
-    "@testring/browser-proxy": "0.8.11",
-    "@testring/cli-config": "0.8.11",
-    "@testring/fs-reader": "0.8.11",
-    "@testring/fs-store": "0.8.11",
-    "@testring/http-api": "0.8.11",
-    "@testring/logger": "0.8.11",
-    "@testring/plugin-api": "0.8.11",
-    "@testring/test-run-controller": "0.8.11",
-    "@testring/test-worker": "0.8.11",
-    "@testring/transport": "0.8.11",
-    "@testring/types": "0.8.11",
-    "@testring/web-application": "0.8.11",
+    "@testring/browser-proxy": "0.8.13",
+    "@testring/cli-config": "0.8.13",
+    "@testring/fs-reader": "0.8.13",
+    "@testring/fs-store": "0.8.13",
+    "@testring/http-api": "0.8.13",
+    "@testring/logger": "0.8.13",
+    "@testring/plugin-api": "0.8.13",
+    "@testring/test-run-controller": "0.8.13",
+    "@testring/test-worker": "0.8.13",
+    "@testring/transport": "0.8.13",
+    "@testring/types": "0.8.13",
+    "@testring/web-application": "0.8.13",
     "@types/node": "22.8.5",
     "@types/yargs": "17.0.33",
     "yargs": "17.7.2"

core/dependencies-builder/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/dependencies-builder",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -17,9 +17,9 @@
   "author": "RingCentral",
   "license": "MIT",
   "dependencies": {
-    "@testring/test-utils": "0.8.11",
-    "@testring/types": "0.8.11",
-    "@testring/utils": "0.8.11",
+    "@testring/test-utils": "0.8.13",
+    "@testring/types": "0.8.13",
+    "@testring/utils": "0.8.13",
     "@types/babel-traverse": "6.25.10",
     "@types/babylon": "6.16.9",
     "babel-traverse": "6.26.0",

core/fs-reader/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/fs-reader",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "repository": {
@@ -18,10 +18,10 @@
     "build": "tsc -p tsconfig.build.json"
   },
   "dependencies": {
-    "@testring/logger": "0.8.11",
-    "@testring/pluggable-module": "0.8.11",
-    "@testring/types": "0.8.11",
+    "@testring/logger": "0.8.13",
+    "@testring/pluggable-module": "0.8.13",
+    "@testring/types": "0.8.13",
     "fast-glob": "3.3.2",
-    "p-limit": "3.1.0"
+    "p-limit": "7.3.1"
   }
 }

core/fs-reader/src/file-resolver.ts

@@ -1,7 +1,29 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import {IFile} from '@testring/types';
-const pLimit = require('p-limit');
+
+// p-limit has been ESM-only since v4; this package's tsconfig targets
+// CommonJS output, so a plain `import`/`require()` can't load it. Loading
+// it via a string-wrapped dynamic import keeps the real ESM `import()`
+// intact at runtime instead of letting tsc downlevel it to `require()` —
+// the same technique `@testring/test-worker`'s worker-controller uses to
+// load native ESM test files.
+const dynamicImport: (specifier: string) => Promise<unknown> = new Function(
+    'specifier',
+    'return import(specifier);',
+) as (specifier: string) => Promise<unknown>;
+
+let pLimitModulePromise: Promise<typeof import('p-limit')> | null = null;
+
+function importPLimit(): Promise<typeof import('p-limit')> {
+    if (pLimitModulePromise === null) {
+        pLimitModulePromise = dynamicImport('p-limit') as Promise<
+            typeof import('p-limit')
+        >;
+    }
+
+    return pLimitModulePromise;
+}
 
 const ERR_NO_FILES = new Error('No test files found');
 
@@ -33,6 +55,7 @@ export async function resolveFiles(files: Array<string>): Promise<IFile[]> {
         throw ERR_NO_FILES;
     }
 
+    const {default: pLimit} = await importPLimit();
     const limit = pLimit(10);
 
     // Limit concurrent file reads

core/fs-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testring/fs-store",
-  "version": "0.8.11",
+  "version": "0.8.13",
   "description": "fs storage & operations service",
   "keywords": [
     "fs",
@@ -19,13 +19,13 @@
   "author": "RingCentral",
   "license": "MIT",
   "dependencies": {
-    "@testring/cli-config": "0.8.11",
-    "@testring/logger": "0.8.11",
-    "@testring/pluggable-module": "0.8.11",
-    "@testring/test-utils": "0.8.11",
-    "@testring/transport": "0.8.11",
-    "@testring/types": "0.8.11",
-    "@testring/utils": "0.8.11"
+    "@testring/cli-config": "0.8.13",
+    "@testring/logger": "0.8.13",
+    "@testring/pluggable-module": "0.8.13",
+    "@testring/test-utils": "0.8.13",
+    "@testring/transport": "0.8.13",
+    "@testring/types": "0.8.13",
+    "@testring/utils": "0.8.13"
   },
   "scripts": {
     "test": "mocha",