ava 3.8.1 → 3.10.1

package/lib/reporters/tap.js

@@ -30,7 +30,7 @@ function dumpError(error) {
 		}
 
 		if (error.values.length > 0) {
-			object.values = error.values.reduce((acc, value) => {
+			object.values = error.values.reduce((acc, value) => { // eslint-disable-line unicorn/no-reduce
 				acc[value.label] = stripAnsi(value.formatted);
 				return acc;
 			}, {});
@@ -158,6 +158,9 @@ 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;

package/lib/runner.js

@@ -23,6 +23,7 @@ class Runner extends Emittery {
 		this.recordNewSnapshots = options.recordNewSnapshots === true;
 		this.runOnlyExclusive = options.runOnlyExclusive === true;
 		this.serial = options.serial === true;
+		this.skippingTests = false;
 		this.snapshotDir = options.snapshotDir;
 		this.updateSnapshots = options.updateSnapshots;
 
@@ -147,6 +148,10 @@ class Runner extends Emittery {
 							task.metadata.exclusive = matcher([title], this.match).length === 1;
 						}
 
+						if (task.metadata.skipped) {
+							this.skippingTests = true;
+						}
+
 						if (task.metadata.exclusive) {
 							this.runOnlyExclusive = true;
 						}
@@ -182,7 +187,7 @@ class Runner extends Emittery {
 				fixedLocation: this.snapshotDir,
 				projectDir: this.projectDir,
 				recordNewSnapshots: this.recordNewSnapshots,
-				updating: this.updateSnapshots
+				updating: this.updateSnapshots && !this.runOnlyExclusive && !this.skippingTests
 			});
 			this.emit('dependency', this.snapshots.snapPath);
 		}
@@ -191,8 +196,12 @@ class Runner extends Emittery {
 	}
 
 	saveSnapshotState() {
+		if (this.updateSnapshots && (this.runOnlyExclusive || this.skippingTests)) {
+			return {cannotSave: true};
+		}
+
 		if (this.snapshots) {
-			return this.snapshots.save();
+			return {touchedFiles: this.snapshots.save()};
 		}
 
 		if (this.updateSnapshots) {
@@ -201,7 +210,7 @@ class Runner extends Emittery {
 			// were skipped. Perhaps emit a warning if this occurs?
 		}
 
-		return null;
+		return {};
 	}
 
 	onRun(runnable) {
@@ -241,7 +250,7 @@ class Runner extends Emittery {
 		};
 
 		let waitForSerial = Promise.resolve();
-		await runnables.reduce((previous, runnable) => {
+		await runnables.reduce((previous, runnable) => { // eslint-disable-line unicorn/no-reduce
 			if (runnable.metadata.serial || this.serial) {
 				waitForSerial = previous.then(() => {
 					// Serial runnables run as long as there was no previous failure, unless
@@ -451,7 +460,7 @@ class Runner extends Emittery {
 				return false;
 			}
 
-			return serialTests.reduce(async (previous, task) => {
+			return serialTests.reduce(async (previous, task) => { // eslint-disable-line unicorn/no-reduce
 				const previousOk = await previous;
 				// Don't start tests after an interrupt.
 				if (this.interrupted) {

package/lib/test.js

@@ -39,7 +39,9 @@ class ExecutionContext extends assert.Assertions {
 			compareWithSnapshot: options => {
 				return test.compareWithSnapshot(options);
 			},
-			powerAssert: test.powerAssert
+			powerAssert: test.powerAssert,
+			experiments: test.experiments,
+			disableSnapshots: test.isHook === true
 		});
 		testMap.set(this, test);
 
@@ -64,8 +66,8 @@ class ExecutionContext extends assert.Assertions {
 
 		this.plan.skip = () => {};
 
-		this.timeout = ms => {
-			test.timeout(ms);
+		this.timeout = (ms, message) => {
+			test.timeout(ms, message);
 		};
 
 		this.teardown = callback => {
@@ -73,6 +75,12 @@ class ExecutionContext extends assert.Assertions {
 		};
 
 		this.try = async (...attemptArgs) => {
+			if (test.isHook) {
+				const error = new Error('`t.try()` can only be used in tests');
+				test.saveFirstError(error);
+				throw error;
+			}
+
 			const {args, buildTitle, implementations, receivedImplementationArray} = parseTestArgs(attemptArgs);
 
 			if (implementations.length === 0) {
@@ -430,7 +438,14 @@ class Test {
 		this.planError = planError;
 	}
 
-	timeout(ms) {
+	timeout(ms, message) {
+		const result = assert.checkAssertionMessage('timeout', message);
+		if (result !== true) {
+			this.saveFirstError(result);
+			// Allow the timeout to be set even when the message is invalid.
+			message = '';
+		}
+
 		if (this.finishing) {
 			return;
 		}
@@ -438,7 +453,7 @@ class Test {
 		this.clearTimeout();
 		this.timeoutMs = ms;
 		this.timeoutTimer = nowAndTimers.setTimeout(() => {
-			this.saveFirstError(new Error('Test timeout exceeded'));
+			this.saveFirstError(new Error(message || 'Test timeout exceeded'));
 
 			if (this.finishDueToTimeout) {
 				this.finishDueToTimeout();
@@ -482,7 +497,13 @@ class Test {
 	}
 
 	async runTeardowns() {
-		for (const teardown of this.teardowns) {
+		const teardowns = [...this.teardowns];
+
+		if (this.experiments.reverseTeardowns) {
+			teardowns.reverse();
+		}
+
+		for (const teardown of teardowns) {
 			try {
 				await teardown(); // eslint-disable-line no-await-in-loop
 			} catch (error) {

package/lib/worker/subprocess.js

@@ -91,8 +91,10 @@ ipc.options.then(async options => {
 
 	runner.on('finish', () => {
 		try {
-			const touchedFiles = runner.saveSnapshotState();
-			if (touchedFiles) {
+			const {cannotSave, touchedFiles} = runner.saveSnapshotState();
+			if (cannotSave) {
+				ipc.send({type: 'snapshot-error'});
+			} else if (touchedFiles) {
 				ipc.send({type: 'touched-files', files: touchedFiles});
 			}
 		} catch (error) {
@@ -196,7 +198,7 @@ ipc.options.then(async options => {
 				if (Reflect.has(mod, Symbol.for('esm:package'))) {
 					requireFn = mod(module);
 				}
-			} catch (_) {}
+			} catch {}
 		}
 
 		// Install dependency tracker after the require configuration has been evaluated

package/package.json

@@ -1,16 +1,16 @@
 {
 	"name": "ava",
-	"version": "3.8.1",
-	"description": "Testing can be a drag. AVA helps you get it done.",
+	"version": "3.10.1",
+	"description": "Node.js test runner that lets you develop with confidence.",
 	"license": "MIT",
 	"repository": "avajs/ava",
 	"homepage": "https://avajs.dev",
 	"bin": "cli.js",
 	"engines": {
-		"node": ">=10.18.0 <11 || >=12.14.0 <13 || >=13.5.0 <14 || >=14.0.0"
+		"node": ">=10.18.0 <11 || >=12.14.0 <12.17.0 || >=12.17.0 <13 || >=14.0.0"
 	},
 	"scripts": {
-		"test": "xo && tsd && c8 tap"
+		"test": "xo && tsd && c8 --report=none tap && c8 --report=none --no-clean test-ava && c8 report"
 	},
 	"files": [
 		"lib",
@@ -56,37 +56,37 @@
 	],
 	"dependencies": {
 		"@concordance/react": "^2.0.0",
-		"acorn": "^7.1.1",
-		"acorn-walk": "^7.1.1",
+		"acorn": "^7.3.1",
+		"acorn-walk": "^7.2.0",
 		"ansi-styles": "^4.2.1",
 		"arrgv": "^1.0.2",
 		"arrify": "^2.0.1",
 		"callsites": "^3.1.0",
-		"chalk": "^4.0.0",
+		"chalk": "^4.1.0",
 		"chokidar": "^3.4.0",
 		"chunkd": "^2.0.1",
 		"ci-info": "^2.0.0",
-		"ci-parallel-vars": "^1.0.0",
+		"ci-parallel-vars": "^1.0.1",
 		"clean-yaml-object": "^0.1.0",
 		"cli-cursor": "^3.1.0",
 		"cli-truncate": "^2.1.0",
-		"code-excerpt": "^2.1.1",
+		"code-excerpt": "^3.0.0",
 		"common-path-prefix": "^3.0.0",
-		"concordance": "^4.0.0",
+		"concordance": "^5.0.0",
 		"convert-source-map": "^1.7.0",
 		"currently-unhandled": "^0.4.1",
 		"debug": "^4.1.1",
 		"del": "^5.1.0",
-		"emittery": "^0.6.0",
+		"emittery": "^0.7.0",
 		"equal-length": "^1.0.0",
 		"figures": "^3.2.0",
-		"globby": "^11.0.0",
-		"ignore-by-default": "^1.0.0",
+		"globby": "^11.0.1",
+		"ignore-by-default": "^2.0.0",
 		"import-local": "^3.0.2",
 		"indent-string": "^4.0.0",
 		"is-error": "^2.2.2",
-		"is-plain-object": "^3.0.0",
-		"is-promise": "^3.0.0",
+		"is-plain-object": "^3.0.1",
+		"is-promise": "^4.0.0",
 		"lodash": "^4.17.15",
 		"matcher": "^3.0.0",
 		"md5-hex": "^3.0.1",
@@ -97,44 +97,45 @@
 		"picomatch": "^2.2.2",
 		"pkg-conf": "^3.1.0",
 		"plur": "^4.0.0",
-		"pretty-ms": "^6.0.1",
+		"pretty-ms": "^7.0.0",
 		"read-pkg": "^5.2.0",
 		"resolve-cwd": "^3.0.0",
 		"slash": "^3.0.0",
 		"source-map-support": "^0.5.19",
-		"stack-utils": "^2.0.1",
+		"stack-utils": "^2.0.2",
 		"strip-ansi": "^6.0.0",
 		"supertap": "^1.0.0",
 		"temp-dir": "^2.0.0",
 		"trim-off-newlines": "^1.0.1",
 		"update-notifier": "^4.1.0",
 		"write-file-atomic": "^3.0.3",
-		"yargs": "^15.3.1"
+		"yargs": "^15.4.0"
 	},
 	"devDependencies": {
 		"@ava/babel": "^1.0.1",
-		"@babel/plugin-proposal-do-expressions": "^7.8.3",
+		"@ava/test": "github:avajs/test",
+		"@babel/plugin-proposal-do-expressions": "^7.10.4",
 		"@sinonjs/fake-timers": "^6.0.1",
 		"ansi-escapes": "^4.3.1",
-		"c8": "^7.1.0",
+		"c8": "^7.2.0",
 		"delay": "^4.3.0",
 		"esm": "^3.2.25",
-		"execa": "^4.0.0",
+		"execa": "^4.0.2",
 		"get-stream": "^5.1.0",
-		"p-event": "^4.1.0",
+		"p-event": "^4.2.0",
 		"proxyquire": "^2.1.3",
 		"react": "^16.13.1",
 		"react-test-renderer": "^16.13.1",
-		"replace-string": "^3.0.0",
+		"replace-string": "^3.1.0",
 		"sinon": "^9.0.2",
 		"source-map-fixtures": "^2.1.0",
 		"tap": "^14.10.7",
 		"temp-write": "^4.0.0",
 		"tempy": "^0.5.0",
 		"touch": "^3.1.0",
-		"tsd": "^0.11.0",
-		"typescript": "^3.8.3",
-		"xo": "^0.30.0",
+		"tsd": "^0.13.1",
+		"typescript": "^3.9.6",
+		"xo": "^0.32.1",
 		"zen-observable": "^0.8.15"
 	}
 }

package/readme.md

@@ -3,7 +3,7 @@
 [![Build Status](https://travis-ci.org/avajs/ava.svg?branch=master)](https://travis-ci.org/avajs/ava)  [![Coverage Status](https://codecov.io/gh/avajs/ava/branch/master/graph/badge.svg)](https://codecov.io/gh/avajs/ava/branch/master) [![XO code style](https://img.shields.io/badge/code_style-XO-5ed9c7.svg)](https://github.com/xojs/xo) [![Join the community on Spectrum](https://withspectrum.github.io/badge/badge.svg)](https://spectrum.chat/ava)
 [![Mentioned in Awesome Node.js](https://awesome.re/mentioned-badge.svg)](https://github.com/sindresorhus/awesome-nodejs)
 
-Testing can be a drag. AVA helps you get it done. AVA is a test runner for Node.js with a concise API, detailed error output, embrace of new language features and process isolation that let you write tests more effectively. So you can ship more awesome code. 🚀
+AVA is a test runner for Node.js with a concise API, detailed error output, embrace of new language features and process isolation that lets you develop with confidence 🚀
 
 Follow the [AVA Twitter account](https://twitter.com/ava__js) for updates.