<div>Stick with the Test prefix, and move ones we come across to conform seems like the right way to go then. I had inferred that this was already VTK convention for tests when I began adding them.</div><div><br></div><div>
David is right that writing this stuff down somewhere central helps new and old developers alike, as well as contributors from the wider community.</div><div><br></div><div>Thanks,</div><div><br></div><div>Marcus</div><br>
<div class="gmail_quote">On Sat, Aug 21, 2010 at 4:12 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Of the 549 cxx test files in vtk, 123 do not start with Test, the<br>
other 426 do start with Test.<br>
<div><div></div><div class="h5"><br>
On Sat, Aug 21, 2010 at 3:54 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> I still prefer TestClassName.cxx.<br>
><br>
> On Sat, Aug 21, 2010 at 2:55 PM, David Doria <<a href="mailto:daviddoria%2Bvtk@gmail.com">daviddoria+vtk@gmail.com</a>> wrote:<br>
>>><br>
>>> Can I suggest we agree upon and adopt a convention moving forward? We can<br>
>>> go back and "correct" existing tests as we have time. It seems reasonable to<br>
>>> have a<br>
>>><br>
>>> TestClassName.cxx (without the vtk- prefix) for every concrete class.<br>
>><br>
>> I have followed this convention in the tests I have added, but in many ways<br>
>> the "Test" seems redundant. The big plus is that in my editor of choice (Qt<br>
>> Creator) I am able to type "TestS" and have every test starting with S<br>
>> listed, but to run a test I have to type,<br>
>><br>
>> ./bin/ChartsCxxTests TestScientificPlot -I<br>
>><br>
>> (e.g vtkPoints -> TestPoints.cxx)<br>
>><br>
>> I doubt the one-to-one mapping of test names to classes will ever be the<br>
>> case. Some classes are very amenable to this, others are not. I agree that<br>
>> it would be good to establish a standard, and stick to it. Now would be a<br>
>> great time to get something set, and do it. I have been working with Bill a<br>
>> little to migrate some tests to use the testing interactor machinery,<br>
>> finding issues and seeing what we can do there too.<br>
>><br>
>> Marcus<br>
>><br>
>> (I started a new thread since the old title (about verdict) wouldn't<br>
>> probably interest many people).<br>
>> Hi all,<br>
>> I started a page here:<br>
>> <a href="http://www.vtk.org/Wiki/TestingGuidelines" target="_blank">http://www.vtk.org/Wiki/TestingGuidelines</a><br>
>> (and linked to it<br>
>> from <a href="http://www.vtk.org/Wiki/VTK/Developers_Corner#Developers_Corner" target="_blank">http://www.vtk.org/Wiki/VTK/Developers_Corner#Developers_Corner</a> )<br>
>> Please add any thoughts and ideas about testing and coverage to that page. I<br>
>> think by having a set of more systematic testing guidelines we will be able<br>
>> to keep much more of the code covered. I've also started a list of questions<br>
>> (about deprecated classes, coverage target, etc) that should be addressed.<br>
>> Thanks,<br>
>><br>
>> David<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>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <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>
</div></div></blockquote></div><br>