diff --git a/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.client.test.tsx b/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.client.test.tsx index 8a7217a74..49f31b8e2 100644 --- a/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.client.test.tsx +++ b/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.client.test.tsx @@ -34,7 +34,6 @@ vi.mock('@/components/system', () => ({ ChartColumnIncreasing: Icon, CircleDollarSign: Icon, Download: Icon, - GitPullRequest: Icon, Spinner: Icon, Button: ({ children, @@ -55,7 +54,7 @@ import { } from './AnalyticsShell'; describe('AnalyticsShell', () => { - it('renders Tasks before PRs in the navigation', () => { + it('omits PRs from the navigation while rendering the PR analytics page', () => { render( { ); const nav = screen.getByRole('navigation'); - const navItems = within(nav).getAllByText(/^(Tasks|PRs|Costs)$/); + const navItems = within(nav).getAllByText(/^(Tasks|Costs)$/); expect(screen.getByRole('heading', { name: 'PRs' })).toBeInTheDocument(); + expect(within(nav).queryByText('PRs')).not.toBeInTheDocument(); expect(navItems.map((item) => item.textContent)).toEqual([ 'Tasks', - 'PRs', 'Costs', ]); }); diff --git a/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.tsx b/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.tsx index 8b28e12c8..2311c1ac7 100644 --- a/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.tsx +++ b/apps/web/src/app/(authenticated)/analytics/AnalyticsShell.tsx @@ -7,7 +7,6 @@ import { ChartColumnIncreasing, CircleDollarSign, Download, - GitPullRequest, Button, Spinner, } from '@/components/system'; @@ -28,7 +27,6 @@ export function getAnalyticsHref(itemId: AnalyticsShellItemId) { const ANALYTICS_SHELL_ITEMS = [ { id: 'tasks', label: 'Tasks', icon: ChartColumnIncreasing }, - { id: 'pullRequests', label: 'PRs', icon: GitPullRequest }, { id: 'costs', label: 'Costs', icon: CircleDollarSign }, ] as const satisfies Array<{ id: AnalyticsObject;