Skip to content

Commit df2606f

Browse files
committed
fix(linter/jest/expect-expect): add support for expect in array expressions
1 parent 6b4a7a7 commit df2606f

File tree

1 file changed

+29
-0
lines changed

1 file changed

+29
-0
lines changed

crates/oxc_linter/src/rules/jest/expect_expect.rs

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,16 @@ fn check_assert_function_used<'a>(
259259
Expression::AwaitExpression(expr) => {
260260
return check_assert_function_used(&expr.argument, assert_function_names, visited, ctx);
261261
}
262+
Expression::ArrayExpression(array_expr) => {
263+
for element in &array_expr.elements {
264+
if let Some(element_expr) = element.as_expression() {
265+
if check_assert_function_used(element_expr, assert_function_names, visited, ctx)
266+
{
267+
return true;
268+
}
269+
}
270+
}
271+
}
262272
_ => {}
263273
}
264274

@@ -440,6 +450,25 @@ fn test() {
440450
"#,
441451
None,
442452
),
453+
(
454+
r"
455+
it('should not warn on expect in Promise.all', async () => {
456+
await Promise.all([
457+
expect(
458+
new Promise((resolve) => {
459+
resolve(1);
460+
})
461+
).resolves.toBe(1),
462+
expect(
463+
new Promise((_, reject) => {
464+
reject(new Error('Failed'));
465+
})
466+
).rejects.toThrowError('Failed'),
467+
]);
468+
});
469+
",
470+
None,
471+
),
443472
(
444473
r#"
445474
it("should not warn on await expect", async () => {

0 commit comments

Comments
 (0)