Add t.timeout to use-t-well rule

Author: pvdlg

rules/assertion-arguments.js

@@ -71,6 +71,10 @@ const expectedNbArguments = {
 	truthy: {
 		min: 1,
 		max: 2
+	},
+	timeout: {
+		min: 1,
+		max: 1
 	}
 };
 

test/assertion-arguments.js

@@ -54,6 +54,7 @@ ruleTester.run('assertion-arguments', rule, {
 		testCase(false, 't.truthy(\'unicorn\', \'message\');'),
 		testCase(false, 't.snapshot(value, \'message\');'),
 		testCase(false, 't.context.plan();'),
+		testCase(false, 't.timeout(100);'),
 		testCase(false, 'foo.t.plan();'),
 		// Shouldn't be triggered since it's not a test file
 		testCase(false, 't.true(true);', false, false),
@@ -103,6 +104,7 @@ ruleTester.run('assertion-arguments', rule, {
 		testCase('always', 't.skip.is(\'same\', \'same\', \'message\');'),
 		testCase('always', 't.is.skip(\'same\', \'same\', \'message\');'),
 		testCase('always', 't.snapshot(value, \'message\');'),
+		testCase('always', 't.timeout(100);'),
 
 		// Shouldn't be triggered since it's not a test file
 		testCase('always', 't.true(true);', [], false),
@@ -131,6 +133,7 @@ ruleTester.run('assertion-arguments', rule, {
 		testCase('never', 't.skip.is(\'same\', \'same\');'),
 		testCase('never', 't.is.skip(\'same\', \'same\');'),
 		testCase('never', 't.snapshot(value);'),
+		testCase('never', 't.timeout(100);'),
 		// Shouldn't be triggered since it's not a test file
 		testCase('never', 't.true(true, \'message\');', [], false),
 
@@ -177,6 +180,7 @@ ruleTester.run('assertion-arguments', rule, {
 		testCase(false, 't.skip.is(\'same\');', tooFewError(2)),
 		testCase(false, 't.is.skip(\'same\');', tooFewError(2)),
 		testCase(false, 't.snapshot();', tooFewError(1)),
+		testCase(false, 't.timeout();', tooFewError(1)),
 
 		// Too many arguments
 		testCase(false, 't.plan(1, \'extra argument\');', tooManyError(1)),
@@ -198,6 +202,7 @@ ruleTester.run('assertion-arguments', rule, {
 		testCase(false, 't.skip.is(\'same\', \'same\', \'message\', \'extra argument\');', tooManyError(3)),
 		testCase(false, 't.is.skip(\'same\', \'same\', \'message\', \'extra argument\');', tooManyError(3)),
 		testCase(false, 't.snapshot(value, \'message\', \'extra argument\');', tooManyError(2)),
+		testCase(false, 't.timeout(1, \'extra argument\');', tooManyError(1)),
 
 		testCase('always', 't.pass();', missingError),
 		testCase('always', 't.fail();', missingError),

test/use-t-well.js

@@ -64,6 +64,7 @@ ruleTester.run('use-t-well', rule, {
 		testCase('a.foo();'),
 		testCase('t.context.foo(a, a);'),
 		testCase('foo.t.bar(a, a);'),
+		testCase('t.timeout(100);'),
 		// Shouldn't be triggered since it's not a test file
 		testCase('t.foo(a, a);', false),
 		testCase('t.foo;', false)

util.js

@@ -123,4 +123,4 @@ const assertionMethodNames = [...assertionMethodsNumArguments.keys()];
 
 exports.assertionMethodsNumArguments = assertionMethodsNumArguments;
 exports.assertionMethods = new Set(assertionMethodNames);
-exports.executionMethods = new Set(assertionMethodNames.concat(['end', 'plan', 'log']));
+exports.executionMethods = new Set(assertionMethodNames.concat(['end', 'plan', 'log', 'timeout']));