ava 3.10.1 → 3.12.1

package/index.d.ts

@@ -433,7 +433,7 @@ export interface CbExecutionContext<Context = unknown> extends ExecutionContext<
 	end(error?: any): void;
 }
 
-export type ImplementationResult = PromiseLike<void> | Subscribable | void; // eslint-disable-line @typescript-eslint/no-invalid-void-type
+export type ImplementationResult = PromiseLike<void> | Subscribable | void;
 export type Implementation<Context = unknown> = (t: ExecutionContext<Context>) => ImplementationResult;
 export type CbImplementation<Context = unknown> = (t: CbExecutionContext<Context>) => ImplementationResult;
 

package/lib/api.js

@@ -147,7 +147,7 @@ class Api extends Emittery {
 				runStatus = new RunStatus(selectedFiles.length, null);
 			}
 
-			const debugWithoutSpecificFile = Boolean(this.options.debug) && selectedFiles.length !== 1;
+			const debugWithoutSpecificFile = Boolean(this.options.debug) && !this.options.debug.active && selectedFiles.length !== 1;
 
 			await this.emit('run', {
 				bailWithoutReporting: debugWithoutSpecificFile,

package/lib/assert.js

@@ -399,15 +399,6 @@ class Assertions {
 		});
 
 		this.like = withSkip((actual, selector, message) => {
-			if (!experiments.likeAssertion) {
-				fail(new AssertionError({
-					assertion: 'like',
-					improperUsage: true,
-					message: 'You must enable the `likeAssertion` experiment in order to use `t.like()`'
-				}));
-				return;
-			}
-
 			if (!checkMessage('like', message)) {
 				return;
 			}

package/lib/cli.js

@@ -88,7 +88,19 @@ exports.run = async () => { // eslint-disable-line complexity
 		confError = error;
 	}
 
-	let debug = null;
+	// Enter debug mode if the main process is being inspected. This assumes the
+	// worker processes are automatically inspected, too. It is not necessary to
+	// run AVA with the debug command, though it's allowed.
+	const activeInspector = require('inspector').url() !== undefined; // eslint-disable-line node/no-unsupported-features/node-builtins
+	let debug = activeInspector ?
+		{
+			active: true,
+			break: false,
+			files: [],
+			host: undefined,
+			port: undefined
+		} : null;
+
 	let resetCache = false;
 	const {argv} = yargs
 		.parserConfiguration({
@@ -122,7 +134,11 @@ exports.run = async () => { // eslint-disable-line complexity
 			array: true,
 			describe: 'Glob patterns to select what test files to run. Leave empty if you want AVA to run all test files instead. Add a colon and specify line numbers of specific tests to run',
 			type: 'string'
-		}))
+		}), argv => {
+			if (activeInspector) {
+				debug.files = argv.pattern || [];
+			}
+		})
 		.command(
 			'debug [<pattern>...]',
 			'Activate Node.js inspector and run a single test file',
@@ -148,6 +164,7 @@ exports.run = async () => { // eslint-disable-line complexity
 			}),
 			argv => {
 				debug = {
+					active: activeInspector,
 					break: argv.break === true,
 					files: argv.pattern,
 					host: argv.host,
@@ -267,6 +284,7 @@ exports.run = async () => { // eslint-disable-line complexity
 	const TapReporter = require('./reporters/tap');
 	const Watcher = require('./watcher');
 	const normalizeExtensions = require('./extensions');
+	const normalizeModuleTypes = require('./module-types');
 	const {normalizeGlobs, normalizePattern} = require('./globs');
 	const normalizeNodeArguments = require('./node-arguments');
 	const validateEnvironmentVariables = require('./environment-variables');
@@ -284,12 +302,6 @@ exports.run = async () => { // eslint-disable-line complexity
 
 	const {type: defaultModuleType = 'commonjs'} = pkg || {};
 
-	const moduleTypes = {
-		cjs: 'commonjs',
-		mjs: 'module',
-		js: defaultModuleType
-	};
-
 	const providers = [];
 	if (Reflect.has(conf, 'babel')) {
 		try {
@@ -331,6 +343,13 @@ exports.run = async () => { // eslint-disable-line complexity
 		exit(error.message);
 	}
 
+	let moduleTypes;
+	try {
+		moduleTypes = normalizeModuleTypes(conf.extensions, defaultModuleType, experiments);
+	} catch (error) {
+		exit(error.message);
+	}
+
 	let globs;
 	try {
 		globs = normalizeGlobs({files: conf.files, ignoredByWatcher: conf.ignoredByWatcher, extensions, providers});
@@ -444,14 +463,14 @@ exports.run = async () => { // eslint-disable-line complexity
 	} else {
 		let debugWithoutSpecificFile = false;
 		api.on('run', plan => {
-			if (plan.debug && plan.files.length !== 1) {
+			if (debug !== null && plan.files.length !== 1) {
 				debugWithoutSpecificFile = true;
 			}
 		});
 
 		const runStatus = await api.run({filter});
 
-		if (debugWithoutSpecificFile) {
+		if (debugWithoutSpecificFile && !debug.active) {
 			exit('Provide the path to the test file you wish to debug');
 			return;
 		}

package/lib/extensions.js

@@ -2,8 +2,11 @@ module.exports = (configuredExtensions, providers = []) => {
 	// Combine all extensions possible for testing. Remove duplicate extensions.
 	const duplicates = new Set();
 	const seen = new Set();
+
+	const normalize = extensions => Array.isArray(extensions) ? extensions : Object.keys(extensions);
+
 	const combine = extensions => {
-		for (const ext of extensions) {
+		for (const ext of normalize(extensions)) {
 			if (seen.has(ext)) {
 				duplicates.add(ext);
 			} else {

package/lib/load-config.js

@@ -7,7 +7,7 @@ const pkgConf = require('pkg-conf');
 
 const NO_SUCH_FILE = Symbol('no ava.config.js file');
 const MISSING_DEFAULT_EXPORT = Symbol('missing default export');
-const EXPERIMENTS = new Set(['disableSnapshotsInHooks', 'likeAssertion', 'reverseTeardowns']);
+const EXPERIMENTS = new Set(['configurableModuleFormat', 'disableSnapshotsInHooks', 'reverseTeardowns']);
 
 // *Very* rudimentary support for loading ava.config.js files containing an `export default` statement.
 const evaluateJsConfig = configFile => {

package/lib/module-types.js

@@ -0,0 +1,75 @@
+const requireTrueValue = value => {
+	if (value !== true) {
+		throw new TypeError('When specifying module types, use `true` for ’cjs’, ’mjs’ and ’js’ extensions');
+	}
+};
+
+const normalize = (extension, type, defaultModuleType) => {
+	switch (extension) {
+		case 'cjs':
+			requireTrueValue(type);
+			return 'commonjs';
+		case 'mjs':
+			requireTrueValue(type);
+			return 'module';
+		case 'js':
+			requireTrueValue(type);
+			return defaultModuleType;
+		default:
+			if (type !== 'commonjs' && type !== 'module') {
+				throw new TypeError(`Module type for ’${extension}’ must be ’commonjs’ or ’module’`);
+			}
+
+			return type;
+	}
+};
+
+const deriveFromObject = (extensionsObject, defaultModuleType) => {
+	const moduleTypes = {};
+	for (const [extension, type] of Object.entries(extensionsObject)) {
+		moduleTypes[extension] = normalize(extension, type, defaultModuleType);
+	}
+
+	return moduleTypes;
+};
+
+const deriveFromArray = (extensions, defaultModuleType) => {
+	const moduleTypes = {};
+	for (const extension of extensions) {
+		switch (extension) {
+			case 'cjs':
+				moduleTypes.cjs = 'commonjs';
+				break;
+			case 'mjs':
+				moduleTypes.mjs = 'module';
+				break;
+			case 'js':
+				moduleTypes.js = defaultModuleType;
+				break;
+			default:
+				moduleTypes[extension] = 'commonjs';
+		}
+	}
+
+	return moduleTypes;
+};
+
+module.exports = (configuredExtensions, defaultModuleType, experiments) => {
+	if (configuredExtensions === undefined) {
+		return {
+			cjs: 'commonjs',
+			mjs: 'module',
+			js: defaultModuleType
+		};
+	}
+
+	if (Array.isArray(configuredExtensions)) {
+		return deriveFromArray(configuredExtensions, defaultModuleType);
+	}
+
+	if (!experiments.configurableModuleFormat) {
+		throw new Error('You must enable the `configurableModuleFormat` experiment in order to specify module types');
+	}
+
+	return deriveFromObject(configuredExtensions, defaultModuleType);
+};

package/lib/provider-manager.js

@@ -21,7 +21,7 @@ function load(providerModule, projectDir) {
 	let level;
 	const provider = makeProvider({
 		negotiateProtocol(identifiers, {version}) {
-			const [identifier] = identifiers.filter(identifier => Reflect.has(levelsByProtocol, identifier));
+			const identifier = identifiers.find(identifier => Reflect.has(levelsByProtocol, identifier));
 
 			if (identifier === undefined) {
 				fatal = new Error(`This version of AVA (${ava.version}) is not compatible with ${providerModule}@${version}`);

package/lib/reporters/default.js

@@ -18,7 +18,6 @@ const colors = require('./colors');
 const formatSerializedError = require('./format-serialized-error');
 const improperUsageMessages = require('./improper-usage-messages');
 const prefixTitle = require('./prefix-title');
-const whileCorked = require('./while-corked');
 
 const nodeInternals = require('stack-utils').nodeInternals();
 
@@ -97,6 +96,48 @@ class LineWriterWithSpinner extends LineWriter {
 	}
 }
 
+function manageCorking(stream) {
+	let corked = false;
+	const cork = () => {
+		corked = true;
+		stream.cork();
+	};
+
+	const uncork = () => {
+		corked = false;
+		stream.uncork();
+	};
+
+	return {
+		decorateFlushingWriter(fn) {
+			return function (...args) {
+				if (corked) {
+					stream.uncork();
+				}
+
+				try {
+					return fn.apply(this, args);
+				} finally {
+					if (corked) {
+						stream.cork();
+					}
+				}
+			};
+		},
+
+		decorateWriter(fn) {
+			return function (...args) {
+				cork();
+				try {
+					return fn.apply(this, args);
+				} finally {
+					uncork();
+				}
+			};
+		}
+	};
+}
+
 class Reporter {
 	constructor({
 		verbose,
@@ -112,13 +153,16 @@ class Reporter {
 		this.stdStream = stdStream;
 		this.watching = watching;
 		this.relativeFile = file => path.relative(projectDir, file);
-		this.consumeStateChange = whileCorked(this.reportStream, this.consumeStateChange);
+
+		const {decorateWriter, decorateFlushingWriter} = manageCorking(this.reportStream);
+		this.consumeStateChange = decorateWriter(this.consumeStateChange);
+		this.endRun = decorateWriter(this.endRun);
 
 		if (this.verbose) {
 			this.durationThreshold = durationThreshold || 100;
 			this.spinner = null;
+			this.clearSpinner = () => {};
 			this.lineWriter = new LineWriter(this.reportStream);
-			this.endRun = whileCorked(this.reportStream, this.endRun);
 		} else {
 			this.spinner = ora({
 				isEnabled: true,
@@ -128,8 +172,8 @@ class Reporter {
 				spinner: spinner || (process.platform === 'win32' ? 'line' : 'dots'),
 				stream: reportStream
 			});
+			this.clearSpinner = decorateFlushingWriter(this.spinner.clear.bind(this.spinner));
 			this.lineWriter = new LineWriterWithSpinner(this.reportStream, this.spinner);
-			this.endRun = whileCorked(this.reportStream, whileCorked(this.lineWriter, this.endRun));
 		}
 
 		this.reset();
@@ -362,9 +406,7 @@ class Reporter {
 
 			case 'worker-stderr': {
 				// Forcibly clear the spinner, writing the chunk corrupts the TTY.
-				if (this.spinner !== null) {
-					this.spinner.clear();
-				}
+				this.clearSpinner();
 
 				this.stdStream.write(event.chunk);
 				// If the chunk does not end with a linebreak, *forcibly* write one to
@@ -386,9 +428,7 @@ class Reporter {
 
 			case 'worker-stdout': {
 				// Forcibly clear the spinner, writing the chunk corrupts the TTY.
-				if (this.spinner !== null) {
-					this.spinner.clear();
-				}
+				this.clearSpinner();
 
 				this.stdStream.write(event.chunk);
 				// If the chunk does not end with a linebreak, *forcibly* write one to

package/lib/worker/subprocess.js

@@ -205,8 +205,14 @@ ipc.options.then(async options => {
 		// to make sure we also track dependencies with custom require hooks
 		dependencyTracking.install(testPath);
 
-		if (options.debug) {
-			require('inspector').open(options.debug.port, options.debug.host, true); // eslint-disable-line node/no-unsupported-features/node-builtins
+		if (options.debug && options.debug.port !== undefined && options.debug.host !== undefined) {
+			// If an inspector was active when the main process started, and is
+			// already active for the worker process, do not open a new one.
+			const inspector = require('inspector'); // eslint-disable-line node/no-unsupported-features/node-builtins
+			if (!options.debug.active || inspector.url() === undefined) {
+				inspector.open(options.debug.port, options.debug.host, true);
+			}
+
 			if (options.debug.break) {
 				debugger; // eslint-disable-line no-debugger
 			}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ava",
-	"version": "3.10.1",
+	"version": "3.12.1",
 	"description": "Node.js test runner that lets you develop with confidence.",
 	"license": "MIT",
 	"repository": "avajs/ava",
@@ -56,14 +56,14 @@
 	],
 	"dependencies": {
 		"@concordance/react": "^2.0.0",
-		"acorn": "^7.3.1",
-		"acorn-walk": "^7.2.0",
+		"acorn": "^8.0.1",
+		"acorn-walk": "^8.0.0",
 		"ansi-styles": "^4.2.1",
 		"arrgv": "^1.0.2",
 		"arrify": "^2.0.1",
 		"callsites": "^3.1.0",
 		"chalk": "^4.1.0",
-		"chokidar": "^3.4.0",
+		"chokidar": "^3.4.2",
 		"chunkd": "^2.0.1",
 		"ci-info": "^2.0.0",
 		"ci-parallel-vars": "^1.0.1",
@@ -72,12 +72,12 @@
 		"cli-truncate": "^2.1.0",
 		"code-excerpt": "^3.0.0",
 		"common-path-prefix": "^3.0.0",
-		"concordance": "^5.0.0",
+		"concordance": "^5.0.1",
 		"convert-source-map": "^1.7.0",
 		"currently-unhandled": "^0.4.1",
 		"debug": "^4.1.1",
 		"del": "^5.1.0",
-		"emittery": "^0.7.0",
+		"emittery": "^0.7.1",
 		"equal-length": "^1.0.0",
 		"figures": "^3.2.0",
 		"globby": "^11.0.1",
@@ -85,14 +85,14 @@
 		"import-local": "^3.0.2",
 		"indent-string": "^4.0.0",
 		"is-error": "^2.2.2",
-		"is-plain-object": "^3.0.1",
+		"is-plain-object": "^4.1.1",
 		"is-promise": "^4.0.0",
-		"lodash": "^4.17.15",
+		"lodash": "^4.17.20",
 		"matcher": "^3.0.0",
 		"md5-hex": "^3.0.1",
 		"mem": "^6.1.0",
 		"ms": "^2.1.2",
-		"ora": "^4.0.4",
+		"ora": "^5.0.0",
 		"p-map": "^4.0.0",
 		"picomatch": "^2.2.2",
 		"pkg-conf": "^3.1.0",
@@ -107,9 +107,9 @@
 		"supertap": "^1.0.0",
 		"temp-dir": "^2.0.0",
 		"trim-off-newlines": "^1.0.1",
-		"update-notifier": "^4.1.0",
+		"update-notifier": "^4.1.1",
 		"write-file-atomic": "^3.0.3",
-		"yargs": "^15.4.0"
+		"yargs": "^15.4.1"
 	},
 	"devDependencies": {
 		"@ava/babel": "^1.0.1",
@@ -117,25 +117,25 @@
 		"@babel/plugin-proposal-do-expressions": "^7.10.4",
 		"@sinonjs/fake-timers": "^6.0.1",
 		"ansi-escapes": "^4.3.1",
-		"c8": "^7.2.0",
-		"delay": "^4.3.0",
+		"c8": "^7.3.0",
+		"delay": "^4.4.0",
 		"esm": "^3.2.25",
-		"execa": "^4.0.2",
-		"get-stream": "^5.1.0",
+		"execa": "^4.0.3",
+		"get-stream": "^6.0.0",
 		"p-event": "^4.2.0",
 		"proxyquire": "^2.1.3",
 		"react": "^16.13.1",
 		"react-test-renderer": "^16.13.1",
 		"replace-string": "^3.1.0",
-		"sinon": "^9.0.2",
+		"sinon": "^9.0.3",
 		"source-map-fixtures": "^2.1.0",
-		"tap": "^14.10.7",
+		"tap": "^14.10.8",
 		"temp-write": "^4.0.0",
-		"tempy": "^0.5.0",
+		"tempy": "^0.6.0",
 		"touch": "^3.1.0",
 		"tsd": "^0.13.1",
-		"typescript": "^3.9.6",
-		"xo": "^0.32.1",
+		"typescript": "^3.9.7",
+		"xo": "^0.33.0",
 		"zen-observable": "^0.8.15"
 	}
 }

package/readme.md

@@ -211,9 +211,9 @@ It's the [Andromeda galaxy](https://simple.wikipedia.org/wiki/Andromeda_galaxy).
 
 ## Team
 
-[![Mark Wubben](https://github.com/novemberborn.png?size=100)](https://github.com/novemberborn) | [![Sindre Sorhus](https://github.com/sindresorhus.png?size=100)](https://github.com/sindresorhus) | [![Vadim Demedes](https://github.com/vadimdemedes.png?size=100)](https://github.com/vadimdemedes)
----|---|---
-[Mark Wubben](https://novemberborn.net) | [Sindre Sorhus](https://sindresorhus.com) | [Vadim Demedes](https://github.com/vadimdemedes)
+[![Mark Wubben](https://github.com/novemberborn.png?size=100)](https://github.com/novemberborn) | [![Sindre Sorhus](https://github.com/sindresorhus.png?size=100)](https://github.com/sindresorhus)
+---|---
+[Mark Wubben](https://novemberborn.net) | [Sindre Sorhus](https://sindresorhus.com)
 
 ###### Former
 
@@ -221,6 +221,7 @@ It's the [Andromeda galaxy](https://simple.wikipedia.org/wiki/Andromeda_galaxy).
 - [James Talmage](https://github.com/jamestalmage)
 - [Juan Soto](https://github.com/sotojuan)
 - [Jeroen Engels](https://github.com/jfmengels)
+- [Vadim Demedes](https://github.com/vadimdemedes)
 
 
 <div align="center">

package/lib/reporters/while-corked.js

@@ -1,13 +0,0 @@
-'use strict';
-function whileCorked(stream, fn) {
-	return function (...args) {
-		stream.cork();
-		try {
-			fn.apply(this, args);
-		} finally {
-			stream.uncork();
-		}
-	};
-}
-
-module.exports = whileCorked;