From 1c2d6ba2a28056b9bef37744ae3bca511b7c3bb9 Mon Sep 17 00:00:00 2001
From: pauladamseniii <padamsen@fiskerinc.com>
Date: Wed, 18 Oct 2023 16:24:33 -0400
Subject: [PATCH] improve tests

---
 .../App/__snapshots__/App.test.js.snap        | 48 ++-----------------
 .../Contexts/__mocks__/CarUpdatesContext.jsx  |  3 ++
 .../Contexts/__mocks__/ManifestsContext.jsx   |  2 +
 src/components/SUMS/Add/index.test.jsx        | 12 ++---
 .../SUMS/__snapshots__/index.test.jsx.snap    |  2 +-
 src/components/SUMS/index.test.jsx            | 12 ++---
 6 files changed, 22 insertions(+), 57 deletions(-)

diff --git a/src/components/App/__snapshots__/App.test.js.snap b/src/components/App/__snapshots__/App.test.js.snap
index 68feacd..c2d990a 100644
--- a/src/components/App/__snapshots__/App.test.js.snap
+++ b/src/components/App/__snapshots__/App.test.js.snap
@@ -4395,48 +4395,6 @@ exports[`App Route /package-deploy authenticated 1`] = `
                     <div
                       class="MuiGrid-root MuiGrid-container MuiGrid-item MuiGrid-justify-content-xs-flex-end MuiGrid-grid-md-4"
                     >
-                      <div
-                        class="MuiFormControl-root MuiFormControl-marginNormal"
-                      >
-                        <label
-                          class="MuiFormLabel-root MuiInputLabel-root makeStyles-whiteBackground-0 MuiInputLabel-formControl MuiInputLabel-animated MuiInputLabel-outlined"
-                          data-shrink="false"
-                        >
-                          Software Version
-                        </label>
-                        <div
-                          class="MuiInputBase-root MuiOutlinedInput-root MuiInputBase-formControl"
-                        >
-                          <select
-                            aria-invalid="false"
-                            class="MuiSelect-root MuiSelect-select MuiSelect-outlined MuiInputBase-input MuiOutlinedInput-input"
-                          />
-                          <svg
-                            aria-hidden="true"
-                            class="MuiSvgIcon-root MuiSelect-icon MuiSelect-iconOutlined"
-                            focusable="false"
-                            viewBox="0 0 24 24"
-                          >
-                            <path
-                              d="M7 10l5 5 5-5z"
-                            />
-                          </svg>
-                          <fieldset
-                            aria-hidden="true"
-                            class="PrivateNotchedOutline-root-0 MuiOutlinedInput-notchedOutline"
-                            style="padding-left: 8px;"
-                          >
-                            <legend
-                              class="PrivateNotchedOutline-legend-0"
-                              style="width: 0.01px;"
-                            >
-                              <span>
-                                ​
-                              </span>
-                            </legend>
-                          </fieldset>
-                        </div>
-                      </div>
                       <button
                         class="MuiButtonBase-root MuiButton-root MuiButton-text MuiButton-textPrimary Mui-disabled Mui-disabled"
                         disabled=""
@@ -6889,8 +6847,10 @@ exports[`App Route /packages authenticated 1`] = `
                       class="MuiTableCell-root MuiTableCell-body MuiTableCell-alignCenter"
                     >
                       <a
-                        href="/sums/undefined"
-                      />
+                        href="/sums/2023.03.00.00.E"
+                      >
+                        2023.03.00.00.E
+                      </a>
                     </td>
                     <td
                       class="MuiTableCell-root MuiTableCell-body MuiTableCell-alignCenter"
diff --git a/src/components/Contexts/__mocks__/CarUpdatesContext.jsx b/src/components/Contexts/__mocks__/CarUpdatesContext.jsx
index c83c7c2..c99e25f 100644
--- a/src/components/Contexts/__mocks__/CarUpdatesContext.jsx
+++ b/src/components/Contexts/__mocks__/CarUpdatesContext.jsx
@@ -21,6 +21,7 @@ let carUpdates = [
       ecu_list: "AGS 1.0.0,AMP 1.0.0",
       created: "2021-08-20T18:37:41.960397Z",
       updated: "2021-08-20T18:37:50.007853Z",
+      sums: "2023.03.00.00.E",
     },
   },
   {
@@ -40,6 +41,7 @@ let carUpdates = [
       ecu_list: "AGS 1.0.0,AMP 1.0.0",
       created: "2021-08-20T18:37:41.960397Z",
       updated: "2021-08-20T18:37:50.007853Z",
+      sums: "2023.03.00.00.E",
     },
   },
   {
@@ -59,6 +61,7 @@ let carUpdates = [
       ecu_list: "AGS 1.0.0,AMP 1.0.0",
       created: "2021-08-20T18:37:41.960397Z",
       updated: "2021-08-20T18:37:50.007853Z",
+      sums: "2023.03.00.00.E",
     },
   },
 ];
