fix: refactor scroll drag event handling and test helper

Author: QdabuliuQ

src/hooks/useScrollDrag.ts

@@ -58,14 +58,6 @@ export default function useScrollDrag(
         }
       };
 
-      const onMouseUp = () => {
-        clearDragState();
-      };
-      
-      const onDragStart = () => {
-        clearDragState();
-      };
-
       const onMouseMove = (e: MouseEvent) => {
         if (mouseDownLock) {
           const mouseY = getPageXY(e, false);
@@ -86,18 +78,13 @@ export default function useScrollDrag(
       };
 
       ele.addEventListener('mousedown', onMouseDown);
-      ele.ownerDocument.addEventListener('mouseup', onMouseUp);
+      ele.ownerDocument.addEventListener('mouseup', clearDragState);
       ele.ownerDocument.addEventListener('mousemove', onMouseMove);
-      
-      ele.ownerDocument.addEventListener('dragstart', onDragStart);
-      ele.ownerDocument.addEventListener('dragend', clearDragState);
 
       return () => {
         ele.removeEventListener('mousedown', onMouseDown);
-        ele.ownerDocument.removeEventListener('mouseup', onMouseUp);
+        ele.ownerDocument.removeEventListener('mouseup', clearDragState);
         ele.ownerDocument.removeEventListener('mousemove', onMouseMove);
-        ele.ownerDocument.removeEventListener('dragstart', onDragStart);
-        ele.ownerDocument.removeEventListener('dragend', clearDragState);
         stopScroll();
       };
     }

tests/scroll.test.js

@@ -734,6 +734,15 @@ describe('List.Scroll', () => {
 
   it('should not scroll after drop table text', () => {
 
+    // Helper function to select text content of an element
+    const selectElementText = (element) => {
+      const range = document.createRange();
+      range.selectNodeContents(element);
+      const selection = window.getSelection();
+      selection.removeAllRanges();
+      selection.addRange(range);
+    };
+
     const onScroll = jest.fn();
     const onDragStart = jest.fn();
     const onDragEnd = jest.fn();
@@ -755,19 +764,11 @@ describe('List.Scroll', () => {
     const fixedItems = container.querySelectorAll('.fixed-item');
     const targetItem = fixedItems[0];
     if (targetItem) {
-      const range = document.createRange();
-      range.selectNodeContents(targetItem);
-      const selection = window.getSelection();
-      selection.removeAllRanges();
-      selection.addRange(range);
+      selectElementText(targetItem);
     }
     const listHolder = container.querySelector('.rc-virtual-list-holder');
     if (targetItem && listHolder) {
-      const range = document.createRange();
-      range.selectNodeContents(targetItem);
-      const selection = window.getSelection();
-      selection.removeAllRanges();
-      selection.addRange(range);
+      selectElementText(targetItem);
 
       fireEvent.dragStart(targetItem, { bubbles: true });