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

Akihiro Suda commented on ZOOKEEPER-2266:
-----------------------------------------

[~phunt]

Thank you for the comment.
I update the patch (based on JaCoCo v0.7.6): ZOOKEEPER-2266-v3.patch.

The new patch doesn't contain the license file, but unfortunately I could not 
find how to ensure the lib is only pulled when a jacoco target is executed, 
without complicating the xml.

The lib is small, so can we pull the lib for all the test targets?

{panel}
$ du -hs
252K    build/test/lib/org.jacoco.agent-0.7.6.201602180812.jar
40K     build/test/lib/org.jacoco.ant-0.7.6.201602180812.jar
136K    build/test/lib/org.jacoco.core-0.7.6.201602180812.jar
140K    build/test/lib/org.jacoco.report-0.7.6.201602180812.jar
372K    build/test/lib/asm-debug-all-5.0.4.jar
{panel}

I will try again if needed.


> Integrate JaCoCo Coverage Library
> ---------------------------------
>
>                 Key: ZOOKEEPER-2266
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2266
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: tests
>            Reporter: Akihiro Suda
>            Assignee: Akihiro Suda
>            Priority: Minor
>         Attachments: ZOOKEEPER-2266-v1.patch, ZOOKEEPER-2266-v2.patch, 
> jacoco-report-example.zip
>
>
> I would like to propose integration of 
> [JaCoCo|http://www.eclemma.org/jacoco/] coverage library with ZooKeeper.
> h4. Purposes
>  - To find poorly covered methods, and improve JUnit testcases to cover them
>  - To estimate causes of flaky testcases (e.g. ZOOKEEPER-2080, 
> ZOOKEEPER-2252, ZOOKEEPER-1868) by comparing reports from succeeded 
> experiments and failed ones (I'm recently interested in how we can 
> systematically realize this.)
> h4. Advantages of JaCoCo
>  - Support recent JDKs (including JDK 8)
>  - Low overhead
>  - Released under EPL
>  -- Note: cobertura has been removed from the code base because it is 
> released under GPL ( ZOOKEEPER-75, 
> http://www.apache.org/legal/resolved.html#category-x )
> h4. Usage
> {panel}
> $ ant test #(plus optionally, -Dtestcase=.. -Dtest.method=..)
> $ ant jacoco-report
> $ x-www-browser build/test/jacoco/reports/index.html &
> {panel}
> Example: jacoco-report-example.zip
> h4. Possible Future Work
> - Integrate to Jenkins buildbot so that we can check coverage after each of 
> builds



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to