feat: update MenuItem test to include itemData handling in onSelect and onClick

Author: 刘欢

tests/MenuItem.spec.tsx

@@ -3,6 +3,7 @@ import { fireEvent, render } from '@testing-library/react';
 import KeyCode from '@rc-component/util/lib/KeyCode';
 import React from 'react';
 import Menu, { MenuItem, MenuItemGroup, SubMenu } from '../src';
+import { MenuItemType } from '@/interface';
 
 describe('MenuItem', () => {
   const subMenuIconText = 'SubMenuIcon';
@@ -181,15 +182,15 @@ describe('MenuItem', () => {
       );
     });
 
-    it('should pass itemData in onSelect and onClick with items', () => {
+    it('should only pass defined itemData properties in onSelect and onClick', () => {
       const onSelect = jest.fn();
       const onClick = jest.fn();
       const { container } = render(
         <Menu
           onSelect={onSelect}
           onClick={onClick}
           selectable
-          items={[{ key: '1', label: 'Menu Item' }]}
+          items={[{ key: '1', label: 'Menu Item', foo: '123' }] as any}
         />,
       );
 
@@ -203,6 +204,14 @@ describe('MenuItem', () => {
           }),
         }),
       );
+      expect(onSelect).toHaveBeenCalledWith(
+        expect.objectContaining({
+          key: '1',
+          itemData: expect.not.objectContaining({
+            foo: '123',
+          }),
+        }),
+      );
       expect(onClick).toHaveBeenCalledWith(
         expect.objectContaining({
           key: '1',
@@ -212,6 +221,14 @@ describe('MenuItem', () => {
           }),
         }),
       );
+      expect(onClick).toHaveBeenCalledWith(
+        expect.objectContaining({
+          key: '1',
+          itemData: expect.not.objectContaining({
+            foo: '123',
+          }),
+        }),
+      );
     });
   });