Fix multiple bugs with evade command: it should spend one unit and no…

…t mark unspent other units

README.md

@@ -40,8 +40,6 @@ image to appear.  The parameters are:
 
 # WIP
 
-    BUG combat with evasion does not "spend" the attacking unit
-
   * use OpenLoop in MCTS: reimplement best commands
   * implement MAP_CENTER and MAP_EAST
 

src/model/commands/EvadeCommand.js

@@ -29,7 +29,8 @@ export class EvadeCommand extends Command {
         } else {
             game.moveUnit(this.toHex, this.fromHex);
         }
-        game.endPhase();
+        game.shiftPhase();
+        game.markUnitSpent(attackingUnit);
         return events;
     }
 

src/model/commands/EvadeCommand.test.js

@@ -39,6 +39,10 @@ describe('evade command', () => {
         expect(game.unitStrength(evadingUnit)).toBe(3);
     });
 
+    test('it spends the attacking unit', () => {
+        expect(game.isSpent(attackingUnit)).toBeTruthy();
+    });
+
 });
 
 test('It can kill the evading unit', () => {