From c3741bc93864918a5a8f13b75419b345effd422d Mon Sep 17 00:00:00 2001
From: deepsikha-dash <deepsikhadash326@gmail.com>
Date: Tue, 9 Jun 2026 23:43:12 +0530
Subject: [PATCH] test(frontend): add AppShell unit coverage

---
 .../testing/unit/components/AppShell.test.tsx | 116 ++++++++++++++++++
 1 file changed, 116 insertions(+)
 create mode 100644 frontend/testing/unit/components/AppShell.test.tsx

diff --git a/frontend/testing/unit/components/AppShell.test.tsx b/frontend/testing/unit/components/AppShell.test.tsx
new file mode 100644
index 00000000..6306f212
--- /dev/null
+++ b/frontend/testing/unit/components/AppShell.test.tsx
@@ -0,0 +1,116 @@
+import React from 'react'
+import { describe, it, expect, beforeEach, vi } from 'vitest'
+import { render, screen, fireEvent } from '@testing-library/react'
+
+let pathname = '/'
+
+vi.mock('react-router-dom', () => ({
+  NavLink: ({ children, to }: any) => (
+    <a href={to}>{children}</a>
+  ),
+  useLocation: () => ({
+    pathname,
+  }),
+}))
+
+vi.mock('../../../src/components/Sidebar', () => ({
+  default: () => <div data-testid="sidebar" />,
+}))
+
+vi.mock('../../../src/components/Background', () => ({
+  default: () => <div data-testid="background" />,
+}))
+
+vi.mock('../../../src/hooks/useShortcuts', () => ({
+  useShortcuts: vi.fn(),
+}))
+
+import AppShell from '../../../src/components/AppShell'
+
+describe('AppShell', () => {
+  beforeEach(() => {
+    pathname = '/'
+    localStorage.clear()
+  })
+
+  it('renders children content', () => {
+    render(
+      <AppShell>
+        <div>Test Content</div>
+      </AppShell>
+    )
+
+    expect(screen.getByText('Test Content')).toBeTruthy()
+  })
+
+  it('opens and closes mobile drawer from menu button', () => {
+    render(
+      <AppShell>
+        <div>Content</div>
+      </AppShell>
+    )
+
+    const button = screen.getByLabelText(/toggle navigation menu/i)
+
+    fireEvent.click(button)
+
+    expect(screen.getByText('Settings')).toBeTruthy()
+
+    fireEvent.click(button)
+
+    expect(screen.queryByText('Settings')).toBeNull()
+  })
+
+  it('renders bottom navigation items', () => {
+    render(
+      <AppShell>
+        <div>Content</div>
+      </AppShell>
+    )
+
+    expect(screen.getByText('Dashboard')).toBeTruthy()
+    expect(screen.getByText('Scans')).toBeTruthy()
+    expect(screen.getByText('Findings')).toBeTruthy()
+    expect(screen.getByText('Reports')).toBeTruthy()
+    expect(screen.getByText('Workflows')).toBeTruthy()
+  })
+
+  it('renders drawer navigation items when menu is opened', () => {
+    render(
+      <AppShell>
+        <div>Content</div>
+      </AppShell>
+    )
+
+    fireEvent.click(
+      screen.getByLabelText(/toggle navigation menu/i)
+    )
+
+    expect(screen.getByText('Settings')).toBeTruthy()
+    expect(screen.getAllByText('Dashboard').length).toBeGreaterThan(0)
+  })
+
+  it('closes mobile drawer when route changes', () => {
+    const { rerender } = render(
+      <AppShell>
+        <div>Content</div>
+      </AppShell>
+    )
+
+    fireEvent.click(
+      screen.getByLabelText(/toggle navigation menu/i)
+    )
+
+    expect(screen.getByText('Settings')).toBeTruthy()
+
+    pathname = '/reports'
+
+    rerender(
+      <AppShell>
+        <div>Content</div>
+      </AppShell>
+    )
+
+    expect(screen.queryByText('Settings')).toBeNull()
+  })
+})
\ No newline at end of file