Fix tests (#123)

example.js

@@ -19,7 +19,7 @@ const tasks = new Listr([
 
 							delay(2000)
 								.then(() => {
-									observer.next(`bar`);
+									observer.next('bar');
 									return delay(2000);
 								})
 								.then(() => {
@@ -79,6 +79,6 @@ const tasks = new Listr([
 	renderer
 });
 
-tasks.run().catch(err => {
-	console.error(err.message);
+tasks.run().catch(error => {
+	console.error(error.message);
 });

index.js

@@ -108,10 +108,10 @@ class Listr {
 
 				return context;
 			})
-			.catch(err => {
-				err.context = context;
-				this._renderer.end(err);
-				throw err;
+			.catch(error => {
+				error.context = context;
+				this._renderer.end(error);
+				throw error;
 			});
 	}
 }

lib/task.js

@@ -171,29 +171,29 @@ class Task extends Subject {
 					this.state = state.COMPLETED;
 				}
 			})
-			.catch(err => {
+			.catch(error => {
 				this.state = state.FAILED;
 
-				if (err instanceof ListrError) {
-					wrapper.report(err);
+				if (error instanceof ListrError) {
+					wrapper.report(error);
 					return;
 				}
 
 				if (!this.hasSubtasks()) {
 					// Do not show the message if we have subtasks as the error is already shown in the subtask
-					this.output = err.message;
+					this.output = error.message;
 				}
 
 				this.next({
 					type: 'DATA',
-					data: err.message
+					data: error.message
 				});
 
-				wrapper.report(err);
+				wrapper.report(error);
 
 				if (this._listr.exitOnError !== false) {
 					// Do not exit when explicitely set to `false`
-					throw err;
+					throw error;
 				}
 			})
 			.then(() => {

test/exit-on-error.js

@@ -30,8 +30,8 @@ test('exit on error', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.message, 'Something went wrong');
+	} catch (error) {
+		t.is(error.message, 'Something went wrong');
 	}
 });
 
@@ -54,9 +54,9 @@ test('set `exitOnError` to false', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.message, 'Something went wrong');
-		t.is(err.errors.length, 1);
+	} catch (error) {
+		t.is(error.message, 'Something went wrong');
+		t.is(error.errors.length, 1);
 	}
 });
 
@@ -110,10 +110,10 @@ test('set `exitOnError` to false in nested list', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.message, 'Something went wrong');
-		t.is(err.errors.length, 1);
-		t.is(err.errors[0].message, 'Unicorn failed');
+	} catch (error) {
+		t.is(error.message, 'Something went wrong');
+		t.is(error.errors.length, 1);
+		t.is(error.errors[0].message, 'Unicorn failed');
 	}
 });
 
@@ -167,11 +167,11 @@ test('set `exitOnError` to false in root', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.name, 'ListrError');
-		t.is(err.errors.length, 2);
-		t.is(err.errors[0].message, 'Foo failed');
-		t.is(err.errors[1].message, 'Unicorn failed');
+	} catch (error) {
+		t.is(error.name, 'ListrError');
+		t.is(error.errors.length, 2);
+		t.is(error.errors[0].message, 'Foo failed');
+		t.is(error.errors[1].message, 'Unicorn failed');
 	}
 });
 
@@ -226,11 +226,11 @@ test('set `exitOnError` to false in root and true in child', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.name, 'ListrError');
-		t.is(err.errors.length, 2);
-		t.is(err.errors[0].message, 'Foo failed');
-		t.is(err.errors[1].message, 'Unicorn failed');
+	} catch (error) {
+		t.is(error.name, 'ListrError');
+		t.is(error.errors.length, 2);
+		t.is(error.errors[0].message, 'Foo failed');
+		t.is(error.errors[1].message, 'Unicorn failed');
 	}
 });
 
@@ -264,10 +264,10 @@ test('exit on error throws error object with context', async t => {
 
 	try {
 		await list.run();
-	} catch (err) {
-		t.is(err.name, 'ListrError');
-		t.is(err.errors.length, 1);
-		t.is(err.errors[0].message, 'Foo failed');
-		t.deepEqual(err.context, {foo: 'bar'});
+	} catch (error) {
+		t.is(error.name, 'ListrError');
+		t.is(error.errors.length, 1);
+		t.is(error.errors[0].message, 'Foo failed');
+		t.deepEqual(error.context, {foo: 'bar'});
 	}
 });

test/test.js

@@ -192,9 +192,9 @@ test('context is attached to error object', async t => {
 	try {
 		await list.run();
 		t.fail('Should throw error');
-	} catch (err) {
-		t.is(err.message, 'foo bar');
-		t.deepEqual(err.context, {
+	} catch (error) {
+		t.is(error.message, 'foo bar');
+		t.deepEqual(error.context, {
 			foo: 'bar'
 		});
 	}