Close #600 PR: Fix AVA hanging when test is used incorrectly. Fixes #574

Author: Juan Soto

lib/runner.js

@@ -63,7 +63,7 @@ optionChain(chainableMethods, function (opts, title, fn) {
 		if (typeof title !== 'string') {
 			throw new TypeError('`todo` tests require a title');
 		}
-	} else if (opts.skipped && typeof fn !== 'function') {
+	} else if (typeof fn !== 'function') {
 		throw new TypeError('Expected a function. Use `test.todo()` for tests without a function.');
 	}
 

test/runner.js

@@ -203,6 +203,16 @@ test('skip test', function (t) {
 	});
 });
 
+test('test throws when given no function', function (t) {
+	t.plan(1);
+
+	var runner = new Runner();
+
+	t.throws(function () {
+		runner.test();
+	}, {message: 'Expected a function. Use `test.todo()` for tests without a function.'});
+});
+
 test('todo test', function (t) {
 	t.plan(5);