Add test for hover state

apache · Apr 4, 2024 · 60cc1c4 · 60cc1c4
1 parent 0d4bf69
commit 60cc1c4
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/superset-frontend/src/dashboard/components/menu/HoverMenu.test.tsx b/superset-frontend/src/dashboard/components/menu/HoverMenu.test.tsx
@@ -17,11 +17,25 @@
  * under the License.
  */
 import React from 'react';
-import { render } from 'spec/helpers/testing-library';
+import { render, screen } from 'spec/helpers/testing-library';
+import userEvent from '@testing-library/user-event';
 
 import HoverMenu from 'src/dashboard/components/menu/HoverMenu';
 
 test('should render a div.hover-menu', () => {
   const { container } = render(<HoverMenu />);
   expect(container.querySelector('.hover-menu')).toBeInTheDocument();
 });
+
+test('should call onHover when mouse enters and leaves', () => {
+  const onHover = jest.fn();
+  render(<HoverMenu onHover={onHover} />);
+
+  const hoverMenu = screen.getByTestId('hover-menu');
+
+  userEvent.hover(hoverMenu);
+  expect(onHover).toBeCalledWith({ isHovered: true });
+
+  userEvent.unhover(hoverMenu);
+  expect(onHover).toBeCalledWith({ isHovered: false });
+});
diff --git a/superset-frontend/src/dashboard/components/menu/HoverMenu.tsx b/superset-frontend/src/dashboard/components/menu/HoverMenu.tsx
@@ -99,6 +99,7 @@ export default class HoverMenu extends React.PureComponent<HoverMenuProps> {
           )}
           onMouseEnter={this.handleMouseEnter}
           onMouseLeave={this.handleMouseLeave}
+          data-test="hover-menu"
         >
           {children}
         </div>