chore: add tests

Author: VladaHarbour

packages/super-editor/src/editors/v1/core/presentation-editor/PresentationEditor.ts

@@ -6512,6 +6512,13 @@ export class PresentationEditor extends EventEmitter {
       }
     }
     if (dom && geometry) {
+      const samePage = dom.pageIndex === geometry.pageIndex;
+      const dx = Math.abs(dom.x - geometry.x);
+      const dy = Math.abs(dom.y - geometry.y);
+      const DOM_GEOMETRY_MISMATCH_PX = 24;
+      if (!samePage || dx > DOM_GEOMETRY_MISMATCH_PX || dy > DOM_GEOMETRY_MISMATCH_PX) {
+        return geometry;
+      }
       return {
         pageIndex: dom.pageIndex,
         x: dom.x,

packages/super-editor/src/editors/v1/core/presentation-editor/tests/PresentationEditor.test.ts

@@ -4,6 +4,8 @@ import { PresentationEditor } from '../PresentationEditor.js';
 import type { Editor as EditorInstance } from '../../Editor.js';
 import { Editor } from '../../Editor.js';
 import { HeaderFooterEditorManager, HeaderFooterLayoutAdapter } from '../../header-footer/HeaderFooterRegistry.js';
+import * as CaretGeometry from '../selection/CaretGeometry.js';
+import * as DomSelectionGeometry from '../../../dom-observer/DomSelectionGeometry.js';
 
 type MockedEditor = Mock<(...args: unknown[]) => EditorInstance> & {
   mock: {
@@ -3785,6 +3787,68 @@ describe('PresentationEditor', () => {
       });
     });
 
+    describe('#computeCaretLayoutRect DOM/geometry mismatch fallback', () => {
+      it('uses DOM caret when DOM and geometry are close', () => {
+        const geometrySpy = vi
+          .spyOn(CaretGeometry, 'computeCaretLayoutRectGeometry')
+          .mockReturnValue({ pageIndex: 0, x: 100, y: 200, height: 18 });
+        const domSpy = vi
+          .spyOn(DomSelectionGeometry, 'computeDomCaretPageLocal')
+          .mockReturnValue({ pageIndex: 0, x: 108, y: 210 });
+
+        editor = new PresentationEditor({
+          element: container,
+          documentId: 'test-doc',
+        });
+
+        const caret = editor.computeCaretLayoutRect(5);
+        expect(caret).toEqual({ pageIndex: 0, x: 108, y: 210, height: 18 });
+
+        geometrySpy.mockRestore();
+        domSpy.mockRestore();
+      });
+
+      it('falls back to geometry when DOM and geometry differ significantly', () => {
+        const geometrySpy = vi
+          .spyOn(CaretGeometry, 'computeCaretLayoutRectGeometry')
+          .mockReturnValue({ pageIndex: 0, x: 100, y: 200, height: 18 });
+        const domSpy = vi
+          .spyOn(DomSelectionGeometry, 'computeDomCaretPageLocal')
+          .mockReturnValue({ pageIndex: 0, x: 180, y: 290 });
+
+        editor = new PresentationEditor({
+          element: container,
+          documentId: 'test-doc',
+        });
+
+        const caret = editor.computeCaretLayoutRect(5);
+        expect(caret).toEqual({ pageIndex: 0, x: 100, y: 200, height: 18 });
+
+        geometrySpy.mockRestore();
+        domSpy.mockRestore();
+      });
+
+      it('falls back to geometry when DOM and geometry point to different pages', () => {
+        const geometrySpy = vi
+          .spyOn(CaretGeometry, 'computeCaretLayoutRectGeometry')
+          .mockReturnValue({ pageIndex: 1, x: 120, y: 220, height: 16 });
+        const domSpy = vi
+          .spyOn(DomSelectionGeometry, 'computeDomCaretPageLocal')
+          .mockReturnValue({ pageIndex: 0, x: 120, y: 220 });
+
+        editor = new PresentationEditor({
+          element: container,
+          documentId: 'test-doc',
+        });
+
+        const caret = editor.computeCaretLayoutRect(5);
+        expect(caret).toEqual({ pageIndex: 1, x: 120, y: 220, height: 16 });
+
+        geometrySpy.mockRestore();
+        domSpy.mockRestore();
+      });
+    });
+
     describe('#updateSelection DOM manipulation error handling', () => {
       it('should handle DOM errors when clearing selection in viewing mode', () => {
         editor = new PresentationEditor({