test: fix tests after version bumps

system-ui · Oct 17, 2024 · e44074e · e44074e
1 parent 14f1620
commit e44074e
Show file tree

Hide file tree

Showing 10 changed files with 250 additions and 202 deletions.
diff --git a/packages/color-modes/test/index.tsx b/packages/color-modes/test/index.tsx
@@ -903,12 +903,11 @@ test('colorMode accepts function from previous state to new one', () => {
 test('warns when localStorage is disabled', () => {
   const restoreConsole = mockConsole()
 
-  const localStorage = window.localStorage
-  Object.defineProperty(window, 'localStorage', {
-    get: jest.fn(() => {
+  const spy = jest
+    .spyOn(Storage.prototype, 'getItem')
+    .mockImplementation(() => {
       throw new Error('SecurityError: The operation is insecure.')
-    }),
-  })
+    })
 
   let mode = ''
   const Consumer = () => {
@@ -927,8 +926,6 @@ test('warns when localStorage is disabled', () => {
 
   expect(mode).toBe(undefined)
 
-  Object.defineProperty(window, 'localStorage', { value: localStorage })
-
   expect((console.warn as jest.Mock).mock.calls[0]).toMatchInlineSnapshot(`
     [
       "localStorage is disabled and color mode might not work as expected.",
@@ -937,6 +934,7 @@ test('warns when localStorage is disabled', () => {
     ]
   `)
 
+  spy.mockClear()
   restoreConsole()
 })
 

diff --git a/packages/css/package.json b/packages/css/package.json
@@ -35,13 +35,13 @@
     "access": "public"
   },
   "dependencies": {
-    "csstype": "^3.0.10"
+    "csstype": "3.0.10"
   },
   "peerDependencies": {
     "@emotion/react": "^11.11.1"
   },
   "devDependencies": {
-    "@emotion/react": "^11.13.3",
+    "@emotion/react": "^11.11.1",
     "@theme-ui/test-utils": "workspace:^",
     "@types/react": "^18.2.12"
   },

diff --git a/packages/css/test/index.ts b/packages/css/test/index.ts
@@ -728,8 +728,11 @@ test('supports vendor properties', () => {
 test('omits empty values', () => {
   expect(
     css({
+      // @ts-ignore TS2873
       color: false && 'blue',
+      // @ts-ignore TS2873
       backgroundColor: undefined && 'whitesmoke',
+      // @ts-ignore TS2873
       textDecoration: null && 'underline',
       border: '1px solid black',
     })(theme)

diff --git a/packages/global/test/__snapshots__/index.tsx.snap b/packages/global/test/__snapshots__/index.tsx.snap
@@ -49,21 +49,6 @@ exports[`renders global styles 1`] = `
       data-s=""
     >
       
-      @font-face{font-family:some-name;}
-    </style>
-    <style
-      data-emotion="css-global"
-      data-s=""
-    >
-      
-      body{font-family:Georgia,serif;margin:0;}
-    </style>
-    <style
-      data-emotion="css-global"
-      data-s=""
-    >
-      
-      h1{color:salmon;}
     </style>
   </head>
   <body>

diff --git a/packages/prism/src/index.tsx b/packages/prism/src/index.tsx
@@ -131,12 +131,17 @@ export default function ThemeUIPrism({
               }
               return (
                 <div {...lineProps}>
-                  {line.map((token, key) => (
-                    <span
-                      {...getTokenProps({ token, key })}
-                      sx={token.empty ? { display: 'inline-block' } : undefined}
-                    />
-                  ))}
+                  {line.map((token, key) => {
+                    return (
+                      <span
+                        {...getTokenProps({ token, key })}
+                        key={key}
+                        sx={
+                          token.empty ? { display: 'inline-block' } : undefined
+                        }
+                      />
+                    )
+                  })}
                 </div>
               )
             })}

diff --git a/packages/prism/test/__snapshots__/index.tsx.snap b/packages/prism/test/__snapshots__/index.tsx.snap
@@ -2,7 +2,7 @@
 
 exports[`highlight start and end 1`] = `
 <pre
-  className="language-js prism-code language-javascript emotion-0"
+  className="language-js prism-code language-javascript css-1avyp1d"
   style={{}}
 >
   <div
@@ -88,7 +88,7 @@ exports[`highlight start and end 1`] = `
 
 exports[`highlights inline comment 1`] = `
 <pre
-  className="language-js prism-code language-javascript emotion-0"
+  className="language-js prism-code language-javascript css-1avyp1d"
   style={{}}
 >
   <div
@@ -262,7 +262,7 @@ exports[`multiple highlights start and end 1`] = `
 
 exports[`no highlight 1`] = `
 <pre
-  className="language-js prism-code language-javascript emotion-0"
+  className="language-js prism-code language-javascript css-1avyp1d"
   style={{}}
 >
   <div
@@ -309,7 +309,7 @@ exports[`no highlight 1`] = `
 
 exports[`renders a code block 1`] = `
 <pre
-  className="language-js prism-code language-javascript emotion-0"
+  className="language-js prism-code language-javascript css-1avyp1d"
   style={{}}
 >
   <div
@@ -351,7 +351,7 @@ exports[`renders a code block 1`] = `
 
 exports[`renders with no className 1`] = `
 <pre
-  className=" prism-code language- emotion-0"
+  className="prism-code language- css-1avyp1d"
   style={{}}
 >
   <div
@@ -368,7 +368,7 @@ exports[`renders with no className 1`] = `
 
 exports[`renders with other languages 1`] = `
 <pre
-  className="language-php prism-code language-php emotion-0"
+  className="language-php prism-code language-php css-1avyp1d"
   style={{}}
 >
   <div