<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 17, 2015 at 2:40 PM, Markus Metz <span dir="ltr"><<a href="mailto:markus.metz.giswork@gmail.com" target="_blank">markus.metz.giswork@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Fri, Jan 16, 2015 at 11:25 PM, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>> wrote:<br>
> On Fri, Jan 16, 2015 at 10:01 AM, Markus Metz<br>
> <<a href="mailto:markus.metz.giswork@gmail.com">markus.metz.giswork@gmail.com</a>> wrote:<br>
>><br>
>> The fixes are all thoroughly tested (I guess I have never<br>
>> before tested vector topology so thoroughly...).<br>
><br>
><br>
> Hi Markus,<br>
><br>
> will you be able to turn your tests into testsuite scripts? It is additional<br>
> work but it gives us possibility to ensure that nobody will break what you<br>
> did and it should save it your time in the long run.<br>
<br>
</span>I modified v.generalize in my local copy to become a topology<br>
debugging tool. The debugging tools could be activated with an<br>
environment variable which would need to be set by the test suite.<br>
<br></blockquote><div>Setting environmental variable should be easy in the test suite. I'm not sure about the v.generalize modification. Topology debugging tool sound's generally useful. Perhaps it could be part of v.generalize interface or a standalone module (build with v.generalize in the same way as r.colors and r3.colors are) but for tests it really doesn't matter.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If I turn the tests into a test suite script, I will use a vector from<br>
the the full version of the North Carolina sample dataset. Is this ok?<br></blockquote><div><br></div><div>This is ok. MarkusN says we should use the new dataset but I think it is not yet stable.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
><br>
> Let me know if you miss something in testing framework which would help you<br>
> to write the tests.<br>
<br>
</span>I would like to provide a command and the test suite must check the<br>
return status. If someone else could then turn this into a testsuite<br>
script, that would be great!<br>
<br></blockquote><div>Any .sh or .py script you put to testsuite directory anywhere in GRASS source code will be executed as test, so in your case, it should be enough just to put the command to .sh file together with the appropriate export command for the environmental variable. I'm afraid this feature is not really documented (except for emails) because I did it in last minute and it is not the best practice. Anyway, posting command is fine if the only thing needed is to check return status.<br><br></div><div>Vaclav<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Markus M<br>
</blockquote></div><br></div></div>