Opened 6 years ago

Last modified 5 years ago

#3816 new defect

test_r_slope_aspect.py is not idempotent

Reported by: pmav99 Owned by: grass-dev@…
Priority: normal Milestone: 7.8.3
Component: Tests Version: svn-trunk
Keywords: Cc:
CPU: Unspecified Platform: Unspecified

Description

test_r_slope_aspect.py is not idempotent, i.e. running the test twice passes the first time but fails the second. The problem is that the maps that are getting created are not removed on cleanUp and as a result, subsequent runs are failing.

The tests can be fixed either by passing --overwrite or by cleaning up the created maps. Unless there are specific reasons preventing us from doing so, I would suggest cleaning up.

PS. My guess is that the tests on fatra are always being run on a fresh copy of the test dataset, that's why this has not been noticed until now.

Change History (3)

comment:1 by sbl, 6 years ago

It seems fatra is using multirunner.py and creates and tears down a mapset per testfile. Still, proper cleanup should be implemented. Patches are welcome...

comment:2 by pmav99, 6 years ago

After removing the following maps, the tests pass again:

fractal_aspect
fractal_slope
limits_aspect
limits_slope
limits_percent_aspect
limits_percent_slope
reference_aspect
reference_slope
sa_separately_aspect
sa_separately_slope
sa_together_aspect
sa_together_slope
small_aspect
small_elevation
small_slope

comment:3 by neteler, 5 years ago

Milestone: 7.8.3
Note: See TracTickets for help on using tickets.