From 5030eb35b6944c4398a8ae33c894721c242788b4 Mon Sep 17 00:00:00 2001
From: John Mitchell <jlmitch5@ncsu.edu>
Date: Thu, 18 Apr 2019 13:10:17 -0400
Subject: [PATCH] migrate App.jsx to mountwithContext

---
 __tests__/App.test.jsx      | 105 +++++++++++-------------------------
 __tests__/enzymeHelpers.jsx |   8 ++-
 2 files changed, 38 insertions(+), 75 deletions(-)

diff --git a/__tests__/App.test.jsx b/__tests__/App.test.jsx
index 9712833be4..29f0f21fa0 100644
--- a/__tests__/App.test.jsx
+++ b/__tests__/App.test.jsx
@@ -1,50 +1,36 @@
 import React from 'react';
-import { MemoryRouter } from 'react-router-dom';
-import { I18nProvider } from '@lingui/react';
 
-import { mount } from 'enzyme';
+import { mountWithContexts } from './enzymeHelpers';
+
 import { asyncFlush } from '../jest.setup';
 
-import { ConfigProvider } from '../src/contexts/Config';
-import { NetworkProvider } from '../src/contexts/Network';
-
-import App, { _App } from '../src/App';
-
-const networkProviderValue = { api: {}, handleHttpError: () => {} };
+import App from '../src/App';
 
 describe('<App />', () => {
   test('expected content is rendered', () => {
-    const appWrapper = mount(
-      <MemoryRouter>
-        <I18nProvider>
-          <NetworkProvider value={networkProviderValue}>
-            <ConfigProvider>
-              <App
-                routeGroups={[
-                  {
-                    groupTitle: 'Group One',
-                    groupId: 'group_one',
-                    routes: [
-                      { title: 'Foo', path: '/foo' },
-                      { title: 'Bar', path: '/bar' },
-                    ],
-                  },
-                  {
-                    groupTitle: 'Group Two',
-                    groupId: 'group_two',
-                    routes: [
-                      { title: 'Fiz', path: '/fiz' },
-                    ]
-                  }
-                ]}
-                render={({ routeGroups }) => (
-                  routeGroups.map(({ groupId }) => (<div key={groupId} id={groupId} />))
-                )}
-              />
-            </ConfigProvider>
-          </NetworkProvider>
-        </I18nProvider>
-      </MemoryRouter>
+    const appWrapper = mountWithContexts(
+      <App
+        routeGroups={[
+          {
+            groupTitle: 'Group One',
+            groupId: 'group_one',
+            routes: [
+              { title: 'Foo', path: '/foo' },
+              { title: 'Bar', path: '/bar' },
+            ],
+          },
+          {
+            groupTitle: 'Group Two',
+            groupId: 'group_two',
+            routes: [
+              { title: 'Fiz', path: '/fiz' },
+            ]
+          }
+        ]}
+        render={({ routeGroups }) => (
+          routeGroups.map(({ groupId }) => (<div key={groupId} id={groupId} />))
+        )}
+      />
     );
 
     // page components
@@ -69,17 +55,7 @@ describe('<App />', () => {
 
     const config = { ansible_version, version };
 
-    const wrapper = mount(
-      <MemoryRouter>
-        <I18nProvider>
-          <NetworkProvider value={networkProviderValue}>
-            <ConfigProvider value={config}>
-              <App />
-            </ConfigProvider>
-          </NetworkProvider>
-        </I18nProvider>
-      </MemoryRouter>
-    );
+    const wrapper = mountWithContexts(<App />, { context: { config } });
 
     // open about modal
     const aboutDropdown = 'Dropdown QuestionCircleIcon';
@@ -106,19 +82,8 @@ describe('<App />', () => {
   });
 
   test('onNavToggle sets state.isNavOpen to opposite', () => {
-    const appWrapper = mount(
-      <MemoryRouter>
-        <I18nProvider>
-          <NetworkProvider value={networkProviderValue}>
-            <ConfigProvider>
-              <App />
-            </ConfigProvider>
-          </NetworkProvider>
-        </I18nProvider>
-      </MemoryRouter>
-    ).find('App');
+    const appWrapper = mountWithContexts(<App />).find('App');
     const { onNavToggle } = appWrapper.instance();
-
     [true, false, true, false, true].forEach(expected => {
       expect(appWrapper.state().isNavOpen).toBe(expected);
       onNavToggle();
@@ -128,17 +93,9 @@ describe('<App />', () => {
   test('onLogout makes expected call to api client', async (done) => {
     const logout = jest.fn(() => Promise.resolve());
 
-    const appWrapper = mount(
-      <MemoryRouter>
-        <I18nProvider>
-          <NetworkProvider value={networkProviderValue}>
-            <ConfigProvider>
-              <_App api={{ logout }} handleHttpError={() => {}} />
-            </ConfigProvider>
-          </NetworkProvider>
-        </I18nProvider>
-      </MemoryRouter>
-    ).find('App');
+    const appWrapper = mountWithContexts(<App />, {
+      context: { network: { api: { logout }, handleHttpError: () => {} } }
+    }).find('App');
 
     appWrapper.instance().onLogout();
     await asyncFlush();
diff --git a/__tests__/enzymeHelpers.jsx b/__tests__/enzymeHelpers.jsx
index c4e31909a7..b49dcb17f7 100644
--- a/__tests__/enzymeHelpers.jsx
+++ b/__tests__/enzymeHelpers.jsx
@@ -45,6 +45,12 @@ const defaultContexts = {
       push: () => {},
       replace: () => {},
       createHref: () => {},
+      location: {
+        hash: '',
+        pathname: '',
+        search: '',
+        state: '',
+      }
     },
     route: {
       location: {
@@ -58,7 +64,7 @@ const defaultContexts = {
         isExact: false,
         path: '',
         url: '',
-      },
+      }
     },
     toJSON: () => '/router/',
   },