add test cases for non-function values

Author: takuji

crates/oxc_linter/src/rules/react/jsx_handler_names.rs

@@ -304,6 +304,8 @@ impl Rule for JsxHandlerNames {
                 if !self.check_local_variables && !value_expr.is_member_expression() {
                     return;
                 }
+                // For other expressions types, use the whole content inside the braces as the handler name,
+                // which will be marked as a bad handler name if the prop key is an event handler prop.
                 let span = expression_container.span.shrink(1);
                 (Some(normalize_handler_name(ctx.source_range(span))), span, false)
             }
@@ -613,6 +615,8 @@ fn test() {
                 serde_json::json!([{ "checkLocalVariables": true, "ignoreComponentNames": ["MyLib*"] }]),
             ),
         ),
+        ("<TestComponent onChange={true} />", None), // ok if not checking local variables (the same behavior as eslint version)
+        ("<TestComponent onChange={'value'} />", None), // ok if not checking local variables (the same behavior as eslint version)
     ];
 
     let fail = vec![
@@ -689,6 +693,14 @@ fn test() {
                 serde_json::json!([{ "checkLocalVariables": true, "ignoreComponentNames": ["MyLibrary*"] }]),
             ),
         ),
+        (
+            "<TestComponent onChange={true} />",
+            Some(serde_json::json!([{ "checkLocalVariables": true }])),
+        ),
+        (
+            "<TestComponent onChange={'value'} />",
+            Some(serde_json::json!([{ "checkLocalVariables": true }])),
+        ),
     ];
 
     Tester::new(JsxHandlerNames::NAME, JsxHandlerNames::PLUGIN, pass, fail).test_and_snapshot();

crates/oxc_linter/src/snapshots/react_jsx_handler_names.snap

@@ -181,3 +181,17 @@ source: crates/oxc_linter/src/tester.rs
  8 │                 </div>
    ╰────
   help: Prop key for handleButton must begin with 'on'
+
+  ⚠ eslint-plugin-react(jsx-handler-names): Invalid handler name: true
+   ╭─[jsx_handler_names.tsx:1:26]
+ 1 │ <TestComponent onChange={true} />
+   ·                          ────
+   ╰────
+  help: Handler function for onChange prop key must be a camelCase name beginning with 'handle' only
+
+  ⚠ eslint-plugin-react(jsx-handler-names): Invalid handler name: 'value'
+   ╭─[jsx_handler_names.tsx:1:26]
+ 1 │ <TestComponent onChange={'value'} />
+   ·                          ───────
+   ╰────
+  help: Handler function for onChange prop key must be a camelCase name beginning with 'handle' only