[ http://jira.codehaus.org/browse/MCLOVER-46?page=comments#action_70592 ] 
            
Andrew Williams commented on MCLOVER-46:
----------------------------------------

Sorry - but I would have to disagree with the closure. It seems to me that the 
line

"Total coverage of did not meet target of 1%"

contains a bug whether you agree with the requested change or not.

All code must have a coverage surely? if there are no tests for code that must 
be tested then the coverage is "0%" not "".

I tend to agree here, that if the code requires no coverage then the absense of 
testing seems to imply 100% not 0% and certainly not ""

The bug here is that if there were class in the code the coverage would be 0% 
but with only interfaces it is broken.

> Coverage reports are incorrect for interface only modules. 
> -----------------------------------------------------------
>
>                 Key: MCLOVER-46
>                 URL: http://jira.codehaus.org/browse/MCLOVER-46
>             Project: Maven 2.x Clover Plugin
>          Issue Type: Bug
>    Affects Versions: 2.2
>         Environment: ubuntu dapper drake. Maven 2.0, clover maven plugin 2.2
>            Reporter: Meghan Claire Pike
>         Assigned To: Vincent Massol
>
> Our projects require a coverage of at least 1% in order to force everyone to 
> at least think about testing. Unfortunately for interface only packages, (due 
> to seperation of concerns) clover just goes into a small spasm and dies. It's 
> output is like this: 
> Clover Version 1.3.12, built on February 08 2006
> loaded from: 
> <path...>.m2/repository/com/cenqua/clover/clover/1.3.12/clover-1.3.12.jar
> Academic License registered to EDINA. This license of Clover is provided to 
> support coursework at EDINA only.
> You have 29 day(s) before your Academic License expires.
> Updating database at 'target/clover/clover.db'
> Processing files at 1.4 source level.
> Instrumented 12 source files.
> ...
> [INFO] [clover:instrument {execution: default}]
> [INFO] [clover:check {execution: default}]
> [INFO] Checking for coverage of [1%] for database [target/clover/clover.db]
> WARN: No coverage data found for '<path to my api>/target/clover/clover.db'.
> [ERROR] Total coverage of   did not meet target of 1%
> [INFO] 
> ------------------------------------------------------------------------
> [ERROR] BUILD ERROR
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Build failed to meet Clover coverage targets
> it doesn't even output 0% or anything like that, which seems to me to be a 
> bug. I think clover should maybe understand a little better that testing 
> api's doesn't make sense, and is quite difficult to do. I don't have any 
> testing classes whatsoever in the project, so it could be from that. But 
> clover should have some ability not to enforce coverage on interfaces. (Or 
> understand that they can't be tested except indirectly.)
> Thanks!

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to