Skip to content

Commit

Permalink
Add test
Browse files Browse the repository at this point in the history
  • Loading branch information
Yueying Lu committed Feb 12, 2025
1 parent cf396d3 commit 317d60d
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 5 deletions.
7 changes: 2 additions & 5 deletions pages/select/select.test.page.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -76,13 +76,10 @@ export default function SelectPage() {
<ScreenshotArea>
<Box padding="l">
<Box padding="s">
<div style={{ height: '700px' }}></div>
<div style={{ height: '500px', backgroundColor: 'lightblue' }}>500px space above the select</div>
<Box variant="h1">Overflow test</Box>
<Box variant="p">Dropdown should not be rendered beyond the container with overflow hidden</Box>
<div
id="smallest_container"
style={{ overflow: 'hidden', blockSize: '500px', padding: 0, backgroundColor: 'red' }}
>
<div id="smallest_container" style={{ overflow: 'hidden', blockSize: '500px', padding: 0 }}>
<div style={{ overflow: 'hidden', blockSize: '900px' }}>
<Select
id="select_overflow"
Expand Down
15 changes: 15 additions & 0 deletions src/select/__integ__/select.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,21 @@ test(
})
);

test(
'should not scroll the container',
useBrowser(async browser => {
const page = new BasePageObject(browser);
await page.setWindowSize({ height: 700, width: 1100 });
await browser.url('/#/light/select/select.test');
const select = createWrapper().findSelect('#select_overflow');
const triggerSelector = select.findTrigger().toSelector();
const { top: containerScrollTop } = await page.getBoundingBox('#select_overflow');
await page.click(triggerSelector);
const { top: containerScrollTopAfter } = await page.getBoundingBox('#select_overflow');
expect(containerScrollTop).toEqual(containerScrollTopAfter);
})
);

test(
'allows the select to be opened and closed using space',
useBrowser(async browser => {
Expand Down

0 comments on commit 317d60d

Please sign in to comment.