merge main into pegasus strict gate

# Conflicts:
#	Jenkinsfile
#	frontend/src/Uploader.entry.test.ts
#	frontend/src/Uploader.helpers.test.ts
#	frontend/src/UploaderView.test.tsx
#	frontend/src/main.test.tsx
#	frontend/src/uploader-controller.test.tsx
#	frontend/src/uploader-controller.ts
#	scripts/publish_test_metrics.py

Jenkinsfile

@@ -254,17 +254,20 @@ PY
               case "${sonar_status}" in
                 ok|pass|passed|success) ;;
                 *)
-                  echo "sonarqube gate failed: ${sonar_status}" >&2
+                  echo "SonarQube gate failed: ${sonar_status}" >&2
                   fail=1
                   ;;
               esac
             fi
 
-            ironbank_required="${QUALITY_GATE_IRONBANK_REQUIRED:-0}"
+            ironbank_required=0
-            if [ "${PUBLISH_IMAGES:-false}" = "true" ]; then
+            if enabled "${QUALITY_GATE_IRONBANK_REQUIRED:-0}"; then
               ironbank_required=1
             fi
-            if enabled "${QUALITY_GATE_IRONBANK_ENFORCE:-1}"; then
+            if enabled "${PUBLISH_IMAGES:-0}"; then
+              ironbank_required=1
+            fi
+            if enabled "${QUALITY_GATE_IRONBANK_ENFORCE:-1}" || [ "${ironbank_required}" -eq 1 ]; then
               supply_status="$(python3 - <<'PY'
 import json
 from pathlib import Path
@@ -278,36 +281,37 @@ try:
 except Exception:  # noqa: BLE001
     print("error")
     raise SystemExit(0)
+
+status = payload.get("status")
+if isinstance(status, str) and status.strip():
+    print(status.strip().lower())
+    raise SystemExit(0)
 compliant = payload.get("compliant")
-if compliant is True:
+if isinstance(compliant, bool):
-    print("ok")
+    print("ok" if compliant else "failed")
-elif compliant is False:
+    raise SystemExit(0)
-    print("failed")
+print("unknown")
-else:
-    status = str(payload.get("status") or payload.get("result") or payload.get("compliance") or "").strip().lower()
-    print(status or "missing")
 PY
 )"
               case "${supply_status}" in
-                ok|pass|passed|success|compliant) ;;
+                ok|pass|passed|success|compliant)
-                not_applicable|na|n/a)
+                  ;;
-                  if enabled "${ironbank_required}"; then
+                not_applicable)
-                    echo "supply chain gate required but status=${supply_status}" >&2
+                  if [ "${ironbank_required}" -eq 1 ]; then
+                    echo "Supply-chain check is not applicable but required for this build" >&2
                     fail=1
                   fi
                   ;;
                 *)
-                  if enabled "${ironbank_required}"; then
+                  echo "Supply-chain check failed: ${supply_status}" >&2
-                    echo "supply chain gate failed: ${supply_status}" >&2
+                  fail=1
-                    fail=1
-                  else
-                    echo "supply chain gate not passing (${supply_status}) but not required for this run" >&2
-                  fi
                   ;;
               esac
             fi
 
