From fefe01ecbf78ac4de5d42c2c0deff77d5f15faec Mon Sep 17 00:00:00 2001 From: Jeroen Ketema Date: Wed, 17 Jun 2026 10:52:38 +0200 Subject: [PATCH] Java: Update expected test results after extractor changes --- .../buildless-erroneous/diagnostics.expected | 15 +++++++++++++++ .../buildless-gradle-boms/diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../java/buildless-gradle/diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../test.py | 2 +- .../java/buildless-maven/diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../buildless-proxy-maven/diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 15 +++++++++++++++ .../java/buildless/diagnostics.expected | 15 +++++++++++++++ .../diagnostics.expected | 18 ++++++++++++++++++ .../test.py | 3 ++- .../diagnostics.expected | 15 +++++++++++++++ .../java/maven-download-failure/test.py | 2 +- 22 files changed, 292 insertions(+), 3 deletions(-) diff --git a/java/ql/integration-tests/java/buildless-erroneous/diagnostics.expected b/java/ql/integration-tests/java/buildless-erroneous/diagnostics.expected index 90aa56bf3f6..ee1b8835665 100644 --- a/java/ql/integration-tests/java/buildless-erroneous/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-erroneous/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Because no usable build tool (Gradle, Maven, etc) was found, build scripts could not be queried for guidance about the appropriate JDK version for the code being extracted, or precise dependency information. The default JDK will be used, and external dependencies will be inferred from the Java package names used.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-gradle-boms/diagnostics.expected b/java/ql/integration-tests/java/buildless-gradle-boms/diagnostics.expected index 976e0eb08fc..d78b3ca081a 100644 --- a/java/ql/integration-tests/java/buildless-gradle-boms/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-gradle-boms/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Gradle to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-gradle-classifiers/diagnostics.expected b/java/ql/integration-tests/java/buildless-gradle-classifiers/diagnostics.expected index 7312fdf95ec..906a8f12990 100644 --- a/java/ql/integration-tests/java/buildless-gradle-classifiers/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-gradle-classifiers/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Gradle to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-gradle-timeout/diagnostics.expected b/java/ql/integration-tests/java/buildless-gradle-timeout/diagnostics.expected index 779ffa91e71..56072f5b90c 100644 --- a/java/ql/integration-tests/java/buildless-gradle-timeout/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-gradle-timeout/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "A Gradle process was aborted because it didn't write to the console for 5 seconds. Consider either lengthening the timeout if appropriate by setting CODEQL_EXTRACTOR_JAVA_BUILDLESS_CHILD_PROCESS_IDLE_TIMEOUT to a higher value or zero for no timeout, or else investigate why Gradle timed out. Java analysis will continue, but the analysis may be of reduced quality.", "severity": "note", diff --git a/java/ql/integration-tests/java/buildless-gradle/diagnostics.expected b/java/ql/integration-tests/java/buildless-gradle/diagnostics.expected index 337fa933808..abe6bfaa24c 100644 --- a/java/ql/integration-tests/java/buildless-gradle/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-gradle/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Gradle to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-inherit-trust-store/diagnostics.expected b/java/ql/integration-tests/java/buildless-inherit-trust-store/diagnostics.expected index 766db40aa62..9faa13e8d8b 100644 --- a/java/ql/integration-tests/java/buildless-inherit-trust-store/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-inherit-trust-store/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-maven-executable-war/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-executable-war/diagnostics.expected index 1058e1528f9..cc4731ad9c2 100644 --- a/java/ql/integration-tests/java/buildless-maven-executable-war/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-executable-war/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-maven-existing-settings-xml/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-existing-settings-xml/diagnostics.expected index f3c89bb842a..fdef7e871f5 100644 --- a/java/ql/integration-tests/java/buildless-maven-existing-settings-xml/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-existing-settings-xml/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-maven-mirrorof/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-mirrorof/diagnostics.expected index f3c89bb842a..fdef7e871f5 100644 --- a/java/ql/integration-tests/java/buildless-maven-mirrorof/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-mirrorof/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-maven-multimodule/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-multimodule/diagnostics.expected index 1058e1528f9..cc4731ad9c2 100644 --- a/java/ql/integration-tests/java/buildless-maven-multimodule/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-multimodule/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-maven-timeout/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-timeout/diagnostics.expected index 5a30189b5e3..04c4fd2c7e5 100644 --- a/java/ql/integration-tests/java/buildless-maven-timeout/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-timeout/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "A Maven process was aborted because it didn't write to the console for 5 seconds. Consider either lenghtening the timeout if appropriate by setting CODEQL_EXTRACTOR_JAVA_BUILDLESS_CHILD_PROCESS_IDLE_TIMEOUT to a higher value or zero for no timeout, or else investigate why Maven timed out. Java analysis will continue, but the analysis may be of reduced quality.", "severity": "note", diff --git a/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/diagnostics.expected index 0ef924eb7c1..49e812dc1e9 100644 --- a/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "At least one dependency JAR suggested by the build system could not be downloaded. This means the analysis will try to satisfy the dependency with its default choice for the required external package name, which may be the wrong version or the wrong package entirely. This may lead to partial analysis of code using this dependency. See the extraction log for full details. If the cause appears to be a temporary outage, consider retrying the analysis.", "severity": "note", diff --git a/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/test.py b/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/test.py index f7673ce3ad1..3a08acad55f 100644 --- a/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/test.py +++ b/java/ql/integration-tests/java/buildless-maven-tolerate-unavailable-dependency/test.py @@ -1,4 +1,4 @@ -def test(codeql, java): +def test(codeql, java, check_diagnostics_java): codeql.database.create( build_mode="none", ) diff --git a/java/ql/integration-tests/java/buildless-maven/diagnostics.expected b/java/ql/integration-tests/java/buildless-maven/diagnostics.expected index f3c89bb842a..fdef7e871f5 100644 --- a/java/ql/integration-tests/java/buildless-maven/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-maven/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-proxy-gradle/diagnostics.expected b/java/ql/integration-tests/java/buildless-proxy-gradle/diagnostics.expected index 337fa933808..abe6bfaa24c 100644 --- a/java/ql/integration-tests/java/buildless-proxy-gradle/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-proxy-gradle/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Gradle to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-proxy-maven/diagnostics.expected b/java/ql/integration-tests/java/buildless-proxy-maven/diagnostics.expected index f3c89bb842a..fdef7e871f5 100644 --- a/java/ql/integration-tests/java/buildless-proxy-maven/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-proxy-maven/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless-sibling-projects/diagnostics.expected b/java/ql/integration-tests/java/buildless-sibling-projects/diagnostics.expected index b3df8a700c3..b821d41e600 100644 --- a/java/ql/integration-tests/java/buildless-sibling-projects/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless-sibling-projects/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis dropped the following dependencies because a sibling project depends on a higher version:\n\n* `junit/junit-4.11`", "severity": "unknown", diff --git a/java/ql/integration-tests/java/buildless/diagnostics.expected b/java/ql/integration-tests/java/buildless/diagnostics.expected index 90aa56bf3f6..ee1b8835665 100644 --- a/java/ql/integration-tests/java/buildless/diagnostics.expected +++ b/java/ql/integration-tests/java/buildless/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Because no usable build tool (Gradle, Maven, etc) was found, build scripts could not be queried for guidance about the appropriate JDK version for the code being extracted, or precise dependency information. The default JDK will be used, and external dependencies will be inferred from the Java package names used.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/diagnostics.expected b/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/diagnostics.expected index f40920e10d6..b23aa85f254 100644 --- a/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/diagnostics.expected +++ b/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/diagnostics.expected @@ -1,3 +1,21 @@ +{ + "attributes": { + "java_vendor": "__REDACTED__", + "java_version": "11.0.31" + }, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Analyzed a Gradle project without the [Gradle wrapper](https://docs.gradle.org/current/userguide/gradle_wrapper.html). This may use an incompatible version of Gradle.", "severity": "warning", diff --git a/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/test.py b/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/test.py index e59277f3ea3..92aedd825ef 100644 --- a/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/test.py +++ b/java/ql/integration-tests/java/gradle-sample-without-wrapper-or-gradle-buildless/test.py @@ -4,7 +4,8 @@ import pathlib # The version of gradle used doesn't work on java 17 -def test(codeql, use_java_11, java, environment): +def test(codeql, use_java_11, java, environment, check_diagnostics): + check_diagnostics.redact += ["attributes.java_vendor"] gradle_override_dir = pathlib.Path(tempfile.mkdtemp()) if runs_on.windows: (gradle_override_dir / "gradle.bat").write_text("@echo off\nexit /b 2\n") diff --git a/java/ql/integration-tests/java/maven-download-failure/diagnostics.expected b/java/ql/integration-tests/java/maven-download-failure/diagnostics.expected index daabe47a9e9..c3e812b3616 100644 --- a/java/ql/integration-tests/java/maven-download-failure/diagnostics.expected +++ b/java/ql/integration-tests/java/maven-download-failure/diagnostics.expected @@ -1,3 +1,18 @@ +{ + "attributes": {}, + "markdownMessage": "Internal telemetry for the Java extractor.\n\nNo action needed.", + "severity": "note", + "source": { + "extractorName": "java", + "id": "java/extractor/summary", + "name": "Java extractor telemetry" + }, + "visibility": { + "cliSummaryTable": false, + "statusPage": false, + "telemetry": true + } +} { "markdownMessage": "Java analysis used build tool Maven to pick a JDK version and/or to recommend external dependencies.", "severity": "unknown", diff --git a/java/ql/integration-tests/java/maven-download-failure/test.py b/java/ql/integration-tests/java/maven-download-failure/test.py index a86d970e3fe..ad9c4bb7840 100644 --- a/java/ql/integration-tests/java/maven-download-failure/test.py +++ b/java/ql/integration-tests/java/maven-download-failure/test.py @@ -2,7 +2,7 @@ import os import os.path import shutil -def test(codeql, java, check_diagnostics): +def test(codeql, java, check_diagnostics_java): # Avoid shutil resolving mvn to the wrapper script in the test dir: os.environ["NoDefaultCurrentDirectoryInExePath"] = "0"