From 175c85d69a21ae0ee61e3481ab23352e15960938 Mon Sep 17 00:00:00 2001
From: Blake Blackshear <blakeb@blakeshome.com>
Date: Sun, 13 Jun 2021 14:49:13 -0500
Subject: [PATCH] fix some test errors

---
 web/src/routes/Camera.jsx                | 38 +++++++++++++-----------
 web/src/routes/__tests__/Camera.test.jsx |  4 +++
 2 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/web/src/routes/Camera.jsx b/web/src/routes/Camera.jsx
index df09ffc7d..7f5527df4 100644
--- a/web/src/routes/Camera.jsx
+++ b/web/src/routes/Camera.jsx
@@ -83,26 +83,30 @@ export default function Camera({ camera }) {
 
   let player;
   if (viewMode === 'live') {
-    player = <>
-      <div>
-        <JSMpegPlayer camera={camera} />
-      </div>
-    </>;
+    player = (
+      <Fragment>
+        <div>
+          <JSMpegPlayer camera={camera} />
+        </div>
+      </Fragment>
+    );
   }
   else if (viewMode === 'debug') {
-    player = <>
-      <div>
-        <AutoUpdatingCameraImage camera={camera} searchParams={searchParams} />
-      </div>
+    player = (
+      <Fragment>
+        <div>
+          <AutoUpdatingCameraImage camera={camera} searchParams={searchParams} />
+        </div>
 
-      <Button onClick={handleToggleSettings} type="text">
-        <span className="w-5 h-5">
-          <SettingsIcon />
-        </span>{' '}
-        <span>{showSettings ? 'Hide' : 'Show'} Options</span>
-      </Button>
-      {showSettings ? <Card header="Options" elevated={false} content={optionContent} /> : null}
-    </>;
+        <Button onClick={handleToggleSettings} type="text">
+          <span className="w-5 h-5">
+            <SettingsIcon />
+          </span>{' '}
+          <span>{showSettings ? 'Hide' : 'Show'} Options</span>
+        </Button>
+        {showSettings ? <Card header="Options" elevated={false} content={optionContent} /> : null}
+      </Fragment>
+    );
   }
 
   return (
diff --git a/web/src/routes/__tests__/Camera.test.jsx b/web/src/routes/__tests__/Camera.test.jsx
index c95a9a1d5..445b45302 100644
--- a/web/src/routes/__tests__/Camera.test.jsx
+++ b/web/src/routes/__tests__/Camera.test.jsx
@@ -32,7 +32,10 @@ describe('Camera Route', () => {
       },
       mockSetOptions,
     ]);
+
     render(<Camera camera="front" />);
+
+    fireEvent.click(screen.queryByText('Debug'));
     fireEvent.click(screen.queryByText('Show Options'));
     expect(screen.queryByTestId('mock-image')).toHaveTextContent(
       'bbox=1&timestamp=0&zones=1&mask=0&motion=1&regions=0'
@@ -47,6 +50,7 @@ describe('Camera Route', () => {
 
     render(<Camera camera="front" />);
 
+    fireEvent.click(screen.queryByText('Debug'));
     fireEvent.click(screen.queryByText('Show Options'));
     fireEvent.change(screen.queryByTestId('bbox-input'), { target: { checked: true } });
     fireEvent.change(screen.queryByTestId('timestamp-input'), { target: { checked: true } });