chore: update vitest and vitest-when (#17149)

* chore: update vitest and vitest-when
Opentrons · Dec 20, 2024 · 04dea6c · 04dea6c
1 parent a80aa00
commit 04dea6c
Show file tree

Hide file tree

Showing 6 changed files with 237 additions and 252 deletions.
diff --git a/...Desktop/Devices/RobotSettings/UpdateBuildroot/__tests__/RobotUpdateProgressModal.test.tsx b/...Desktop/Devices/RobotSettings/UpdateBuildroot/__tests__/RobotUpdateProgressModal.test.tsx
@@ -52,6 +52,7 @@ describe('DownloadUpdateModal', () => {
 
   let props: React.ComponentProps<typeof RobotUpdateProgressModal>
   const mockCreateLiveCommand = vi.fn()
+  const mockDispatchStartRobotUpdate = vi.fn()
 
   beforeEach(() => {
     mockCreateLiveCommand.mockResolvedValue(null)
@@ -68,7 +69,9 @@ describe('DownloadUpdateModal', () => {
       progressPercent: 50,
     })
     vi.mocked(getRobotSessionIsManualFile).mockReturnValue(false)
-    vi.mocked(useDispatchStartRobotUpdate).mockReturnValue(vi.fn)
+    vi.mocked(useDispatchStartRobotUpdate).mockReturnValue(
+      mockDispatchStartRobotUpdate
+    )
     vi.mocked(getRobotUpdateDownloadError).mockReturnValue(null)
   })
 

diff --git a/package.json b/package.json
@@ -69,7 +69,7 @@
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "@vitejs/plugin-react": "4.2.0",
-    "@vitest/coverage-v8": "1.3.0",
+    "@vitest/coverage-v8": "2.1.8",
     "ajv": "6.12.3",
     "aws-sdk": "^2.493.0",
     "babel-loader": "^8.2.2",
@@ -151,8 +151,8 @@
     "typescript": "5.3.3",
     "url-loader": "^2.1.0",
     "vite": "5.3.2",
-    "vitest": "1.2.2",
-    "vitest-when": "0.3.1",
+    "vitest": "2.1.8",
+    "vitest-when": "0.5.0",
     "wait-on": "^4.0.2",
     "webpack": "^4.41.6",
     "webpack-bundle-analyzer": "^3.6.0",

diff --git a/protocol-designer/src/__tests__/persist.test.ts b/protocol-designer/src/__tests__/persist.test.ts
@@ -4,12 +4,14 @@ import type { MockInstance } from 'vitest'
 import * as persist from '../persist'
 
 describe('persist', () => {
-  let getItemSpy: MockInstance<unknown[], unknown>
-  let setItemSpy: MockInstance<unknown[], unknown>
+  let getItemSpy: MockInstance<(key: string) => string | null>
+  let setItemSpy: MockInstance<(key: string, value: string) => void>
 
   beforeEach(() => {
     const LocalStorageProto = Object.getPrototypeOf(global.localStorage)
-    getItemSpy = vi.spyOn(LocalStorageProto, 'getItem')
+    getItemSpy = vi.spyOn(LocalStorageProto, 'getItem') as MockInstance<
+      (key: string) => string | null
+    >
     setItemSpy = vi.spyOn(LocalStorageProto, 'setItem')
   })
 

diff --git a/setup-vitest.ts → setup-vitest.mts b/setup-vitest.ts → setup-vitest.mts
diff --git a/vitest.config.ts → vitest.config.mts b/vitest.config.ts → vitest.config.mts
@@ -13,7 +13,12 @@ export default mergeConfig(
       environment: 'jsdom',
       allowOnly: true,
       exclude: [...configDefaults.exclude, '**/node_modules/**', '**/dist/**'],
-      setupFiles: ['./setup-vitest.ts'],
+      setupFiles: ['./setup-vitest.mts'],
+      coverage: {
+        exclude: ['**/node_modules/**', '**/dist/**', '**/__tests__/**'],
+        provider: 'v8',
+        reporter: ['text', 'json', 'html'],
+      },
     },
     resolve: {
       alias: {