hover: on the <span> only fired on the 20×20px visual circle, not the full 44×44px touch target. Add `group` + `focus-visible:ring-*` to the outer button; switch to `group-hover:` on the inner span so the visual response covers the entire interactive area. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
115 lines
4.6 KiB
TypeScript
115 lines
4.6 KiB
TypeScript
import { describe, it, expect, afterEach, vi } from 'vitest';
|
|
import { cleanup, render } from 'vitest-browser-svelte';
|
|
import { page, userEvent } from 'vitest/browser';
|
|
import HelpPopover from './HelpPopover.svelte';
|
|
|
|
afterEach(cleanup);
|
|
|
|
function renderPopover(label = 'Help') {
|
|
return render(HelpPopover, { props: { label } });
|
|
}
|
|
|
|
describe('HelpPopover — initial state', () => {
|
|
it('renders a trigger button with the given label', async () => {
|
|
renderPopover();
|
|
const btn = page.getByRole('button', { name: /Help/ });
|
|
await expect.element(btn).toBeInTheDocument();
|
|
});
|
|
|
|
it('starts closed: aria-expanded is false, popover not in DOM', async () => {
|
|
renderPopover();
|
|
const btn = page.getByRole('button', { name: /Help/ });
|
|
await expect.element(btn).toHaveAttribute('aria-expanded', 'false');
|
|
expect(document.querySelector('[role="region"]')).toBeNull();
|
|
});
|
|
});
|
|
|
|
describe('HelpPopover — open / close interactions', () => {
|
|
it('opens on click: aria-expanded true, popover in DOM', async () => {
|
|
renderPopover();
|
|
await page.getByRole('button', { name: /Help/ }).click();
|
|
const btn = page.getByRole('button', { name: /Help/ });
|
|
await expect.element(btn).toHaveAttribute('aria-expanded', 'true');
|
|
expect(document.querySelector('[role="region"]')).not.toBeNull();
|
|
});
|
|
|
|
it('closes on Esc key', async () => {
|
|
renderPopover();
|
|
await page.getByRole('button', { name: /Help/ }).click();
|
|
expect(document.querySelector('[role="region"]')).not.toBeNull();
|
|
|
|
document.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape', bubbles: true }));
|
|
await vi.waitFor(() => expect(document.querySelector('[role="region"]')).toBeNull());
|
|
});
|
|
|
|
it('closes on outside click', async () => {
|
|
renderPopover();
|
|
await page.getByRole('button', { name: /Help/ }).click();
|
|
expect(document.querySelector('[role="region"]')).not.toBeNull();
|
|
|
|
document.dispatchEvent(new PointerEvent('pointerdown', { bubbles: true }));
|
|
await vi.waitFor(() => expect(document.querySelector('[role="region"]')).toBeNull());
|
|
});
|
|
|
|
it('opens on Enter key', async () => {
|
|
renderPopover();
|
|
(document.querySelector('button[aria-expanded]') as HTMLButtonElement).focus();
|
|
await userEvent.keyboard('{Enter}');
|
|
await vi.waitFor(() => expect(document.querySelector('[role="region"]')).not.toBeNull());
|
|
});
|
|
|
|
it('opens on Space key', async () => {
|
|
renderPopover();
|
|
(document.querySelector('button[aria-expanded]') as HTMLButtonElement).focus();
|
|
await userEvent.keyboard('{Space}');
|
|
await vi.waitFor(() => expect(document.querySelector('[role="region"]')).not.toBeNull());
|
|
});
|
|
});
|
|
|
|
describe('HelpPopover — hover-target', () => {
|
|
it('hover styles propagate from 44px button group to inner span, not from span itself', () => {
|
|
const { container } = renderPopover();
|
|
const btn = container.querySelector('button[aria-expanded]')!;
|
|
const span = btn.querySelector('span')!;
|
|
const btnClasses = btn.className.split(/\s+/);
|
|
const spanClasses = span.className.split(/\s+/);
|
|
expect(btnClasses).toContain('group');
|
|
expect(spanClasses).not.toContain('hover:border-brand-navy');
|
|
expect(spanClasses).toContain('group-hover:border-brand-navy');
|
|
expect(spanClasses).not.toContain('hover:text-brand-navy');
|
|
expect(spanClasses).toContain('group-hover:text-brand-navy');
|
|
});
|
|
|
|
it('outer button has focus-visible ring for keyboard users', () => {
|
|
const { container } = renderPopover();
|
|
const btn = container.querySelector('button[aria-expanded]')!;
|
|
expect(btn.className).toContain('focus-visible:ring-2');
|
|
expect(btn.className).toContain('focus-visible:ring-brand-navy');
|
|
});
|
|
});
|
|
|
|
describe('HelpPopover — aria wiring', () => {
|
|
it('trigger aria-controls matches popover element id', async () => {
|
|
renderPopover();
|
|
await page.getByRole('button', { name: /Help/ }).click();
|
|
const btn = document.querySelector('button[aria-expanded]') as HTMLButtonElement;
|
|
const controls = btn.getAttribute('aria-controls');
|
|
expect(controls).toBeTruthy();
|
|
const popover = document.getElementById(controls!);
|
|
expect(popover).not.toBeNull();
|
|
});
|
|
|
|
it('two renders produce different, predictable IDs (no Math.random — SSR safe)', async () => {
|
|
const { container: c1 } = render(HelpPopover, { props: { label: 'A' } });
|
|
const { container: c2 } = render(HelpPopover, { props: { label: 'B' } });
|
|
const id1 = c1.querySelector('button[aria-controls]')?.getAttribute('aria-controls');
|
|
const id2 = c2.querySelector('button[aria-controls]')?.getAttribute('aria-controls');
|
|
expect(id1).toBeTruthy();
|
|
expect(id2).toBeTruthy();
|
|
expect(id1).not.toBe(id2);
|
|
// IDs must be deterministic (counter-based), not random hex
|
|
expect(id1).toMatch(/^help-popover-\d+$/);
|
|
expect(id2).toMatch(/^help-popover-\d+$/);
|
|
});
|
|
});
|