Right now, you must call the MoveDelegate twice in each turn because certain things only happen during the "non-combat" version which runs the second time (clearing unit movement, etc.). To add support for 1914, it must be possible to have a single move phase in each turn.
This is the first step towards moving this end-of-turn clean-up to the EndTurnDelegate. The RevisedTest assumes that calling the move delegate's start/end multiple times will clear transport loading/unloading limits. Call the EndTurnDelegate from the test so that when the code is moved there it will continue to work.
This had an issue where the EndTurnDelegate tries to display the report, and the call to RemotePlayer returns null. I'm not entirely sure why since the unit test infrastructure creates a DummyPlayer. Perhaps there's a better way to fix this, but the solution I found was to move the report display to EndTurnDelegate's end()
Log in to post a comment.