test(linter/no-fallthrough): add test for switch with logical operators + break (#14811)

fixes #6417

Author: camc314

crates/oxc_linter/src/rules/eslint/no_fallthrough.rs

@@ -585,6 +585,9 @@ fn test() {
                 "reportUnusedFallthroughComment": false
             }])),
         ),
+        // Issue #6417: switch with logical operators should work correctly with break
+        ("switch(true) { case x === 1 || x === 2: a(); break; case x === 3: b(); }", None),
+        ("switch(true) { case x === 1 && y: a(); break; case x === 3: b(); }", None),
     ];
 
     let fail = vec![
@@ -648,6 +651,9 @@ fn test() {
                 "reportUnusedFallthroughComment": true
             }])),
         ),
+        // Issue #6417: switch with logical operators should detect fallthrough
+        ("switch(true) { case x === 1 || x === 2: a(); case x === 3: b(); }", None),
+        ("switch(true) { case x === 1 && y: a(); case x === 3: b(); }", None),
         // TODO: it should fail but doesn't, we ignore conditional discriminants for now.
         // ("switch (a === b ? c : d) { case 1: ; case 2: ; case 3: ; }", None)
     ];

crates/oxc_linter/src/snapshots/eslint_no_fallthrough.snap

@@ -174,3 +174,15 @@ source: crates/oxc_linter/src/tester.rs
  1 │ switch(foo) { case 0: a(); break; /* falls through */ case 1: b(); }
    ·                                  ─────────────────────
    ╰────
+
+  ⚠ eslint(no-fallthrough): Expected a 'break' statement before 'case'.
+   ╭─[no_fallthrough.tsx:1:46]
+ 1 │ switch(true) { case x === 1 || x === 2: a(); case x === 3: b(); }
+   ·                                              ──────────────────
+   ╰────
+
+  ⚠ eslint(no-fallthrough): Expected a 'break' statement before 'case'.
+   ╭─[no_fallthrough.tsx:1:40]
+ 1 │ switch(true) { case x === 1 && y: a(); case x === 3: b(); }
+   ·                                        ──────────────────
+   ╰────