Phase 2: Add component tests for all 8 shared frontend components

[Copilot]

• Phase 1: Vitest test infrastructure (merged in PR PR 1: Add Vitest test infrastructure for frontend #409)
• Phase 2: Component tests for src/components/ — 64 tests, all passing, 0 errors
  • src/__mocks__/render.jsx — manual mock (explicit vi import)
  • src/test/setup.js — fixed: getClientRects/getBoundingClientRect stubs are now plain functions, not vi.fn(), so vi.clearAllMocks() in beforeEach doesn't reset them (was causing 1 unhandled error in CI)
  • RequiredTextField.test.jsx
  • ThemeSwitcher.test.jsx
  • BottomDrawer.test.jsx
  • FileUpload.test.jsx
  • ImageUpload.test.jsx
  • MenuBar.test.jsx
  • ContentEditor.test.jsx
  • Base.test.jsx

[payamnj]

@copilot the frontend test failed

[Copilot]

@copilot the frontend test failed

Fixed in commit 3bdb6a1. The root cause was that the getClientRects/getBoundingClientRect stubs in setup.js were created with vi.fn(), so vi.clearAllMocks() in beforeEach wiped their implementations after the first test — leaving ProseMirror's scroll logic calling undefined() and throwing an unhandled error (exit code 1).

Changed those stubs to plain functions so clearAllMocks leaves them intact. All 64 tests now pass with 0 errors.