Vaclav, Your test script is working great. I think it's ok to overwrite the first method because the second one actually tests the new g.mremove anyway. I deleted the first method and it worked fine.
Huidae On Tue, Jun 3, 2014 at 9:15 AM, Vaclav Petras <[email protected]> wrote: > > > > On Tue, Jun 3, 2014 at 4:08 AM, Martin Landa <[email protected]> > wrote: > >> 2014-06-03 10:04 GMT+02:00 Huidae Cho <[email protected]>: >> > you mean the new g.mremove with the g.mlist interface >> > (http://trac.osgeo.org/grass/attachment/ticket/2228/g-mremove-main.diff >> )? >> >> I had in mind replacing g.remove with g.mremove, but right, >> consolidating interfaces would be also nice... >> >> Agreed. Note also that there are is good reasoning behind the new > interface by Glynn and Huidae [1] besides the fact the old g.mremove was > used for a long time. Also I've tried to test the new interface [2]. > > Vaclav > > [1] http://trac.osgeo.org/grass/ticket/2228#comment:6 > [2] http://trac.osgeo.org/grass/attachment/ticket/2228/test_g_mremove.py > > PS: Now I see that the test contains a bug, the first method is not > executed because I gave the same name to the second one. I should use > pylint next time. > > Martin >> >> -- >> Martin Landa * http://geo.fsv.cvut.cz/gwiki/Landa >> > >
_______________________________________________ grass-dev mailing list [email protected] http://lists.osgeo.org/mailman/listinfo/grass-dev
