[material-ui][Chip] Fix handling on event handlers (mui#46263)

Author: sai6855

packages/mui-material/src/Chip/Chip.js

@@ -531,15 +531,15 @@ const Chip = React.forwardRef(function Chip(inProps, ref) {
       ...handlers,
       onClick: (event) => {
         handlers.onClick?.(event);
-        onClick(event);
+        onClick?.(event);
       },
       onKeyDown: (event) => {
         handlers.onKeyDown?.(event);
-        handleKeyDown(event);
+        handleKeyDown?.(event);
       },
       onKeyUp: (event) => {
         handlers.onKeyUp?.(event);
-        handleKeyUp(event);
+        handleKeyUp?.(event);
       },
     }),
   });

packages/mui-material/src/Chip/Chip.test.js

@@ -733,4 +733,36 @@ describe('<Chip />', () => {
       ).not.to.throw();
     });
   });
+
+  it('should not throw on clicking Chip when onClick is not provided', () => {
+    expect(() => {
+      const { getByTestId } = render(<Chip data-testid="chip" />);
+      const chip = getByTestId('chip');
+      fireEvent.click(chip);
+    }).not.throw();
+  });
+
+  it('should not throw on keydown when onKeyDown is not provided', () => {
+    expect(() => {
+      const { getByTestId } = render(<Chip data-testid="chip" onClick={() => {}} />);
+      const chip = getByTestId('chip');
+      act(() => {
+        chip.focus();
+      });
+
+      fireEvent.keyDown(chip, { key: 'Enter' });
+    }).not.throw();
+  });
+
+  it('should not throw on keyup when onKeyUp is not provided', () => {
+    expect(() => {
+      const { getByTestId } = render(<Chip data-testid="chip" onClick={() => {}} />);
+      const chip = getByTestId('chip');
+      act(() => {
+        chip.focus();
+      });
+
+      fireEvent.keyUp(chip, { key: ' ' });
+    }).not.throw();
+  });
 });