Edited tests for ActionContainer to test functionality of its direct child, RouteDescription, instead of its grandchild Action. Tests still need written for RouteDescription.

Co-authored-by: Chris LeBrett <[email protected]>
Co-authored-by: Robby Tipton <[email protected]>
Co-authored-by: Joseph Park <[email protected]>
Co-authored-by: David Kim <[email protected]>
Co-authored-by: Kevin HoEun Lee <[email protected]>

Author: 5 people

package.json

@@ -112,10 +112,9 @@
     "jest": "^26.1.0",
     "jest-cli": "^26.1.0",
     "jest-diff": "^26.1.0",
-    "jest-puppeteer": "^3.4.0",
     "jest-runner-eslint": "^0.7.7",
     "jscharting-react": "^1.2.1",
-    "puppeteer": "^1.9.0",
+    "puppeteer": "^14.3.0",
     "sass": "^1.26.10",
     "sass-loader": "^7.3.1",
     "sinon-chrome": "^3.0.1",

src/app/__tests__/ActionContainer.test.tsx

@@ -8,6 +8,7 @@ import ActionContainer from '../containers/ActionContainer';
 import { useStoreContext } from '../store';
 import { emptySnapshots } from '../actions/actions';
 import Action from '../components/Action';
+import RouteDescription from '../components/RouteDescription';
 
 configure({ adapter: new (Adapter as any)() });
 
@@ -76,7 +77,7 @@ const state = {
                 }],
                 route: {
                   id: 4,
-                  url: 'http://localhost:8080/',
+                  url: 'http://localhost:8080/test/',
                 },
               },
               children: [],
@@ -103,11 +104,12 @@ useStoreContext.mockImplementation(() => [state, dispatch]);
 
 let wrapper;
 
-//actionView={true} must be passed in to <ActionContainer /> in beforeEach() to deal with new
-//conditional rendering in ActionContainer that shows/hides time-travel functionality
+// actionView={true} must be passed in to <ActionContainer /> in beforeEach() to deal with new
+// conditional rendering in ActionContainer that shows/hides time-travel functionality
 
 beforeEach(() => {
   wrapper = shallow(<ActionContainer actionView={true} />);
+  // wrapper2 = shallow(<RouteDescription />);
   useStoreContext.mockClear();
   dispatch.mockClear();
 });
@@ -123,6 +125,6 @@ describe('testing the emptySnapshot button', () => {
   });
 });
 
-test('number of actions should reflect snapshots array', () => {
-  expect(wrapper.find(Action).length).toBe(state.tabs[state.currentTab].snapshots.length);
+test('number of RouteDescription components should reflect number of unique routes', () => {
+  expect(wrapper.find(RouteDescription).length).toBe(2);
 });