Java: add extra test for java/alert-suppression-annotations

yh-semmle · yh-semmle · commit ee2c97f147fc · 2019-10-17T22:09:04.000-04:00
diff --git a/java/ql/test/query-tests/AlertSuppression/AlertSuppressionAnnotations.expected b/java/ql/test/query-tests/AlertSuppression/AlertSuppressionAnnotations.expected
@@ -1,4 +1,6 @@
-| TestSuppressWarnings.java:2:1:2:49 | SuppressWarnings | lgtm[java/non-sync-override] | lgtm[java/non-sync-override] | TestSuppressWarnings.java:2:1:17:5 | suppression range |
+| TestSuppressWarnings.java:2:1:2:49 | SuppressWarnings | lgtm[java/non-sync-override] | lgtm[java/non-sync-override] | TestSuppressWarnings.java:2:1:21:5 | suppression range |
 | TestSuppressWarnings.java:5:5:5:31 | SuppressWarnings | lgtm[] | lgtm[] | TestSuppressWarnings.java:5:5:8:5 | suppression range |
 | TestSuppressWarnings.java:10:5:10:104 | SuppressWarnings | lgtm[java/confusing-method-name] not confusing | lgtm[java/confusing-method-name] | TestSuppressWarnings.java:9:5:13:5 | suppression range |
 | TestSuppressWarnings.java:10:5:10:104 | SuppressWarnings | lgtm[java/non-sync-override] | lgtm[java/non-sync-override] | TestSuppressWarnings.java:9:5:13:5 | suppression range |
+| TestSuppressWarnings.java:18:5:18:98 | SuppressWarnings | lgtm[java/confusing-method-name] blah blah lgtm[java/non-sync-override] | lgtm[java/confusing-method-name] | TestSuppressWarnings.java:18:5:21:5 | suppression range |
+| TestSuppressWarnings.java:18:5:18:98 | SuppressWarnings | lgtm[java/confusing-method-name] blah blah lgtm[java/non-sync-override] | lgtm[java/non-sync-override] | TestSuppressWarnings.java:18:5:21:5 | suppression range |
diff --git a/java/ql/test/query-tests/AlertSuppression/TestSuppressWarnings.java b/java/ql/test/query-tests/AlertSuppression/TestSuppressWarnings.java
@@ -14,5 +14,9 @@ public void test2() {
     @SuppressWarnings("lgtm")
     public void test3() {
         
+    }
+    @SuppressWarnings({"lgtm[java/confusing-method-name] blah blah lgtm[java/non-sync-override]"})
+    public void test4() {
+        
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -14,5 +14,9 @@ public void test2() {`
`14`	`14`	`@SuppressWarnings("lgtm")`
`15`	`15`	`public void test3() {`
`16`	`16`
	`17`	`+ }`
	`18`	`+ @SuppressWarnings({"lgtm[java/confusing-method-name] blah blah lgtm[java/non-sync-override]"})`
	`19`	`+ public void test4() {`
	`20`	`+`
`17`	`21`	`}`
`18`	`22`	`}`