Skip to content

Commit 6237cf0

Browse files
committed
test(menu): enable Axe tests but skip scrollable-region-focusable
1 parent 32f0e1a commit 6237cf0

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

core/src/components/menu/test/a11y/menu.e2e.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// import AxeBuilder from '@axe-core/playwright';
1+
import AxeBuilder from '@axe-core/playwright';
22
import { expect } from '@playwright/test';
33
import { configs, test } from '@utils/test/playwright';
44

@@ -19,10 +19,12 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
1919
await expect(heading).toHaveText('Open Menu');
2020

2121
/**
22-
* started to fail after update to Stencil v4.36.1
22+
* Skip the rule that checks if the menu is scrollable and focusable.
23+
* This test is missing the `ion-app` wrapper which allows the
24+
* menu content to be focusable.
2325
*/
24-
// const results = await new AxeBuilder({ page }).analyze();
25-
// expect(results.violations).toEqual([]);
26+
const results = await new AxeBuilder({ page }).disableRules('scrollable-region-focusable').analyze();
27+
expect(results.violations).toEqual([]);
2628
});
2729
});
2830
});

0 commit comments

Comments
 (0)