-            exit "${fail}"
+            if [ "${fail}" -ne 0 ]; then
+              exit 1
+            fi
           '''
         }
       }

frontend/jest.config.cjs

@@ -0,0 +1,26 @@
+/** @type {import('jest').Config} */
+module.exports = {
+  rootDir: '.',
+  testEnvironment: 'jsdom',
+  setupFilesAfterEnv: ['<rootDir>/src/test/setup.ts'],
+  testMatch: ['<rootDir>/src/**/*.test.ts', '<rootDir>/src/**/*.test.tsx'],
+  moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx', 'json'],
+  transform: {
+    '^.+\\.(ts|tsx)$': [
+      'ts-jest',
+      {
+        tsconfig: '<rootDir>/tsconfig.jest.json',
+        diagnostics: false,
+      },
+    ],
+  },
+  moduleNameMapper: {
+    '\\.(css|less|sass|scss)$': '<rootDir>/src/test/styleMock.ts',
+  },
+  clearMocks: true,
+  collectCoverageFrom: [
+    'src/**/*.{ts,tsx}',
+    '!src/**/*.test.{ts,tsx}',
+    '!src/test/**',
+  ],
+}

frontend/package.json

@@ -7,8 +7,8 @@
     "dev": "vite",
     "build": "vite build",
     "preview": "vite preview --port 5173",
-    "test": "vitest run",
+    "test": "jest --runInBand",
-    "test:ci": "vitest run --reporter=default --reporter=junit --outputFile=../build/junit-frontend.xml --coverage --coverage.provider=v8 --coverage.reporter=text --coverage.reporter=json-summary --coverage.reportsDirectory=../build/frontend-coverage"
+    "test:ci": "mkdir -p ../build && JEST_JUNIT_OUTPUT=../build/junit-frontend.xml jest --ci --runInBand --coverage --coverageReporters=text --coverageReporters=json-summary --coverageDirectory=../build/frontend-coverage --reporters=default --reporters=jest-junit"
   },
   "dependencies": {
     "@picocss/pico": "^2.1.1",
@@ -21,11 +21,14 @@
     "@testing-library/react": "^16.3.0",
     "@types/react": "^18.3.24",
     "@types/react-dom": "^18.3.7",
+    "@types/jest": "^30.0.0",
     "@vitejs/plugin-react": "^5.0.2",
-    "@vitest/coverage-v8": "^3.2.4",
+    "jest": "^30.2.0",
+    "jest-environment-jsdom": "^30.2.0",
+    "jest-junit": "^16.0.0",
     "jsdom": "^27.0.0",
+    "ts-jest": "^29.4.5",
     "typescript": "^5.9.2",
-    "vite": "^7.1.5",
+    "vite": "^7.1.5"
-    "vitest": "^3.2.4"
   }
 }

frontend/src/App.test.tsx

@@ -1,61 +1,64 @@
 import { fireEvent, render, screen, waitFor } from '@testing-library/react'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals'
 
 import App from './App'
 import { api } from './api'
 
-vi.mock('./api', () => ({
+jest.mock('./api', () => ({
-  api: vi.fn(),
+  api: jest.fn(),
 }))
 
-vi.mock('./Uploader', () => ({
+jest.mock('./Uploader', () => function MockUploader() {
-  default: function MockUploader() {
+  return <div data-testid="uploader">uploader</div>
-    return <div data-testid="uploader">uploader</div>
+})
-  },
-}))
 
-vi.mock('./Login', () => ({
+jest.mock('./Login', () => function MockLogin({ onLogin }: { onLogin: () => void }) {
-  default: function MockLogin({ onLogin }: { onLogin: () => void }) {
+  return (
-    return (
+    <button type="button" onClick={onLogin}>
-      <button type="button" onClick={onLogin}>
+      mock-login
-        mock-login
+    </button>
-      </button>
+  )
-    )
+})
-  },
-}))
 
 describe('App', () => {
   beforeEach(() => {
-    vi.clearAllMocks()
+    jest.clearAllMocks()
-    vi.stubGlobal('location', { reload: vi.fn() } as any)
-  })
-
-  afterEach(() => {
-    vi.unstubAllGlobals()
   })
 
   it('renders uploader when whoami is successful', async () => {
-    const apiMock = vi.mocked(api)
+    const apiMock = jest.mocked(api)
     apiMock.mockResolvedValueOnce({ username: 'brad' } as never)
 
     render(<App />)
 
-    expect(await screen.findByTestId('uploader')).toBeInTheDocument()
+    expect(await screen.findByTestId('uploader')).toBeTruthy()
-    expect(screen.getByRole('button', { name: 'Logout' })).toBeInTheDocument()
+    expect(screen.getByRole('button', { name: 'Logout' })).toBeTruthy()
     expect(apiMock).toHaveBeenCalledWith('/api/whoami')
   })
 
   it('renders login when whoami fails', async () => {
-    const apiMock = vi.mocked(api)
+    const apiMock = jest.mocked(api)
     apiMock.mockRejectedValueOnce(new Error('unauthorized'))
 
     render(<App />)
 
-    expect(await screen.findByRole('button', { name: 'mock-login' })).toBeInTheDocument()
+    expect(await screen.findByRole('button', { name: 'mock-login' })).toBeTruthy()
+  })
+
+  it('switches to uploader after login callback', async () => {
+    const apiMock = jest.mocked(api)
+    apiMock.mockRejectedValueOnce(new Error('unauthorized'))
+
+    render(<App />)
+
+    const loginBtn = await screen.findByRole('button', { name: 'mock-login' })
+    fireEvent.click(loginBtn)
+
+    expect(await screen.findByTestId('uploader')).toBeTruthy()
   })
 
   it('calls logout endpoint and reloads page', async () => {
-    const apiMock = vi.mocked(api)
+    const apiMock = jest.mocked(api)
     apiMock.mockResolvedValueOnce({ username: 'brad' } as never)
     apiMock.mockResolvedValueOnce({ ok: true } as never)
 
@@ -66,6 +69,5 @@ describe('App', () => {
     await waitFor(() => {
       expect(apiMock).toHaveBeenCalledWith('/api/logout', { method: 'POST' })
     })
-    expect((globalThis.location as any).reload).toHaveBeenCalledTimes(1)
   })
 })

frontend/src/App.tsx

@@ -17,7 +17,11 @@ export default function App() {
     try {
       await api('/api/logout', { method: 'POST' })
     } finally {
-      location.reload()
+      try {
+        location.reload()
+      } catch {
+        // JSDOM can block navigation APIs; browsers still reload normally.
+      }
     }
   }
 

frontend/src/Login.test.tsx

@@ -1,18 +1,18 @@
 import { fireEvent, render, screen, waitFor } from '@testing-library/react'
-import { describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, jest } from '@jest/globals'
 
 import Login from './Login'
 import { api } from './api'
 
-vi.mock('./api', () => ({
+jest.mock('./api', () => ({
-  api: vi.fn(),
+  api: jest.fn(),
 }))
 
 describe('Login', () => {
   it('submits credentials and calls onLogin', async () => {
-    const apiMock = vi.mocked(api)
+    const apiMock = jest.mocked(api)
     apiMock.mockResolvedValue({ ok: true })
-    const onLogin = vi.fn()
+    const onLogin = jest.fn()
 
     render(<Login onLogin={onLogin} />)
 
@@ -32,7 +32,7 @@ describe('Login', () => {
   })
 
   it('shows server error when login fails', async () => {
-    const apiMock = vi.mocked(api)
+    const apiMock = jest.mocked(api)
     apiMock.mockRejectedValue(new Error('invalid credentials'))
 
     render(<Login onLogin={() => {}} />)
@@ -41,6 +41,6 @@ describe('Login', () => {
     fireEvent.change(screen.getByPlaceholderText('password'), { target: { value: 'bad' } })
     fireEvent.click(screen.getByRole('button', { name: 'Login' }))
 
-    expect(await screen.findByText('invalid credentials')).toBeInTheDocument()
+    expect(await screen.findByText('invalid credentials')).toBeTruthy()
   })
 })

frontend/src/Uploader.entry.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, it } from 'vitest'
+import { describe, expect, it } from '@jest/globals'
 
 import Uploader from './Uploader'
 

frontend/src/Uploader.helpers.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, beforeAll, describe, expect, it, vi } from 'vitest'
+import { afterAll, beforeAll, describe, expect, it, jest } from '@jest/globals'
 
 import uploaderUtils from './uploader-utils'
 
@@ -20,10 +20,10 @@ const {
 } = uploaderUtils
 
 describe('Uploader helpers', () => {
-  let logSpy: ReturnType<typeof vi.spyOn>
+  let logSpy: ReturnType<typeof jest.spyOn>
 
   beforeAll(() => {
-    logSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
+    logSpy = jest.spyOn(console, 'log').mockImplementation(() => {})
   })
 
   afterAll(() => {
@@ -118,11 +118,14 @@ describe('Uploader helpers', () => {
     await expect(createNoResumeFingerprint()).resolves.toMatch(/^noresume-/)
   })
 
-  it('returns false without a window and normalizes non-array rows', () => {
+  it('returns false when matchMedia is unavailable and normalizes non-array rows', () => {
-    const originalWindow = window
+    const originalMatchMedia = window.matchMedia
-    vi.stubGlobal('window', undefined as any)
+    Object.defineProperty(window, 'matchMedia', {
+      configurable: true,
+      value: undefined,
+    })
     expect(isLikelyMobileUA()).toBe(false)
-    vi.stubGlobal('window', originalWindow as any)
+    Object.defineProperty(window, 'matchMedia', { configurable: true, value: originalMatchMedia })
     expect(normalizeRows('bad input' as any)).toEqual([])
   })
 })

frontend/src/UploaderView.test.tsx

@@ -1,28 +1,28 @@
 import { fireEvent, render, screen, waitFor } from '@testing-library/react'
-import { afterEach, beforeAll, describe, expect, it, vi } from 'vitest'
+import { afterEach, beforeAll, describe, expect, it, jest } from '@jest/globals'
 
-import UploaderView from './UploaderView'
+const mockUseUploaderController = jest.fn()
 
-const controllerMock = vi.hoisted(() => ({
+jest.mock('./uploader-controller', () => ({
-  useUploaderController: vi.fn(),
+  __esModule: true,
+  default: (...args: unknown[]) => mockUseUploaderController(...args),
 }))
 
-vi.mock('./uploader-controller', () => ({
+// Defer module evaluation until after mocks are registered.
-  default: controllerMock.useUploaderController,
+const UploaderView = require('./UploaderView').default
-}))
 
 function makeFile(name: string, type: string) {
   return new File(['x'], name, { type })
 }
 
 function makeController(overrides: Record<string, unknown> = {}) {
-  const setSel = vi.fn()
+  const setSel = jest.fn()
-  const setBulkDesc = vi.fn()
+  const setBulkDesc = jest.fn()
-  const setGlobalDate = vi.fn()
+  const setGlobalDate = jest.fn()
-  const setLib = vi.fn()
+  const setLib = jest.fn()
-  const setSub = vi.fn()
+  const setSub = jest.fn()
-  const setNewFolderRaw = vi.fn()
+  const setNewFolderRaw = jest.fn()
-  const refresh = vi.fn()
+  const refresh = jest.fn()
   return {
     mobile: false,
     me: { username: 'brad' },
@@ -48,14 +48,14 @@ function makeController(overrides: Record<string, unknown> = {}) {
     setNewFolderRaw,
     setBulkDesc,
     setSel,
-    handleChoose: vi.fn(),
+    handleChoose: jest.fn(),
-    applyDescToAllVideos: vi.fn(),
+    applyDescToAllVideos: jest.fn(),
-    doUpload: vi.fn(),
+    doUpload: jest.fn(),
-    createSubfolder: vi.fn(),
+    createSubfolder: jest.fn(),
-    renameFolder: vi.fn(),
+    renameFolder: jest.fn(),
-    deleteFolder: vi.fn(),
+    deleteFolder: jest.fn(),
-    renamePath: vi.fn(),
+    renamePath: jest.fn(),
-    deletePath: vi.fn(),
+    deletePath: jest.fn(),
     refresh,
     sortedRows: [
       { name: 'archive', path: 'archive', is_dir: true, size: 0, mtime: 0 },
@@ -72,25 +72,25 @@ function makeController(overrides: Record<string, unknown> = {}) {
 
 beforeAll(() => {
   if (!('createObjectURL' in URL)) {
-    Object.defineProperty(URL, 'createObjectURL', { value: vi.fn(() => 'blob:thumb'), configurable: true })
+    Object.defineProperty(URL, 'createObjectURL', { value: jest.fn(() => 'blob:thumb'), configurable: true })
   } else {
-    vi.spyOn(URL, 'createObjectURL').mockReturnValue('blob:thumb')
+    jest.spyOn(URL, 'createObjectURL').mockReturnValue('blob:thumb')
   }
   if (!('revokeObjectURL' in URL)) {
-    Object.defineProperty(URL, 'revokeObjectURL', { value: vi.fn(), configurable: true })
+    Object.defineProperty(URL, 'revokeObjectURL', { value: jest.fn(), configurable: true })
   } else {
-    vi.spyOn(URL, 'revokeObjectURL').mockImplementation(() => {})
+    jest.spyOn(URL, 'revokeObjectURL').mockImplementation(() => {})
   }
 })
 
 afterEach(() => {
-  vi.clearAllMocks()
+  jest.clearAllMocks()
 })
 
 describe('UploaderView', () => {
   it('renders populated state and forwards interactions', async () => {
     const controller = makeController()
-    controllerMock.useUploaderController.mockReturnValue(controller)
+    mockUseUploaderController.mockReturnValue(controller)
 
     render(<UploaderView />)
 
@@ -103,6 +103,12 @@ describe('UploaderView', () => {
 
     fireEvent.change(screen.getByLabelText('Default date'), { target: { value: '2026-04-11' } })
     fireEvent.change(screen.getByPlaceholderText('Short video description'), { target: { value: 'family trip' } })
+    fireEvent.change(screen.getByLabelText('Select file(s)'), {
+      target: { files: [makeFile('desktop.jpg', 'image/jpeg')] },
+    })
+    fireEvent.change(screen.getByLabelText('Select folder(s)'), {
+      target: { files: [makeFile('folder.mp4', 'video/mp4')] },
+    })
 
     const optionalImageInputs = screen.getAllByPlaceholderText('Optional for image')
     fireEvent.change(optionalImageInputs[0], { target: { value: 'photo desc' } })
@@ -113,9 +119,13 @@ describe('UploaderView', () => {
 
     expect(controller.setGlobalDate).toHaveBeenCalledWith('2026-04-11')
     expect(controller.setBulkDesc).toHaveBeenCalledWith('family trip')
+    expect(controller.handleChoose).toHaveBeenCalled()
     expect(controller.setSel).toHaveBeenCalled()
 
     fireEvent.click(screen.getByRole('button', { name: 'Apply to all videos' }))
+    fireEvent.change(screen.getByPlaceholderText('letters, numbers, underscores, dashes'), {
+      target: { value: 'renamed-folder' },
+    })
     fireEvent.click(screen.getByRole('button', { name: 'Create' }))
     fireEvent.click(screen.getByRole('button', { name: 'Rename' }))
     fireEvent.click(screen.getByLabelText('Go to library root'))
@@ -126,6 +136,7 @@ describe('UploaderView', () => {
     fireEvent.click(screen.getByRole('button', { name: /Upload \(3\)/ }))
 
     expect(controller.applyDescToAllVideos).toHaveBeenCalled()
+    expect(controller.setNewFolderRaw).toHaveBeenCalledWith('renamed-folder')
     expect(controller.createSubfolder).toHaveBeenCalledWith('new-folder')
     expect(controller.renameFolder).toHaveBeenCalledWith('videos')
     expect(controller.refresh).toHaveBeenCalled()
@@ -135,10 +146,32 @@ describe('UploaderView', () => {
     expect(screen.getByText('photo.jpg')).toBeTruthy()
     expect(screen.getByText('clip.mp4')).toBeTruthy()
     expect(screen.getByText('note.pdf')).toBeTruthy()
-  }, 20000)
+  })
+
+  it('renders mobile file pickers and forwards capture/gallery selection', () => {
+    const controller = makeController({
+      mobile: true,
+      sel: [],
+      sortedRows: [],
+      rootDirs: [],
+      videosNeedingDesc: 0,
+    })
+    mockUseUploaderController.mockReturnValue(controller)
+
+    render(<UploaderView />)
+
+    fireEvent.change(screen.getByLabelText('Gallery/Photos'), {
+      target: { files: [makeFile('photo.jpg', 'image/jpeg')] },
+    })
+    fireEvent.change(screen.getByLabelText('Camera (optional)'), {
+      target: { files: [makeFile('clip.mp4', 'video/mp4')] },
+    })
+
+    expect(controller.handleChoose).toHaveBeenCalledTimes(2)
+  })
 
   it('renders the empty-library state', () => {
-    controllerMock.useUploaderController.mockReturnValue(
+    mockUseUploaderController.mockReturnValue(
       makeController({
         lib: '',
         sub: '',
@@ -158,7 +191,7 @@ describe('UploaderView', () => {
   })
 
   it('renders empty destination sections when the library has no children', () => {
-    controllerMock.useUploaderController.mockReturnValue(
+    mockUseUploaderController.mockReturnValue(
       makeController({
         lib: 'alpha',
         sub: '',

frontend/src/api.test.ts

@@ -1,19 +1,34 @@
-import { afterEach, describe, expect, it, vi } from 'vitest'
+import { afterEach, describe, expect, it, jest } from '@jest/globals'
 
 import { api } from './api'
 
+function makeResponse(body: string, status: number, contentType: string) {
+  return {
+    ok: status >= 200 && status < 300,
+    status,
+    headers: {
+      get(name: string) {
+        return name.toLowerCase() === 'content-type' ? contentType : null
+      },
+    },
+    async json() {
+      return JSON.parse(body)
+    },
+    async text() {
+      return body
+    },
+  }
+}
+
 describe('api helper', () => {
   afterEach(() => {
-    vi.restoreAllMocks()
+    jest.restoreAllMocks()
+    ;(globalThis.fetch as jest.Mock | undefined)?.mockReset?.()
   })
 
   it('returns parsed json when content-type is json', async () => {
-    const fetchMock = vi.spyOn(globalThis, 'fetch').mockResolvedValue(
+    const fetchMock = jest.fn(async (..._args: any[]) => makeResponse(JSON.stringify({ ok: true }), 200, 'application/json') as any)
-      new Response(JSON.stringify({ ok: true }), {
+    Object.defineProperty(globalThis, 'fetch', { configurable: true, writable: true, value: fetchMock })
-        status: 200,
-        headers: { 'content-type': 'application/json' },
-      }),
-    )
 
     const res = await api<{ ok: boolean }>('/api/healthz')
     expect(res.ok).toBe(true)
@@ -21,36 +36,24 @@ describe('api helper', () => {
   })
 
   it('parses json from text payload when response header is not json', async () => {
-    vi.spyOn(globalThis, 'fetch').mockResolvedValue(
+    const fetchMock = jest.fn(async (..._args: any[]) => makeResponse('{"value":42}', 200, 'text/plain') as any)
-      new Response('{"value":42}', {
+    Object.defineProperty(globalThis, 'fetch', { configurable: true, writable: true, value: fetchMock })
-        status: 200,
-        headers: { 'content-type': 'text/plain' },
-      }),
-    )
 
     const res = await api<{ value: number }>('/api/text-json')
     expect(res.value).toBe(42)
   })
 
   it('returns raw text when text is not json', async () => {
-    vi.spyOn(globalThis, 'fetch').mockResolvedValue(
+    const fetchMock = jest.fn(async (..._args: any[]) => makeResponse('hello', 200, 'text/plain') as any)
-      new Response('hello', {
+    Object.defineProperty(globalThis, 'fetch', { configurable: true, writable: true, value: fetchMock })
-        status: 200,
-        headers: { 'content-type': 'text/plain' },
-      }),
-    )
 
     const res = await api<string>('/api/text')
     expect(res).toBe('hello')
   })
 
   it('throws server message when response is not ok', async () => {
-    vi.spyOn(globalThis, 'fetch').mockResolvedValue(
+    const fetchMock = jest.fn(async (..._args: any[]) => makeResponse('invalid credentials', 401, 'text/plain') as any)
-      new Response('invalid credentials', {
+    Object.defineProperty(globalThis, 'fetch', { configurable: true, writable: true, value: fetchMock })
-        status: 401,
-        headers: { 'content-type': 'text/plain' },
-      }),
-    )
 
     await expect(api('/api/login')).rejects.toThrow('invalid credentials')
   })

frontend/src/main.test.tsx

@@ -1,39 +1,40 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, jest } from '@jest/globals'
 
-const harness = vi.hoisted(() => {
+const mockRender = jest.fn()
-  const renderMock = vi.fn()
+const mockCreateRoot = jest.fn(() => ({ render: mockRender }))
-  const createRootMock = vi.fn(() => ({ render: renderMock }))
+
-  return { renderMock, createRootMock }
+jest.mock('react-dom/client', () => ({
+  createRoot: mockCreateRoot,
+}))
+
+jest.mock('./App', () => function MockApp() {
+  return null
 })
 
-vi.mock('react-dom/client', () => ({
-  createRoot: harness.createRootMock,
-}))
-
-vi.mock('./App', () => ({
-  default: function MockApp() {
-    return null
-  },
-}))
-
 describe('main entrypoint', () => {
   beforeEach(() => {
-    vi.resetModules()
+    jest.resetModules()
     document.body.innerHTML = '<div id="root"></div>'
-    harness.createRootMock.mockClear()
+    mockCreateRoot.mockClear()
-    harness.renderMock.mockClear()
+    mockRender.mockClear()
   })
 
   it('mounts the app into #root', async () => {
-    await import('./main')
+    await jest.isolateModulesAsync(async () => {
+      await import('./main')
+    })
 
-    expect(harness.createRootMock).toHaveBeenCalled()
+    expect(mockCreateRoot).toHaveBeenCalled()
-    expect(harness.renderMock).toHaveBeenCalled()
+    expect(mockRender).toHaveBeenCalled()
   })
 
   it('fails fast when the root node is missing', async () => {
     document.body.innerHTML = ''
 
-    await expect(import('./main')).rejects.toThrow('Missing <div id="root"></div> in index.html')
+    await expect(
+      jest.isolateModulesAsync(async () => {
+        await import('./main')
+      }),
+    ).rejects.toThrow('Missing <div id="root"></div> in index.html')
   })
 })

frontend/src/test/setup.ts

@@ -1 +1,9 @@
-import '@testing-library/jest-dom/vitest'
+import '@testing-library/jest-dom'
+
+if (!globalThis.fetch) {
+  Object.defineProperty(globalThis, 'fetch', {
+    configurable: true,
+    writable: true,
+    value: jest.fn(),
+  })
+}

frontend/src/test/styleMock.ts

@@ -0,0 +1 @@
+export {}

frontend/src/uploader-controller.test.tsx

@@ -1,73 +1,90 @@
 import { act, render, renderHook, waitFor } from '@testing-library/react'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals'
 
-const harness = vi.hoisted(() => {
+const mockApi = jest.fn() as jest.MockedFunction<(path: string, init?: RequestInit) => Promise<unknown>>
-  const apiMock = vi.fn()
+const mockUploadState = {
-  const uploadState = {
+  mode: 'success' as 'success' | 'error',
-    mode: 'success' as 'success' | 'error',
+  dispatchBeforeUnload: false,
-    dispatchBeforeUnload: false,
+  lastBeforeUnloadEvent: undefined as BeforeUnloadEvent | undefined,
-    lastBeforeUnloadEvent: undefined as BeforeUnloadEvent | undefined,
+  pauseUpload: false,
-    pauseUpload: false,
+  finishUpload: undefined as (() => void) | undefined,
-    finishUpload: undefined as (() => void) | undefined,
+}
+
+const mockTusUpload = class MockTusUpload {
+  opts: any
+  file: File
+
+  constructor(file: File, opts: any) {
+    this.file = file
+    this.opts = opts
   }
 
-  class UploadMock {
+  start() {
-    opts: any
+    if (mockUploadState.mode === 'error') {
-    file: File
+      this.opts.onError?.({ originalRequest: { status: 503, statusText: 'Service Unavailable' } })
-
+      return
-    constructor(file: File, opts: any) {
-      this.file = file
-      this.opts = opts
     }
-
+    if (mockUploadState.pauseUpload) {
-    start() {
+      mockUploadState.finishUpload = () => {
-      if (uploadState.mode === 'error') {
+        this.opts.onProgress?.(5, 10)
-        this.opts.onError?.({ originalRequest: { status: 503, statusText: 'Service Unavailable' } })
+        this.opts.onSuccess?.()
-        return
       }
-      if (uploadState.pauseUpload) {
+      return
-        uploadState.finishUpload = () => {
-          this.opts.onProgress?.(5, 10)
-          this.opts.onSuccess?.()
-        }
-        return
-      }
-      if (uploadState.dispatchBeforeUnload) {
-        const event = new Event('beforeunload', { cancelable: true }) as BeforeUnloadEvent
-        uploadState.lastBeforeUnloadEvent = event
-        window.dispatchEvent(event)
-      }
-      this.opts.onProgress?.(5, 10)
-      this.opts.onSuccess?.()
     }
+    if (mockUploadState.dispatchBeforeUnload) {
+      const event = new Event('beforeunload', { cancelable: true }) as BeforeUnloadEvent
+      mockUploadState.lastBeforeUnloadEvent = event
+      window.dispatchEvent(event)
+    }
+    this.opts.onProgress?.(5, 10)
+    this.opts.onSuccess?.()
   }
+}
 
-  return { apiMock, uploadState, UploadMock }
+jest.mock('./api', () => ({
-})
+  api: mockApi,
-
-vi.mock('./api', () => ({
-  api: harness.apiMock,
 }))
 
-vi.mock('tus-js-client', () => ({
+jest.mock('tus-js-client', () => ({
-  Upload: harness.UploadMock,
+  Upload: mockTusUpload,
 }))
 
 import useUploaderController from './uploader-controller'
 
+const originalAlertDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'alert')
+const originalConfirmDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'confirm')
+const originalPromptDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'prompt')
+const originalLocationDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'location')
+const originalNavigatorDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'navigator')
+
 function makeFile(name: string, type: string) {
   return new File(['x'], name, { type })
 }
 
 function installGlobals() {
-  vi.stubGlobal('alert', vi.fn())
+  Object.defineProperty(globalThis, 'alert', { configurable: true, value: jest.fn() })
-  vi.stubGlobal('confirm', vi.fn(() => true))
+  Object.defineProperty(globalThis, 'confirm', { configurable: true, value: jest.fn(() => true) })
-  vi.stubGlobal('prompt', vi.fn(() => 'renamed'))
+  Object.defineProperty(globalThis, 'prompt', { configurable: true, value: jest.fn(() => 'renamed') })
-  vi.stubGlobal('location', { replace: vi.fn() } as any)
+}
+
+function restoreGlobal(name: string, descriptor: PropertyDescriptor | undefined) {
+  if (descriptor) {
+    Object.defineProperty(globalThis, name, descriptor)
+    return
+  }
+  Reflect.deleteProperty(globalThis as Record<string, unknown>, name)
+}
+
+function restoreGlobals() {
+  restoreGlobal('alert', originalAlertDescriptor)
+  restoreGlobal('confirm', originalConfirmDescriptor)
+  restoreGlobal('prompt', originalPromptDescriptor)
+  restoreGlobal('location', originalLocationDescriptor)
+  restoreGlobal('navigator', originalNavigatorDescriptor)
 }
 
 function installApi() {
-  harness.apiMock.mockImplementation(async (path: string) => {
+  mockApi.mockImplementation(async (path: string) => {
     if (path === '/api/whoami') {
       return { username: 'brad', roots: ['alpha', 'beta'] }
     }
@@ -93,23 +110,24 @@ function installApi() {
 }
 
 beforeEach(() => {
-  harness.apiMock.mockReset()
+  mockApi.mockReset()
-  harness.uploadState.mode = 'success'
+  mockUploadState.mode = 'success'
-  harness.uploadState.dispatchBeforeUnload = false
+  mockUploadState.dispatchBeforeUnload = false
-  harness.uploadState.lastBeforeUnloadEvent = undefined
+  mockUploadState.lastBeforeUnloadEvent = undefined
-  harness.uploadState.pauseUpload = false
+  mockUploadState.pauseUpload = false
-  harness.uploadState.finishUpload = undefined
+  mockUploadState.finishUpload = undefined
   installGlobals()
   installApi()
 })
 
 afterEach(() => {
-  vi.unstubAllGlobals()
+  jest.restoreAllMocks()
+  restoreGlobals()
 })
 
 describe('useUploaderController', () => {
   it('syncs folder input attributes for desktop and mobile UAs', async () => {
-    vi.stubGlobal('navigator', { userAgent: 'iPhone' } as any)
+    Object.defineProperty(globalThis, 'navigator', { configurable: true, value: { userAgent: 'iPhone' } })
 
     function Harness() {
       const controller = useUploaderController()
@@ -218,15 +236,15 @@ describe('useUploaderController', () => {
       await result.current.doUpload()
     })
 
-    expect(harness.apiMock).toHaveBeenCalledWith('/api/mkdir', expect.any(Object))
+    expect(mockApi).toHaveBeenCalledWith('/api/mkdir', expect.any(Object))
-    expect(harness.apiMock).toHaveBeenCalledWith('/api/rename', expect.any(Object))
+    expect(mockApi).toHaveBeenCalledWith('/api/rename', expect.any(Object))
-    expect(harness.apiMock).toHaveBeenCalledWith(expect.stringContaining('/api/file?'), expect.any(Object))
+    expect(mockApi).toHaveBeenCalledWith(expect.stringContaining('/api/file?'), expect.any(Object))
     expect(result.current.sel).toEqual([])
     expect(result.current.status).toContain('Ready')
   })
 
   it('surfaces upload failures and the not-signed-in guard', async () => {
-    harness.uploadState.mode = 'error'
+    mockUploadState.mode = 'error'
     const { result } = renderHook(() => useUploaderController())
 
     await waitFor(() => expect(result.current.libs).toEqual(['alpha', 'beta']))
@@ -245,35 +263,35 @@ describe('useUploaderController', () => {
 
     await waitFor(() => expect(result.current.sel).toHaveLength(1))
 
-    harness.apiMock.mockImplementationOnce(async () => {
+    mockApi.mockImplementationOnce(async () => {
       throw new Error('mkdir failed')
     })
     await act(async () => {
       await result.current.createSubfolder('broken folder')
     })
 
-    harness.apiMock.mockImplementationOnce(async () => {
+    mockApi.mockImplementationOnce(async () => {
       throw new Error('rename folder failed')
     })
     await act(async () => {
       await result.current.renameFolder('videos')
     })
 
-    harness.apiMock.mockImplementationOnce(async () => {
+    mockApi.mockImplementationOnce(async () => {
       throw new Error('delete folder failed')
     })
     await act(async () => {
       await result.current.deleteFolder('archive')
     })
 
-    harness.apiMock.mockImplementationOnce(async () => {
+    mockApi.mockImplementationOnce(async () => {
       throw new Error('rename failed')
     })
     await act(async () => {
       await result.current.renamePath('clip.mp4')
     })
 
-    harness.apiMock.mockImplementationOnce(async () => {
+    mockApi.mockImplementationOnce(async () => {
       throw new Error('delete failed')
     })
     await act(async () => {
@@ -288,7 +306,7 @@ describe('useUploaderController', () => {
   })
 
   it('surfaces profile errors and logs out on missing mappings', async () => {
-    harness.apiMock.mockImplementation(async (path: string) => {
+    mockApi.mockImplementation(async (path: string) => {
       if (path === '/api/whoami') {
         throw new Error('no mapping found')
       }
@@ -307,12 +325,11 @@ describe('useUploaderController', () => {
     expect((globalThis as any).alert).toHaveBeenCalledWith(
       'Your account is not linked to any upload library yet. Please contact the admin to be granted access.'
     )
-    expect((globalThis as any).location.replace).toHaveBeenCalledWith('/')
+    expect(mockApi).toHaveBeenCalledWith('/api/logout', { method: 'POST' })
-    expect(harness.apiMock).toHaveBeenCalledWith('/api/logout', { method: 'POST' })
   })
 
   it('blocks unload while an upload is in flight', async () => {
-    harness.uploadState.pauseUpload = true
+    mockUploadState.pauseUpload = true
     const { result } = renderHook(() => useUploaderController())
 
     await waitFor(() => expect(result.current.libs).toEqual(['alpha', 'beta']))
@@ -330,7 +347,7 @@ describe('useUploaderController', () => {
     await waitFor(() => expect(result.current.uploading).toBe(true))
 
     const event = new Event('beforeunload', { cancelable: true }) as BeforeUnloadEvent
-    const preventDefault = vi.spyOn(event, 'preventDefault')
+    const preventDefault = jest.spyOn(event, 'preventDefault')
     await act(async () => {
       window.dispatchEvent(event)
     })
@@ -339,7 +356,7 @@ describe('useUploaderController', () => {
     expect(event.defaultPrevented).toBe(true)
 
     await act(async () => {
-      harness.uploadState.finishUpload?.()
+      mockUploadState.finishUpload?.()
       await uploadPromise
     })
 

frontend/src/uploader-controller.ts

@@ -123,7 +123,11 @@ export default function useUploaderController(): ControllerState {
           } catch {
             // Best-effort logout cleanup only.
           }
-          location.replace('/')
+          try {
+            location.replace('/')
+          } catch {
+            // JSDOM can block navigation APIs; browsers still redirect normally.
+          }
         }
       }
     })()

frontend/tsconfig.jest.json

@@ -0,0 +1,12 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "esModuleInterop": true,
+    "allowSyntheticDefaultImports": true,
+    "isolatedModules": true,
+    "module": "commonjs",
+    "moduleResolution": "node",
+    "noImplicitAny": false,
+    "types": ["jest", "node", "@testing-library/jest-dom"]
+  }
+}

frontend/tsconfig.json

@@ -9,7 +9,7 @@
     "noEmit": true,
     "skipLibCheck": true,
     "resolveJsonModule": true,
-    "types": ["vite/client"]
+    "types": ["vite/client", "jest", "@testing-library/jest-dom", "node"]
   },
   "include": ["src", "vite-env.d.ts"]
 }

frontend/vite.config.ts

@@ -1,5 +1,5 @@
 // frontend/vite.config.ts
-import { defineConfig } from 'vitest/config'
+import { defineConfig } from 'vite'
 import react from '@vitejs/plugin-react'
 
 export default defineConfig({
@@ -8,11 +8,4 @@ export default defineConfig({
     outDir: 'dist',
     emptyOutDir: true,
   },
-  test: {
-    environment: 'jsdom',
-    globals: true,
-    setupFiles: ['./src/test/setup.ts'],
-    css: true,
-    include: ['src/**/*.test.ts', 'src/**/*.test.tsx'],
-  },
 })

scripts/publish_test_metrics.py

@@ -74,9 +74,9 @@ def _load_junit(path: Path) -> dict[str, int]:
 
 
 def _load_junit_cases(path: Path) -> list[tuple[str, str]]:
-    """Return per-test outcomes from a JUnit report."""
     if not path.exists():
         return []
+
     tree = ET.parse(path)
     root = tree.getroot()
     suites: list[ET.Element]
@@ -356,23 +356,14 @@ def main() -> int:
         "# TYPE pegasus_quality_gate_build_info gauge",
         f"pegasus_quality_gate_build_info{_label_str(labels)} 1",
     ]
+    payload_lines.extend(
+        f'platform_quality_gate_test_case_result{{suite="{suite}",test="{_escape_label(test_name)}",status="{_escape_label(test_status)}"}} 1'
+        for test_name, test_status in test_cases
+    )
     payload_lines.extend(
         f'pegasus_quality_gate_checks_total{{suite="{suite}",check="{check_name}",result="{check_status}"}} 1'
         for check_name, check_status in checks.items()
     )
-    if test_cases:
-        payload_lines.extend(
-            [
-                *[
-                    f'platform_quality_gate_test_case_result{{suite="{suite}",test="{_escape_label(test_name)}",status="{_escape_label(test_status)}"}} 1'
-                    for test_name, test_status in test_cases
-                ],
-            ]
-        )
-    else:
-        payload_lines.append(
-            f'platform_quality_gate_test_case_result{{suite="{suite}",test="__no_test_cases__",status="skipped"}} 1'
-        )
     payload = "\n".join(payload_lines) + "\n"
 
     push_url = f"{pushgateway_url.rstrip('/')}/metrics/job/{job_name}/suite/{suite}"