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