update _undo to reflect current ctx (#393)

* update _undo to reflect current ctx

add tests for undo and redo of actions that change phase.

* group undo/redo tests

Author: mgrau

src/core/reducer.js

@@ -310,6 +310,7 @@ export function CreateGameReducer({ game, multiplayer }) {
           action.payload
         );
         state = apiCtx.updateAndDetach(state, true);
+        state._undo[state._undo.length - 1].ctx = state.ctx;
 
         return state;
       }

src/core/reducer.test.js

@@ -289,6 +289,16 @@ test('undo / redo', () => {
   let game = Game({
     moves: {
       move: (G, ctx, arg) => ({ ...G, [arg]: true }),
+      nextPhase: (G, ctx) => {
+        ctx.events.endPhase({ next: 'phase2' });
+      },
+    },
+    flow: {
+      startingPhase: 'phase1',
+      phases: {
+        phase1: {},
+        phase2: {},
+      },
     },
   });
 
@@ -336,6 +346,42 @@ test('undo / redo', () => {
   state = reducer(state, makeMove('move', 'A'));
   expect(state.G).toEqual({ A: true });
 
+  state = reducer(state, makeMove('nextPhase'));
+  expect(state.G).toEqual({ A: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, makeMove('move', 'B'));
+  expect(state.G).toEqual({ A: true, B: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, undo());
+  expect(state.G).toEqual({ A: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, redo());
+  expect(state.G).toEqual({ A: true, B: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, undo());
+  expect(state.G).toEqual({ A: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, undo());
+  expect(state.G).toEqual({ A: true });
+  expect(state.ctx.phase).toEqual('phase1');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, redo());
+  expect(state.G).toEqual({ A: true });
+  expect(state.ctx.phase).toEqual('phase2');
+  expect(state.ctx).toEqual(state._undo[state._undo.length - 1].ctx);
+
+  state = reducer(state, undo());
   state = reducer(state, gameEvent('endTurn'));
   state = reducer(state, undo());
   expect(state.G).toEqual({ A: true });