update

Author: li-jia-nan

tests/arrow.test.jsx

@@ -6,6 +6,7 @@ import {
   spyElementPrototypes,
 } from '@rc-component/util/lib/test/domHook';
 import Trigger from '../src';
+import Arrow from '../src/Popup/Arrow';
 
 describe('Trigger.Arrow', () => {
   beforeAll(() => {
@@ -42,6 +43,22 @@ describe('Trigger.Arrow', () => {
     );
   });
 
+  it('uses default arrow options and position', () => {
+    render(
+      <Arrow
+        prefixCls="rc-trigger-popup"
+        align={{ points: ['tl', 'bl'] }}
+        arrow={undefined}
+        arrowPos={{}}
+      />,
+    );
+
+    expect(document.querySelector('.rc-trigger-popup-arrow')).toHaveStyle({
+      left: 0,
+      top: 0,
+    });
+  });
+
   describe('direction', () => {
     let divSpy;
     let windowSpy;

tests/basic.test.jsx

@@ -298,6 +298,67 @@ describe('Trigger.Basic', () => {
         expect(isPopupHidden()).toBeTruthy();
       });
 
+      it('cancels refresh close detection when mouse returns to safe hover area', () => {
+        const { container } = render(
+          <Trigger
+            action={['hover']}
+            mouseLeaveDelay={0.1}
+            popup={<strong>trigger</strong>}
+          >
+            <div className="target">hover</div>
+          </Trigger>,
+        );
+
+        const target = container.querySelector('.target');
+
+        fireEvent.mouseEnter(target, { clientX: 50, clientY: 10 });
+        act(() => jest.runAllTimers());
+
+        const popup = document.querySelector('.rc-trigger-popup');
+
+        mockRect(target, { left: 0, top: 0, width: 100, height: 20 });
+        mockRect(popup, { left: 20, top: 60, width: 60, height: 30 });
+
+        fireEvent.mouseLeave(target, { clientX: 50, clientY: 20 });
+        fireEvent.mouseMove(document, { clientX: 150, clientY: 40 });
+        fireEvent.mouseMove(document, { clientX: 50, clientY: 40 });
+
+        act(() => jest.advanceTimersByTime(100));
+
+        expect(isPopupHidden()).toBeFalsy();
+      });
+
+      it('keeps pending refresh close detection while mouse remains unsafe', () => {
+        const { container } = render(
+          <Trigger
+            action={['hover']}
+            mouseLeaveDelay={0.1}
+            popup={<strong>trigger</strong>}
+          >
+            <div className="target">hover</div>
+          </Trigger>,
+        );
+
+        const target = container.querySelector('.target');
+
+        fireEvent.mouseEnter(target, { clientX: 50, clientY: 10 });
+        act(() => jest.runAllTimers());
+
+        const popup = document.querySelector('.rc-trigger-popup');
+
+        mockRect(target, { left: 0, top: 0, width: 100, height: 20 });
+        mockRect(popup, { left: 20, top: 60, width: 60, height: 30 });
+
+        fireEvent.mouseLeave(target, { clientX: 50, clientY: 20 });
+        fireEvent.mouseMove(document, { clientX: 50, clientY: 150 });
+        fireEvent.mouseMove(document, { clientX: 50, clientY: 150 });
+
+        mockRect(popup, { left: 20, top: 60, width: 60, height: 200 });
+        act(() => jest.advanceTimersByTime(100));
+
+        expect(isPopupHidden()).toBeFalsy();
+      });
+
       it('waits for mousemove to start refresh close detection after safe hover area disappears', () => {
         const { container } = render(
           <Trigger

tests/flip.test.tsx

@@ -4,7 +4,7 @@ import { spyElementPrototypes } from '@rc-component/util/lib/test/domHook';
 import * as React from 'react';
 import type { AlignType, TriggerProps } from '../src';
 import Trigger from '../src';
-import { getVisibleArea } from '../src/util';
+import { collectScroller, getVisibleArea } from '../src/util';
 
 const flush = async () => {
   for (let i = 0; i < 10; i += 1) {
@@ -366,6 +366,47 @@ describe('Trigger.Align', () => {
     window.getComputedStyle = oriGetComputedStyle;
   });
 
+  it('visible area skips document body and html elements', () => {
+    const initArea = {
+      left: 0,
+      right: 500,
+      top: 0,
+      bottom: 500,
+    };
+
+    expect(getVisibleArea(initArea)).toEqual(initArea);
+    expect(
+      getVisibleArea(initArea, [document.body, document.documentElement]),
+    ).toEqual(initArea);
+  });
+
+  it('handles elements without a document window', () => {
+    const detachedDocument = document.implementation.createHTMLDocument();
+    const scroller = detachedDocument.createElement('div');
+    const child = detachedDocument.createElement('div');
+
+    detachedDocument.body.appendChild(scroller);
+    scroller.appendChild(child);
+
+    expect(collectScroller(child)).toEqual([]);
+    expect(
+      getVisibleArea(
+        {
+          left: 0,
+          right: 500,
+          top: 0,
+          bottom: 500,
+        },
+        [scroller],
+      ),
+    ).toEqual({
+      left: 0,
+      right: 100,
+      top: 0,
+      bottom: 100,
+    });
+  });
+
   // e.g. adjustY + shiftX may make popup out but push back in screen
   // which should keep flip
   /*

tests/safeHover.test.ts

@@ -1,4 +1,5 @@
 import {
+  getSafeHoverAreaPolygons,
   getSafeHoverSide,
   isPointInSafeHoverArea,
   type SafeHoverRect,
@@ -138,4 +139,22 @@ describe('safeHover util', () => {
       ),
     ).toBe(true);
   });
+
+  it('supports custom safe hover buffer', () => {
+    expect(
+      getSafeHoverAreaPolygons(
+        [50, 20],
+        rect(0, 0, 100, 20),
+        rect(20, 60, 60, 30),
+      ),
+    ).toHaveLength(2);
+    expect(
+      getSafeHoverAreaPolygons(
+        [50, 20],
+        rect(0, 0, 100, 20),
+        rect(20, 60, 60, 30),
+        4,
+      ),
+    ).toHaveLength(2);
+  });
 });

tests/useOffsetStyle.test.tsx

@@ -0,0 +1,29 @@
+import { renderHook } from '@testing-library/react';
+import useOffsetStyle from '../src/hooks/useOffsetStyle';
+
+describe('useOffsetStyle', () => {
+  it('uses right and bottom offsets with dynamic inset alignment', () => {
+    const { result } = renderHook(() =>
+      useOffsetStyle(
+        false,
+        true,
+        true,
+        {
+          points: ['br', 'tr'],
+          dynamicInset: true,
+        },
+        12,
+        34,
+        56,
+        78,
+      ),
+    );
+
+    expect(result.current).toEqual({
+      left: 'auto',
+      top: 'auto',
+      right: 12,
+      bottom: 34,
+    });
+  });
+});