fix(linter/prefer-called-once): avoid panic on trailing comma

crates/oxc_linter/src/rules/vitest/prefer_called_once.rs

@@ -112,11 +112,14 @@ impl PreferCalledOnce {
             ctx.diagnostic_with_fix(
                 prefer_called_once_diagnostic(matcher_and_args_span, new_matcher_name.as_ref()),
                 |fixer| {
+                    let argument_without_parenthesis_span =
+                        get_inside_parenthesis_span(called_times_value.span, ctx);
+
                     let multi_fix = fixer.for_multifix();
                     let mut fixes = multi_fix.new_fix_with_capacity(2);
 
                     fixes.push(fixer.replace(matcher_to_be_fixed.span, new_matcher_name));
-                    fixes.push(fixer.delete(&called_times_value.span));
+                    fixes.push(fixer.delete(&argument_without_parenthesis_span));
 
                     fixes.with_message("Replace API with preferOnce instead of Times")
                 },
@@ -125,6 +128,28 @@ impl PreferCalledOnce {
     }
 }
 
+fn is_open_parenthesis(source_text: &str) -> bool {
+    source_text.starts_with('(')
+}
+
+fn is_close_parenthesis(source_text: &str) -> bool {
+    source_text.ends_with(')')
+}
+
+fn get_inside_parenthesis_span(argument_span: Span, ctx: &LintContext<'_>) -> Span {
+    let mut inside_parentehsis_span = Span::new(argument_span.start, argument_span.end);
+
+    while !is_open_parenthesis(ctx.source_range(inside_parentehsis_span.expand_left(1))) {
+        inside_parentehsis_span = inside_parentehsis_span.expand_left(1);
+    }
+
+    while !is_close_parenthesis(ctx.source_range(inside_parentehsis_span.expand_right(1))) {
+        inside_parentehsis_span = inside_parentehsis_span.expand_right(1);
+    }
+
+    inside_parentehsis_span
+}
+
 #[test]
 fn test() {
     use crate::tester::Tester;
@@ -153,6 +178,9 @@ fn test() {
         "expect(fn).resolves.toBeCalledTimes(1);",
         "expect(fn).resolves.toHaveBeenCalledTimes(1);",
         "expect(fn).resolves.toHaveBeenCalledTimes(/*comment*/1);",
+        "expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledTimes(
+              1,
+            );",
     ];
 
     let fix = vec![
@@ -165,6 +193,12 @@ fn test() {
             "expect(fn).resolves.toHaveBeenCalledTimes(1);",
             "expect(fn).resolves.toHaveBeenCalledOnce();",
         ),
+        (
+            "expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledTimes(
+            1,
+            );",
+            "expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledOnce();",
+        ),
     ];
     Tester::new(PreferCalledOnce::NAME, PreferCalledOnce::PLUGIN, pass, fail)
         .expect_fix(fix)

crates/oxc_linter/src/snapshots/vitest_prefer_called_once.snap

@@ -49,3 +49,11 @@ source: crates/oxc_linter/src/tester.rs
    ·                     ───────────────────────────────────
    ╰────
   help: Prefer `toHaveBeenCalledOnce()`.
+
+  ⚠ eslint-plugin-vitest(prefer-called-once): The use of `toBeCalledTimes(1)` and `toHaveBeenCalledTimes(1)` is discouraged.
+   ╭─[prefer_called_once.tsx:1:53]
+ 1 │ ╭─▶ expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledTimes(
+ 2 │ │                 1,
+ 3 │ ╰─▶             );
+   ╰────
+  help: Prefer `toHaveBeenCalledOnce()`.