From 384f35bc7f2673e852e288ddacc26262a3ba28bd Mon Sep 17 00:00:00 2001
From: Alexandr Ovchinnikov <alexigov@gmail.com>
Date: Mon, 8 Jan 2024 15:55:21 +0100
Subject: [PATCH] fix(react): added onClick type to SwitcherItem (#15457)

updated author name and email

Co-authored-by: Guilherme Datilio Ribeiro <guilhermedatilio@gmail.com>
---
 .all-contributorsrc                              |  9 +++++++++
 README.md                                        |  1 +
 .../__snapshots__/PublicAPI-test.js.snap         |  3 +++
 .../src/components/UIShell/SwitcherItem.tsx      |  8 ++++++++
 .../UIShell/__tests__/SwitcherItem-test.js       | 16 +++++++++++++++-
 5 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/.all-contributorsrc b/.all-contributorsrc
index 0b8113115af6..7d58177cff28 100644
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -1397,6 +1397,15 @@
       "contributions": [
         "code"
       ]
+    },
+    {
+      "login": "amercury",
+      "name": "Alexandr Ovchinnikov",
+      "avatar_url": "https://avatars.githubusercontent.com/u/17834588?v=4",
+      "profile": "https://github.com/amercury",
+      "contributions": [
+        "code"
+      ]
     }
   ],
   "commitConvention": "none"
diff --git a/README.md b/README.md
index ddc01897ed09..3636d7c35e58 100644
--- a/README.md
+++ b/README.md
@@ -271,6 +271,7 @@ check out our [Contributing Guide](/.github/CONTRIBUTING.md) and our
     <td align="center"><a href="https://samuelechinellato.com/#/"><img src="https://avatars.githubusercontent.com/u/49278203?v=4?s=100" width="100px;" alt=""/><br /><sub><b>SamChinellato</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=SamChinellato" title="Code">💻</a></td>
     <td align="center"><a href="https://github.com/stevenpatrick009"><img src="https://avatars.githubusercontent.com/u/106097350?v=4?s=100" width="100px;" alt=""/><br /><sub><b>stevenpatrick009</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=stevenpatrick009" title="Code">💻</a></td>
     <td align="center"><a href="https://github.com/xalc"><img src="https://avatars.githubusercontent.com/u/18441947?v=4?s=100" width="100px;" alt=""/><br /><sub><b>HunterXalc</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=xalc" title="Code">💻</a></td>
+    <td align="center"><a href="https://github.com/amercury"><img src="https://avatars.githubusercontent.com/u/17834588?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Alexandr Ovchinnikov</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=amercury" title="Code">💻</a></td>
     <td align="center"><a href="https://mattborghi.github.io/"><img src="https://avatars.githubusercontent.com/u/11933424?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Matias Borghi</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=mattborghi" title="Code">💻</a></td>
   </tr>
 </table>
diff --git a/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap b/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap
index ed212e4292e5..8b4192c9b5f6 100644
--- a/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap
+++ b/packages/react/__tests__/__snapshots__/PublicAPI-test.js.snap
@@ -7379,6 +7379,9 @@ Map {
       "index": Object {
         "type": "number",
       },
+      "onClick": Object {
+        "type": "func",
+      },
       "onKeyDown": Object {
         "type": "func",
       },
diff --git a/packages/react/src/components/UIShell/SwitcherItem.tsx b/packages/react/src/components/UIShell/SwitcherItem.tsx
index 0f4af096f6c9..90f303c73ade 100644
--- a/packages/react/src/components/UIShell/SwitcherItem.tsx
+++ b/packages/react/src/components/UIShell/SwitcherItem.tsx
@@ -30,6 +30,10 @@ interface BaseSwitcherItemProps {
    * event handlers
    */
   onKeyDown?: (event: KeyboardEvent) => void;
+  /**
+   * event handlers
+   */
+  onClick?: (event: MouseEvent) => void;
   /**
    * Specify the tab index of the Link
    */
@@ -143,6 +147,10 @@ SwitcherItem.propTypes = {
    * Specify the index of the SwitcherItem
    */
   index: PropTypes.number,
+  /**
+   * event handlers
+   */
+  onClick: PropTypes.func,
   /**
    * event handlers
    */
diff --git a/packages/react/src/components/UIShell/__tests__/SwitcherItem-test.js b/packages/react/src/components/UIShell/__tests__/SwitcherItem-test.js
index 45599f0bbe80..dd1b8d35e884 100644
--- a/packages/react/src/components/UIShell/__tests__/SwitcherItem-test.js
+++ b/packages/react/src/components/UIShell/__tests__/SwitcherItem-test.js
@@ -7,7 +7,7 @@
 
 import React from 'react';
 import SwitcherItem from '../SwitcherItem';
-import { render, screen } from '@testing-library/react';
+import { render, screen, fireEvent } from '@testing-library/react';
 
 describe('SwitcherItem', () => {
   describe('renders as expected - Component API', () => {
@@ -78,5 +78,19 @@ describe('SwitcherItem', () => {
         '-1'
       );
     });
+
+    it('should handle onClick event', () => {
+      const dummyHandler = jest.fn();
+
+      render(
+        <SwitcherItem aria-label="dummy-aria-label" onClick={dummyHandler}>
+          Dummy child
+        </SwitcherItem>
+      );
+
+      fireEvent.click(screen.getByRole('listitem').firstChild);
+
+      expect(dummyHandler).toHaveBeenCalledTimes(1);
+    });
   });
 });