Team,

While running code coverage analysis with sonar for integration tests, based 
upon the errors thrown, i could see the below issues\notes with CS project.

Issue1:

The coverage tool is complaining about duplicate sources  for below files. 
These are available with same name under folders 
./cloudstack/plugins/network-elements/juniper-srx and as well under 
..../palo-alto.

./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/AddExternalFirewallCmd.java
./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/DeleteExternalFirewallCmd.java
./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/ListExternalFirewallCmd.java

I renamed one while running analysis to a different name.  It proceeded further 
with its analysis once renamed.

Is it intentional to have same name or can be renamed?

Issue2:

The source directory does not correspond to package declaration for  code files 
under
/root/softwares/cscode/cloudstack/services/console-proxy-rdp/rdpconsole/src/main/java/rdpclient/

This error when compared to other files in the similar path has a different 
package convention and usage.
/root/softwares/cscode/cloudstack/services/secondary-storage/src/org/apache/cloudstack/storage/

Changing tool configuration properties worked to over come this, but Is it 
intentional to have a different package structure for rdpclient code base 
against others?



Thanks!
Santhosh

Reply via email to