fix: explicit vi import in render mock; patch Text/Range.getClientRects for ProseMirror

Agent-Logs-Url: https://github.com/AvaCodeSolutions/django-email-learning/sessions/441cafc3-06cd-4339-aade-0b3dd794af20

Co-authored-by: payamnj <11951509+payamnj@users.noreply.github.com>

Author: Copilot

frontend/src/__mocks__/render.jsx

@@ -1,5 +1,7 @@
 // Manual mock for src/render.jsx
 // Re-exports useAppContext backed by the test AppContext so components
 // rendered via renderWithProviders(…) receive the correct context values.
+import { vi } from 'vitest';
+
 export { useAppContext } from '../test/test-utils.jsx';
 export default vi.fn();

frontend/src/test/setup.js

@@ -61,7 +61,16 @@ beforeEach(() => {
   global.fetch.mockResolvedValue({ ok: true, json: () => Promise.resolve({}) });
 });
 
-// ProseMirror calls Element.getClientRects() when scrolling to selection.
-// jsdom does not implement this method, which causes unhandled errors during
-// ContentEditor tests.  Provide a minimal stub so ProseMirror exits cleanly.
-Element.prototype.getClientRects = vi.fn(() => []);
+// ProseMirror's scroll-to-selection requires getClientRects / getBoundingClientRect
+// on Text nodes and Range objects — neither is implemented by jsdom.
+// Provide minimal stubs so the EditorView exits cleanly instead of throwing.
+const _emptyRects = [];
+if (typeof Text !== 'undefined') {
+  Text.prototype.getClientRects = vi.fn(() => _emptyRects);
+  Text.prototype.getBoundingClientRect = vi.fn(() => new DOMRect(0, 0, 0, 0));
+}
+if (typeof Range !== 'undefined' && !Range.prototype.getClientRects) {
+  Range.prototype.getClientRects = vi.fn(() => _emptyRects);
+}
+// Also patch Element in case jsdom's own stub is absent in some test envs
+Element.prototype.getClientRects = vi.fn(() => _emptyRects);