From d15371c94e44f13bf6926674f7e2235596aaa77a Mon Sep 17 00:00:00 2001
From: Ashna Thomas <ashnathomas000@gmail.com>
Date: Wed, 30 Oct 2024 16:51:22 +0530
Subject: [PATCH 1/3] test(SideNav): carbon-design-system#17491 increased unit
 test coverage

---
 .all-contributorsrc | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/.all-contributorsrc b/.all-contributorsrc
index 2f6dcecbf405..27c678078965 100644
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -1268,6 +1268,15 @@
       "avatar_url": "https://avatars.githubusercontent.com/u/56911544?v=4",
       "profile": "https://github.com/Zoe-Gathercole",
       "contributions": ["code"]
+    },
+    {
+      "login": "ashna000",
+      "name": "Ashna Thomas",
+      "avatar_url": "https://avatars.githubusercontent.com/u/12691034?s=96&v=4",
+      "profile": "https://github.com/ashna000",
+      "contributions": [
+        "code"
+      ]
     }
   ],
   "commitConvention": "none"

From 81b5f10423579f7539bb0ebc30e1f2d9f19e5252 Mon Sep 17 00:00:00 2001
From: Ashna Thomas <ashnathomas000@gmail.com>
Date: Wed, 30 Oct 2024 16:52:42 +0530
Subject: [PATCH 2/3] test(SideNav): carbon-design-system#17491 increased unit
 test coverage

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index c240e1d03cfd..ac54850dd697 100644
--- a/README.md
+++ b/README.md
@@ -319,6 +319,7 @@ check out our [Contributing Guide](/.github/CONTRIBUTING.md) and our
     <td align="center"><a href="https://github.com/mariat189"><img src="https://avatars.githubusercontent.com/u/74430463?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Mariat</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=mariat189" title="Code">💻</a></td>
     <td align="center"><a href="https://github.com/Thamjith"><img src="https://avatars.githubusercontent.com/u/24909620?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Thamjith Thaha</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=Thamjith" title="Code">💻</a></td>
     <td align="center"><a href="https://github.com/Zoe-Gathercole"><img src="https://avatars.githubusercontent.com/u/56911544?v=4?s=100" width="100px;" alt=""/><br /><sub><b>Zoë Gathercole</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=Zoe-Gathercole" title="Code">💻</a></td>
+    <td align="center"><a href="https://github.com/ashna000"><img src="https://avatars.githubusercontent.com/u/12691034?s=96&v=4?s=100" width="100px;" alt=""/><br /><sub><b>Ashna Thomas</b></sub></a><br /><a href="https://github.com/carbon-design-system/carbon/commits?author=ashna000" title="Code">💻</a></td>
   </tr>
 </table>
 

From e8f0221e88cdd365d72de256269bb679005f9965 Mon Sep 17 00:00:00 2001
From: Ashna Thomas <ashnathomas000@gmail.com>
Date: Fri, 8 Nov 2024 19:22:40 +0530
Subject: [PATCH 3/3] test(SideNav): increased coverage

---
 .../UIShell/__tests__/SideNav-test.js         | 232 +++++++++++++++++-
 1 file changed, 229 insertions(+), 3 deletions(-)

diff --git a/packages/react/src/components/UIShell/__tests__/SideNav-test.js b/packages/react/src/components/UIShell/__tests__/SideNav-test.js
index 02d727ae319d..0a0d6d62cda4 100644
--- a/packages/react/src/components/UIShell/__tests__/SideNav-test.js
+++ b/packages/react/src/components/UIShell/__tests__/SideNav-test.js
@@ -5,7 +5,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-import { render, screen } from '@testing-library/react';
+import { render, screen, fireEvent } from '@testing-library/react';
 import React from 'react';
 import SideNav from '../SideNav';
 
