fix(test): align context menu tests with current paste and cursor behavior

- ContextMenu: test now expects moveCursorToMouseEvent NOT to be called
  when right-clicking outside a range selection (code preserves range
  selection to avoid presentation mode hit-testing issues)
- menuItems paste: test reflects that text branch prefers pasteText over
  pasteHTML and passes empty html to handleClipboardPaste for URL detection
- menuItems fallback: test reflects that no insertContent fallback exists
  when pasteText is unavailable in the text branch

Author: caio-pizzol

packages/super-editor/src/components/context-menu/tests/ContextMenu.test.js

@@ -294,7 +294,10 @@ describe('ContextMenu.vue', () => {
       expect(moveCursorToMouseEvent).not.toHaveBeenCalled();
     });
 
-    it('should move cursor when right-click happens outside the active selection', async () => {
+    it('should not move cursor when right-click happens outside a range selection', async () => {
+      // When a range selection exists, the context menu preserves it — coordinate-based
+      // hit testing in presentation mode can misreport whether the click is inside the
+      // selection, which would collapse it unexpectedly.
       mount(ContextMenu, { props: mockProps });
 
       const { moveCursorToMouseEvent } = await import('../../cursor-helpers.js');
@@ -313,7 +316,7 @@ describe('ContextMenu.vue', () => {
 
       await contextMenuHandler(rightClickEvent);
 
-      expect(moveCursorToMouseEvent).toHaveBeenCalledWith(rightClickEvent, mockEditor);
+      expect(moveCursorToMouseEvent).not.toHaveBeenCalled();
     });
 
     it('should allow native context menu when modifier is pressed', async () => {

packages/super-editor/src/components/context-menu/tests/menuItems.test.js

@@ -567,12 +567,16 @@ describe('menuItems.js', () => {
   });
 
   describe('getItems - paste action behavior', () => {
-    it('should not force plain-text insert when HTML paste is unhandled', async () => {
+    it('should prefer pasteText over pasteHTML when both html and text are available', async () => {
+      // When plain text is available, the paste action prefers view.pasteText
+      // because Chromium wraps writeText() output in HTML, which routes through
+      // handleHtmlPaste and strips leading/trailing spaces.
       const insertContent = vi.fn();
       mockEditor = createMockEditor({
         commands: { insertContent },
       });
       mockEditor.view.dom.focus = vi.fn();
+      mockEditor.view.pasteText = vi.fn();
       mockEditor.view.pasteHTML = vi.fn();
       mockContext = createMockContext({
         editor: mockEditor,
@@ -593,12 +597,14 @@ describe('menuItems.js', () => {
       expect(pasteAction).toBeTypeOf('function');
       await pasteAction(mockEditor);
 
+      // Passes empty html to handleClipboardPaste for URL detection only
       expect(clipboardMocks.handleClipboardPaste).toHaveBeenCalledWith(
         { editor: mockEditor, view: mockEditor.view },
-        '<p>word html</p>',
+        '',
         'word html',
       );
-      expect(mockEditor.view.pasteHTML).toHaveBeenCalledWith('<p>word html</p>', expect.any(Object));
+      expect(mockEditor.view.pasteText).toHaveBeenCalledWith('word html', expect.any(Object));
+      expect(mockEditor.view.pasteHTML).not.toHaveBeenCalled();
       expect(insertContent).not.toHaveBeenCalled();
     });
 
@@ -631,11 +637,15 @@ describe('menuItems.js', () => {
       expect(insertContent).not.toHaveBeenCalled();
     });
 
-    it('should fall back to insertContent when view has no pasteHTML or pasteText', async () => {
+    it('should not paste when view has no pasteText and handleClipboardPaste returns false', async () => {
+      // When pasteText is unavailable and URL detection returns false,
+      // the text branch has no further action. The code does not fall
+      // back to insertContent for the text branch.
       const insertContent = vi.fn();
       mockEditor = createMockEditor({
         commands: { insertContent },
       });
+      mockEditor.view.dom.focus = vi.fn();
       // No pasteHTML or pasteText on view
       delete mockEditor.view.pasteHTML;
       delete mockEditor.view.pasteText;
@@ -657,7 +667,13 @@ describe('menuItems.js', () => {
 
       await pasteAction(mockEditor);
 
-      expect(insertContent).toHaveBeenCalledWith('fallback text', { contentType: 'text' });
+      expect(clipboardMocks.handleClipboardPaste).toHaveBeenCalledWith(
+        { editor: mockEditor, view: mockEditor.view },
+        '',
+        'fallback text',
+      );
+      // No pasteText available, URL detection returned false — nothing else happens
+      expect(insertContent).not.toHaveBeenCalled();
     });
   });