Updated tests to fix build Closed piotrwitek#95 (piotrwitek#133)

Author: piotrwitek

README.md

@@ -1135,41 +1135,45 @@ export default combineReducers<TodosState, TodosAction>({
 ### Testing reducer
 
 ```tsx
-import { todosReducer as reducer, todosActions as actions } from './';
+import {
+  todosReducer as reducer,
+  todosActions as actions,
+  TodosState,
+} from './';
 
 /**
  * FIXTURES
  */
-const activeTodo = { id: '1', completed: false, title: 'active todo' };
-const completedTodo = { id: '2', completed: true, title: 'completed todo' };
-
-const initialState = reducer(undefined, {} as any);
+const getInitialState = (initial?: Partial<TodosState>) =>
+  reducer(initial as TodosState, {} as any);
 
 /**
  * STORIES
  */
 describe('Todos Stories', () => {
   describe('initial state', () => {
     it('should match a snapshot', () => {
+      const initialState = getInitialState();
       expect(initialState).toMatchSnapshot();
     });
   });
 
   describe('adding todos', () => {
     it('should add a new todo as the first element', () => {
-      const action = actions.add('new todo');
-      const state = reducer(initialState, action);
+      const initialState = getInitialState();
+      expect(initialState.todos).toHaveLength(0);
+      const state = reducer(initialState, actions.add('new todo'));
       expect(state.todos).toHaveLength(1);
-      expect(state.todos[0].id).toEqual(action.payload.id);
+      expect(state.todos[0].title).toEqual('new todo');
     });
   });
 
   describe('toggling completion state', () => {
     it('should mark active todo as complete', () => {
-      const action = actions.toggle(activeTodo.id);
-      const state0 = { ...initialState, todos: [activeTodo] };
-      expect(state0.todos[0].completed).toBeFalsy();
-      const state1 = reducer(state0, action);
+      const activeTodo = { id: '1', completed: false, title: 'active todo' };
+      const initialState = getInitialState({ todos: [activeTodo] });
+      expect(initialState.todos[0].completed).toBeFalsy();
+      const state1 = reducer(initialState, actions.toggle(activeTodo.id));
       expect(state1.todos[0].completed).toBeTruthy();
     });
   });

playground/src/features/todos-typesafe/__snapshots__/reducer.spec.ts.snap

@@ -1,38 +1,42 @@
-import { todosReducer as reducer, todosActions as actions } from './';
+import {
+  todosReducer as reducer,
+  todosActions as actions,
+  TodosState,
+} from './';
 
 /**
  * FIXTURES
  */
-const activeTodo = { id: '1', completed: false, title: 'active todo' };
-const completedTodo = { id: '2', completed: true, title: 'completed todo' };
-
-const initialState = reducer(undefined, {} as any);
+const getInitialState = (initial?: Partial<TodosState>) =>
+  reducer(initial as TodosState, {} as any);
 
 /**
  * STORIES
  */
 describe('Todos Stories', () => {
   describe('initial state', () => {
     it('should match a snapshot', () => {
+      const initialState = getInitialState();
       expect(initialState).toMatchSnapshot();
     });
   });
 
   describe('adding todos', () => {
     it('should add a new todo as the first element', () => {
-      const action = actions.add('new todo');
-      const state = reducer(initialState, action);
+      const initialState = getInitialState();
+      expect(initialState.todos).toHaveLength(0);
+      const state = reducer(initialState, actions.add('new todo'));
       expect(state.todos).toHaveLength(1);
-      expect(state.todos[0].id).toEqual(action.payload.id);
+      expect(state.todos[0].title).toEqual('new todo');
     });
   });
 
   describe('toggling completion state', () => {
     it('should mark active todo as complete', () => {
-      const action = actions.toggle(activeTodo.id);
-      const state0 = { ...initialState, todos: [activeTodo] };
-      expect(state0.todos[0].completed).toBeFalsy();
-      const state1 = reducer(state0, action);
+      const activeTodo = { id: '1', completed: false, title: 'active todo' };
+      const initialState = getInitialState({ todos: [activeTodo] });
+      expect(initialState.todos[0].completed).toBeFalsy();
+      const state1 = reducer(initialState, actions.toggle(activeTodo.id));
       expect(state1.todos[0].completed).toBeTruthy();
     });
   });