fix: wire up TabMenu.Item onClick handler which was being swallowed a…

…fter TS refactor (#270)
cision · Sep 14, 2020 · 0830a18 · 0830a18
1 parent 5fe895e
commit 0830a18
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/components/TabMenu/Item/Item.tsx b/src/components/TabMenu/Item/Item.tsx
@@ -26,7 +26,7 @@ const Item: React.FC<ItemProps> = ({
     [styles.active]: active || isChildActive(children),
   });
   return (
-    <li className={className} {...props}>
+    <li className={className} onClick={onClick} {...props}>
       {children}
     </li>
   );

diff --git a/src/components/TabMenu/TabMenu.test.tsx b/src/components/TabMenu/TabMenu.test.tsx
@@ -27,6 +27,18 @@ describe('TabMenu.Item', () => {
 
     expect(inactive.prop('active')).not.toBe(true);
   });
+
+  it('triggers onClick handler when clicking', () => {
+    const fakeClickHandler = jest.fn();
+    const menuItem = mount(
+      <TabMenu.Item onClick={fakeClickHandler}>
+        Clickable menu item
+      </TabMenu.Item>
+    );
+    menuItem.simulate('click');
+
+    expect(fakeClickHandler).toHaveBeenCalled();
+  });
 });
 
 describe('EasyTabMenu', () => {