Reword error message for when assertions are used after test has fini…

…shed
avajs · Apr 5, 2017 · d2a294d · d2a294d
1 parent ecdadfa
commit d2a294d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/test.js b/lib/test.js
@@ -154,15 +154,15 @@ class Test {
 
 	countPassedAssertion() {
 		if (this.finishing) {
-			this.saveFirstError(new Error('Assertion passed, but test has already ended'));
+			this.saveFirstError(new Error('Assertion passed, but test has already finished'));
 		}
 
 		this.assertCount++;
 	}
 
 	addPendingAssertion(promise) {
 		if (this.finishing) {
-			this.saveFirstError(new Error('Assertion passed, but test has already ended'));
+			this.saveFirstError(new Error('Assertion passed, but test has already finished'));
 		}
 
 		this.assertCount++;
@@ -171,7 +171,7 @@ class Test {
 
 	addFailedAssertion(error) {
 		if (this.finishing) {
-			this.saveFirstError(new Error('Assertion failed, but test has already ended'));
+			this.saveFirstError(new Error('Assertion failed, but test has already finished'));
 		}
 
 		this.assertCount++;

diff --git a/test/test.js b/test/test.js
@@ -591,7 +591,7 @@ test('number of assertions matches t.plan when the test exits, but before all pe
 		result = r;
 	}).run().then(passed => {
 		t.is(passed, false);
-		t.match(result.reason.message, /Assertion passed, but test has already ended/);
+		t.match(result.reason.message, /Assertion passed, but test has already finished/);
 		t.is(result.reason.name, 'Error');
 		t.end();
 	});
@@ -610,7 +610,7 @@ test('number of assertions matches t.plan when the test exits, but before all pe
 		result = r;
 	}).run().then(passed => {
 		t.is(passed, false);
-		t.match(result.reason.message, /Assertion failed, but test has already ended/);
+		t.match(result.reason.message, /Assertion failed, but test has already finished/);
 		t.is(result.reason.name, 'Error');
 		t.end();
 	});