Skip to content

DNS Provider URL Validation - #13821

Open
DaanHoogland wants to merge 14 commits into
mainfrom
dnsProviderUrlValidate
Open

DNS Provider URL Validation#13821
DaanHoogland wants to merge 14 commits into
mainfrom
dnsProviderUrlValidate

server: optimize DnsProviderManagerImplTest

22ff345
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 24, 2026 in 4m 5s

Quality Gate passed

Issues
10 New issues
0 Accepted issues

Measures
0 Security Hotspots
69.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 184 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "trim". 

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

Check failure on line 168 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "isBlank". 

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

Check warning on line 979 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

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

Check failure on line 278 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "trim". 

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

Check warning on line 961 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

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

Check warning on line 960 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

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

Check warning on line 922 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check failure on line 277 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "isNotBlank". 

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

Check warning on line 635 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.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=AaA0a3kSRSd_MmyHAoor&open=AaA0a3kSRSd_MmyHAoor&pullRequest=13821

Check warning on line 949 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ_b59q_GDdNQ584wbmu&open=AZ_b59q_GDdNQ584wbmu&pullRequest=13821
View more details on SonarQubeCloud