fix: global duration not work(Also includes some other attributes) (#365)

* fix: some properties changed by useState are not work

* test: add test

Author: yellowryan

src/hooks/useNotification.tsx

@@ -107,11 +107,19 @@ export default function useNotification(
 
   const [taskQueue, setTaskQueue] = React.useState<Task[]>([]);
 
+  // memorized shareConfig
+  const { closeIcon, closable, duration, showProgress, pauseOnHover } = shareConfig;
+  const memoShareConfig = React.useMemo<typeof shareConfig>(() => {
+    return {
+      ...shareConfig,
+    };
+  }, [closeIcon, closable, duration, showProgress, pauseOnHover]);
+
   // ========================= Refs =========================
   const api = React.useMemo<NotificationAPI>(() => {
     return {
       open: (config) => {
-        const mergedConfig = mergeConfig(shareConfig, config);
+        const mergedConfig = mergeConfig(memoShareConfig, config);
         if (mergedConfig.key === null || mergedConfig.key === undefined) {
           mergedConfig.key = `rc-notification-${uniqueKey}`;
           uniqueKey += 1;
@@ -126,7 +134,7 @@ export default function useNotification(
         setTaskQueue((queue) => [...queue, { type: 'destroy' }]);
       },
     };
-  }, []);
+  }, [memoShareConfig]);
 
   // ======================= Container ======================
   // React 18 should all in effect that we will check container in each render

tests/index.test.tsx

@@ -849,4 +849,47 @@ describe('Notification.Basic', () => {
       expect(document.querySelector('.rc-notification-notice-progress')).toBeFalsy();
     });
   });
+
+  describe('Modifying properties through useState can take effect', () => {
+    it('should show notification and disappear after 5 seconds', async () => {
+      const Demo: React.FC = () => {
+        const [duration, setDuration] = React.useState(0);
+        const [api, holder] = useNotification({ duration });
+
+        return (
+          <>
+            <button data-testid="change-duration" onClick={() => setDuration(5)}>
+              change duration
+            </button>
+            <button
+              data-testid="show-notification"
+              onClick={() => {
+                api.open({
+                  content: `Test Notification`,
+                });
+              }}
+            >
+              show notification
+            </button>
+            {holder}
+          </>
+        );
+      };
+
+      const { getByTestId } = render(<Demo />);
+
+      fireEvent.click(getByTestId('show-notification'));
+
+      expect(document.querySelectorAll('.rc-notification-notice').length).toBe(1);
+      fireEvent.click(getByTestId('change-duration'));
+      fireEvent.click(getByTestId('show-notification'));
+      expect(document.querySelectorAll('.rc-notification-notice').length).toBe(2);
+
+      act(() => {
+        vi.advanceTimersByTime(5000);
+      });
+
+      expect(document.querySelectorAll('.rc-notification-notice').length).toBe(1);
+    });
+  });
 });