[ 
https://issues.apache.org/jira/browse/RANGER-3539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17464497#comment-17464497
 ] 

Pradeep Agrawal commented on RANGER-3539:
-----------------------------------------

1) Yes, jacoco need to be installed but its just to view the details in report 
format. This is optional and for those who want to use it. The link i had given 
has the steps to install jacoco. it will not take more than 5 minutes. Other 
alternate approach for developers is to use *EclEmma* eclipse plugin to get the 
coverage of each java file. EclEmma tool also uses jacoco libraries internally.

2) If you refer the given links for installation, it seems jacoco installation 
could be specific to every one's environment so not sure it will be a good idea 
to plug it with build process, causing a ranger build process to be a blocker 
if its does not fit to any environment. Proposed plugin does not force build 
process to have jacoco installed. Previous commands (mvn clean install) will 
continue working without referring to jacoco plugin. As of now i don't want to 
make it mandatory unless we have some decent level of coverage. 

3) As i mentioned above no need to make it part of build process and its a 
standard mvn command for jacoco plugin and easily available on the net, so no 
need to remember.

4) Yes, But currently overall coverage is too low, we can not force anything as 
of now. Yes, it can be addressed separately but people need to contribute for 
missing test cases to increase the coverage to a decent level.

5) Use EclEmma eclipse plugin for eclipse IDE, we can find same for other 
IDE's.  Refer [https://www.eclemma.org/]

 

> Add jacoco-maven-plugin for code coverage
> -----------------------------------------
>
>                 Key: RANGER-3539
>                 URL: https://issues.apache.org/jira/browse/RANGER-3539
>             Project: Ranger
>          Issue Type: Improvement
>          Components: Ranger
>            Reporter: Pradeep Agrawal
>            Assignee: Pradeep Agrawal
>            Priority: Major
>             Fix For: 3.0.0
>
>         Attachments: 
> 0001-RANGER-3539-Add-jacoco-maven-plugin-for-code-coverag.patch
>
>
> References :
>  # [https://www.baeldung.com/jacoco]
>  # [https://www.baeldung.com/sonarqube-jacoco-code-coverage]
>  # [https://mkyong.com/maven/maven-jacoco-code-coverage-example/]
>  # 
> [https://medium.com/backend-habit/generate-codecoverage-report-with-jacoco-and-sonarqube-ed15c4045885]
>  



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to