For ITK, we did have do have a directory similar to what David suggests. We called put it in<br>NonUnit/IntegratedTest module<br>These tests pretty much depend on everything.<br><br>Bill<br><br><div class="gmail_quote">On Mon, Mar 4, 2013 at 11:46 AM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yeah, I suspect that a few of the tests aren't in the right module<br>
yet, but from skimming through the list, it looks like most of them<br>
are already in the right location.<br>
<br>
I'm not sure why circular dependencies would be a problem, since the<br>
TEST_DEPENDS are listed separately from the main dependencies?<br>
Unless the tests always have to be built at the same time as the module<br>
itself, but I can't imagine that being the case since the tests are in<br>
a separate directory.<br>
<br>
At the very worst, we could make a module called Testing/Legacy and<br>
use it as a catch-all for tests that have complex dependency issues.<br>
<span class="HOEnZb"><font color="#888888"><br>
 - David<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Mon, Mar 4, 2013 at 6:49 AM, Nikhil Shetty <<a href="mailto:nikhil.shetty@kitware.com">nikhil.shetty@kitware.com</a>> wrote:<br>
> Here is my working list which I used to categorize all the tests. It gives a<br>
> little more information about the include files the test depends on which<br>
> were not modularized at that time.<br>
><br>
><br>
> <a href="https://docs.google.com/a/kitware.com/spreadsheet/ccc?key=0AkuX7fTJrmgydFdRVTUtckJCVjl1amtzNGtUUUhhLUE#gid=0" target="_blank">https://docs.google.com/a/kitware.com/spreadsheet/ccc?key=0AkuX7fTJrmgydFdRVTUtckJCVjl1amtzNGtUUUhhLUE#gid=0</a><br>

><br>
> So it just may be that many test are not in simply because at the time some<br>
> of the files they were dependent on were not modularized.<br>
><br>
> From what I remember some others are disabled because they were creating a<br>
> circular dependencies among modules.<br>
><br>
><br>
><br>
> On Mon, Mar 4, 2013 at 8:32 AM, Nikhil Shetty <<a href="mailto:nikhil.shetty@kitware.com">nikhil.shetty@kitware.com</a>><br>
> wrote:<br>
>><br>
>> I wrote one in Common Lisp. I used it to categorize tests. Basically it<br>
>> first makes a map of all the files to modules. Then for each test finds the<br>
>> headers and candidate modules it can belong to. I can send it to you if you<br>
>> like.<br>
>><br>
>> Here is the output.<br>
>><br>
>><br>
>> <a href="https://docs.google.com/a/kitware.com/spreadsheet/ccc?key=0AkuX7fTJrmgydE5LaXZvbXdjT2Q5bDRyUUVFaWU0VXc#gid=0" target="_blank">https://docs.google.com/a/kitware.com/spreadsheet/ccc?key=0AkuX7fTJrmgydE5LaXZvbXdjT2Q5bDRyUUVFaWU0VXc#gid=0</a><br>

>><br>
>> On Mon, Mar 4, 2013 at 1:32 AM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>> wrote:<br>
>>><br>
>>> Is there a script that can automatically list what modules a<br>
>>> particular .cxx file depends on?<br>
>>><br>
>>> On Sat, Mar 2, 2013 at 4:49 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>> wrote:<br>
>>> > I suspect that many of these were left out during modularization.<br>
>>> > Perhaps<br>
>>> > because they caused dependencies on other modules.<br>
>>> ><br>
>>> ><br>
>>> > On Saturday, March 2, 2013, David Gobbi wrote:<br>
>>> >><br>
>>> >> I just sent a list of files to vtk-devel that shows what is<br>
>>> >> newly-disabled in master as compared to release.<br>
>>> >><br>
>>> >> And I've attached another file that includes the full path<br>
>>> >> to each disabled test.<br>
>>> >><br>
>>> >>  - David<br>
>>> >><br>
>>> >> On Sat, Mar 2, 2013 at 4:10 PM, Berk Geveci <<a href="mailto:berk.geveci@kitware.com">berk.geveci@kitware.com</a>><br>
>>> >> wrote:<br>
>>> >> > Can you generate this as a list of files? Then one or more of us has<br>
>>> >> > to go through these and either remove them or make them work again.<br>
>>> >> ><br>
>>> >> > -berk<br>
>>> >> ><br>
>>> >> > On Sat, Mar 2, 2013 at 5:21 PM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>><br>
>>> >> > wrote:<br>
>>> >> >> There are also a number of tests that aren't being run.  I wrote a<br>
>>> >> >> python script (attached) that looks through all the Testing<br>
>>> >> >> directories and identifies test programs that don't appear in a<br>
>>> >> >> CMakeLists.txt file.  It reports:<br>
>>> >> >><br>
>>> >> >>  166 out of 1770 tests are not enabled<br>
>>> >> >>  44 are commented out, the rest are unlisted in cmake<br>
>>> >> >><br>
>>> >> >> Compare this to VTK 5.10:<br>
>>> >> >><br>
>>> >> >>  54 out of 1213 tests are not enabled<br>
>>> >> >>  21 are commented out, the rest are unlisted in cmake<br>
>>> >> >><br>
>>> >> >> It's probably more informative to list the tests by language:<br>
>>> >> >><br>
>>> >> >> Cxx: 132 not enabled (39 in cmake but commented out)<br>
>>> >> >> Tcl: 19 not enabled (2 in cmake but commented out)<br>
>>> >> >> Python: 15 not enabled (3 in cmake but commented out)<br>
>>> >> >><br>
>>> >> >> Recently Bill and others have done an awesome job of reactivating<br>
>>> >> >> tcl<br>
>>> >> >> and python tests, but a large number of C++ tests are still<br>
>>> >> >> missing/disabled.<br>
>>> >> >><br>
>>> >> >>  - David<br>
>>> >> >><br>
>>> >> >><br>
>>> >> >> On Sat, Mar 2, 2013 at 1:46 PM, Berk Geveci<br>
>>> >> >> <<a href="mailto:berk.geveci@kitware.com">berk.geveci@kitware.com</a>><br>
>>> >> >> wrote:<br>
>>> >> >>> Yup. One other thing that we would like to try in the near future<br>
>>> >> >>> is<br>
>>> >> >>> to look at combined VTK + ParaView coverage of VTK. ParaView tests<br>
>>> >> >>> some of VTK more thoroughly and that should increase coverage<br>
>>> >> >>> further.<br>
>>> >> >>> I am hoping for 70%. Beyond that, we will have to write more<br>
>>> >> >>> tests.<br>
>>> >> >>><br>
>>> >> >>> -berk<br>
>>> >> >>><br>
>>> >> >>> On Sat, Mar 2, 2013 at 10:08 AM, Bill Lorensen<br>
>>> >> >>> <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
>>> >> >>>> Berk,<br>
>>> >> >>>><br>
>>> >> >>>> Looks like tcl and python were added to ubuntu coverage. Now it<br>
>>> >> >>>> is<br>
>>> >> >>>> showing<br>
>>> >> >>>> 66%.<br>
>>> >> >>>><br>
>>> >> >>>> Bill<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > --<br>
>>> > Unpaid intern in BillsBasement at noware dot com<br>
>>> ><br>
>>> _______________________________________________<br>
>>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>><br>
>>> Visit other Kitware open-source projects at<br>
>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>><br>
>>> Follow this link to subscribe/unsubscribe:<br>
>>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Regards<br>
>> Nikhil<br>
><br>
><br>
><br>
><br>
> --<br>
> Regards<br>
> Nikhil<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br>