chore: Should keep trigger didUpdate with Portal render

Author: zombieJ

src/Portal.tsx

@@ -11,7 +11,6 @@ import canUseDom from './Dom/canUseDom';
 export type PortalRef = {};
 
 export interface PortalProps {
-  /** @deprecated Not know who use this? */
   didUpdate?: (prevProps: PortalProps) => void;
   getContainer: () => HTMLElement;
   children?: React.ReactNode;
@@ -32,10 +31,12 @@ const Portal = forwardRef<PortalRef, PortalProps>((props, ref) => {
     initRef.current = true;
   }
 
+  // [Legacy] Used by `rc-trigger`
   useEffect(() => {
-    // Not know who use this. Just keep it here
     didUpdate?.(props);
+  });
 
+  useEffect(() => {
     return () => {
       if (containerRef.current) {
         containerRef.current.parentNode.removeChild(containerRef.current);

tests/Portal.test.tsx

@@ -1,6 +1,7 @@
 import React from 'react';
 import { mount } from 'enzyme';
 import PortalWrapper from '../src/PortalWrapper';
+import Portal from '../src/Portal';
 
 describe('Portal', () => {
   let container: HTMLDivElement;
@@ -27,4 +28,22 @@ describe('Portal', () => {
 
     wrapper.unmount();
   });
+
+  it('didUpdate', () => {
+    const didUpdate = jest.fn();
+
+    const wrapper = mount(
+      <Portal
+        didUpdate={didUpdate}
+        getContainer={() => document.createElement('div')}
+      >
+        light
+      </Portal>,
+    );
+
+    expect(didUpdate).toHaveBeenCalledTimes(1);
+
+    wrapper.setProps({ justForceUpdate: true });
+    expect(didUpdate).toHaveBeenCalledTimes(2);
+  });
 });