Review: Approve

I looked at this again right now and went ahead and merged. 

I thought it was a good idea to split the tests into two separates, but it had 
some code duplication that I decided in the end it is better to keep the test 
together. I just made the tests a bit tighter by directly comparing the 
immovables instead of just != nil.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1380286/+merge/242975
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/bug-1380286.

_______________________________________________
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

Reply via email to