<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 17, 2014 at 8:05 AM, 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 Mon, Nov 17, 2014 at 1:05 PM, Markus Neteler <<a href="mailto:neteler@osgeo.org">neteler@osgeo.org</a>> wrote:<br>
> Hi Markus,<br>
><br>
> please revert these changes: nc_spm is not used but nc_basic!<br>
<br></span></blockquote><div>No they run on full NS SPM. I want to add basic NC SPM but I need to do some changes in top level run and report system. The PyGRASS tests were written for basic NC, the rest for full NC, the issues is that they are different.<br><br></div><div>The ultimate goal is to use the new sample location but it does not exist yet. We've discussed with Helena and Hamish what Helena and MarkusN started but there are still just the ideas.<br></div><div><br><a href="http://lists.osgeo.org/pipermail/grass-dev/2014-September/070999.html">http://lists.osgeo.org/pipermail/grass-dev/2014-September/070999.html</a><br><a href="http://osgeo-org.1560.x6.nabble.com/Heading-towards-unified-dataset-td5165053.html">http://osgeo-org.1560.x6.nabble.com/Heading-towards-unified-dataset-td5165053.html</a><br></div><div> </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="">
</span>ok, reverted in r62772<br>
<br></blockquote><div>You could have actually leave it there, it needs to be changed or revisited anyway. But revert is fine too.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
According to the manual, tests are based on NC sample data [0], e.g.<br>
<br>
python -m grass.gunittest.main --location nc_spm_grass7 --location-type nc<br>
<br></blockquote><div>Sorry, but the examples in the manual are just examples. It is defined nowhere so far which location is the right one or how often they should run. You can run tests in whichever location you want and in fact I'm running them in empty XY location and Piemonte UTM32 location. The issue here is selecting tests which should run with which data, it needs just a little bit more work to be functional (note: --location-type is currently ignored).<br><br></div><div>See also "GSoC final report":<br><br>tests should specify the location they can run in<br> * some code already exists for this but parsing of LOCATION variable or tag needs to added<br> * the missing piece is determination of which location types we need (e.g., same maps but different projections)<br><br></div><div>Thanks for being involved in this, the ideal is that all developers would run the tests (considering that we don't have common infrastructure to run tests after each commit),<br></div><div>Vaclav<br></div><div><br><a href="http://fatra.cnr.ncsu.edu/grassgistests/summary_report/index.html">http://fatra.cnr.ncsu.edu/grassgistests/summary_report/index.html</a><br><a href="http://grass.osgeo.org/grass71/manuals/libpython/gunittest_testing.html">http://grass.osgeo.org/grass71/manuals/libpython/gunittest_testing.html</a><br><a href="http://grass.osgeo.org/grass71/manuals/libpython/gunittest_running_tests.html">http://grass.osgeo.org/grass71/manuals/libpython/gunittest_running_tests.html</a><br></div><div><a href="http://trac.osgeo.org/grass/wiki/GSoC/2014/TestingFrameworkForGRASS#Finalreport">http://trac.osgeo.org/grass/wiki/GSoC/2014/TestingFrameworkForGRASS#Finalreport</a><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>
<br>
[0] <a href="http://grass.osgeo.org/grass71/manuals/libpython/gunittest_testing.html" target="_blank">http://grass.osgeo.org/grass71/manuals/libpython/gunittest_testing.html</a><br>
<div class=""><div class="h5"><br>
> <a href="http://grass.osgeo.org/download/sample-data/" target="_blank">http://grass.osgeo.org/download/sample-data/</a><br>
> --> smaller subset NC location:<br>
>       --> GRASS 7 data subset tar.gz | zip (nc_basic...)<br>
><br>
> Otherwise the online tests will be completely broken.<br>
><br>
> markusN<br>
><br>
><br>
> On Mon, Nov 17, 2014 at 12:43 PM,  <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>> wrote:<br>
>> Author: mmetz<br>
>> Date: 2014-11-17 03:43:25 -0800 (Mon, 17 Nov 2014)<br>
>> New Revision: 62770<br>
>><br>
>> Modified:<br>
>>    grass/trunk/lib/python/pygrass/raster/testsuite/test_category.py<br>
>>    grass/trunk/lib/python/pygrass/raster/testsuite/test_history.py<br>
>> Log:<br>
>> pygrass: testsuite raster geology does not exist in nc_spm<br>
>><br>
>> Modified: grass/trunk/lib/python/pygrass/raster/testsuite/test_category.py<br>
>> ===================================================================<br>
>> --- grass/trunk/lib/python/pygrass/raster/testsuite/test_category.py    2014-11-17 11:34:40 UTC (rev 62769)<br>
>> +++ grass/trunk/lib/python/pygrass/raster/testsuite/test_category.py    2014-11-17 11:43:25 UTC (rev 62770)<br>
>> @@ -14,7 +14,7 @@<br>
>><br>
>>  class RasterCategoryTestCate(TestCase):<br>
>><br>
>> -    name = 'geology'<br>
>> +    name = 'geology_30m'<br>
>>      catsname = ''<br>
>><br>
>>      def testCategory(self):<br>
>><br>
>> Modified: grass/trunk/lib/python/pygrass/raster/testsuite/test_history.py<br>
>> ===================================================================<br>
>> --- grass/trunk/lib/python/pygrass/raster/testsuite/test_history.py     2014-11-17 11:34:40 UTC (rev 62769)<br>
>> +++ grass/trunk/lib/python/pygrass/raster/testsuite/test_history.py     2014-11-17 11:43:25 UTC (rev 62770)<br>
>> @@ -13,7 +13,7 @@<br>
>><br>
>>  class RasterHistoryTestCate(TestCase):<br>
>><br>
>> -    name = 'geology'<br>
>> +    name = 'geology_30m'<br>
>><br>
>>      def testHistory(self):<br>
>>          r = RasterRow(<a href="http://self.name" target="_blank">self.name</a>)<br>
>><br>
>> _______________________________________________<br>
>> grass-commit mailing list<br>
>> <a href="mailto:grass-commit@lists.osgeo.org">grass-commit@lists.osgeo.org</a><br>
>> <a href="http://lists.osgeo.org/mailman/listinfo/grass-commit" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-commit</a><br>
> _______________________________________________<br>
> grass-dev mailing list<br>
> <a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</div></div></blockquote></div><br></div></div>