refactor(betterer 🔧): fix test mock

package.json

@@ -59,6 +59,7 @@
     "@phenomnomnominal/commitlint-plugin": "^1.1.1",
     "@types/eslint": "^9.6.1",
     "@types/node": "^18.19.50",
+    "@typescript-eslint/parser": "^8.4.0",
     "@vitest/coverage-v8": "^1.5.0",
     "@vitest/ui": "^1.5.0",
     "cz-conventional-changelog": "^3.3.0",
@@ -86,4 +87,4 @@
       "path": "cz-conventional-changelog"
     }
   }
-}
+}

packages/betterer/src/config/validate.ts

@@ -1,5 +1,4 @@
 import assert from 'node:assert';
-import os from 'node:os';
 
 import { BettererError } from '@betterer/errors';
 
@@ -68,8 +67,9 @@ export function validateStringRegExpArray<Config extends object>(config: Config)
   return config;
 }
 
-export function validateWorkers(workers: number | boolean = true): number {
-  const totalCPUs = os.cpus().length;
+export async function validateWorkers(workers: number | boolean = true): Promise<number> {
+  const { cpus } = await import('node:os');
+  const totalCPUs = cpus().length;
 
   if (workers === true || isUndefined(workers)) {
     workers = totalCPUs >= 4 ? totalCPUs - 2 : false;

packages/betterer/src/context/config.ts

@@ -10,7 +10,7 @@ import {
   validateWorkers
 } from '../config/index.js';
 
-export function createContextConfig(options: BettererOptionsContext): BettererConfigContext {
+export async function createContextConfig(options: BettererOptionsContext): Promise<BettererConfigContext> {
   const ci = options.ci ?? false;
   const filters = toRegExps(toArray<string | RegExp>(options.filters));
   const excludes = toRegExps(toArray<string | RegExp>(options.excludes));
@@ -27,7 +27,7 @@ export function createContextConfig(options: BettererOptionsContext): BettererCo
   validateStringRegExpArray({ filters });
   validateStringArray({ includes });
 
-  const workers = validateWorkers(options.workers);
+  const workers = await validateWorkers(options.workers);
 
   return {
     ci,

packages/betterer/src/globals.ts

@@ -44,7 +44,7 @@ export async function createGlobals(
   let reporter = await loadDefaultReporter();
 
   try {
-    const configContext = createContextConfig(options);
+    const configContext = await createContextConfig(options);
     const configFS = await createFSConfig(options);
     const configReporter = await createReporterConfig(configFS, options);
     const configWatcher = createWatcherConfig(configFS, optionsWatch);

test/__snapshots__/config-validation.spec.ts.snap

@@ -315,23 +315,23 @@ Error: "workers" must be a number. Received \`{}\`.
 exports[`betterer > should throw when there is invalid config: { workers: -1 } 1`] = `
 [
   "
-Error: "workers" must be more than zero and not more than the number of available CPUs (10). To disable workers, set to \`false\`. Received \`-1\`.
+Error: "workers" must be more than zero and not more than the number of available CPUs (1). To disable workers, set to \`false\`. Received \`-1\`.
 ",
 ]
 `;
 
 exports[`betterer > should throw when there is invalid config: { workers: 1000 } 1`] = `
 [
   "
-Error: "workers" must be more than zero and not more than the number of available CPUs (10). To disable workers, set to \`false\`. Received \`1000\`.
+Error: "workers" must be more than zero and not more than the number of available CPUs (1). To disable workers, set to \`false\`. Received \`1000\`.
 ",
 ]
 `;
 
 exports[`betterer > should throw when there is invalid config: { workers: NaN } 1`] = `
 [
   "
-Error: "workers" must be more than zero and not more than the number of available CPUs (10). To disable workers, set to \`false\`. Received \`null\`.
+Error: "workers" must be more than zero and not more than the number of available CPUs (1). To disable workers, set to \`false\`. Received \`null\`.
 ",
 ]
 `;