widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #04265
Re: ship behaviour and regression tests
It was done - had to be, and now it waits for merge approval...
2015-08-04 7:39 GMT+02:00 Holger Rapp <HolgerRapp@xxxxxxx>:
>
> > On 30.07.2015, at 12:26, Tibor Bamhor <tiborb95@xxxxxxxxx> wrote:
> >
> > When reworking a ship behaviour a bit I found small problem when testing
> > test_cancel_started_expedition_underway_two_ships.lua.
> >
> > Two ships and a port requesting a ship. The old implementation sends to
> the port only first ship in m_ships, that is by chance the one farer from
> port.
> >
> > My implementation sends both ships there and of course the wrong ship is
> first there (the one that is closer).
> >
> > I will code it the way that regression test is satisfied, but this does
> not make good sense. Why insist on first ship in m_ships when any ship
> would do the job. Or rather why dont expect the closer one to come to port
> - it is more reasonable….
>
> I think it would be fine to change the test to accept just any ship.
>
> >
> > Please think about it
> >
> > Tibor
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~widelands-dev
> > Post to : widelands-dev@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~widelands-dev
> > More help : https://help.launchpad.net/ListHelp
>
>
References