ava 3.5.2 → 3.6.0

package/index.d.ts

@@ -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;
@@ -343,29 +346,29 @@ 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>, ...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.
-	*/
+	 * 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[]>;
 }
 

package/lib/runner.js

@@ -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() {

package/lib/test.js

@@ -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 = [];
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ava",
-	"version": "3.5.2",
+	"version": "3.6.0",
 	"description": "Testing can be a drag. AVA helps you get it done.",
 	"license": "MIT",
 	"repository": "avajs/ava",
@@ -59,7 +59,7 @@
 		"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",
@@ -75,9 +75,9 @@
 		"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",
@@ -90,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",
@@ -105,34 +105,32 @@
 		"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",
-		"lolex": "^5.1.2",
-		"nyc": "^15.0.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.28.1",
+		"xo": "^0.28.2",
 		"zen-observable": "^0.8.15"
 	},
 	"xo": {

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)