Skip to content

npe guard on missing session attribute - #13871

Open
DaanHoogland wants to merge 2 commits into
4.20from
ghi13815-NPEwhileSkippingSSOcheck
Open

npe guard on missing session attribute#13871
DaanHoogland wants to merge 2 commits into
4.20from
ghi13815-NPEwhileSkippingSSOcheck

deal with sonarcube/-cloud warnings

0ab78ce
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 14, 2026 in 9m 32s

Annotations

Check warning on line 277 in server/src/test/java/com/cloud/api/ApiServletTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless "eq(...)" invocation; pass the values directly.

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ__u_NPDoP9WiI4fUg3&open=AZ__u_NPDoP9WiI4fUg3&pullRequest=13871

Check warning on line 346 in server/src/test/java/com/cloud/api/ApiServletTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "mock".

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ_6-d1yyvrdxWit8HOp&open=AZ_6-d1yyvrdxWit8HOp&pullRequest=13871

Check warning on line 268 in server/src/test/java/com/cloud/api/ApiServletTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless "eq(...)" invocation; pass the values directly.

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ__u_NPDoP9WiI4fUg2&open=AZ__u_NPDoP9WiI4fUg2&pullRequest=13871

Check warning on line 264 in server/src/test/java/com/cloud/api/ApiServletTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace these 3 tests with a single Parameterized one.

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ__u_NPDoP9WiI4fUg1&open=AZ__u_NPDoP9WiI4fUg1&pullRequest=13871

Check warning on line 294 in server/src/test/java/com/cloud/api/ApiServletTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless "eq(...)" invocation; pass the values directly.

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ__u_NPDoP9WiI4fUg4&open=AZ__u_NPDoP9WiI4fUg4&pullRequest=13871