diff --git a/src/renderer/components/metrics/MetricPill.test.tsx b/src/renderer/components/metrics/MetricPill.test.tsx index 4528894e8..9715b5dc6 100644 --- a/src/renderer/components/metrics/MetricPill.test.tsx +++ b/src/renderer/components/metrics/MetricPill.test.tsx @@ -1,3 +1,6 @@ +import { screen } from '@testing-library/react'; +import userEvent from '@testing-library/user-event'; + import { MarkGithubIcon } from '@primer/octicons-react'; import { renderWithProviders } from '../../__helpers__/test-utils'; @@ -31,4 +34,19 @@ describe('renderer/components/metrics/MetricPill.tsx', () => { expect(tree.container).toMatchSnapshot(); }); + + it('has a single tab stop when the pill is not clickable', async () => { + renderWithProviders( + <> + + + , + ); + + await userEvent.tab(); + expect(screen.getByRole('button', { name: /Mock Pill|1/ })).toHaveFocus(); + + await userEvent.tab(); + expect(screen.getByRole('button', { name: 'Next' })).toHaveFocus(); + }); }); diff --git a/src/renderer/components/metrics/MetricPill.tsx b/src/renderer/components/metrics/MetricPill.tsx index 06271bfb5..a161ebf97 100644 --- a/src/renderer/components/metrics/MetricPill.tsx +++ b/src/renderer/components/metrics/MetricPill.tsx @@ -25,12 +25,7 @@ export function MetricPill(props: Readonly) { // @ts-expect-error: We overload text with a ReactNode