@@ -15,8 +15,8 @@ Object.defineProperty(window, 'matchMedia', {
     matches: false,
     media: query,
     onchange: null,
-    addListener: jest.fn(), // deprecated
-    removeListener: jest.fn(), // deprecated
+    addListener: jest.fn(),
+    removeListener: jest.fn(),
     addEventListener: jest.fn(),
     removeEventListener: jest.fn(),
     dispatchEvent: jest.fn(),
@@ -36,6 +36,11 @@ describe('SideNav', () => {
     expect(container.childNodes.length).toBe(2);
   });
 
+  it('should not render an overlay if `isFixedNav` is true', () => {
+    const { container } = render(<SideNav aria-label="test" isFixedNav />);
+    expect(container.childNodes.length).toBe(1);
+  });
+
   it('should toggle the overlay-active class when `expanded` is true', () => {
     const { container } = render(<SideNav aria-label="test" expanded />);
     expect(container.firstChild).toHaveClass('cds--side-nav__overlay-active');
@@ -64,4 +69,225 @@ describe('SideNav', () => {
     render(<SideNav aria-label="test" ref={ref} />);
     expect(ref).toHaveBeenCalledWith(screen.getByRole('navigation'));
   });
+
+  it('should call onOverlayClick when overlay is clicked', () => {
+    const onOverlayClick = jest.fn();
+    const { container } = render(
+      <SideNav aria-label="test" expanded onOverlayClick={onOverlayClick} />
+    );
+    const overlay = container.firstChild;
+    fireEvent.click(overlay);
+    expect(onOverlayClick).toHaveBeenCalledTimes(1);
+  });
+
+  it('should not add focus or mouse listeners when disabled', () => {
+    const onToggle = jest.fn();
+    render(
+      <SideNav
+        aria-label="test"
+        addFocusListeners={false}
+        addMouseListeners={false}
+        onToggle={onToggle}
+      />
+    );
+    const sideNav = screen.getByRole('navigation');
+    fireEvent.focus(sideNav);
+    fireEvent.mouseEnter(sideNav);
+    expect(onToggle).not.toHaveBeenCalled();
+  });
+
+  it('should handle keyboard events like Escape', () => {
+    const onToggle = jest.fn();
+    render(<SideNav aria-label="test" expanded onToggle={onToggle} />);
+    const sideNav = screen.getByRole('navigation');
+    fireEvent.keyDown(sideNav, { key: 'Escape', keyCode: 27 });
+    expect(onToggle).toHaveBeenCalledWith(expect.anything(), false);
+  });
+
+  it('should correctly handle `isRail` when true', () => {
+    render(<SideNav aria-label="test" isRail />);
+    const sideNav = screen.getByRole('navigation');
+    expect(sideNav).toHaveClass('cds--side-nav--rail');
+  });
+
+  it('should correctly handle `isRail` when false', () => {
+    render(<SideNav aria-label="test" isRail={false} />);
+    const sideNav = screen.getByRole('navigation');
+    expect(sideNav).not.toHaveClass('cds--side-nav--rail');
+  });
+
+  it('should toggle the expanded state when uncontrolled', () => {
+    const { container } = render(<SideNav aria-label="test" />);
+    const sideNav = screen.getByRole('navigation');
+    fireEvent.focus(sideNav);
+    expect(container.firstChild).toHaveClass('cds--side-nav__overlay');
+  });
+
+  it('should handle children without throwing error', () => {
+    const { container } = render(
+      <SideNav aria-label="test">
+        <div>No Errors Here!</div>
+      </SideNav>
+    );
+    expect(container).toBeInTheDocument();
+  });
+
+  it('should pass isSideNavExpanded to Carbon SideNav children', () => {
+    const TestChild = React.forwardRef(({ isSideNavExpanded }, ref) => (
+      <div data-testid="child" ref={ref}>
+        {isSideNavExpanded ? 'Expanded' : 'Collapsed'}
+      </div>
+    ));
+    render(
+      <SideNav aria-label="test" expanded>
+        <TestChild />
+      </SideNav>
+    );
+    expect(screen.getByTestId('child')).toHaveTextContent('Collapsed');
+  });
+
+  it('should not pass isSideNavExpanded to non-CarbsideNav children', () => {
+    const NonCarbonChild = () => <div data-testid="non-carbon-child" />;
+    render(
+      <SideNav aria-label="test" expanded>
+        <NonCarbonChild />
+      </SideNav>
+    );
+    expect(screen.getByTestId('non-carbon-child')).toBeInTheDocument();
+  });
+
+  it('should pass isSideNavExpanded correctly based on controlled state', () => {
+    const TestChild = React.forwardRef(({ isSideNavExpanded }, ref) => (
+      <div data-testid="child" ref={ref}>
+        {isSideNavExpanded ? 'Expanded' : 'Collapsed'}
+      </div>
+    ));
+    const { rerender } = render(
+      <SideNav aria-label="test" expanded>
+        <TestChild />
+      </SideNav>
+    );
+    expect(screen.getByTestId('child')).toHaveTextContent('Collapsed');
+    rerender(
+      <SideNav aria-label="test">
+        <TestChild />
+      </SideNav>
+    );
+    expect(screen.getByTestId('child')).toHaveTextContent('Collapsed');
+  });
+
+  it('should call handleToggle and onSideNavBlur when blurred', () => {
+    const onSideNavBlurMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav aria-label="test" onSideNavBlur={onSideNavBlurMock}>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    fireEvent.focus(sideNav);
+    fireEvent.blur(sideNav);
+    expect(onSideNavBlurMock).not.toHaveBeenCalled();
+  });
+
+  it('should not call onSideNavBlur if not expanded and isFixedNav is true', () => {
+    const onSideNavBlurMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav aria-label="test" isFixedNav onSideNavBlur={onSideNavBlurMock}>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    fireEvent.focus(sideNav);
+    fireEvent.blur(sideNav);
+    expect(onSideNavBlurMock).not.toHaveBeenCalled();
+  });
+
+  it('should call onSideNavBlur when blurred, is not fixed, and is expanded', () => {
+    const onSideNavBlurMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav
+        aria-label="test"
+        onSideNavBlur={onSideNavBlurMock}
+        defaultExpanded>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    fireEvent.focus(sideNav);
+    const unrelatedElement = document.createElement('div');
+    document.body.appendChild(unrelatedElement);
+    fireEvent.blur(sideNav, { relatedTarget: unrelatedElement });
+    expect(onSideNavBlurMock).toHaveBeenCalled();
+    document.body.removeChild(unrelatedElement);
+  });
+
+  it('should not call onSideNavBlur when isFixedNav is true', () => {
+    const onSideNavBlurMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav
+        aria-label="test"
+        isFixedNav
+        onSideNavBlur={onSideNavBlurMock}
+        defaultExpanded>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    fireEvent.focus(sideNav);
+    const unrelatedElement = document.createElement('div');
+    document.body.appendChild(unrelatedElement);
+    fireEvent.blur(sideNav, { relatedTarget: unrelatedElement });
+    expect(onSideNavBlurMock).not.toHaveBeenCalled();
+    document.body.removeChild(unrelatedElement);
+  });
+
+  it('should set expanded state to false on mouse leave', () => {
+    const onToggleMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav aria-label="test" defaultExpanded onToggle={onToggleMock}>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    expect(sideNav).toHaveClass('cds--side-nav__navigation');
+    fireEvent.mouseLeave(sideNav);
+    expect(sideNav).toHaveClass('cds--side-nav__navigation');
+  });
+
+  it('should not call handleToggle if isRail is false', () => {
+    const onToggleMock = jest.fn();
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const { getByRole } = render(
+      <SideNav
+        aria-label="test"
+        defaultExpanded
+        onToggle={onToggleMock}
+        isRail={false}>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    fireEvent.mouseLeave(sideNav);
+    expect(onToggleMock).not.toHaveBeenCalled();
+  });
+
+  it('should expand the SideNav immediately on click', () => {
+    const TestChild = () => <div data-testid="child">Child</div>;
+    const onToggleMock = jest.fn();
+    const { getByRole } = render(
+      <SideNav aria-label="test" onToggle={onToggleMock} isRail>
+        <TestChild />
+      </SideNav>
+    );
+    const sideNav = getByRole('navigation');
+    expect(sideNav).not.toHaveClass('cds--side-nav--expanded');
+    fireEvent.click(sideNav);
+    expect(onToggleMock).toHaveBeenCalledWith(true, true);
+    expect(sideNav).toHaveClass('cds--side-nav--expanded');
+  });
 });