← Back to team overview

widelands-dev team mailing list archive

[Bug 916545] Re: About depending on GGZ packages

 

As I said, I wasn't sure how easy it would be. It just seemed like a
nice opportunity to add tests to one of the important components. Like
SirVer mentioned, I imagined something like running a local server and
seeing if that responds in the expected way would be the easist way.
(Mocking [1] might also be appropriate.)

I have to admit I wasn't really sure to what extent there would be to
create an identical replacement or rather a server performing the same
tasks. (If that makes sense?) In the latter case, tests for the old
server would obviously not work for the new one. So that part is
probably not relevant.

All in all though, it is probably more important to get something which
work than to be able to automatically check whether it works. ;) So
while I ideally would appreciate to see a test suite added at the same
time, I'm perfectly happy that someone is capable of adding a
replacement. It can also be argued that as long as for instance the
protocol is still discussed writing tests might be a waste of time as
those parts end up being changed later anyways, so it might make more
sense to add them later, when we have a working implementation. I also
realize that developer time is limited.

[1] http://en.wikipedia.org/wiki/Mock_object

-- 
You received this bug notification because you are a member of Widelands
Developers, which is a bug assignee.
https://bugs.launchpad.net/bugs/916545

Title:
  About depending on GGZ packages

Status in Widelands:
  Confirmed
Status in “widelands” package in Debian:
  New

Bug description:
  I noticed a bug report in Debian which basically suggests to drop the
  GGZ packages because it is unmaintained upstreams. Since Widelands
  depends on ggz, this of course affects us there (not to mention Debian
  derivaties like Ubuntu and Linux Mint).

  I know the ggz dependency has been discussed a bit before, but we
  should probably figure out what the long-term plan is.

To manage notifications about this bug go to:
https://bugs.launchpad.net/widelands/+bug/916545/+subscriptions