Skip to content

Commit 420c862

Browse files
Support assertion message in `t.timeout()
See avajs/ava@ca8ea45.
1 parent 26a3732 commit 420c862

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

rules/assertion-arguments.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ const expectedNbArguments = {
7878
},
7979
timeout: {
8080
min: 1,
81-
max: 1
81+
max: 2
8282
}
8383
};
8484

test/assertion-arguments.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ ruleTester.run('assertion-arguments', rule, {
5555
testCase(false, 't.snapshot(value, \'message\');'),
5656
testCase(false, 't.context.plan();'),
5757
testCase(false, 't.teardown(() => {});'),
58-
testCase(false, 't.timeout(100);'),
58+
testCase(false, 't.timeout(100, \'message\');'),
5959
testCase(false, 'foo.t.plan();'),
6060
// Shouldn't be triggered since it's not a test file
6161
testCase(false, 't.true(true);', false, false),
@@ -106,7 +106,7 @@ ruleTester.run('assertion-arguments', rule, {
106106
testCase('always', 't.is.skip(\'same\', \'same\', \'message\');'),
107107
testCase('always', 't.snapshot(value, \'message\');'),
108108
testCase('always', 't.teardown(() => {});'),
109-
testCase('always', 't.timeout(100);'),
109+
testCase('always', 't.timeout(100, \'message\');'),
110110
testCase('always', 't.try(tt => tt.pass());'),
111111
testCase('always', 't.try(tt => tt.pass(), 1, 2);'),
112112
testCase('always', 't.try(\'title\', tt => tt.pass(), 1, 2);'),
@@ -215,7 +215,7 @@ ruleTester.run('assertion-arguments', rule, {
215215
testCase(false, 't.is.skip(\'same\', \'same\', \'message\', \'extra argument\');', tooManyError(3)),
216216
testCase(false, 't.snapshot(value, \'message\', \'extra argument\');', tooManyError(2)),
217217
testCase(false, 't.teardown(() => {}, \'extra argument\');', tooManyError(1)),
218-
testCase(false, 't.timeout(1, \'extra argument\');', tooManyError(1)),
218+
testCase(false, 't.timeout(1, \'message\', \'extra argument\');', tooManyError(2)),
219219

220220
testCase('always', 't.pass();', missingError),
221221
testCase('always', 't.fail();', missingError),

0 commit comments

Comments
 (0)