[GRASS-dev] [GRASS GIS] #3822: Tests for r.basins.fill are broken

GRASS GIS trac at osgeo.org
Fri Apr 19 05:54:30 PDT 2019


#3822: Tests for r.basins.fill are broken
-------------------------+-------------------------
 Reporter:  pmav99       |      Owner:  grass-dev@…
     Type:  defect       |     Status:  new
 Priority:  normal       |  Milestone:
Component:  Tests        |    Version:  svn-trunk
 Keywords:               |        CPU:  Unspecified
 Platform:  Unspecified  |
-------------------------+-------------------------
 These tests are broken both for python 2 and python 3:
 - {{{raster/r.basins.fill/testsuite/testrbf.py}}}
 -
 http://fatra.cnr.ncsu.edu/grassgistests/reports_for_date-2019-03-26-07-00/report_for_nc_spm_full_v2alpha_nc/raster/r.basins.fill/testrbf/index.html

 Moreover, the testsuite [https://github.com/GRASS-GIS/grass-
 ci/blob/8161cae6d4df311d6560e305644673834c8f7f01/raster/r.basins.fill/testsuite/testrbf.py#L24-L25
 overwrites the "elevation" and "geology"] maps thus ruining the test
 dataset (at least on python 3):

 before running the tests:
 {{{
 ./exec_grass.sh r.univar elevation
 Starting GRASS GIS...
 Cleaning up temporary files...
 Executing <r.univar elevation> ...
  100%
 total null and non-null cells: 29535
 total null cells: 0

 Of the non-null cells:
 ----------------------
 n: 29535
 minimum: 89.6269
 maximum: 134.301
 range: 44.6739
 mean: 116.855
 mean of absolute values: 116.855
 standard deviation: 9.32828
 variance: 87.0169
 variation coefficient: 7.98278 %
 sum: 3451315.7405014
 Execution of <r.univar elevation> finished.
 Cleaning up default sqlite database ...
 Cleaning up temporary files...
 }}}
 after:
 {{{
 ./exec_grass.sh r.univar elevation
 Starting GRASS GIS...
 Cleaning up temporary files...
 Executing <r.univar elevation> ...
  100%
 total null and non-null cells: 29535
 total null cells: 29535

 Of the non-null cells:
 ----------------------
 n: 0
 minimum: -nan
 maximum: -nan
 range: -nan
 mean: -nan
 mean of absolute values: -nan
 standard deviation: -nan
 variance: -nan
 variation coefficient: -nan %
 sum: -nan
 Execution of <r.univar elevation> finished.
 Cleaning up default sqlite database ...
 Cleaning up temporary files...
 }}}

 Until someone finds the time to fix them, please add the following
 decorator on the Test class:
 {{{
 import unittest

 @unittest.skip("See #3822")
 class TestRasterbasin(TestCase):
     # ...
 }}}

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/3822>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list