fix: theming tests (#159)

* Fix window.matchMedia may not exist in test environment * Fix getting of potentially more than one "Duality" text
duality-labs · Oct 19, 2022 · 983dfcf · 983dfcf
1 parent 20a59ec
commit 983dfcf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/src/lib/themeProvider.tsx b/src/lib/themeProvider.tsx
@@ -55,11 +55,12 @@ export function ThemeProvider({ children }: { children: React.ReactNode }) {
 
   // preference change (if the browser settings were changed)
   useEffect(() => {
-    const mediaQuery = window.matchMedia('(prefers-color-scheme: dark)');
-    mediaQuery.addEventListener('change', onPreferenceChange, false);
-    return () =>
-      mediaQuery.removeEventListener('change', onPreferenceChange, false);
-
+    const mediaQuery = window.matchMedia?.('(prefers-color-scheme: dark)');
+    if (mediaQuery) {
+      mediaQuery.addEventListener('change', onPreferenceChange, false);
+      return () =>
+        mediaQuery.removeEventListener('change', onPreferenceChange, false);
+    }
     function onPreferenceChange() {
       setDefaultTheme(getDefaultBrowserTheme());
     }

diff --git a/src/pages/App/App.test.tsx b/src/pages/App/App.test.tsx
@@ -3,6 +3,6 @@ import App from './App';
 
 test('renders learn react link', () => {
   render(<App />);
-  const page = screen.getByText(/Duality/i);
-  expect(page).toBeInTheDocument();
+  const page = screen.getAllByText(/Duality/i);
+  expect(page[0]).toBeInTheDocument();
 });