ava 3.15.0 → 4.0.0

package/lib/reporters/format-serialized-error.js

@@ -1,27 +1,16 @@
-'use strict';
-const trimOffNewlines = require('trim-off-newlines');
-const chalk = require('../chalk').get();
+export default function formatSerializedError(error) {
+	const printMessage = error.values.length === 0
+		? Boolean(error.message)
+		: !error.values[0].label.startsWith(error.message);
 
-function formatSerializedError(error) {
-	const printMessage = error.values.length === 0 ?
-		Boolean(error.message) :
-		!error.values[0].label.startsWith(error.message);
-
-	if (error.statements.length === 0 && error.values.length === 0) {
+	if (error.values.length === 0) {
 		return {formatted: null, printMessage};
 	}
 
 	let formatted = '';
 	for (const value of error.values) {
-		formatted += `${value.label}\n\n${trimOffNewlines(value.formatted)}\n\n`;
-	}
-
-	for (const statement of error.statements) {
-		formatted += `${statement[0]}\n${chalk.grey('=>')} ${trimOffNewlines(statement[1])}\n\n`;
+		formatted += `${value.label}\n\n${value.formatted}\n\n`;
 	}
 
-	formatted = trimOffNewlines(formatted);
-	return {formatted, printMessage};
+	return {formatted: formatted.trim(), printMessage};
 }
-
-module.exports = formatSerializedError;

package/lib/reporters/improper-usage-messages.js

@@ -1,8 +1,7 @@
-'use strict';
-const chalk = require('../chalk').get();
-const pkg = require('../../package.json');
+import {chalk} from '../chalk.js';
+import pkg from '../pkg.cjs';
 
-exports.forError = error => {
+export default function buildMessage(error) {
 	if (!error.improperUsage) {
 		return null;
 	}
@@ -21,7 +20,7 @@ Visit the following URL for more details:
 	if (assertion === 'snapshot') {
 		const {name, snapPath} = error.improperUsage;
 
-		if (name === 'ChecksumError') {
+		if (name === 'ChecksumError' || name === 'InvalidSnapshotError') {
 			return `The snapshot file is corrupted.
 
 File path: ${chalk.yellow(snapPath)}
@@ -39,9 +38,9 @@ Please run AVA again with the ${chalk.cyan('--update-snapshots')} flag to upgrad
 
 		if (name === 'VersionMismatchError') {
 			const {snapVersion, expectedVersion} = error.improperUsage;
-			const upgradeMessage = snapVersion < expectedVersion ?
-				`Please run AVA again with the ${chalk.cyan('--update-snapshots')} flag to upgrade.` :
-				'You should upgrade AVA.';
+			const upgradeMessage = snapVersion < expectedVersion
+				? `Please run AVA again with the ${chalk.cyan('--update-snapshots')} flag to upgrade.`
+				: 'You should upgrade AVA.';
 
 			return `The snapshot file is v${snapVersion}, but only v${expectedVersion} is supported.
 
@@ -52,4 +51,4 @@ ${upgradeMessage}`;
 	}
 
 	return null;
-};
+}

package/lib/reporters/prefix-title.js

@@ -1,21 +1,23 @@
-'use strict';
-const path = require('path');
-const figures = require('figures');
-const chalk = require('../chalk').get();
+import path from 'node:path';
 
-const SEPERATOR = ' ' + chalk.gray.dim(figures.pointerSmall) + ' ';
+import figures from 'figures';
 
-module.exports = (base, file, title) => {
-	const prefix = file
+import {chalk} from '../chalk.js';
+
+const SEPARATOR = ' ' + chalk.gray.dim(figures.pointerSmall) + ' ';
+
+export default function prefixTitle(extensions, base, file, title) {
+	const parts = file
 		// Only replace base if it is found at the start of the path
 		.replace(base, (match, offset) => offset === 0 ? '' : match)
-		.replace(/\.spec/, '')
-		.replace(/\.test/, '')
-		.replace(/test-/g, '')
-		.replace(/\.js$/, '')
 		.split(path.sep)
-		.filter(p => p !== '__tests__')
-		.join(SEPERATOR);
+		.filter(p => p !== '__tests__');
+
+	const filename = parts.pop()
+		.replace(/\.spec\./, '.')
+		.replace(/\.test\./, '.')
+		.replace(/test-/, '')
+		.replace(new RegExp(`.(${extensions.join('|')})$`), '');
 
-	return prefix + SEPERATOR + title;
-};
+	return [...parts, filename, title].join(SEPARATOR);
+}

package/lib/reporters/tap.js

@@ -1,14 +1,13 @@
-'use strict';
-const os = require('os');
-const path = require('path');
+import os from 'node:os';
+import path from 'node:path';
 
-const plur = require('plur');
-const stripAnsi = require('strip-ansi');
-const supertap = require('supertap');
-const indentString = require('indent-string');
+import indentString from 'indent-string';
+import plur from 'plur';
+import stripAnsi from 'strip-ansi';
+import supertap from 'supertap';
 
-const beautifyStack = require('./beautify-stack');
-const prefixTitle = require('./prefix-title');
+import beautifyStack from './beautify-stack.js';
+import prefixTitle from './prefix-title.js';
 
 function dumpError(error) {
 	const object = {...error.object};
@@ -30,10 +29,7 @@ function dumpError(error) {
 		}
 
 		if (error.values.length > 0) {
-			object.values = error.values.reduce((acc, value) => { // eslint-disable-line unicorn/no-reduce
-				acc[value.label] = stripAnsi(value.formatted);
-				return acc;
-			}, {});
+			object.values = Object.fromEntries(error.values.map(({label, formatted}) => [label, stripAnsi(formatted)]));
 		}
 	}
 
@@ -49,10 +45,11 @@ function dumpError(error) {
 	return object;
 }
 
-class TapReporter {
+export default class TapReporter {
 	constructor(options) {
 		this.i = 0;
 
+		this.extensions = options.extensions;
 		this.stdStream = options.stdStream;
 		this.reportStream = options.reportStream;
 
@@ -65,7 +62,7 @@ class TapReporter {
 
 	startRun(plan) {
 		if (plan.files.length > 1) {
-			this.prefixTitle = (testFile, title) => prefixTitle(plan.filePathPrefix, testFile, title);
+			this.prefixTitle = (testFile, title) => prefixTitle(this.extensions, plan.filePathPrefix, testFile, title);
 		}
 
 		plan.status.on('stateChange', evt => this.consumeStateChange(evt));
@@ -80,7 +77,7 @@ class TapReporter {
 				failed: this.stats.failedTests + this.stats.remainingTests,
 				passed: this.stats.passedTests + this.stats.passedKnownFailingTests,
 				skipped: this.stats.skippedTests,
-				todo: this.stats.todoTests
+				todo: this.stats.todoTests,
 			}) + os.EOL);
 
 			if (this.stats.parallelRuns) {
@@ -93,7 +90,7 @@ class TapReporter {
 				failed: 0,
 				passed: 0,
 				skipped: 0,
-				todo: 0
+				todo: 0,
 			}) + os.EOL);
 		}
 	}
@@ -105,7 +102,7 @@ class TapReporter {
 			index: ++this.i,
 			passed: flags.passed,
 			skip: flags.skip,
-			todo: flags.todo
+			todo: flags.todo,
 		}) + os.EOL);
 	}
 
@@ -117,7 +114,7 @@ class TapReporter {
 			index: ++this.i,
 			passed: false,
 			skip: false,
-			todo: false
+			todo: false,
 		}) + os.EOL);
 	}
 
@@ -132,11 +129,11 @@ class TapReporter {
 	}
 
 	writeTimeout(evt) {
-		const err = new Error(`Exited because no new tests completed within the last ${evt.period}ms of inactivity`);
+		const error = new Error(`Exited because no new tests completed within the last ${evt.period}ms of inactivity`);
 
 		for (const [testFile, tests] of evt.pendingTests) {
 			for (const title of tests) {
-				this.writeTest({testFile, title, err}, {passed: false, todo: false, skip: false});
+				this.writeTest({testFile, title, err: error}, {passed: false, todo: false, skip: false});
 			}
 		}
 	}
@@ -168,9 +165,6 @@ class TapReporter {
 					this.writeTest(evt, {passed: false, todo: true, skip: false});
 				}
 
-				break;
-			case 'snapshot-error':
-				this.writeComment(evt, {title: 'Could not update snapshots'});
 				break;
 			case 'stats':
 				this.stats = evt.stats;
@@ -219,4 +213,3 @@ class TapReporter {
 		}
 	}
 }
-module.exports = TapReporter;

package/lib/run-status.js

@@ -1,17 +1,21 @@
-'use strict';
-const Emittery = require('emittery');
-const cloneDeep = require('lodash/cloneDeep');
+import v8 from 'node:v8';
 
-class RunStatus extends Emittery {
-	constructor(files, parallelRuns) {
+import Emittery from 'emittery';
+
+const copyStats = stats => v8.deserialize(v8.serialize(stats));
+
+export default class RunStatus extends Emittery {
+	constructor(files, parallelRuns, selectionInsights) {
 		super();
 
 		this.pendingTests = new Map();
 
-		this.emptyParallelRun = parallelRuns &&
-			parallelRuns.currentFileCount === 0 &&
-			parallelRuns.totalRuns > 1 &&
-			files > 0;
+		this.emptyParallelRun = parallelRuns
+			&& parallelRuns.currentFileCount === 0
+			&& parallelRuns.totalRuns > 1
+			&& files > 0;
+
+		this.selectionInsights = selectionInsights;
 
 		this.stats = {
 			byFile: new Map(),
@@ -32,7 +36,7 @@ class RunStatus extends Emittery {
 			timeouts: 0,
 			todoTests: 0,
 			uncaughtExceptions: 0,
-			unhandledRejections: 0
+			unhandledRejections: 0,
 		};
 	}
 
@@ -51,7 +55,7 @@ class RunStatus extends Emittery {
 			todoTests: 0,
 			uncaughtExceptions: 0,
 			unhandledRejections: 0,
-			...stats
+			...stats,
 		});
 
 		this.pendingTests.set(testFile, new Set());
@@ -147,7 +151,7 @@ class RunStatus extends Emittery {
 		}
 
 		if (changedStats) {
-			this.emit('stateChange', {type: 'stats', stats: cloneDeep(stats)});
+			this.emit('stateChange', {type: 'stats', stats: copyStats(stats)});
 		}
 
 		this.emit('stateChange', event);
@@ -163,15 +167,15 @@ class RunStatus extends Emittery {
 		}
 
 		if (
-			this.stats.declaredTests === 0 ||
-			this.stats.internalErrors > 0 ||
-			this.stats.failedHooks > 0 ||
-			this.stats.failedTests > 0 ||
-			this.stats.failedWorkers > 0 ||
-			this.stats.sharedWorkerErrors > 0 ||
-			this.stats.timeouts > 0 ||
-			this.stats.uncaughtExceptions > 0 ||
-			this.stats.unhandledRejections > 0
+			this.stats.declaredTests === 0
+			|| this.stats.internalErrors > 0
+			|| this.stats.failedHooks > 0
+			|| this.stats.failedTests > 0
+			|| this.stats.failedWorkers > 0
+			|| this.stats.sharedWorkerErrors > 0
+			|| this.stats.timeouts > 0
+			|| this.stats.uncaughtExceptions > 0
+			|| this.stats.unhandledRejections > 0
 		) {
 			return 1;
 		}
@@ -194,6 +198,8 @@ class RunStatus extends Emittery {
 			this.pendingTests.get(event.testFile).delete(event.title);
 		}
 	}
-}
 
-module.exports = RunStatus;
+	getFailedTestFiles() {
+		return [...this.stats.byFile].filter(statByFile => statByFile[1].failedTests).map(statByFile => statByFile[0]);
+	}
+}