fix: do not forward disabled to trigger

Author: zombieJ

src/Dropdown.tsx

@@ -65,8 +65,9 @@ const Dropdown = React.forwardRef<TriggerRef, DropdownProps>((props, ref) => {
     overlay,
     children,
     onVisibleChange,
+    disabled,
     ...otherProps
-  } = props;
+  } = props as DropdownProps & { disabled?: boolean };
 
   const [triggerVisible, setTriggerVisible] = React.useState<boolean>();
   const mergedVisible = 'visible' in props ? visible : triggerVisible;

tests/props.test.tsx

@@ -0,0 +1,33 @@
+import { render } from '@testing-library/react';
+import * as React from 'react';
+import Dropdown from '../src';
+
+const mockTriggerRender = jest.fn();
+
+jest.mock('@rc-component/trigger', () => {
+  const ReactModule: typeof React = jest.requireActual('react');
+
+  return {
+    __esModule: true,
+    default: ReactModule.forwardRef<HTMLElement, Record<string, unknown>>(
+      (props, ref) => {
+        mockTriggerRender(props);
+        return ReactModule.createElement('div', { ref });
+      },
+    ),
+  };
+});
+
+it('does not forward disabled to Trigger', () => {
+  const runtimeProps = { disabled: true };
+
+  render(
+    <Dropdown {...runtimeProps} visible overlay={<div />}>
+      <button type="button">open</button>
+    </Dropdown>,
+  );
+
+  const triggerProps = mockTriggerRender.mock.calls[0][0];
+  expect(triggerProps).not.toHaveProperty('disabled');
+  expect(triggerProps).toHaveProperty('popupVisible', true);
+});