fix: sonar coverage (#2765)

Author: mishraomp

.github/workflows/analysis.yml

@@ -75,11 +75,11 @@ jobs:
           dir: frontend
           node_version: "22"
           sonar_args: >
-            -Dsonar.exclusions=**/coverage/**,**/node_modules/**,**/*spec.ts
+            -Dsonar.exclusions=**/coverage/**,**/node_modules/**,**/*spec.ts,**/*test.tsx
             -Dsonar.organization=bcgov-sonarcloud
             -Dsonar.projectKey=quickstart-openshift_frontend
             -Dsonar.sources=src
-            -Dsonar.test.inclusions=**/*spec.ts
+            -Dsonar.test.inclusions=**/*spec.ts,**/*test.tsx
             -Dsonar.javascript.lcov.reportPaths=./coverage/lcov.info
           sonar_token: ${{ env.sonar_token }}
           dep_scan: warn

frontend/src/components/__tests__/Dashboard.test.tsx

@@ -1,5 +1,6 @@
 import { vi } from 'vitest'
-import { render, screen } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
 import Dashboard from '@/components/Dashboard'
 
 vi.mock('@tanstack/react-router', () => ({
@@ -8,12 +9,24 @@ vi.mock('@tanstack/react-router', () => ({
 
 describe('Dashboard', () => {
   test('renders a heading with the correct text', () => {
-    const navigate = vi.fn()
-    const useNavigateMock = vi.fn(() => navigate)
-    vi.doMock('@tanstack/react-router', () => ({
-      useNavigate: useNavigateMock,
-    }))
     render(<Dashboard />)
     expect(screen.getByText(/Employee ID/i)).toBeInTheDocument()
   })
+
+  test('opens and closes the row details modal', async () => {
+    const user = userEvent.setup()
+    render(<Dashboard />)
+
+    // Wait for the mocked user data to load and render the View Details button.
+    const viewDetailsButton = await screen.findByRole('button', { name: /view details/i })
+    await user.click(viewDetailsButton)
+
+    expect(screen.getByText(/Row Details/i)).toBeInTheDocument()
+
+    const closeButtons = screen.getAllByRole('button', { name: 'Close' })
+    await user.click(closeButtons.at(-1)!)
+    await waitFor(() => {
+      expect(screen.queryByText(/Row Details/i)).not.toBeInTheDocument()
+    })
+  })
 })

frontend/src/components/__tests__/NotFound.test.tsx

@@ -1,20 +1,29 @@
 import { vi } from 'vitest'
 import { render, screen } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
 import NotFound from '@/components/NotFound'
 
+const navigate = vi.fn()
+
 vi.mock('@tanstack/react-router', () => ({
-  useNavigate: vi.fn(),
+  useNavigate: () => navigate,
 }))
 
 describe('NotFound', () => {
+  beforeEach(() => {
+    navigate.mockClear()
+  })
+
   test('renders a heading with the correct text', () => {
-    const navigate = vi.fn()
-    const useNavigateMock = vi.fn(() => navigate)
-    vi.doMock('@tanstack/react-router', () => ({
-      useNavigate: useNavigateMock,
-    }))
     render(<NotFound />)
     const headingElement = screen.getByRole('heading', { name: /404/i })
     expect(headingElement).toBeInTheDocument()
   })
+
+  test('navigates home when the Back Home button is clicked', async () => {
+    const user = userEvent.setup()
+    render(<NotFound />)
+    await user.click(screen.getByRole('button', { name: /back home/i }))
+    expect(navigate).toHaveBeenCalledWith({ to: '/' })
+  })
 })