Merge pull request #17137 from smowton/smowton/admin/ecj-annotations-…

…bug-test

Java: Add test for ECJ compiling an enum with a constructor parameter annotation

java/ql/integration-tests/all-platforms/java/buildless-dependency-different-repository/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-inherit-trust-store/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-maven-multimodule/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-maven-tolerate-unavailable-dependency/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-maven/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-proxy-maven/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/buildless-snapshot-repository/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/Diagnostics.ql

@@ -0,0 +1,4 @@
+import semmle.code.java.Diagnostics
+
+from Diagnostic d
+select d, d.getSeverity(), d.getMessage()

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/Test.java

@@ -0,0 +1,9 @@
+public enum Test {
+
+  A("A"), B("B"), C("C");
+
+  private Test(@Ann String x) { }
+
+}
+
+@interface Ann {}

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/Test2.java

@@ -0,0 +1,2 @@
+
+public class Test2 { Test t; }

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/test.expected

@@ -0,0 +1 @@
+| Test.java:5:16:5:28 | x | Test.java:5:16:5:19 | Ann |

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/test.py

@@ -0,0 +1,8 @@
+import urllib.request
+from create_database_utils import *
+
+urllib.request.urlretrieve("https://repo1.maven.org/maven2/org/eclipse/jdt/ecj/3.38.0/ecj-3.38.0.jar", "ecj.jar")
+
+# This tests the case where ECJ emits a RuntimeIn/VisibleAnnotations attribute that isn't the same size as the corresponding method argument list, in particular due to forgetting to include the synthetic parameters added to explicit enumeration constructors.
+
+run_codeql_database_create(["java -cp ecj.jar org.eclipse.jdt.internal.compiler.batch.Main Test.java -d out -source 8", "java -cp ecj.jar org.eclipse.jdt.internal.compiler.batch.Main Test2.java -cp out -source 8"], lang="java")

java/ql/integration-tests/all-platforms/java/ecj-tolerate-enum-annotations/test.ql

@@ -0,0 +1,4 @@
+import java
+
+from Parameter p
+select p, p.getAnAnnotation()

java/ql/integration-tests/all-platforms/java/java-web-jsp/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-enforcer/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-extract-properties/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-large-xml-files/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-small-xml-files/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-xml-mode-all/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-xml-mode-byname/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-xml-mode-disabled/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample-xml-mode-smart/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially

java/ql/integration-tests/all-platforms/java/maven-sample/force_sequential_test_execution

@@ -0,0 +1 @@
+# Concurrent Maven processes using ~/.m2/repository is not safe, so this test must run sequentially