Removed duplicate onChange (#17754)

* fix: removed duplicate onChange

* test: added test to make sure onchange is called once

* test: fixed spelling

* fix: removed test story

Author: guidari

packages/react/src/components/Menu/Menu-test.js

@@ -254,4 +254,28 @@ describe('MenuItem', () => {
       spy.mockRestore();
     });
   });
+
+  it('should call onChange once', async () => {
+    const onChange = jest.fn();
+
+    render(
+      <Menu open label="Menu">
+        <MenuItem label="Menu">
+          <MenuItemRadioGroup
+            label="MenuItemRadioGroup"
+            items={[
+              { label: 'Item 1', value: '1' },
+              { label: 'Item 2', value: '2' },
+            ]}
+            onChange={onChange}
+            itemToString={(item) => item.label}
+          />
+        </MenuItem>
+      </Menu>
+    );
+
+    await userEvent.click(screen.getByTitle('Menu'));
+    await userEvent.click(screen.getByTitle('Item 1'));
+    expect(onChange).toHaveBeenCalledTimes(1);
+  });
 });

packages/react/src/components/Menu/MenuItem.tsx

@@ -557,10 +557,6 @@ export const MenuItemRadioGroup = forwardRef(function MenuItemRadioGroup<Item>(
 
   function handleClick(item, e) {
     setSelection(item);
-
-    if (onChange) {
-      onChange(e);
-    }
   }
 
   useEffect(() => {