ava 6.1.3 → 6.2.0

package/lib/assert.js

@@ -423,7 +423,7 @@ export class Assertions {
 				retval = fn();
 				if (isPromise(retval)) {
 					// Here isPromise() checks if something is "promise like". Cast to an actual promise.
-					Promise.resolve(retval).catch(noop);
+					Promise.resolve(retval).catch(noop); // eslint-disable-line promise/prefer-await-to-then
 					throw fail(new AssertionError(message, {
 						assertion: 't.throws()',
 						formattedDetails: [formatWithLabel('Function returned a promise. Use `t.throwsAsync()` instead:', retval)],
@@ -462,7 +462,10 @@ export class Assertions {
 			try {
 				assertMessage(message, 't.throwsAsync()');
 			} catch (error) {
-				Promise.resolve(thrower).catch(noop);
+				try {
+					await thrower;
+				} catch {}
+
 				throw error;
 			}
 
@@ -476,7 +479,10 @@ export class Assertions {
 			try {
 				expectations = validateExpectations('t.throwsAsync()', expectations, args.length, experiments);
 			} catch (error) {
-				Promise.resolve(thrower).catch(noop);
+				try {
+					await thrower;
+				} catch {}
+
 				throw fail(error);
 			}
 
@@ -484,7 +490,7 @@ export class Assertions {
 				// Record the stack before it gets lost in the promise chain.
 				const assertionStack = getAssertionStack();
 				// Handle "promise like" objects by casting to a real Promise.
-				const intermediate = Promise.resolve(promise).then(value => {
+				const intermediate = Promise.resolve(promise).then(value => { // eslint-disable-line promise/prefer-await-to-then
 					throw failPending(new AssertionError(message, {
 						assertion: 't.throwsAsync()',
 						assertionStack,
@@ -568,7 +574,10 @@ export class Assertions {
 			try {
 				assertMessage(message, 't.notThrowsAsync()');
 			} catch (error) {
-				Promise.resolve(nonThrower).catch(noop);
+				try {
+					await nonThrower;
+				} catch {}
+
 				throw error;
 			}
 
@@ -583,7 +592,7 @@ export class Assertions {
 				// Create an error object to record the stack before it gets lost in the promise chain.
 				const assertionStack = getAssertionStack();
 				// Handle "promise like" objects by casting to a real Promise.
-				const intermediate = Promise.resolve(promise).then(noop, error => {
+				const intermediate = Promise.resolve(promise).then(noop, error => { // eslint-disable-line promise/prefer-await-to-then
 					throw failPending(new AssertionError(message, {
 						assertion: 't.notThrowsAsync()',
 						assertionStack,

package/lib/cli.js

@@ -388,9 +388,15 @@ export default async function loadCli() { // eslint-disable-line complexity
 		exit(error.message);
 	}
 
+	const workerThreads = combined.workerThreads !== false;
+
 	let nodeArguments;
 	try {
 		nodeArguments = normalizeNodeArguments(conf.nodeArguments, argv['node-arguments']);
+		if (workerThreads && 'filterNodeArgumentsForWorkerThreads' in conf) {
+			const {filterNodeArgumentsForWorkerThreads: filter} = conf;
+			nodeArguments = nodeArguments.filter(argument => filter(argument));
+		}
 	} catch (error) {
 		exit(error.message);
 	}

package/lib/eslint-plugin-helper-worker.js

@@ -41,10 +41,9 @@ const buildGlobs = ({conf, providers, projectDir, overrideExtensions, overrideFi
 
 const resolveGlobs = async (projectDir, overrideExtensions, overrideFiles) => {
 	if (!configCache.has(projectDir)) {
-		configCache.set(projectDir, loadConfig({resolveFrom: projectDir}).then(async ({config: conf}) => {
-			const providers = await collectProviders({conf, projectDir});
-			return {conf, providers};
-		}));
+		const {config: conf} = await loadConfig({resolveFrom: projectDir});
+		const providers = await collectProviders({conf, projectDir});
+		configCache.set(projectDir, {conf, providers});
 	}
 
 	const {conf, providers} = await configCache.get(projectDir);

package/lib/load-config.js

@@ -157,6 +157,13 @@ export async function loadConfig({configFile, resolveFrom = process.cwd(), defau
 		...defaults, nonSemVerExperiments: {}, ...fileConf, ...packageConf, projectDir, configFile,
 	};
 
+	if (
+		'filterNodeArgumentsForWorkerThreads' in config
+		&& typeof config.filterNodeArgumentsForWorkerThreads !== 'function'
+	) {
+		throw new Error(`filterNodeArgumentsForWorkerThreads from ${fileForErrorMessage} must be a function`);
+	}
+
 	const {nonSemVerExperiments: experiments} = config;
 	if (!isPlainObject(experiments)) {
 		throw new Error(`nonSemVerExperiments from ${fileForErrorMessage} must be an object`);

package/lib/plugin-support/shared-workers.js

@@ -16,6 +16,11 @@ const waitForAvailable = async worker => {
 	}
 };
 
+const waitForError = async worker => {
+	const [error] = await events.once(worker, 'error');
+	return tagWorkerError(error);
+};
+
 function launchWorker(filename, initialData) {
 	if (launchedWorkers.has(filename)) {
 		return launchedWorkers.get(filename);
@@ -34,7 +39,7 @@ function launchWorker(filename, initialData) {
 	const launched = {
 		statePromises: {
 			available: waitForAvailable(worker),
-			error: events.once(worker, 'error').then(([error]) => tagWorkerError(error)),
+			error: waitForError(worker),
 		},
 		exited: false,
 		worker,
@@ -79,7 +84,7 @@ export async function observeWorkerProcess(fork, runStatus) {
 		}
 	};
 
-	fork.promise.finally(() => {
+	fork.promise.finally(() => { // eslint-disable-line promise/prefer-await-to-then
 		removeAllInstances();
 	});
 
@@ -94,7 +99,7 @@ export async function observeWorkerProcess(fork, runStatus) {
 			}
 		};
 
-		launched.statePromises.error.then(error => {
+		launched.statePromises.error.then(error => { // eslint-disable-line promise/prefer-await-to-then
 			launched.worker.off('message', handleWorkerMessage);
 			removeAllInstances();
 			runStatus.emitStateChange({type: 'shared-worker-error', err: serializeError(error)});
@@ -113,7 +118,7 @@ export async function observeWorkerProcess(fork, runStatus) {
 				port,
 			}, [port]);
 
-			fork.promise.finally(() => {
+			fork.promise.finally(() => { // eslint-disable-line promise/prefer-await-to-then
 				launched.worker.postMessage({
 					type: 'deregister-test-worker',
 					id: fork.threadId,

package/lib/runner.js

@@ -489,7 +489,7 @@ export default class Runner extends Emittery {
 
 		// Note that the hooks and tests always begin running asynchronously.
 		const beforePromise = this.runHooks(this.tasks.before, contextRef);
-		const serialPromise = beforePromise.then(beforeHooksOk => {
+		const serialPromise = beforePromise.then(beforeHooksOk => { // eslint-disable-line promise/prefer-await-to-then
 			// Don't run tests if a `before` hook failed.
 			if (!beforeHooksOk) {
 				return false;
@@ -511,7 +511,7 @@ export default class Runner extends Emittery {
 				return this.runTest(task, contextRef.copy());
 			}, true);
 		});
-		const concurrentPromise = Promise.all([beforePromise, serialPromise]).then(async ([beforeHooksOk, serialOk]) => {
+		const concurrentPromise = Promise.all([beforePromise, serialPromise]).then(async ([beforeHooksOk, serialOk]) => { // eslint-disable-line promise/prefer-await-to-then
 			// Don't run tests if a `before` hook failed, or if `failFast` is enabled
 			// and a previous serial test failed.
 			if (!beforeHooksOk || (!serialOk && this.failFast)) {

package/lib/test.js

@@ -590,7 +590,7 @@ export default class Test {
 				};
 
 				promise
-					.catch(error => {
+					.catch(error => { // eslint-disable-line promise/prefer-await-to-then
 						if (this.testFailure !== null && error === this.testFailure) {
 							return;
 						}
@@ -607,7 +607,7 @@ export default class Test {
 							}));
 						}
 					})
-					.then(() => resolve(this.finish()));
+					.then(() => resolve(this.finish())); // eslint-disable-line promise/prefer-await-to-then
 			});
 		}
 

package/lib/watcher.js

@@ -72,7 +72,7 @@ async function * plan({api, filter, globs, projectDir, providers, stdin, abortSi
 	};
 
 	// Begin a file trace in the background.
-	fileTracer.update(findTests(cwdAndGlobs).then(testFiles => testFiles.map(path => ({
+	fileTracer.update(findTests(cwdAndGlobs).then(testFiles => testFiles.map(path => ({ // eslint-disable-line promise/prefer-await-to-then
 		path: nodePath.relative(projectDir, path),
 		isTest: true,
 		exists: true,
@@ -187,7 +187,7 @@ async function * plan({api, filter, globs, projectDir, providers, stdin, abortSi
 		// If the file tracer is still analyzing dependencies, wait for that to
 		// complete.
 		if (fileTracer.busy !== null) {
-			fileTracer.busy.then(() => debounce.refresh());
+			fileTracer.busy.then(() => debounce.refresh()); // eslint-disable-line promise/prefer-await-to-then
 			takeCoverageForSelfTests?.();
 			return;
 		}
@@ -526,7 +526,7 @@ class FileTracer {
 	}
 
 	update(changes) {
-		const current = this.#update(changes).finally(() => {
+		const current = this.#update(changes).finally(() => { // eslint-disable-line promise/prefer-await-to-then
 			if (this.#pendingTrace === current) {
 				this.#pendingTrace = null;
 				this.#updateRunning = new Promise(resolve => {

package/lib/worker/base.js

@@ -1,6 +1,6 @@
 import {mkdir} from 'node:fs/promises';
 import {createRequire} from 'node:module';
-import {join as joinPath, resolve as resolvePath} from 'node:path';
+import path from 'node:path';
 import process from 'node:process';
 import {pathToFileURL} from 'node:url';
 import {workerData} from 'node:worker_threads';
@@ -89,7 +89,7 @@ const run = async options => {
 
 	refs.runnerChain = runner.chain;
 
-	channel.peerFailed.then(() => {
+	channel.peerFailed.then(() => { // eslint-disable-line promise/prefer-await-to-then
 		runner.interrupt();
 	});
 
@@ -187,7 +187,7 @@ const run = async options => {
 
 		// Try to load the module as a file, relative to the project directory.
 		// Match load() behavior.
-		const fullPath = resolvePath(projectDir, ref);
+		const fullPath = path.resolve(projectDir, ref);
 		try {
 			for (const extension of extensionsToLoadAsModules) {
 				if (fullPath.endsWith(`.${extension}`)) {
@@ -208,9 +208,9 @@ const run = async options => {
 
 	let importFromProject = async ref => {
 		// Do not use the cacheDir since it's not guaranteed to be inside node_modules.
-		const avaCacheDir = joinPath(projectDir, 'node_modules', '.cache', 'ava');
+		const avaCacheDir = path.join(projectDir, 'node_modules', '.cache', 'ava');
 		await mkdir(avaCacheDir, {recursive: true});
-		const stubPath = joinPath(avaCacheDir, 'import-from-project.mjs');
+		const stubPath = path.join(avaCacheDir, 'import-from-project.mjs');
 		await writeFileAtomic(stubPath, 'export const importFromProject = ref => import(ref);\n');
 		({importFromProject} = await import(pathToFileURL(stubPath)));
 		return importFromProject(ref);

package/lib/worker/channel.cjs

@@ -158,9 +158,9 @@ function registerSharedWorker(filename, initialData) {
 	// The attaching of message listeners will cause the port to be referenced by
 	// Node.js. In order to keep track, explicitly reference before attaching.
 	sharedWorkerHandle.ref();
-	const ready = selectAvaMessage(ourPort, 'ready').then(() => {
+	const ready = selectAvaMessage(ourPort, 'ready').then(() => { // eslint-disable-line promise/prefer-await-to-then
 		currentlyAvailable = error === null;
-	}).finally(() => {
+	}).finally(() => { // eslint-disable-line promise/prefer-await-to-then
 		// Once ready, it's up to user code to subscribe to messages, which (see
 		// below) causes us to reference the port.
 		sharedWorkerHandle.unref();
@@ -170,7 +170,7 @@ function registerSharedWorker(filename, initialData) {
 
 	// Errors are received over the test worker channel, not the message port
 	// dedicated to the shared worker.
-	events.once(channelEmitter, 'shared-worker-error').then(() => {
+	events.once(channelEmitter, 'shared-worker-error').then(() => { // eslint-disable-line promise/prefer-await-to-then
 		unsubscribe();
 		sharedWorkerHandle.forceUnref();
 		error = new Error('The shared worker is no longer available');

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ava",
-	"version": "6.1.3",
+	"version": "6.2.0",
 	"description": "Node.js test runner that lets you develop with confidence.",
 	"license": "MIT",
 	"repository": "avajs/ava",
@@ -36,7 +36,7 @@
 	},
 	"type": "module",
 	"engines": {
-		"node": "^18.18 || ^20.8 || ^21 || ^22"
+		"node": "^18.18 || ^20.8 || ^22 || >=23"
 	},
 	"scripts": {
 		"test": "./scripts/test.sh"
@@ -83,14 +83,14 @@
 		"typescript"
 	],
 	"dependencies": {
-		"@vercel/nft": "^0.26.2",
-		"acorn": "^8.11.3",
-		"acorn-walk": "^8.3.2",
+		"@vercel/nft": "^0.27.5",
+		"acorn": "^8.13.0",
+		"acorn-walk": "^8.3.4",
 		"ansi-styles": "^6.2.1",
 		"arrgv": "^1.0.2",
 		"arrify": "^3.0.0",
-		"callsites": "^4.1.0",
-		"cbor": "^9.0.1",
+		"callsites": "^4.2.0",
+		"cbor": "^9.0.2",
 		"chalk": "^5.3.0",
 		"chunkd": "^2.0.1",
 		"ci-info": "^4.0.0",
@@ -100,10 +100,10 @@
 		"common-path-prefix": "^3.0.0",
 		"concordance": "^5.0.4",
 		"currently-unhandled": "^0.4.1",
-		"debug": "^4.3.4",
-		"emittery": "^1.0.1",
-		"figures": "^6.0.1",
-		"globby": "^14.0.0",
+		"debug": "^4.3.7",
+		"emittery": "^1.0.3",
+		"figures": "^6.1.0",
+		"globby": "^14.0.2",
 		"ignore-by-default": "^2.1.0",
 		"indent-string": "^5.0.0",
 		"is-plain-object": "^5.0.0",
@@ -111,34 +111,34 @@
 		"matcher": "^5.0.0",
 		"memoize": "^10.0.0",
 		"ms": "^2.1.3",
-		"p-map": "^7.0.1",
+		"p-map": "^7.0.2",
 		"package-config": "^5.0.0",
-		"picomatch": "^3.0.1",
+		"picomatch": "^4.0.2",
 		"plur": "^5.1.0",
-		"pretty-ms": "^9.0.0",
+		"pretty-ms": "^9.1.0",
 		"resolve-cwd": "^3.0.0",
 		"stack-utils": "^2.0.6",
 		"strip-ansi": "^7.1.0",
 		"supertap": "^3.0.1",
 		"temp-dir": "^3.0.0",
-		"write-file-atomic": "^5.0.1",
+		"write-file-atomic": "^6.0.0",
 		"yargs": "^17.7.2"
 	},
 	"devDependencies": {
 		"@ava/test": "github:avajs/test",
-		"@ava/typescript": "^4.1.0",
-		"@sindresorhus/tsconfig": "^5.0.0",
-		"@types/node": "^20.11.10",
-		"ansi-escapes": "^6.2.0",
-		"c8": "^9.1.0",
-		"execa": "^8.0.1",
+		"@ava/typescript": "^5.0.0",
+		"@sindresorhus/tsconfig": "^5.1.1",
+		"@types/node": "^22.8.1",
+		"ansi-escapes": "^7.0.0",
+		"c8": "^10.1.2",
+		"execa": "^9.5.0",
 		"expect": "^29.7.0",
-		"sinon": "^17.0.1",
-		"tap": "^18.7.0",
+		"sinon": "^19.0.2",
+		"tap": "^20.0.3",
 		"tempy": "^3.1.0",
-		"tsd": "^0.30.4",
-		"typescript": "~5.3.3",
-		"xo": "^0.57.0",
+		"tsd": "^0.31.2",
+		"typescript": "~5.6.3",
+		"xo": "^0.59.3",
 		"zen-observable": "^0.10.0"
 	},
 	"peerDependencies": {
@@ -150,6 +150,6 @@
 		}
 	},
 	"volta": {
-		"node": "20.10.0"
+		"node": "22.10.0"
 	}
 }