[ITK Community] [Insight-developers] Raising code coverage of LabelMap filters

[Bradley Lowekamp]

Luis,

In short no reason. The patch should just be taken as a list of 14 tests, which currently are not being run. Arguments need to be determined for them.

If I recall correctly, my first goal was to determine the list of test which were being compiled but not being run. So I commented out all that appears to not be running, and tested. Then as I added tests, I un-commented the test file line. I was stashing and add via a patch, so I clearly made a mistake during the tedious process.

I hope this list is useful.

Brad



On Feb 14, 2014, at 7:32 AM, Luis Ibanez <luis.ibanez at kitware.com> wrote:

> Brad,
> 
> Why are you commenting out the .cxx files in lines 42-45 ?
> 
> http://review.source.kitware.com/#/c/13646/2/Modules/Filtering/LabelMap/test/CMakeLists.txt
> 
> Wouldn't that reduce the code coverage instead of increasing it ?
> 
> 
>     Luis
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/community/attachments/20140214/fc6b0e68/attachment-0002.html>
-------------- next part --------------
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Kitware offers ITK Training Courses, for more information visit:
http://kitware.com/products/protraining.php

Please keep messages on-topic and check the ITK FAQ at:
http://www.itk.org/Wiki/ITK_FAQ

Follow this link to subscribe/unsubscribe:
http://www.itk.org/mailman/listinfo/insight-developers