ava 6.1.0 → 6.1.2

package/lib/assert.js

@@ -417,6 +417,7 @@ export class Assertions {
 			}
 
 			let retval;
+			let threw = false;
 			let actual = null;
 			try {
 				retval = fn();
@@ -429,10 +430,11 @@ export class Assertions {
 					}));
 				}
 			} catch (error) {
+				threw = true;
 				actual = error;
 			}
 
-			if (!actual) {
+			if (!threw) {
 				throw fail(new AssertionError(message, {
 					assertion: 't.throws()',
 					formattedDetails: [formatWithLabel('Function returned:', retval)],

package/lib/cli.js

@@ -381,7 +381,9 @@ export default async function loadCli() { // eslint-disable-line complexity
 
 	let globs;
 	try {
-		globs = normalizeGlobs({files: conf.files, ignoredByWatcher: conf.watchMode?.ignoreChanges, extensions, providers});
+		globs = normalizeGlobs({
+			files: conf.files, ignoredByWatcher: conf.watchMode?.ignoreChanges, extensions, providers,
+		});
 	} catch (error) {
 		exit(error.message);
 	}

package/lib/create-chain.js

@@ -11,9 +11,7 @@ function startChain(name, call, defaults) {
 }
 
 function extendChain(previous, name, flag) {
-	if (!flag) {
-		flag = name;
-	}
+	flag ||= name;
 
 	const fn = (...args) => {
 		callWithFlag(previous, flag, args);
@@ -89,7 +87,9 @@ export default function createChain(fn, defaults, meta) {
 	// "todo" tests cannot be chained. Allow todo tests to be flagged as needing
 	// to be serial.
 	root.todo = startChain('test.todo', fn, {...defaults, type: 'test', todo: true});
-	root.serial.todo = startChain('test.serial.todo', fn, {...defaults, serial: true, type: 'test', todo: true});
+	root.serial.todo = startChain('test.serial.todo', fn, {
+		...defaults, serial: true, type: 'test', todo: true,
+	});
 
 	root.macro = options => {
 		if (typeof options === 'function') {

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

@@ -48,7 +48,9 @@ const resolveGlobs = async (projectDir, overrideExtensions, overrideFiles) => {
 	}
 
 	const {conf, providers} = await configCache.get(projectDir);
-	return buildGlobs({conf, providers, projectDir, overrideExtensions, overrideFiles});
+	return buildGlobs({
+		conf, providers, projectDir, overrideExtensions, overrideFiles,
+	});
 };
 
 const data = new Uint8Array(workerData.dataBuffer);

package/lib/load-config.js

@@ -35,7 +35,7 @@ const loadConfigFile = async ({projectDir, configFile}) => {
 
 function resolveConfigFile(configFile) {
 	if (configFile) {
-		configFile = path.resolve(configFile); // Relative to CWD
+		return path.resolve(configFile); // Relative to CWD
 	}
 
 	return configFile;
@@ -153,7 +153,9 @@ export async function loadConfig({configFile, resolveFrom = process.cwd(), defau
 		}
 	}
 
-	const config = {...defaults, nonSemVerExperiments: {}, ...fileConf, ...packageConf, projectDir, configFile};
+	const config = {
+		...defaults, nonSemVerExperiments: {}, ...fileConf, ...packageConf, projectDir, configFile,
+	};
 
 	const {nonSemVerExperiments: experiments} = config;
 	if (!isPlainObject(experiments)) {

package/lib/plugin-support/shared-worker-loader.js

@@ -238,7 +238,7 @@ try {
 		},
 	});
 } catch (error) {
-	fatal = fatal ?? error;
+	fatal ??= error;
 } finally {
 	if (fatal !== undefined) {
 		process.nextTick(() => {

package/lib/snapshot-manager.js

@@ -220,7 +220,9 @@ export function extractCompressedSnapshot(buffer, snapPath) {
 	const compressedOffset = sha256sumOffset + SHA_256_HASH_LENGTH;
 	const compressed = buffer.slice(compressedOffset);
 
-	return {version, compressed, sha256sumOffset, compressedOffset};
+	return {
+		version, compressed, sha256sumOffset, compressedOffset,
+	};
 }
 
 function decodeSnapshots(buffer, snapPath) {
@@ -290,11 +292,7 @@ class Manager {
 	}
 
 	recordSerialized({data, label, belongsTo, index}) {
-		let block = this.newBlocksByTitle.get(belongsTo);
-		if (!block) {
-			block = {snapshots: []};
-		}
-
+		const block = this.newBlocksByTitle.get(belongsTo) ?? {snapshots: []};
 		const {snapshots} = block;
 
 		if (index > snapshots.length) {
@@ -319,7 +317,9 @@ class Manager {
 
 		return () => { // Must be called in order!
 			this.hasChanges = true;
-			this.recordSerialized({data, label, belongsTo, index});
+			this.recordSerialized({
+				data, label, belongsTo, index,
+			});
 		};
 	}
 
@@ -338,11 +338,7 @@ class Manager {
 
 	skipSnapshot({belongsTo, index, deferRecording}) {
 		const oldBlock = this.oldBlocksByTitle.get(belongsTo);
-		let snapshot = oldBlock?.snapshots[index];
-
-		if (!snapshot) {
-			snapshot = {};
-		}
+		const snapshot = oldBlock?.snapshots[index] ?? {};
 
 		// Retain the label from the old snapshot, so as not to assume that the
 		// snapshot.skip() arguments are well-formed.

package/lib/test.js

@@ -2,7 +2,9 @@ import concordance from 'concordance';
 import isPromise from 'is-promise';
 import plur from 'plur';
 
-import {AssertionError, Assertions, checkAssertionMessage, getAssertionStack} from './assert.js';
+import {
+	AssertionError, Assertions, checkAssertionMessage, getAssertionStack,
+} from './assert.js';
 import concordanceOptions from './concordance-options.js';
 import nowAndTimers from './now-and-timers.cjs';
 import parseTestArgs from './parse-test-args.js';
@@ -277,7 +279,9 @@ export default class Test {
 
 			const {deferredSnapshotRecordings, error, logs, passed, assertCount, snapshotCount} = await attempt.run();
 			const errors = error ? [error] : [];
-			return {assertCount, deferredSnapshotRecordings, errors, logs, passed, snapshotCount, startingSnapshotCount};
+			return {
+				assertCount, deferredSnapshotRecordings, errors, logs, passed, snapshotCount, startingSnapshotCount,
+			};
 		};
 
 		this.assertCount = 0;

package/lib/watcher.js

@@ -7,7 +7,9 @@ import {nodeFileTrace} from '@vercel/nft';
 import createDebug from 'debug';
 
 import {chalk} from './chalk.js';
-import {applyTestFileFilter, classify, buildIgnoreMatcher, findTests} from './globs.js';
+import {
+	applyTestFileFilter, classify, buildIgnoreMatcher, findTests,
+} from './globs.js';
 import {levels as providerLevels} from './provider-manager.js';
 
 const debug = createDebug('ava:watcher');
@@ -34,7 +36,9 @@ export function available(projectDir) {
 
 export async function start({api, filter, globs, projectDir, providers, reporter, stdin, signal}) {
 	providers = providers.filter(({level}) => level >= providerLevels.ava6);
-	for await (const {files, ...runtimeOptions} of plan({api, filter, globs, projectDir, providers, stdin, abortSignal: signal})) {
+	for await (const {files, ...runtimeOptions} of plan({
+		api, filter, globs, projectDir, providers, stdin, abortSignal: signal,
+	})) {
 		await api.run({files, filter, runtimeOptions});
 		reporter.endRun();
 		reporter.lineWriter.writeLine(END_MESSAGE);
@@ -62,7 +66,9 @@ async function * plan({api, filter, globs, projectDir, providers, stdin, abortSi
 	const changeFromPath = path => {
 		const {isTest} = classify(path, cwdAndGlobs);
 		const stats = fileStats(path);
-		return {path, isTest, exists: stats !== undefined, isFile: stats?.isFile() ?? false};
+		return {
+			path, isTest, exists: stats !== undefined, isFile: stats?.isFile() ?? false,
+		};
 	};
 
 	// Begin a file trace in the background.
@@ -106,7 +112,8 @@ async function * plan({api, filter, globs, projectDir, providers, stdin, abortSi
 				case 'uncaught-exception':
 				case 'unhandled-rejection':
 				case 'worker-failed': {
-					failureCounts.set(evt.testFile, 1 + (failureCounts.get(evt.testFile) ?? 0));
+					const path = nodePath.relative(projectDir, evt.testFile);
+					failureCounts.set(path, 1 + (failureCounts.get(path) ?? 0));
 					break;
 				}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ava",
-	"version": "6.1.0",
+	"version": "6.1.2",
 	"description": "Node.js test runner that lets you develop with confidence.",
 	"license": "MIT",
 	"repository": "avajs/ava",
@@ -83,9 +83,9 @@
 		"typescript"
 	],
 	"dependencies": {
-		"@vercel/nft": "^0.24.4",
-		"acorn": "^8.11.2",
-		"acorn-walk": "^8.3.0",
+		"@vercel/nft": "^0.26.2",
+		"acorn": "^8.11.3",
+		"acorn-walk": "^8.3.2",
 		"ansi-styles": "^6.2.1",
 		"arrgv": "^1.0.2",
 		"arrify": "^3.0.0",
@@ -111,11 +111,11 @@
 		"matcher": "^5.0.0",
 		"memoize": "^10.0.0",
 		"ms": "^2.1.3",
-		"p-map": "^6.0.0",
+		"p-map": "^7.0.1",
 		"package-config": "^5.0.0",
 		"picomatch": "^3.0.1",
 		"plur": "^5.1.0",
-		"pretty-ms": "^8.0.0",
+		"pretty-ms": "^9.0.0",
 		"resolve-cwd": "^3.0.0",
 		"stack-utils": "^2.0.6",
 		"strip-ansi": "^7.1.0",
@@ -128,17 +128,17 @@
 		"@ava/test": "github:avajs/test",
 		"@ava/typescript": "^4.1.0",
 		"@sindresorhus/tsconfig": "^5.0.0",
-		"@types/node": "^20.10.3",
+		"@types/node": "^20.11.10",
 		"ansi-escapes": "^6.2.0",
-		"c8": "^8.0.1",
+		"c8": "^9.1.0",
 		"execa": "^8.0.1",
 		"expect": "^29.7.0",
 		"sinon": "^17.0.1",
-		"tap": "^18.6.1",
+		"tap": "^18.7.0",
 		"tempy": "^3.1.0",
-		"tsd": "^0.29.0",
-		"typescript": "~5.3.2",
-		"xo": "^0.56.0",
+		"tsd": "^0.30.4",
+		"typescript": "~5.3.3",
+		"xo": "^0.57.0",
 		"zen-observable": "^0.10.0"
 	},
 	"peerDependencies": {