You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2020/09/07 20:20:53 UTC

[GitHub] [zookeeper] nkalmar commented on pull request #1451: ZOOKEEPER-2266: Integrate JaCoCo Coverage Library

nkalmar commented on pull request #1451:
URL: https://github.com/apache/zookeeper/pull/1451#issuecomment-688498585


   We've seen demand for code coverage, and some apache projects are already using jacoco. 
   I found an old jira to integrate jacoco with ant. 
   
   I know this doesn't really tell us how good are testing is (specially that for many tests we spin up a whole zk server), but there is a demand to see this in reports, so.. here it is.
   
   I did several tests (all test, one test, jacoco on/off and see generated sources - should be only in zk-server), a good starting point:
   'mvn clean install -Dtest=WatcherTest -DfailIfNoTests=false -Dcodecoverage'
   
   By default code coverage is off, enable it by adding -Dcodecoverage to mvn args.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org