SONARJAVA-5324 FP, modify S6856 to support PatchMapping annotation (#5029)

Author: erwan-serandour

its/autoscan/src/test/resources/autoscan/diffs/diff_S6856.json

@@ -1,6 +1,6 @@
 {
   "ruleKey": "S6856",
   "hasTruePositives": false,
-  "falseNegatives": 43,
+  "falseNegatives": 44,
   "falsePositives": 0
 }

java-checks-test-sources/default/src/main/java/checks/spring/MissingPathVariableAnnotationCheckSample.java

@@ -5,6 +5,7 @@
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.ModelAttribute;
+import org.springframework.web.bind.annotation.PatchMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.PutMapping;
@@ -51,6 +52,11 @@ public String request(String id) {
     return "Hello World";
   }
 
+  @PatchMapping("/{id}") // Noncompliant
+  public String patch(String id) {
+    return "Hello World";
+  }
+
   @PutMapping("/id")
   @DeleteMapping("/{id}") // Noncompliant
   public String deletePut(String id) {
@@ -84,6 +90,11 @@ public String requestCompliant(@PathVariable String id) {
     return "Hello World";
   }
 
+  @PatchMapping("/{id}")
+  public String patchCompliant(@PathVariable  String id) {
+    return "Hello World";
+  }
+
   @GetMapping("/{id}")
   @DeleteMapping({"/{id}"})
   public String deleteGetCompliant(@PathVariable String id) { // compliant

java-checks/src/main/java/org/sonar/java/checks/spring/MissingPathVariableAnnotationCheck.java

@@ -47,7 +47,8 @@ public class MissingPathVariableAnnotationCheck extends IssuableSubscriptionVisi
     "org.springframework.web.bind.annotation.GetMapping",
     "org.springframework.web.bind.annotation.PostMapping",
     "org.springframework.web.bind.annotation.PutMapping",
-    "org.springframework.web.bind.annotation.DeleteMapping");
+    "org.springframework.web.bind.annotation.DeleteMapping",
+    "org.springframework.web.bind.annotation.PatchMapping");
 
   @Override
   public List<Tree.Kind> nodesToVisit() {