ava 3.4.0 → 3.6.0

package/eslint-plugin-helper.js

@@ -49,7 +49,7 @@ function load(projectDir, overrides) {
 	let helperPatterns = [];
 	if (overrides && overrides.helpers !== undefined) {
 		if (!Array.isArray(overrides.helpers) || overrides.helpers.length === 0) {
-			throw new Error('The \'helpers\' override must be an array containing glob patterns.');
+			throw new Error('The ’helpers’ override must be an array containing glob patterns.');
 		}
 
 		helperPatterns = normalizePatterns(overrides.helpers);

package/index.d.ts

@@ -1,11 +1,11 @@
 export interface Subscribable {
 	subscribe(observer: {
-		error(err: any): void,
-		complete(): void,
-	}): void
+		error(err: any): void;
+		complete(): void;
+	}): void;
 }
 
-export type Constructor = (new (...args: Array<any>) => any);
+export type Constructor = (new (...args: any[]) => any);
 
 /** Specify one or more expectations the thrown error must satisfy. */
 export type ThrowsExpectation = {
@@ -29,8 +29,8 @@ export type CommitDiscardOptions = {
 	/**
 	 * Whether the logs should be included in those of the parent test.
 	 */
-	retainLogs?: boolean
-}
+	retainLogs?: boolean;
+};
 
 /** Options that can be passed to the `t.snapshot()` assertion. */
 export type SnapshotOptions = {
@@ -308,6 +308,9 @@ export interface ExecutionContext<Context = unknown> extends Assertions {
 	/** Title of the test or hook. */
 	readonly title: string;
 
+	/** Whether the test has passed. Only accurate in afterEach hooks. */
+	readonly passed: boolean;
+
 	log: LogFn;
 	plan: PlanFn;
 	timeout: TimeoutFn;
@@ -316,10 +319,10 @@ export interface ExecutionContext<Context = unknown> extends Assertions {
 
 export interface LogFn {
 	/** Log one or more values. */
-	(...values: Array<any>): void;
+	(...values: any[]): void;
 
 	/** Skip logging. */
-	skip(...values: Array<any>): void;
+	skip(...values: any[]): void;
 }
 
 export interface PlanFn {
@@ -343,30 +346,30 @@ export interface TimeoutFn {
 
 export interface TryFn<Context = unknown> {
 	/**
-	 * Requires opt-in. Attempt to run some assertions. The result must be explicitly committed or discarded or else
+	 * Attempt to run some assertions. The result must be explicitly committed or discarded or else
 	 * the test will fail. A macro may be provided. The title may help distinguish attempts from
 	 * one another.
 	 */
 	<Args extends any[]>(title: string, fn: EitherMacro<Args, Context>, ...args: Args): Promise<TryResult>;
 
 	/**
-	* Requires opt-in. Attempt to run some assertions. The result must be explicitly committed or discarded or else
+	 * Attempt to run some assertions. The result must be explicitly committed or discarded or else
 	 * the test will fail. A macro may be provided. The title may help distinguish attempts from
 	 * one another.
 	 */
-	<Args extends any[]>(title: string, fn: [EitherMacro<Args, Context>, ...EitherMacro<Args, Context>[]], ...args: Args): Promise<TryResult[]>;
+	<Args extends any[]>(title: string, fn: [EitherMacro<Args, Context>, ...Array<EitherMacro<Args, Context>>], ...args: Args): Promise<TryResult[]>;
 
 	/**
-	* Requires opt-in. Attempt to run some assertions. The result must be explicitly committed or discarded or else
-	* the test will fail. A macro may be provided.
-	*/
+	 * Attempt to run some assertions. The result must be explicitly committed or discarded or else
+	 * the test will fail. A macro may be provided.
+	 */
 	<Args extends any[]>(fn: EitherMacro<Args, Context>, ...args: Args): Promise<TryResult>;
 
 	/**
-	* Requires opt-in. Attempt to run some assertions. The result must be explicitly committed or discarded or else
-	* the test will fail. A macro may be provided.
-	*/
-	<Args extends any[]>(fn: [EitherMacro<Args, Context>, ...EitherMacro<Args, Context>[]], ...args: Args): Promise<TryResult[]>;
+	 * Attempt to run some assertions. The result must be explicitly committed or discarded or else
+	 * the test will fail. A macro may be provided.
+	 */
+	<Args extends any[]>(fn: [EitherMacro<Args, Context>, ...Array<EitherMacro<Args, Context>>], ...args: Args): Promise<TryResult[]>;
 }
 
 export interface AssertionError extends Error {}
@@ -427,32 +430,32 @@ export type Macro<Args extends any[], Context = unknown> = UntitledMacro<Args, C
 	 * the title provided when the test or hook was declared. Also receives the remaining test arguments.
 	 */
 	title?: (providedTitle: string | undefined, ...args: Args) => string;
-}
+};
 
 export type EitherMacro<Args extends any[], Context> = Macro<Args, Context> | UntitledMacro<Args, Context>;
 
 /** Alias for a single macro, or an array of macros. */
-export type OneOrMoreMacros<Args extends any[], Context> = EitherMacro<Args, Context> | [EitherMacro<Args, Context>, ...EitherMacro<Args, Context>[]];
+export type OneOrMoreMacros<Args extends any[], Context> = EitherMacro<Args, Context> | [EitherMacro<Args, Context>, ...Array<EitherMacro<Args, Context>>];
 
 /** A reusable test or hook implementation, for tests & hooks declared with the `.cb` modifier. */
-export type UntitledCbMacro<Args extends any[], Context = unknown> = (t: CbExecutionContext<Context>, ...args: Args) => ImplementationResult
+export type UntitledCbMacro<Args extends any[], Context = unknown> = (t: CbExecutionContext<Context>, ...args: Args) => ImplementationResult;
 
 /** A reusable test or hook implementation, for tests & hooks declared with the `.cb` modifier. */
 export type CbMacro<Args extends any[], Context = unknown> = UntitledCbMacro<Args, Context> & {
 	title?: (providedTitle: string | undefined, ...args: Args) => string;
-}
+};
 
 export type EitherCbMacro<Args extends any[], Context> = CbMacro<Args, Context> | UntitledCbMacro<Args, Context>;
 
 /** Alias for a single macro, or an array of macros, used for tests & hooks declared with the `.cb` modifier. */
-export type OneOrMoreCbMacros<Args extends any[], Context> = EitherCbMacro<Args, Context> | [EitherCbMacro<Args, Context>, ...EitherCbMacro<Args, Context>[]];
+export type OneOrMoreCbMacros<Args extends any[], Context> = EitherCbMacro<Args, Context> | [EitherCbMacro<Args, Context>, ...Array<EitherCbMacro<Args, Context>>];
 
 export interface TestInterface<Context = unknown> {
 	/** Declare a concurrent test. */
 	(title: string, implementation: Implementation<Context>): void;
 
 	/** Declare a concurrent test that uses one or more macros. Additional arguments are passed to the macro. */
-	<T extends any[]>(title: string, macros: OneOrMoreMacros<T, Context>, ...rest: T): void
+	<T extends any[]>(title: string, macros: OneOrMoreMacros<T, Context>, ...rest: T): void;
 
 	/** Declare a concurrent test that uses one or more macros. The macro is responsible for generating a unique test title. */
 	<T extends any[]>(macros: OneOrMoreMacros<T, Context>, ...rest: T): void;

package/lib/api.js

@@ -3,7 +3,6 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 const commonPathPrefix = require('common-path-prefix');
-const isCi = require('is-ci');
 const resolveCwd = require('resolve-cwd');
 const debounce = require('lodash/debounce');
 const arrify = require('arrify');
@@ -13,6 +12,7 @@ const Emittery = require('emittery');
 const pMap = require('p-map');
 const tempDir = require('temp-dir');
 const globs = require('./globs');
+const isCi = require('./is-ci');
 const RunStatus = require('./run-status');
 const fork = require('./fork');
 const serializeError = require('./serialize-error');
@@ -22,7 +22,7 @@ function resolveModules(modules) {
 		const modulePath = resolveCwd.silent(name);
 
 		if (modulePath === undefined) {
-			throw new Error(`Could not resolve required module '${name}'`);
+			throw new Error(`Could not resolve required module ’${name}’`);
 		}
 
 		return modulePath;

package/lib/assert.js

@@ -29,34 +29,34 @@ function formatPowerAssertValue(value) {
 	return concordance.format(value, concordanceOptions);
 }
 
-const hasOwnProperty = (obj, prop) => Object.prototype.hasOwnProperty.call(obj, prop);
+const hasOwnProperty = (object, prop) => Object.prototype.hasOwnProperty.call(object, prop);
 const noop = () => {};
 const notImplemented = () => {
 	throw new Error('not implemented');
 };
 
 class AssertionError extends Error {
-	constructor(opts) {
-		super(opts.message || '');
+	constructor(options) {
+		super(options.message || '');
 		this.name = 'AssertionError';
 
-		this.assertion = opts.assertion;
-		this.fixedSource = opts.fixedSource;
-		this.improperUsage = opts.improperUsage || false;
-		this.actualStack = opts.actualStack;
-		this.operator = opts.operator;
-		this.values = opts.values || [];
+		this.assertion = options.assertion;
+		this.fixedSource = options.fixedSource;
+		this.improperUsage = options.improperUsage || false;
+		this.actualStack = options.actualStack;
+		this.operator = options.operator;
+		this.values = options.values || [];
 
 		// Raw expected and actual objects are stored for custom reporters
 		// (such as wallaby.js), that manage worker processes directly and
 		// use the values for custom diff views
-		this.raw = opts.raw;
+		this.raw = options.raw;
 
 		// Reserved for power-assert statements
 		this.statements = [];
 
-		if (opts.savedError) {
-			this.savedError = opts.savedError;
+		if (options.savedError) {
+			this.savedError = options.savedError;
 		} else {
 			this.savedError = getErrorWithLongStackTrace();
 		}
@@ -72,8 +72,8 @@ function getErrorWithLongStackTrace() {
 	return err;
 }
 
-function validateExpectations(assertion, expectations, numArgs) { // eslint-disable-line complexity
-	if (numArgs === 1 || expectations === null || expectations === undefined) {
+function validateExpectations(assertion, expectations, numberArgs) { // eslint-disable-line complexity
+	if (numberArgs === 1 || expectations === null || expectations === undefined) {
 		expectations = {};
 	} else if (
 		typeof expectations === 'function' ||

package/lib/cli.js

@@ -5,8 +5,8 @@ const updateNotifier = require('update-notifier');
 const figures = require('figures');
 const arrify = require('arrify');
 const yargs = require('yargs');
-const isCi = require('is-ci');
 const readPkg = require('read-pkg');
+const isCi = require('./is-ci');
 const loadConfig = require('./load-config');
 
 function exit(message) {
@@ -106,8 +106,8 @@ exports.run = async () => { // eslint-disable-line complexity
 			'strip-aliased': true,
 			'unknown-options-as-args': false
 		})
-		.usage('$0 [<pattern>...]')
-		.usage('$0 debug [<pattern>...]')
+		.usage('$0 [<pattern>...]') // eslint-disable-line unicorn/string-content
+		.usage('$0 debug [<pattern>...]') // eslint-disable-line unicorn/string-content
 		.usage('$0 reset-cache')
 		.options({
 			color: {
@@ -118,19 +118,24 @@ exports.run = async () => { // eslint-disable-line complexity
 				description: 'Specific JavaScript file for AVA to read its config from, instead of using package.json or ava.config.* files'
 			}
 		})
-		.command('* [<pattern>...]', 'Run tests', yargs => yargs.options(FLAGS).positional('pattern', {
+		.command('* [<pattern>...]', 'Run tests', yargs => yargs.options(FLAGS).positional('pattern', { // eslint-disable-line unicorn/string-content
 			array: true,
 			describe: 'Glob patterns to select what test files to run. Leave empty if you want AVA to run all test files instead',
 			type: 'string'
 		}))
 		.command(
-			'debug [<pattern>...]',
+			'debug [<pattern>...]', // eslint-disable-line unicorn/string-content
 			'Activate Node.js inspector and run a single test file',
 			yargs => yargs.options(FLAGS).options({
 				break: {
 					description: 'Break before the test file is loaded',
 					type: 'boolean'
 				},
+				host: {
+					default: '127.0.0.1',
+					description: 'Address or hostname through which you can connect to the inspector',
+					type: 'string'
+				},
 				port: {
 					default: 9229,
 					description: 'Port on which you can connect to the inspector',
@@ -145,12 +150,13 @@ exports.run = async () => { // eslint-disable-line complexity
 				debug = {
 					break: argv.break === true,
 					files: argv.pattern,
+					host: argv.host,
 					port: argv.port
 				};
 			})
 		.command(
 			'reset-cache',
-			'Reset AVA\'s compilation cache and exit',
+			'Reset AVA’s compilation cache and exit',
 			yargs => yargs,
 			() => {
 				resetCache = true;
@@ -243,11 +249,11 @@ exports.run = async () => { // eslint-disable-line complexity
 	}
 
 	if (Reflect.has(conf, 'helpers')) {
-		exit('AVA no longer compiles helpers. Add exclusion patterns to the \'files\' configuration and specify \'compileAsTests\' in the Babel options instead.');
+		exit('AVA no longer compiles helpers. Add exclusion patterns to the ’files’ configuration and specify ’compileAsTests’ in the Babel options instead.');
 	}
 
 	if (Reflect.has(conf, 'sources')) {
-		exit('\'sources\' has been removed. Use \'ignoredByWatcher\' to provide glob patterns of files that the watcher should ignore.');
+		exit('’sources’ has been removed. Use ’ignoredByWatcher’ to provide glob patterns of files that the watcher should ignore.');
 	}
 
 	const ciParallelVars = require('ci-parallel-vars');

package/lib/concordance-options.js

@@ -88,7 +88,7 @@ const colorTheme = {
 	string: {
 		open: ansiStyles.blue.open,
 		close: ansiStyles.blue.close,
-		line: {open: forceColor.blue('\''), close: forceColor.blue('\'')},
+		line: {open: forceColor.blue('\''), close: forceColor.blue('\'')}, // eslint-disable-line unicorn/string-content
 		multiline: {start: forceColor.blue('`'), end: forceColor.blue('`')},
 		controlPicture: ansiStyles.grey,
 		diff: {

package/lib/create-chain.js

@@ -11,35 +11,35 @@ function startChain(name, call, defaults) {
 	return fn;
 }
 
-function extendChain(prev, name, flag) {
+function extendChain(previous, name, flag) {
 	if (!flag) {
 		flag = name;
 	}
 
 	const fn = (...args) => {
-		callWithFlag(prev, flag, args);
+		callWithFlag(previous, flag, args);
 	};
 
-	const fullName = `${chainRegistry.get(prev).fullName}.${name}`;
+	const fullName = `${chainRegistry.get(previous).fullName}.${name}`;
 	Object.defineProperty(fn, 'name', {value: fullName});
-	prev[name] = fn;
+	previous[name] = fn;
 
-	chainRegistry.set(fn, {flag, fullName, prev});
+	chainRegistry.set(fn, {flag, fullName, prev: previous});
 	return fn;
 }
 
-function callWithFlag(prev, flag, args) {
+function callWithFlag(previous, flag, args) {
 	const combinedFlags = {[flag]: true};
 	do {
-		const step = chainRegistry.get(prev);
+		const step = chainRegistry.get(previous);
 		if (step.call) {
 			step.call({...step.defaults, ...combinedFlags}, args);
-			prev = null;
+			previous = null;
 		} else {
 			combinedFlags[step.flag] = true;
-			prev = step.prev;
+			previous = step.prev;
 		}
-	} while (prev);
+	} while (previous);
 }
 
 function createHookChain(hook, isAfterHook) {

package/lib/environment-variables.js

@@ -6,7 +6,7 @@ function validateEnvironmentVariables(environmentVariables) {
 
 	for (const value of Object.values(environmentVariables)) {
 		if (typeof value !== 'string') {
-			throw new TypeError('The \'environmentVariables\' configuration must be an object containing string values.');
+			throw new TypeError('The ’environmentVariables’ configuration must be an object containing string values.');
 		}
 	}
 

package/lib/extensions.js

@@ -21,7 +21,7 @@ module.exports = (configuredExtensions, providers = []) => {
 	}
 
 	if (duplicates.size > 0) {
-		throw new Error(`Unexpected duplicate extensions in options: '${[...duplicates].join('\', \'')}'.`);
+		throw new Error(`Unexpected duplicate extensions in options: ’${[...duplicates].join('’, ’')}’.`);
 	}
 
 	// Unless the default was used by providers, as long as the extensions aren't explicitly set, set the default.

package/lib/fork.js

@@ -14,7 +14,7 @@ const AVA_PATH = path.resolve(__dirname, '..');
 
 const workerPath = require.resolve('./worker/subprocess');
 
-module.exports = (file, opts, execArgv = process.execArgv) => {
+module.exports = (file, options, execArgv = process.execArgv) => {
 	let finished = false;
 
 	const emitter = new Emittery();
@@ -24,16 +24,16 @@ module.exports = (file, opts, execArgv = process.execArgv) => {
 		}
 	};
 
-	opts = {
+	options = {
 		file,
 		baseDir: process.cwd(),
-		...opts
+		...options
 	};
 
-	const subprocess = childProcess.fork(workerPath, opts.workerArgv, {
-		cwd: opts.projectDir,
+	const subprocess = childProcess.fork(workerPath, options.workerArgv, {
+		cwd: options.projectDir,
 		silent: true,
-		env: {NODE_ENV: 'test', ...process.env, ...opts.environmentVariables, AVA_PATH},
+		env: {NODE_ENV: 'test', ...process.env, ...options.environmentVariables, AVA_PATH},
 		execArgv
 	});
 
@@ -64,7 +64,7 @@ module.exports = (file, opts, execArgv = process.execArgv) => {
 			}
 
 			if (message.ava.type === 'ready-for-options') {
-				send({type: 'options', options: opts});
+				send({type: 'options', options});
 				return;
 			}
 

package/lib/globs.js

@@ -46,11 +46,11 @@ exports.normalizePatterns = normalizePatterns;
 
 function normalizeGlobs({extensions, files: filePatterns, ignoredByWatcher: ignoredByWatcherPatterns, providers}) {
 	if (filePatterns !== undefined && (!Array.isArray(filePatterns) || filePatterns.length === 0)) {
-		throw new Error('The \'files\' configuration must be an array containing glob patterns.');
+		throw new Error('The ’files’ configuration must be an array containing glob patterns.');
 	}
 
 	if (ignoredByWatcherPatterns !== undefined && (!Array.isArray(ignoredByWatcherPatterns) || ignoredByWatcherPatterns.length === 0)) {
-		throw new Error('The \'ignoredByWatcher\' configuration must be an array containing glob patterns.');
+		throw new Error('The ’ignoredByWatcher’ configuration must be an array containing glob patterns.');
 	}
 
 	const extensionPattern = buildExtensionPattern(extensions);

package/lib/is-ci.js

@@ -0,0 +1,5 @@
+const info = require('ci-info');
+
+const {AVA_FORCE_CI} = process.env;
+
+module.exports = AVA_FORCE_CI === 'not-ci' ? false : AVA_FORCE_CI === 'ci' || info.isCI;

package/lib/load-config.js

@@ -12,7 +12,7 @@ const EXPERIMENTS = new Set();
 // *Very* rudimentary support for loading ava.config.js files containing an `export default` statement.
 const evaluateJsConfig = configFile => {
 	const contents = fs.readFileSync(configFile, 'utf8');
-	const script = new vm.Script(`'use strict';(()=>{let __export__;\n${contents.replace(/export default/g, '__export__ =')};return __export__;})()`, {
+	const script = new vm.Script(`'use strict';(()=>{let __export__;\n${contents.replace(/export default/g, '__export__ =')};return __export__;})()`, { // eslint-disable-line unicorn/string-content
 		filename: configFile,
 		lineOffset: -1
 	});
@@ -137,7 +137,7 @@ function loadConfig({configFile, resolveFrom = process.cwd(), defaults = {}} = {
 		}
 
 		if ('ava' in fileConf) {
-			throw new Error(`Encountered 'ava' property in ${fileForErrorMessage}; avoid wrapping the configuration`);
+			throw new Error(`Encountered ’ava’ property in ${fileForErrorMessage}; avoid wrapping the configuration`);
 		}
 	}
 

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

@@ -30,7 +30,7 @@ Please run AVA again with the ${chalk.cyan('--update-snapshots')} flag to recrea
 		}
 
 		if (name === 'LegacyError') {
-			return `The snapshot file was created with AVA 0.19. It's not supported by this AVA version.
+			return `The snapshot file was created with AVA 0.19. It’s not supported by this AVA version.
 
 File path: ${chalk.yellow(snapPath)}
 

package/lib/reporters/mini.js

@@ -51,23 +51,23 @@ class LineWriter extends stream.Writable {
 		callback();
 	}
 
-	_writeWithSpinner(str) {
+	_writeWithSpinner(string) {
 		if (!this.spinner.id) {
-			this.dest.write(str);
+			this.dest.write(string);
 			return;
 		}
 
-		this.lastSpinnerText = str;
+		this.lastSpinnerText = string;
 		// Ignore whitespace at the end of the chunk. We're continiously rewriting
 		// the last line through the spinner. Also be careful to remove the indent
 		// as the spinner adds its own.
-		this.spinner.text = str.trimEnd().slice(2);
+		this.spinner.text = string.trimEnd().slice(2);
 		this.spinner.render();
 	}
 
-	writeLine(str) {
-		if (str) {
-			this.write(indentString(str, 2) + os.EOL);
+	writeLine(string) {
+		if (string) {
+			this.write(indentString(string, 2) + os.EOL);
 		} else {
 			this.write(os.EOL);
 		}
@@ -237,14 +237,14 @@ class MiniReporter {
 		}
 	}
 
-	writeWithCounts(str) {
+	writeWithCounts(string) {
 		if (!this.stats) {
-			return this.lineWriter.writeLine(str);
+			return this.lineWriter.writeLine(string);
 		}
 
-		str = str || '';
-		if (str !== '') {
-			str += os.EOL;
+		string = string || '';
+		if (string !== '') {
+			string += os.EOL;
 		}
 
 		let firstLinePostfix = this.watching ?
@@ -252,33 +252,33 @@ class MiniReporter {
 			'';
 
 		if (this.stats.passedTests > 0) {
-			str += os.EOL + colors.pass(`${this.stats.passedTests} passed`) + firstLinePostfix;
+			string += os.EOL + colors.pass(`${this.stats.passedTests} passed`) + firstLinePostfix;
 			firstLinePostfix = '';
 		}
 
 		if (this.stats.passedKnownFailingTests > 0) {
-			str += os.EOL + colors.error(`${this.stats.passedKnownFailingTests} ${plur('known failure', this.stats.passedKnownFailingTests)}`);
+			string += os.EOL + colors.error(`${this.stats.passedKnownFailingTests} ${plur('known failure', this.stats.passedKnownFailingTests)}`);
 		}
 
 		if (this.stats.failedHooks > 0) {
-			str += os.EOL + colors.error(`${this.stats.failedHooks} ${plur('hook', this.stats.failedHooks)} failed`) + firstLinePostfix;
+			string += os.EOL + colors.error(`${this.stats.failedHooks} ${plur('hook', this.stats.failedHooks)} failed`) + firstLinePostfix;
 			firstLinePostfix = '';
 		}
 
 		if (this.stats.failedTests > 0) {
-			str += os.EOL + colors.error(`${this.stats.failedTests} ${plur('test', this.stats.failedTests)} failed`) + firstLinePostfix;
+			string += os.EOL + colors.error(`${this.stats.failedTests} ${plur('test', this.stats.failedTests)} failed`) + firstLinePostfix;
 			firstLinePostfix = '';
 		}
 
 		if (this.stats.skippedTests > 0) {
-			str += os.EOL + colors.skip(`${this.stats.skippedTests} skipped`);
+			string += os.EOL + colors.skip(`${this.stats.skippedTests} skipped`);
 		}
 
 		if (this.stats.todoTests > 0) {
-			str += os.EOL + colors.todo(`${this.stats.todoTests} todo`);
+			string += os.EOL + colors.todo(`${this.stats.todoTests} todo`);
 		}
 
-		this.lineWriter.writeLine(str);
+		this.lineWriter.writeLine(string);
 	}
 
 	writeErr(evt) {
@@ -379,13 +379,13 @@ class MiniReporter {
 		cliCursor.show(this.reportStream);
 
 		if (!this.stats) {
-			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn't find any files to test`));
+			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn’t find any files to test`));
 			this.lineWriter.writeLine();
 			return;
 		}
 
 		if (this.matching && this.stats.selectedTests === 0) {
-			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn't find any matching tests`));
+			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn’t find any matching tests`));
 			this.lineWriter.writeLine();
 			return;
 		}

package/lib/reporters/tap.js

@@ -10,26 +10,26 @@ const indentString = require('indent-string');
 const prefixTitle = require('./prefix-title');
 
 function dumpError(error) {
-	const obj = {...error.object};
+	const object = {...error.object};
 	if (error.name) {
-		obj.name = error.name;
+		object.name = error.name;
 	}
 
 	if (error.message) {
-		obj.message = error.message;
+		object.message = error.message;
 	}
 
 	if (error.avaAssertionError) {
 		if (error.assertion) {
-			obj.assertion = error.assertion;
+			object.assertion = error.assertion;
 		}
 
 		if (error.operator) {
-			obj.operator = error.operator;
+			object.operator = error.operator;
 		}
 
 		if (error.values.length > 0) {
-			obj.values = error.values.reduce((acc, value) => {
+			object.values = error.values.reduce((acc, value) => {
 				acc[value.label] = stripAnsi(value.formatted);
 				return acc;
 			}, {});
@@ -37,15 +37,15 @@ function dumpError(error) {
 	}
 
 	if (error.nonErrorObject) {
-		obj.message = 'Non-error object';
-		obj.formatted = stripAnsi(error.formatted);
+		object.message = 'Non-error object';
+		object.formatted = stripAnsi(error.formatted);
 	}
 
 	if (error.stack) {
-		obj.at = error.stack;
+		object.at = error.stack;
 	}
 
-	return obj;
+	return object;
 }
 
 class TapReporter {

package/lib/reporters/verbose.js

@@ -31,9 +31,9 @@ class LineWriter extends stream.Writable {
 		callback();
 	}
 
-	writeLine(str) {
-		if (str) {
-			this.write(indentString(str, 2) + os.EOL);
+	writeLine(string) {
+		if (string) {
+			this.write(indentString(string, 2) + os.EOL);
 			this.lastLineIsEmpty = false;
 		} else {
 			this.write(os.EOL);
@@ -321,13 +321,13 @@ class VerboseReporter {
 
 	endRun() { // eslint-disable-line complexity
 		if (!this.stats) {
-			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn't find any files to test`));
+			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn’t find any files to test`));
 			this.lineWriter.writeLine();
 			return;
 		}
 
 		if (this.matching && this.stats.selectedTests === 0) {
-			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn't find any matching tests`));
+			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn’t find any matching tests`));
 			this.lineWriter.writeLine();
 			return;
 		}

package/lib/runner.js

@@ -232,9 +232,9 @@ class Runner extends Emittery {
 		};
 
 		let waitForSerial = Promise.resolve();
-		await runnables.reduce((prev, runnable) => {
+		await runnables.reduce((previous, runnable) => {
 			if (runnable.metadata.serial || this.serial) {
-				waitForSerial = prev.then(() => {
+				waitForSerial = previous.then(() => {
 					// Serial runnables run as long as there was no previous failure, unless
 					// the runnable should always be run.
 					return (allPassed || runnable.metadata.always) && runAndStoreResult(runnable);
@@ -243,7 +243,7 @@ class Runner extends Emittery {
 			}
 
 			return Promise.all([
-				prev,
+				previous,
 				waitForSerial.then(() => {
 					// Concurrent runnables are kicked off after the previous serial
 					// runnables have completed, as long as there was no previous failure
@@ -266,7 +266,7 @@ class Runner extends Emittery {
 		return result;
 	}
 
-	async runHooks(tasks, contextRef, titleSuffix) {
+	async runHooks(tasks, contextRef, titleSuffix, testPassed) {
 		const hooks = tasks.map(task => new Runnable({
 			contextRef,
 			experiments: this.experiments,
@@ -278,7 +278,8 @@ class Runner extends Emittery {
 			updateSnapshots: this.updateSnapshots,
 			metadata: task.metadata,
 			powerAssert: this.powerAssert,
-			title: `${task.title}${titleSuffix || ''}`
+			title: `${task.title}${titleSuffix || ''}`,
+			testPassed
 		}));
 		const outcome = await this.runMultiple(hooks, this.serial);
 		for (const result of outcome.storedResults) {
@@ -304,10 +305,11 @@ class Runner extends Emittery {
 	}
 
 	async runTest(task, contextRef) {
-		let hooksAndTestOk = false;
-
 		const hookSuffix = ` for ${task.title}`;
-		if (await this.runHooks(this.tasks.beforeEach, contextRef, hookSuffix)) {
+		let hooksOk = await this.runHooks(this.tasks.beforeEach, contextRef, hookSuffix);
+
+		let testOk = false;
+		if (hooksOk) {
 			// Only run the test if all `beforeEach` hooks passed.
 			const test = new Runnable({
 				contextRef,
@@ -325,7 +327,9 @@ class Runner extends Emittery {
 			});
 
 			const result = await this.runSingle(test);
-			if (result.passed) {
+			testOk = result.passed;
+
+			if (testOk) {
 				this.emit('stateChange', {
 					type: 'test-passed',
 					title: result.title,
@@ -333,7 +337,8 @@ class Runner extends Emittery {
 					knownFailing: result.metadata.failing,
 					logs: result.logs
 				});
-				hooksAndTestOk = await this.runHooks(this.tasks.afterEach, contextRef, hookSuffix);
+
+				hooksOk = await this.runHooks(this.tasks.afterEach, contextRef, hookSuffix, testOk);
 			} else {
 				this.emit('stateChange', {
 					type: 'test-failed',
@@ -347,8 +352,8 @@ class Runner extends Emittery {
 			}
 		}
 
-		const alwaysOk = await this.runHooks(this.tasks.afterEachAlways, contextRef, hookSuffix);
-		return hooksAndTestOk && alwaysOk;
+		const alwaysOk = await this.runHooks(this.tasks.afterEachAlways, contextRef, hookSuffix, testOk);
+		return alwaysOk && hooksOk && testOk;
 	}
 
 	async start() {
@@ -424,16 +429,16 @@ class Runner extends Emittery {
 				return false;
 			}
 
-			return serialTests.reduce(async (prev, task) => {
-				const prevOk = await prev;
+			return serialTests.reduce(async (previous, task) => {
+				const previousOk = await previous;
 				// Don't start tests after an interrupt.
 				if (this.interrupted) {
-					return prevOk;
+					return previousOk;
 				}
 
 				// Prevent subsequent tests from running if `failFast` is enabled and
 				// the previous test failed.
-				if (!prevOk && this.failFast) {
+				if (!previousOk && this.failFast) {
 					return false;
 				}
 

package/lib/snapshot-manager.js

@@ -169,9 +169,9 @@ function encodeSnapshots(buffersByHash) {
 	byteOffset += 4;
 
 	// Allows 65535 hashes (tests or identified snapshots) per file.
-	const numHashes = Buffer.alloc(2);
-	numHashes.writeUInt16LE(buffersByHash.size);
-	buffers.push(numHashes);
+	const numberHashes = Buffer.alloc(2);
+	numberHashes.writeUInt16LE(buffersByHash.size);
+	buffers.push(numberHashes);
 	byteOffset += 2;
 
 	const entries = [];
@@ -183,9 +183,9 @@ function encodeSnapshots(buffersByHash) {
 		byteOffset += MD5_HASH_LENGTH;
 
 		// Allows 65535 snapshots per hash.
-		const numSnapshots = Buffer.alloc(2);
-		numSnapshots.writeUInt16LE(snapshotBuffers.length, 0);
-		buffers.push(numSnapshots);
+		const numberSnapshots = Buffer.alloc(2);
+		numberSnapshots.writeUInt16LE(snapshotBuffers.length, 0);
+		buffers.push(numberSnapshots);
 		byteOffset += 2;
 
 		for (const value of snapshotBuffers) {
@@ -255,18 +255,18 @@ function decodeSnapshots(buffer, snapPath) {
 	byteOffset += 4;
 
 	const snapshotsByHash = new Map();
-	const numHashes = decompressed.readUInt16LE(byteOffset);
+	const numberHashes = decompressed.readUInt16LE(byteOffset);
 	byteOffset += 2;
 
-	for (let count = 0; count < numHashes; count++) {
+	for (let count = 0; count < numberHashes; count++) {
 		const hash = decompressed.toString('hex', byteOffset, byteOffset + MD5_HASH_LENGTH);
 		byteOffset += MD5_HASH_LENGTH;
 
-		const numSnapshots = decompressed.readUInt16LE(byteOffset);
+		const numberSnapshots = decompressed.readUInt16LE(byteOffset);
 		byteOffset += 2;
 
-		const snapshotsBuffers = new Array(numSnapshots);
-		for (let index = 0; index < numSnapshots; index++) {
+		const snapshotsBuffers = new Array(numberSnapshots);
+		for (let index = 0; index < numberSnapshots; index++) {
 			const start = decompressed.readUInt32LE(byteOffset) + headerLength;
 			byteOffset += 4;
 			const end = decompressed.readUInt32LE(byteOffset) + headerLength;

package/lib/test.js

@@ -108,7 +108,7 @@ class ExecutionContext extends assert.Assertions {
 						}
 
 						if (discarded) {
-							test.saveFirstError(new Error('Can\'t commit a result that was previously discarded'));
+							test.saveFirstError(new Error('Can’t commit a result that was previously discarded'));
 							return;
 						}
 
@@ -127,7 +127,7 @@ class ExecutionContext extends assert.Assertions {
 					},
 					discard: ({retainLogs = false} = {}) => {
 						if (committed) {
-							test.saveFirstError(new Error('Can\'t discard a result that was previously committed'));
+							test.saveFirstError(new Error('Can’t discard a result that was previously committed'));
 							return;
 						}
 
@@ -174,6 +174,10 @@ class ExecutionContext extends assert.Assertions {
 		testMap.get(this).contextRef.set(context);
 	}
 
+	get passed() {
+		return testMap.get(this).testPassed;
+	}
+
 	_throwsArgStart(assertion, file, line) {
 		testMap.get(this).trackThrows({assertion, file, line});
 	}
@@ -192,6 +196,7 @@ class Test {
 		this.metadata = options.metadata;
 		this.powerAssert = options.powerAssert;
 		this.title = options.title;
+		this.testPassed = options.testPassed;
 		this.registerUniqueTitle = options.registerUniqueTitle;
 		this.logs = [];
 
@@ -592,7 +597,7 @@ class Test {
 		if (this.metadata.callback) {
 			if (returnedObservable || returnedPromise) {
 				const asyncType = returnedObservable ? 'observables' : 'promises';
-				this.saveFirstError(new Error(`Do not return ${asyncType} from tests declared via \`test.cb(...)\`. Use \`test.cb(...)\` for legacy callback APIs. When using promises, observables or async functions, use \`test(...)\`.`));
+				this.saveFirstError(new Error(`Do not return ${asyncType} from tests declared via \`test.cb(…)\`. Use \`test.cb(…)\` for legacy callback APIs. When using promises, observables or async functions, use \`test(…)\`.`));
 				return this.finishPromised();
 			}
 

package/lib/worker/ensure-forked.js

@@ -13,6 +13,6 @@ if (!isForked) {
 
 		process.exit(1); // eslint-disable-line unicorn/no-process-exit
 	} else {
-		throw new Error('The \'ava\' module can only be imported in test files');
+		throw new Error('The ’ava’ module can only be imported in test files');
 	}
 }

package/lib/worker/subprocess.js

@@ -8,7 +8,7 @@ const ipc = require('./ipc');
 
 const supportsESM = async () => {
 	try {
-		await import('data:text/javascript,');
+		await import('data:text/javascript,'); // eslint-disable-line node/no-unsupported-features/es-syntax
 		return true;
 	} catch {}
 
@@ -155,7 +155,7 @@ ipc.options.then(async options => {
 				}
 
 				if (isESMSupported) {
-					return import(pathToFileURL(ref));
+					return import(pathToFileURL(ref)); // eslint-disable-line node/no-unsupported-features/es-syntax
 				}
 
 				ipc.send({type: 'internal-error', err: serializeError('Internal runner error', false, new Error('ECMAScript Modules are not supported in this Node.js version.'))});
@@ -189,7 +189,7 @@ ipc.options.then(async options => {
 		dependencyTracking.install(testPath);
 
 		if (options.debug) {
-			require('inspector').open(options.debug.port, '127.0.0.1', true);
+			require('inspector').open(options.debug.port, options.debug.host, true); // eslint-disable-line node/no-unsupported-features/node-builtins
 			if (options.debug.break) {
 				debugger; // eslint-disable-line no-debugger
 			}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ava",
-	"version": "3.4.0",
+	"version": "3.6.0",
 	"description": "Testing can be a drag. AVA helps you get it done.",
 	"license": "MIT",
 	"repository": "avajs/ava",
@@ -10,14 +10,12 @@
 		"node": ">=10.18.0 <11 || >=12.14.0 <13 || >=13.5.0"
 	},
 	"scripts": {
-		"lint": "xo",
-		"test:tap": "tap",
-		"test:typescript": "tsc --noEmit -p test/ts-types",
-		"test": "npm run lint && npm run test:typescript && npm run test:tap"
+		"test": "xo && tsd && nyc tap"
 	},
 	"files": [
 		"lib",
 		"*.js",
+		"!*.config.js",
 		"index.d.ts"
 	],
 	"keywords": [
@@ -61,9 +59,10 @@
 		"ansi-styles": "^4.2.1",
 		"arrgv": "^1.0.2",
 		"arrify": "^2.0.1",
-		"chalk": "^3.0.0",
+		"chalk": "^4.0.0",
 		"chokidar": "^3.3.1",
 		"chunkd": "^2.0.1",
+		"ci-info": "^2.0.0",
 		"ci-parallel-vars": "^1.0.0",
 		"clean-stack": "^2.2.0",
 		"clean-yaml-object": "^0.1.0",
@@ -76,14 +75,13 @@
 		"currently-unhandled": "^0.4.1",
 		"debug": "^4.1.1",
 		"del": "^5.1.0",
-		"emittery": "^0.5.1",
+		"emittery": "^0.6.0",
 		"equal-length": "^1.0.0",
-		"figures": "^3.1.0",
+		"figures": "^3.2.0",
 		"globby": "^11.0.0",
 		"ignore-by-default": "^1.0.0",
 		"import-local": "^3.0.2",
 		"indent-string": "^4.0.0",
-		"is-ci": "^2.0.0",
 		"is-error": "^2.2.2",
 		"is-plain-object": "^3.0.0",
 		"is-promise": "^2.1.0",
@@ -92,11 +90,11 @@
 		"md5-hex": "^3.0.1",
 		"ms": "^2.1.2",
 		"ora": "^4.0.3",
-		"p-map": "^3.0.0",
-		"picomatch": "^2.2.1",
+		"p-map": "^4.0.0",
+		"picomatch": "^2.2.2",
 		"pkg-conf": "^3.1.0",
 		"plur": "^4.0.0",
-		"pretty-ms": "^6.0.0",
+		"pretty-ms": "^6.0.1",
 		"read-pkg": "^5.2.0",
 		"resolve-cwd": "^3.0.0",
 		"slash": "^3.0.0",
@@ -107,77 +105,76 @@
 		"temp-dir": "^2.0.0",
 		"trim-off-newlines": "^1.0.1",
 		"update-notifier": "^4.1.0",
-		"write-file-atomic": "^3.0.1",
-		"yargs": "^15.1.0"
+		"write-file-atomic": "^3.0.3",
+		"yargs": "^15.3.1"
 	},
 	"devDependencies": {
 		"@ava/babel": "^1.0.1",
-		"@sinonjs/fake-timers": "^6.0.0",
-		"ansi-escapes": "^4.3.0",
+		"@sinonjs/fake-timers": "^6.0.1",
+		"ansi-escapes": "^4.3.1",
 		"delay": "^4.3.0",
 		"esm": "^3.2.25",
 		"execa": "^4.0.0",
 		"get-stream": "^5.1.0",
+		"nyc": "^15.0.1",
 		"p-event": "^4.1.0",
 		"proxyquire": "^2.1.3",
-		"react": "^16.12.0",
-		"react-test-renderer": "^16.12.0",
+		"react": "^16.13.1",
+		"react-test-renderer": "^16.13.1",
 		"replace-string": "^3.0.0",
-		"sinon": "^8.1.1",
+		"sinon": "^9.0.1",
 		"source-map-fixtures": "^2.1.0",
-		"tap": "^14.10.6",
+		"tap": "^14.10.7",
 		"temp-write": "^4.0.0",
-		"tempy": "^0.4.0",
+		"tempy": "^0.5.0",
 		"touch": "^3.1.0",
-		"ts-node": "^8.6.2",
+		"tsd": "^0.11.0",
 		"typescript": "^3.7.5",
-		"xo": "^0.26.1",
+		"xo": "^0.28.2",
 		"zen-observable": "^0.8.15"
 	},
 	"xo": {
 		"ignores": [
 			"media/**",
-			"test/fixture/ava-paths/target/test.js",
-			"test/fixture/{source-map-initial,syntax-error}.js",
-			"test/fixture/snapshots/test-sourcemaps/build/**",
-			"test/fixture/power-assert.js",
-			"**/*.ts"
+			"test-tap/fixture/ava-paths/target/test.js",
+			"test-tap/fixture/{source-map-initial,syntax-error}.js",
+			"test-tap/fixture/snapshots/test-sourcemaps/build/**",
+			"test-tap/fixture/power-assert.js"
 		],
 		"rules": {
-			"no-use-extend-native/no-use-extend-native": "off"
+			"no-use-extend-native/no-use-extend-native": "off",
+			"@typescript-eslint/no-var-requires": "off"
 		},
 		"overrides": [
 			{
-				"files": "test/**/*.js",
+				"files": "*.d.ts",
+				"rules": {
+					"@typescript-eslint/member-ordering": "off",
+					"@typescript-eslint/prefer-function-type": "off",
+					"@typescript-eslint/unified-signatures": "off"
+				}
+			},
+			{
+				"files": "test-d/*.ts",
+				"rules": {
+					"@typescript-eslint/explicit-function-return-type": "off",
+					"@typescript-eslint/no-empty-function": "off",
+					"@typescript-eslint/no-unused-vars": "off",
+					"no-unused-vars": "off"
+				}
+			},
+			{
+				"files": "test-tap/**/*.js",
 				"rules": {
 					"promise/prefer-await-to-then": "off"
 				}
 			},
 			{
-				"files": "test/fixture/**/*.js",
+				"files": "test-tap/fixture/**/*.js",
 				"rules": {
 					"import/no-extraneous-dependencies": "off"
 				}
 			}
 		]
-	},
-	"tap": {
-		"browser": false,
-		"coverage-report": [
-			"html",
-			"lcov",
-			"text"
-		],
-		"esm": false,
-		"files": [
-			"test/*.js",
-			"test/reporters/*.js",
-			"test/integration/*.js"
-		],
-		"flow": false,
-		"jobs": 2,
-		"jsx": false,
-		"timeout": 300,
-		"ts": false
 	}
 }

package/readme.md

@@ -156,7 +156,6 @@ We have a growing list of [common pitfalls](docs/08-common-pitfalls.md) you may
 - [Passing arguments to your test files](docs/recipes/passing-arguments-to-your-test-files.md)
 - [Testing React components](docs/recipes/react.md)
 - [Testing Vue.js components](docs/recipes/vue.md)
-- [JSPM and SystemJS](docs/recipes/jspm-systemjs.md)
 - [Debugging tests with Chrome DevTools](docs/recipes/debugging-with-chrome-devtools.md)
 - [Debugging tests with VSCode](docs/recipes/debugging-with-vscode.md)
 - [Debugging tests with WebStorm](docs/recipes/debugging-with-webstorm.md)