SONARJAVA-4826 S6880 Use switch instead of if else for pattern matchi…

…ng (#4726) * SONARJAVA-4826 S6880 Use switch instead of if else for pattern matching * Prototype implementation of quickfix * Refactor + Improve quickfix indents * Test first 3 quickfixes * More rigorous implementation without quickfix * Make test cases follow the spec more closely * Implement quickfix * Add test for guards + Test first quickfix * Don't generate guards for constant cases + add tests * Fix bugs with constant cases + Refactor * Add test * Inline constant argument * Handle non-block branches + add tests * Add test * Add tests * Fix autoscan * Apply requested changes * Lift returns when possible * Add test case * Reformatting
SonarSource · Mar 20, 2024 · 2f70d6b · 2f70d6b
1 parent 61ef9d2
commit 2f70d6b
Show file tree

Hide file tree

Showing 9 changed files with 890 additions and 0 deletions.
diff --git a/its/autoscan/src/test/resources/autoscan/autoscan-diff-by-rules.json b/its/autoscan/src/test/resources/autoscan/autoscan-diff-by-rules.json
@@ -2957,6 +2957,12 @@
     "falseNegatives": 6,
     "falsePositives": 0
   },
+  {
+    "ruleKey": "S6880",
+    "hasTruePositives": true,
+    "falseNegatives": 0,
+    "falsePositives": 0
+  },
   {
     "ruleKey": "S6885",
     "hasTruePositives": true,

diff --git a/its/autoscan/src/test/resources/autoscan/diffs/diff_S6880.json b/its/autoscan/src/test/resources/autoscan/diffs/diff_S6880.json
@@ -0,0 +1,6 @@
+{
+  "ruleKey": "S6880",
+  "hasTruePositives": true,
+  "falseNegatives": 0,
+  "falsePositives": 0
+}