Skip to content

Commit 9c4bd42

Browse files
committed
fix(linter/jest/expect-expect): add support for expect in array expressions (#12877)
closes #12869
1 parent 982275a commit 9c4bd42

File tree

1 file changed

+21
-0
lines changed

1 file changed

+21
-0
lines changed

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

Lines changed: 21 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,17 @@ 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(new Promise((resolve) => { resolve(1); }) ).resolves.toBe(1),
458+
expect(new Promise((_, reject) => { reject(new Error('Failed')); })).rejects.toThrowError('Failed'),
459+
]);
460+
});
461+
",
462+
None,
463+
),
443464
(
444465
r#"
445466
it("should not warn on await expect", async () => {

0 commit comments

Comments
 (0)