zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ZOOKEEPER-2955) Enable Clover code coverage report
Date Mon, 22 Jan 2018 20:45:00 GMT

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

ASF GitHub Bot commented on ZOOKEEPER-2955:
-------------------------------------------

Github user phunt commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/443#discussion_r163062466
  
    --- Diff: build.xml ---
    @@ -132,21 +169,16 @@ xmlns:cs="antlib:com.puppycrawl.tools.checkstyle.ant">
         <property name="ant_task_repo_url"
             value="${mvnrepo}${tsk.org}${ant-task.version}/maven-ant-tasks-${ant-task.version}.jar"/>
         <property name="ant_task.jar" location="${ivy.lib}/maven-ant-tasks-${ant-task.version}.jar"/>
    -    
    -    <available property="clover.present"
    -               classname="com.cenqua.clover.CloverInstr"
    -               classpath="${clover.home}/lib/clover.jar"/>
     
    -    <available file="${c.src.dir}/Makefile" property="Makefile.present"/>
    -
    -    <!-- check if clover reports should be generated -->
    -    <condition property="clover.enabled">
    -      <and>
    -        <isset property="run.clover"/>
    -        <isset property="clover.present"/>
    -      </and>
    -    </condition>
    +    <!-- clover property set -->
    +    <property name="clover.home" location="${test.java.build.dir}"/>
    --- End diff --
    
    Similar as comment above - can't we contain this all in it's own subdir?


> Enable Clover code coverage report
> ----------------------------------
>
>                 Key: ZOOKEEPER-2955
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2955
>             Project: ZooKeeper
>          Issue Type: Test
>          Components: tests
>    Affects Versions: 3.5.3, 3.4.11
>            Reporter: Mark Fenes
>            Assignee: Mark Fenes
>            Priority: Major
>
> We have limited code coverage support in ZK. Clover for Java was running in the past
but was turned off. 
> Enable Clover code coverage report to make us more confident on the quality, stability
and compatibility of future ZK releases.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message