feat: primer v37 upgrade

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/renderer/__helpers__/jest.setup.ts

@@ -44,12 +44,10 @@ process.env.OAUTH_CLIENT_ID = 'FAKE_CLIENT_ID_123';
 process.env.OAUTH_CLIENT_SECRET = 'FAKE_CLIENT_SECRET_123';
 
 /**
- * Primer (@primer/react) Setup
+ * Primer (@primer/react) Setup - START
  *
  * Borrowed from https://github.com/primer/react/blob/main/packages/react/src/utils/test-helpers.tsx
  */
-// @ts-expect-error: prevent ReferenceError: TextEncoder is not defined
-global.TextEncoder = TextEncoder;
 
 // JSDOM doesn't mock ResizeObserver
 global.ResizeObserver = jest.fn().mockImplementation(() => {
@@ -68,18 +66,22 @@ global.CSS = {
   }),
 };
 
+// @ts-expect-error: prevent ReferenceError: TextEncoder is not defined
+global.TextEncoder = TextEncoder;
+
+/**
+ * Primer (@primer/react) Setup - END
+ */
+
 window.HTMLMediaElement.prototype.play = jest.fn();
 
-// JSDOM doesn't implement matchMedia; needed for Primer media queries (useMedia hook)
-if (!window.matchMedia) {
-  window.matchMedia = (query: string): MediaQueryList => ({
-    matches: false,
-    media: query,
-    onchange: null,
-    addListener: () => {}, // deprecated
-    removeListener: () => {}, // deprecated
-    addEventListener: () => {},
-    removeEventListener: () => {},
-    dispatchEvent: () => false,
-  });
-}
+window.matchMedia = (query: string): MediaQueryList => ({
+  matches: false,
+  media: query,
+  onchange: null,
+  addListener: () => {}, // deprecated
+  removeListener: () => {}, // deprecated
+  addEventListener: () => {},
+  removeEventListener: () => {},
+  dispatchEvent: () => false,
+});

src/renderer/routes/Accounts.test.tsx

@@ -3,27 +3,6 @@ import userEvent from '@testing-library/user-event';
 
 import { BaseStyles, ThemeProvider } from '@primer/react';
 
-// Polyfills for DOM APIs used by Primer components that jsdom doesn't implement
-// Minimal no-op implementations if jsdom environment lacks them
-const g = global as unknown as Record<string, unknown>;
-if (typeof g.ResizeObserver === 'undefined') {
-  g.ResizeObserver = class {
-    observe() {}
-    unobserve() {}
-    disconnect() {}
-  };
-}
-if (typeof g.IntersectionObserver === 'undefined') {
-  g.IntersectionObserver = class {
-    observe() {}
-    unobserve() {}
-    disconnect() {}
-    takeRecords() {
-      return [];
-    }
-  };
-}
-
 import {
   mockAuth,
   mockGitHubAppAccount,
@@ -38,6 +17,8 @@ import * as links from '../utils/links';
 import * as storage from '../utils/storage';
 import { AccountsRoute } from './Accounts';
 
+// import { setupMatchMedia } from '../__helpers__/jest.setup';
+
 const mockNavigate = jest.fn();
 jest.mock('react-router-dom', () => ({
   ...jest.requireActual('react-router-dom'),
@@ -47,6 +28,7 @@ jest.mock('react-router-dom', () => ({
 describe('renderer/routes/Accounts.tsx', () => {
   afterEach(() => {
     jest.clearAllMocks();
+    // setupMatchMedia()
   });
 
   describe('General', () => {