Skip to content

chore: port changes in google-cloud-java since 2026-03-24 #4170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
diegomarquezp wants to merge 10 commits into
base: main
Choose a base branch
from

chore: add missing settings.xml for GraalVM job

8fc92f5
Select commit
Loading
Failed to load commit list.
Open

chore: port changes in google-cloud-java since 2026-03-24 #4170

chore: add missing settings.xml for GraalVM job
8fc92f5
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [java_showcase_integration_tests] SonarCloud Code Analysis failed Apr 8, 2026 in 1m 17s

Quality Gate failed

Failed conditions
2 Security Hotspots
48.5% Coverage on New Code (required ≥ 80%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 318 in api-common-java/src/main/java/com/google/api/pathtemplate/PathTemplate.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

This block of commented-out lines of code should be removed. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqzDSeDQPUo8jO-d&open=AZ1uvqzDSeDQPUo8jO-d&pullRequest=4170

Check warning on line 232 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/logging/ITActionableErrorsLogging.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqmdSeDQPUo8jO9h&open=AZ1uvqmdSeDQPUo8jO9h&pullRequest=4170

Check warning on line 68 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelTracePropagation.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove this unused "responseTrailers" private field. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqSSeDQPUo8jO9_&open=AZ1uvqqSSeDQPUo8jO9_&pullRequest=4170

Check failure on line 519 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO92&open=AZ1uvqqISeDQPUo8jO92&pullRequest=4170

Check warning on line 536 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelTracing.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "errorSpan". 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqmSeDQPUo8jO-A&open=AZ1uvqqmSeDQPUo8jO-A&pullRequest=4170

Check warning on line 191 in gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracerContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqvCSeDQPUo8jO-X&open=AZ1uvqvCSeDQPUo8jO-X&pullRequest=4170

Check warning on line 52 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/logging/ITActionableErrorsLogging.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove this 'public' modifier. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqmdSeDQPUo8jO9g&open=AZ1uvqmdSeDQPUo8jO9g&pullRequest=4170

Check warning on line 964 in api-common-java/src/test/java/com/google/api/pathtemplate/PathTemplateTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Replace these 6 tests with a single Parameterized one. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqzgSeDQPUo8jO-j&open=AZ1uvqzgSeDQPUo8jO-j&pullRequest=4170

Check warning on line 109 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO9j&open=AZ1uvqqISeDQPUo8jO9j&pullRequest=4170

Check warning on line 71 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelTracePropagation.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqSSeDQPUo8jO97&open=AZ1uvqqSSeDQPUo8jO97&pullRequest=4170

Check failure on line 170 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO9n&open=AZ1uvqqISeDQPUo8jO9n&pullRequest=4170

Check failure on line 559 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelTracing.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqmSeDQPUo8jO-E&open=AZ1uvqqmSeDQPUo8jO-E&pullRequest=4170

Check failure on line 167 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO9m&open=AZ1uvqqISeDQPUo8jO9m&pullRequest=4170

Check warning on line 409 in api-common-java/src/main/java/com/google/api/pathtemplate/PathTemplate.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Refactor the code in order to not assign to this loop counter from within the loop body. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqzDSeDQPUo8jO-b&open=AZ1uvqzDSeDQPUo8jO-b&pullRequest=4170

Check warning on line 208 in gax-java/gax/src/main/java/com/google/api/gax/tracing/ApiTracer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvquCSeDQPUo8jO-T&open=AZ1uvquCSeDQPUo8jO-T&pullRequest=4170

Check failure on line 67 in gax-java/gax/src/test/java/com/google/api/gax/tracing/ErrorTypeUtilTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Refactor the body of this try/catch to not have multiple invocations throwing the same checked exception. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqrvSeDQPUo8jO-J&open=AZ1uvqrvSeDQPUo8jO-J&pullRequest=4170

Check warning on line 112 in gax-java/gax/src/test/java/com/google/api/gax/tracing/CompositeTracerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Replace this lambda with method reference 'compositeScope::close'. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqsKSeDQPUo8jO-N&open=AZ1uvqsKSeDQPUo8jO-N&pullRequest=4170

Check failure on line 82 in gax-java/gax/src/test/java/com/google/api/gax/tracing/ErrorTypeUtilTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Refactor the body of this try/catch to not have multiple invocations throwing the same checked exception. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqrvSeDQPUo8jO-K&open=AZ1uvqrvSeDQPUo8jO-K&pullRequest=4170

Check warning on line 345 in gax-java/gax-grpc/src/test/java/com/google/api/gax/grpc/GrpcClientCallsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqwESeDQPUo8jO-Y&open=AZ1uvqwESeDQPUo8jO-Y&pullRequest=4170

Check warning on line 356 in gax-java/gax-grpc/src/test/java/com/google/api/gax/grpc/GrpcClientCallsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Rename "mockChannel" which hides the field declared at line 91. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqwESeDQPUo8jO-Z&open=AZ1uvqwESeDQPUo8jO-Z&pullRequest=4170

Check warning on line 120 in gax-java/gax/src/main/java/com/google/api/gax/tracing/CompositeTracer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqtiSeDQPUo8jO-Q&open=AZ1uvqtiSeDQPUo8jO-Q&pullRequest=4170

Check warning on line 98 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelTracePropagation.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqSSeDQPUo8jO99&open=AZ1uvqqSSeDQPUo8jO99&pullRequest=4170

Check warning on line 418 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO9w&open=AZ1uvqqISeDQPUo8jO9w&pullRequest=4170

Check failure on line 586 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO95&open=AZ1uvqqISeDQPUo8jO95&pullRequest=4170

Check warning on line 502 in java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITOtelGoldenMetrics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [java_showcase_integration_tests] SonarCloud Code Analysis

Rename this generic name to match the regular expression '^[A-Z][0-9]?$'. 

See more on https://sonarcloud.io/project/issues?id=googleapis_gapic-generator-java_integration_tests&issues=AZ1uvqqISeDQPUo8jO9z&open=AZ1uvqqISeDQPUo8jO9z&pullRequest=4170
View more details on SonarQubeCloud