Merge pull request #86 from path2finding/feature/reducer-action-tests

Adds more remainder+action tests

Author: Shawn Toubeau

src/actions/mazeActions/mazeActions.spec.ts

@@ -4,12 +4,19 @@ import {
   handleChangeEnd,
   makeWall,
   makeEmpty,
+  progressBFS,
+  progressDFS,
+  progressAstar,
   CHANGE_START,
   CHANGE_END,
   MAKE_WALL,
   MAKE_EMPTY,
+  PROGRESS_BFS,
+  PROGRESS_DFS,
+  PROGRESS_ASTAR,
 } from "./mazeActions";
 import { initialState } from "../../models/maze/initialState";
+import { Coord } from "../../models/maze";
 
 describe("Maze Action Tests", () => {
   const mockStore = configureStore();
@@ -79,9 +86,74 @@ describe("Maze Action Tests", () => {
     });
   });
 
-  // TODO: progressBFS
+  describe("Progress BFS", () => {
+    it("Should call PROGRESS_BFS action", () => {
+      const queue: Coord[] = [];
+      const coord = { x: 0, y: 0 };
+      const neighbors = [
+        { x: 1, y: 0 },
+        { x: 0, y: 1 },
+      ];
+
+      const expectedActions = [
+        {
+          type: PROGRESS_BFS,
+          payload: {
+            queue: queue,
+            coord: coord,
+            neighbors: neighbors,
+          },
+        },
+      ];
+      reduxStore.dispatch(progressBFS(queue, coord, neighbors));
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
+
+  describe("Progress DFS", () => {
+    it("Should call PROGRESS_DFS action", () => {
+      const stack: Coord[] = [];
+      const coord = { x: 0, y: 0 };
+      const neighbors = [
+        { x: 1, y: 0 },
+        { x: 0, y: 1 },
+      ];
+
+      const expectedActions = [
+        {
+          type: PROGRESS_DFS,
+          payload: {
+            stack: stack,
+            coord: coord,
+            neighbors: neighbors,
+          },
+        },
+      ];
+      reduxStore.dispatch(progressDFS(stack, coord, neighbors));
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
 
-  // TODO: progressDFS
+  describe("Progress A*", () => {
+    it("Should call PROGRESS_ASTAR action", () => {
+      const openSet: Coord[] = [];
+      const closedSet: Coord[] = [];
+      const newMazeInfo = initialState.mazeInfo;
+      const end = { x: 1, y: 1 };
 
-  // TODO: progressASTAR
+      const expectedActions = [
+        {
+          type: PROGRESS_ASTAR,
+          payload: {
+            openSet: openSet,
+            closedSet: closedSet,
+            newMazeInfo: newMazeInfo,
+            end: end,
+          },
+        },
+      ];
+      reduxStore.dispatch(progressAstar(openSet, closedSet, newMazeInfo, end));
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
 });

src/actions/menuActions/menuActions.spec.ts

@@ -16,6 +16,12 @@ import {
   handleChangeAlgo,
   loadMaze,
   LOAD_MAZE,
+  CHANGE_SPEED,
+  handleChangeSpeed,
+  UPDATE_GRID_SIZE,
+  handleChangeGridSize,
+  RANDOMIZE_WALLS,
+  randomizeWalls,
 } from "./menuActions";
 import { ButtonProps, DropdownProps } from "semantic-ui-react";
 import { initialState } from "../../models/menu/initialState";
@@ -30,7 +36,7 @@ describe("Navbar Action Tests", () => {
   });
 
   describe("Start Visualization Test", () => {
-    it("Should dispatch correct action", () => {
+    it("Should call START_VISUALIZATION action", () => {
       const expectedActions = [
         {
           type: START_VISUALIZATION,
@@ -48,7 +54,7 @@ describe("Navbar Action Tests", () => {
   });
 
   describe("Pause Visualization Test", () => {
-    it("Should modify redux store", () => {
+    it("Should call PAUSE_VISUALIZATION action", () => {
       const expectedActions = [
         {
           type: PAUSE_VISUALIZATION,
@@ -66,7 +72,7 @@ describe("Navbar Action Tests", () => {
   });
 
   describe("Stop Visualization Test", () => {
-    it("Should dispatch correct action", () => {
+    it("Should call STOP_VISUALIZATION action", () => {
       const expectedActions = [
         {
           type: STOP_VISUALIZATION,
@@ -84,10 +90,26 @@ describe("Navbar Action Tests", () => {
     });
   });
 
-  // TODO: change speed
+  describe("Change Visualization Speed", () => {
+    it("Should call CHANGE_SPEED action", () => {
+      const expectedActions = [
+        {
+          type: CHANGE_SPEED,
+          payload: undefined,
+        },
+      ];
+      reduxStore.dispatch(
+        handleChangeSpeed(
+          { target: {} } as React.MouseEvent<HTMLButtonElement, MouseEvent>,
+          {} as DropdownProps
+        )
+      );
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
 
   describe("Change Algorithm Test", () => {
-    it("Should dispatch correct action", () => {
+    it("Should call CHANGE_ALGO action", () => {
       const expectedActions = [
         {
           type: CHANGE_ALGO,
@@ -104,10 +126,27 @@ describe("Navbar Action Tests", () => {
     });
   });
 
-  // TODO: change grid size
+  describe("Change Grid Size Test", () => {
+    it("Should call UPDATE_GRID_SIZE action", () => {
+      const cols = 20;
+      const rows = 20;
+
+      const expectedActions = [
+        {
+          type: UPDATE_GRID_SIZE,
+          payload: {
+            cols: cols,
+            rows: rows,
+          },
+        },
+      ];
+      reduxStore.dispatch(handleChangeGridSize(cols, rows));
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
 
   describe("Move Start Point Test", () => {
-    it("Should modify redux store", () => {
+    it("Should call TOGGLE_MOVE_START action", () => {
       const expectedActions = [
         {
           type: TOGGLE_MOVE_START,
@@ -125,7 +164,7 @@ describe("Navbar Action Tests", () => {
   });
 
   describe("Move End Point Test", () => {
-    it("Should modify redux store", () => {
+    it("Should call TOGGLE_MOVE_END action", () => {
       const expectedActions = [
         {
           type: TOGGLE_MOVE_END,
@@ -142,10 +181,21 @@ describe("Navbar Action Tests", () => {
     });
   });
 
-  // TODO: randomize walls
+  describe("Randomize Walls Test", () => {
+    it("Should call RANDOMIZE_WALLS action", () => {
+      const expectedActions = [
+        {
+          type: RANDOMIZE_WALLS,
+          payload: null,
+        },
+      ];
+      reduxStore.dispatch(randomizeWalls());
+      expect(reduxStore.getActions()).toEqual(expectedActions);
+    });
+  });
 
   describe("Clear Grid Test", () => {
-    it("Should dispatch correct action", () => {
+    it("Should call CLEAR_GRID action", () => {
       const expectedActions = [
         {
           type: CLEAR_GRID,
@@ -162,8 +212,6 @@ describe("Navbar Action Tests", () => {
     });
   });
 
-  // TODO: save maze
-
   describe("Load Maze Test", () => {
     it("Should call LOAD_MAZE action", () => {
       const maze = generateMaze(5, 5, false);

src/components/Maze/algorithms/A*.ts

@@ -8,7 +8,7 @@ import {
   getCoord,
 } from "./";
 
-const heuristic = (a: Coord, b: Coord): number => {
+export const heuristic = (a: Coord, b: Coord): number => {
   // Manhattan distance formula
   return Math.abs(a.x - b.x) + Math.abs(a.y - b.y);
 

src/components/Menu/index.tsx

@@ -282,7 +282,6 @@ class MenuBar extends React.Component<Props, State> {
               />
               <span>Stop</span>
             </Button>
-            {/* TODO: fix */}
             {isPlaying && startTime && !endTime ? (
               <Menu.Item>
                 <Moment date={startTime} durationFromNow></Moment>

src/components/Menu/menu.spec.tsx

@@ -52,7 +52,6 @@ describe("<Menu />", () => {
         <Menu {...props} />
       </Provider>
     );
-    // console.log('wrapper is', wrapper.debug());
     expect(wrapper).toBeDefined();
   });
 
@@ -114,21 +113,4 @@ describe("<Menu />", () => {
     wrapper.find(SemanticButton).at(3).simulate("click");
     expect(props.onClear).toHaveBeenCalled();
   });
-
-  // TODO: This fails because it's not hooked up to a reducer
-  // it("Save Maze button calls action", () => {
-  //   wrapper
-  //     .find(SemanticButton)
-  //     .at(5)
-  //     .simulate("click");
-  //   expect(props.saveMaze).toHaveBeenCalled();
-  // });
-
-  // it("Change dropdown selection", () => {
-  //   wrapper
-  //     .find(SemanticDropdown)
-  //     .simulate("click")
-  //     .simulate("keypress", { key: "Tab" });
-  //   expect(wrapper.find(SemanticDropdown).selectedAlgo).toBeDefined();
-  // });
 });