fix: [Tooltip] modify delay visible logic in componentDidUpdate (#1402)

* fix: [Tooltip] modify delay visible logic in componentDidUpdate
* fix: remove broken snapShot test

---------

Co-authored-by: Lion <liangbj01.yuanfudao.com>
Co-authored-by: pointhalo <[email protected]>

packages/semi-ui/datePicker/__test__/datePicker.test.js

@@ -58,6 +58,7 @@ describe(`DatePicker`, () => {
 
         const elem = mount(<DatePicker motion={motion} defaultOpen={open} defaultValue={defaultValue} />);
 
+        await sleep();
         expect(document.querySelectorAll(popupSelector).length).toBe(1);
 
         // document.body.click();

packages/semi-ui/slider/__test__/__snapshots__/slider.test.js.snap

@@ -1,5 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`Slider should show tooltip when hovering slider handler 1`] = `null`;
-
-exports[`Slider should show tooltip when hovering slider handler 2`] = `null`;

packages/semi-ui/slider/__test__/slider.test.js

@@ -86,19 +86,20 @@ describe('Slider', () => {
     //     expect(wrapper.state('currentValue')).toBe(54);
     // });
 
-    it('should show tooltip when hovering slider handler', () => {
-        const wrapper = mount(<Slider defaultValue={30} />);
-        wrapper
-            .find(`.${BASE_CLASS_PREFIX}-slider-handle`)
-            .at(0)
-            .simulate('mouseEnter');
-        expect(render(wrapper.find(`.${BASE_CLASS_PREFIX}-tooltip-wrapper`))).toMatchSnapshot();
-        wrapper
-            .find(`.${BASE_CLASS_PREFIX}-slider-handle`)
-            .at(0)
-            .simulate('mouseLeave');
-        expect(render(wrapper.find(`.${BASE_CLASS_PREFIX}-tooltip-wrapper`))).toMatchSnapshot();
-    });
+    // Cannot test snapShot becase of random generated wrapperId in tooltip
+    // it('should show tooltip when hovering slider handler', () => {
+    //     const wrapper = mount(<Slider defaultValue={30} />);
+    //     wrapper
+    //         .find(`.${BASE_CLASS_PREFIX}-slider-handle`)
+    //         .at(0)
+    //         .simulate('mouseEnter');
+    //     expect(render(wrapper.find(`.${BASE_CLASS_PREFIX}-tooltip-wrapper`))).toMatchSnapshot();
+    //     wrapper
+    //         .find(`.${BASE_CLASS_PREFIX}-slider-handle`)
+    //         .at(0)
+    //         .simulate('mouseLeave');
+    //     expect(render(wrapper.find(`.${BASE_CLASS_PREFIX}-tooltip-wrapper`))).toMatchSnapshot();
+    // });
 
     it('when hover into slider', () => {
         let wrapper = mount(<Slider showBoundary={true} />);

packages/semi-ui/tooltip/index.tsx

@@ -496,7 +496,11 @@ export default class Tooltip extends BaseComponent<TooltipProps, TooltipState> {
             "[Semi Tooltip] 'mouseLeaveDelay' cannot be less than 'mouseEnterDelay', which may cause the dropdown layer to not be hidden."
         );
         if (prevProps.visible !== this.props.visible) {
-            this.props.visible ? this.foundation.delayShow() : this.foundation.delayHide();
+            if (["hover", "focus"].includes(this.props.trigger)) {
+                this.props.visible ? this.foundation.delayShow() : this.foundation.delayHide();
+            } else {
+                this.props.visible ? this.foundation.show() : this.foundation.hide();
+            }
         }
         if (!isEqual(prevProps.rePosKey, this.props.rePosKey)) {
             this.rePosition();