← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1281823 into lp:widelands

 

Do the rename here I'd say.

> I dont understand a meaning of this comment. Why should we care that resource_amount!=starting_resource_amount here. In fact they will probably be equal. Depending on a map used for testing.

I would then suggest that you test one more value and also test the absolute value:

self.f.resource_amount=10
assert_equal(self.f.starting_resource_amount, self.f.resource_amount)
assert_equal(10, self.f.starting_resource_amount)

self.f.resource_amount=20
assert_equal(self.f.starting_resource_amount, self.f.resource_amount)
assert_equal(20, self.f.starting_resource_amount)

-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1281823/+merge/242837
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1281823.


References