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 _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
