search

FreeCol / freecol

FreeCol: FreeCol is a turn-based strategy game based on the old game Colonization, and similar to Civilization. The objective of the game is to create an independent nation.
GNU General Public License v2.0
585 stars 170 forks source link

Simple move test case enhanced. #60

Closed hjayyang94 closed 4 years ago

hjayyang94 commented 4 years ago

The existing test case, testSimpleMove(), was not actually testing the movement. It merely tested whether or not tile 2 was in the direction.NE of tile1.

Added additional assertion statement to test if automatic movement (a.k.a. simple movement) occurs.

mpope042 commented 4 years ago

I am not sure what you are trying to do here. The patch has an unrelated file and a single boolean check which would be better bundled into its own routine that tests the automatic move functionality rather than tacked on to the simple "does it work at all" tests.