diff --git a/src/components/Contexts/__mocks__/ManifestsContext.jsx b/src/components/Contexts/__mocks__/ManifestsContext.jsx
index a4e8b4e..5c02afc 100644
--- a/src/components/Contexts/__mocks__/ManifestsContext.jsx
+++ b/src/components/Contexts/__mocks__/ManifestsContext.jsx
@@ -53,6 +53,7 @@ let manifest = {
   created: "2021-12-09T22:38:11.679943Z",
   updated: "2022-01-14T00:47:08.996451Z",
   active: true,
+  sums: "2023.03.00.00.E",
 };
 let manifests = [
   {
@@ -62,6 +63,7 @@ let manifests = [
     created: "2021-07-01T22:40:07.778509Z",
     updated: "2021-07-12T18:22:13.736755Z",
     active: true,
+    sums: "2023.03.00.00.E",
   },
 ];
 let totalManifests = 1;
diff --git a/src/components/SUMS/Add/index.test.jsx b/src/components/SUMS/Add/index.test.jsx
index dd85da0..f44513f 100644
--- a/src/components/SUMS/Add/index.test.jsx
+++ b/src/components/SUMS/Add/index.test.jsx
@@ -4,10 +4,8 @@ jest.mock("../../Contexts/UserContext");
 jest.mock("@material-ui/core/utils/unstable_useId", () =>
   jest.fn().mockReturnValue("mui-test-id")
 );
-
 import { render, waitFor } from "@testing-library/react";
-import { BrowserRouter } from "react-router-dom";
-
+import { MemoryRouter, Route } from "react-router-dom";
 import { CarUpdatesProvider } from "../../Contexts/CarUpdatesContext";
 import { StatusProvider } from "../../Contexts/StatusContext";
 import { UserProvider, setToken } from "../../Contexts/UserContext";
@@ -20,9 +18,11 @@ const renderSUMSAdd = async () => {
     <CarUpdatesProvider>
       <StatusProvider>
         <UserProvider>
-          <BrowserRouter>
-            <MainForm />
-          </BrowserRouter>
+          <MemoryRouter initialEntries={["/sums/add-rxswin/2023.03.00.00.E"]}>
+            <Route path="/sums/add-rxswin/:sums_version">
+              <MainForm sums_version="2023.03.00.00.E" />
+            </Route>
+          </MemoryRouter>
         </UserProvider>
       </StatusProvider>
     </CarUpdatesProvider>
diff --git a/src/components/SUMS/__snapshots__/index.test.jsx.snap b/src/components/SUMS/__snapshots__/index.test.jsx.snap
index f68b68c..e79d3c8 100644
--- a/src/components/SUMS/__snapshots__/index.test.jsx.snap
+++ b/src/components/SUMS/__snapshots__/index.test.jsx.snap
@@ -23,7 +23,7 @@ exports[`SUMS Render 1`] = `
               >
                 <a
                   class="makeStyles-labelInline-0"
-                  href="/sums-rxswin-add/undefined"
+                  href="/sums-rxswin-add/2023.03.00.00.E"
                 >
                   <svg
                     aria-hidden="true"
diff --git a/src/components/SUMS/index.test.jsx b/src/components/SUMS/index.test.jsx
index 7e00a34..cfd5390 100644
--- a/src/components/SUMS/index.test.jsx
+++ b/src/components/SUMS/index.test.jsx
@@ -4,10 +4,8 @@ jest.mock("../Contexts/UserContext");
 jest.mock("@material-ui/core/utils/unstable_useId", () =>
   jest.fn().mockReturnValue("mui-test-id")
 );
-
 import { render, waitFor } from "@testing-library/react";
-import { BrowserRouter } from "react-router-dom";
-
+import { MemoryRouter, Route } from "react-router-dom";
 import { CarUpdatesProvider } from "../Contexts/CarUpdatesContext";
 import { StatusProvider } from "../Contexts/StatusContext";
 import { UserProvider, setToken } from "../Contexts/UserContext";
@@ -20,9 +18,11 @@ const renderSUMS = async () => {
     <CarUpdatesProvider>
       <StatusProvider>
         <UserProvider>
-          <BrowserRouter>
-            <MainForm />
-          </BrowserRouter>
+          <MemoryRouter initialEntries={["/sums/2023.03.00.00.E"]}>
+            <Route path="/sums/:sums_version">
+              <MainForm sums_version="2023.03.00.00.E" />
+            </Route>
+          </MemoryRouter>
         </UserProvider>
       </StatusProvider>
     </CarUpdatesProvider>