Skip to content

Commit 83b4141

Browse files
improving data highlighted tests
1 parent e8c2832 commit 83b4141

File tree

1 file changed

+22
-35
lines changed

1 file changed

+22
-35
lines changed

packages/kit-headless/src/components/select/select.test.ts

Lines changed: 22 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -321,65 +321,52 @@ test.describe('Keyboard Behavior', () => {
321321
test(`GIVEN an open hero select
322322
WHEN the first option is highlighted and the down arrow key is pressed
323323
THEN the second option should have data-highlighted`, async ({ page }) => {
324-
const {
325-
getTrigger,
326-
getItemAt: getOptionAt,
327-
openListbox,
328-
} = await setup(page, 'hero');
324+
const { driver: d } = await setup(page, 'hero');
329325

330-
await openListbox('Enter');
326+
await d.openListbox('Enter');
331327

332328
// first index highlighted
333329

334-
await expect(getOptionAt(0)).toHaveAttribute('data-highlighted');
330+
await expect(d.getItemAt(0)).toHaveAttribute('data-highlighted');
335331

336-
await getTrigger().focus();
337-
await getTrigger().press('ArrowDown');
338-
await expect(getOptionAt(1)).toHaveAttribute('data-highlighted');
332+
await d.getItemAt(0).focus();
333+
await d.getItemAt(0).press('ArrowDown');
334+
await expect(d.getItemAt(1)).toHaveAttribute('data-highlighted');
339335
});
340336

341337
test(`GIVEN an open hero select
342338
WHEN the third option is highlighted and the up arrow key is pressed
343339
THEN the second option should have data-highlighted`, async ({ page }) => {
344-
const {
345-
getTrigger,
346-
getItemAt: getOptionAt,
347-
openListbox,
348-
} = await setup(page, 'hero');
340+
const { driver: d } = await setup(page, 'hero');
349341

350-
await openListbox('Enter');
342+
await d.openListbox('Enter');
351343

352-
await expect(getOptionAt(0)).toHaveAttribute('data-highlighted');
353-
await getTrigger().press('ArrowDown');
354-
await getTrigger().press('ArrowDown');
344+
await expect(d.getItemAt(0)).toHaveAttribute('data-highlighted');
345+
await d.getItemAt(0).press('ArrowDown');
346+
await d.getItemAt(1).press('ArrowDown');
347+
1;
355348

356-
await getTrigger().press('ArrowUp');
357-
await expect(getOptionAt(1)).toHaveAttribute('data-highlighted');
349+
await d.getItemAt(2).press('ArrowUp');
350+
await expect(d.getItemAt(1)).toHaveAttribute('data-highlighted');
358351
});
359352

360353
test(`GIVEN a hero select with a chosen option
361354
AND the down arrow key is pressed
362355
THEN the data-highlighted option should not change on re-open`, async ({
363356
page,
364357
}) => {
365-
const {
366-
getTrigger,
367-
getListbox,
368-
getItemAt: getOptionAt,
369-
openListbox,
370-
} = await setup(page, 'hero');
358+
const { driver: d } = await setup(page, 'hero');
371359

372-
await openListbox('Enter');
360+
await d.openListbox('Enter');
373361

374362
// second option highlighted
363+
await d.getItemAt(0).press('ArrowDown');
364+
await expect(d.getItemAt(1)).toHaveAttribute('data-highlighted');
365+
await d.getItemAt(1).press('Enter');
366+
await expect(d.getListbox()).toBeHidden();
375367

376-
await getTrigger().press('ArrowDown');
377-
await expect(getOptionAt(1)).toHaveAttribute('data-highlighted');
378-
await getTrigger().press('Enter');
379-
await expect(getListbox()).toBeHidden();
380-
381-
await getTrigger().press('ArrowDown');
382-
await expect(getOptionAt(1)).toHaveAttribute('data-highlighted');
368+
await d.getTrigger().press('ArrowDown');
369+
await expect(d.getItemAt(1)).toHaveAttribute('data-highlighted');
383370
});
384371
});
385372

0 commit comments

Comments
 (0)