[material-ui][SwitchBase] Fix spreading of handlers in getSlotProps (…

…#45197)
mui · Feb 4, 2025 · 9074d47 · 9074d47
1 parent 1fb6063
commit 9074d47
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/packages/mui-material/src/internal/SwitchBase.js b/packages/mui-material/src/internal/SwitchBase.js
@@ -212,6 +212,7 @@ const SwitchBase = React.forwardRef(function SwitchBase(props, ref) {
     className: classes.input,
     externalForwardedProps,
     getSlotProps: (handlers) => ({
+      ...handlers,
       onChange: (event) => {
         handlers.onChange?.(event);
         handleInputChange(event);

diff --git a/packages/mui-material/src/internal/SwitchBase.test.js b/packages/mui-material/src/internal/SwitchBase.test.js
@@ -479,4 +479,31 @@ describe('<SwitchBase />', () => {
       expect(getByRole('checkbox')).to.have.property('value', 'red');
     });
   });
+
+  it('should call event handlers in slotProps when provided', () => {
+    const rootOnClick = spy();
+    const inputOnClick = spy();
+    const { getByRole } = render(
+      <SwitchBase
+        icon="unchecked"
+        checkedIcon="checked"
+        type="checkbox"
+        slotProps={{
+          root: {
+            onClick: rootOnClick,
+          },
+          input: {
+            onClick: inputOnClick,
+          },
+        }}
+      />,
+    );
+
+    act(() => {
+      getByRole('checkbox').click();
+    });
+
+    expect(rootOnClick.callCount).to.equal(1);
+    expect(inputOnClick.callCount).to.equal(1);
+  });
 });