Skip to content

Commit 67b95c1

Browse files
siggerzzBen Siggery
authored andcommitted
refactor(pie-text-input): DSW-2384 tests to use @playwright/test (#2194)
* text input tests * refactor components tests * add variation stories * add slot variations * visual tests * update import --------- Co-authored-by: Ben Siggery <[email protected]>
1 parent ac698f4 commit 67b95c1

File tree

8 files changed

+1113
-1066
lines changed

8 files changed

+1113
-1066
lines changed

apps/pie-storybook/stories/testing/pie-text-input.test.stories.ts

+402
Large diffs are not rendered by default.
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { defineConfig } from '@sand4rt/experimental-ct-web';
2-
import { getPlaywrightVisualConfig } from '@justeattakeaway/pie-components-config';
1+
import { defineConfig } from '@playwright/test';
2+
import { getPlaywrightNativeVisualConfig } from '@justeattakeaway/pie-components-config';
33

4-
export default defineConfig(getPlaywrightVisualConfig());
4+
export default defineConfig(getPlaywrightNativeVisualConfig());
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { defineConfig } from '@sand4rt/experimental-ct-web';
2-
import { getPlaywrightConfig } from '@justeattakeaway/pie-components-config';
1+
import { defineConfig } from '@playwright/test';
2+
import { getPlaywrightNativeConfig } from '@justeattakeaway/pie-components-config';
33

4-
export default defineConfig(getPlaywrightConfig());
4+
export default defineConfig(getPlaywrightNativeConfig());

packages/components/pie-text-input/test/accessibility/pie-text-input.spec.ts

+5-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11

2-
import { test, expect } from '@justeattakeaway/pie-webc-testing/src/playwright/webc-fixtures.ts';
3-
import { PieTextInput } from '../../src/index.ts';
2+
import { test, expect } from '@justeattakeaway/pie-webc-testing/src/playwright/playwright-fixtures.ts';
3+
import { BasePage } from '@justeattakeaway/pie-webc-testing/src/helpers/page-object/base-page.ts';
44

55
test.describe('PieTextInput - Accessibility tests', () => {
6-
test('a11y - should test the PieTextInput component WCAG compliance', async ({ makeAxeBuilder, mount }) => {
7-
await mount(PieTextInput);
6+
test('a11y - should test the PieTextInput component WCAG compliance', async ({ makeAxeBuilder, page }) => {
7+
const textInputDefaultPage = new BasePage(page, 'text-input--default');
8+
await textInputDefaultPage.load();
89

910
const results = await makeAxeBuilder().analyze();
1011

packages/components/pie-text-input/test/component/pie-text-input.spec.ts

+624-610
Large diffs are not rendered by default.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
const textInput = {
2+
selectors: {
3+
container: {
4+
dataTestId: 'pie-text-input-container',
5+
},
6+
assistiveText: {
7+
dataTestId: 'pie-text-input-assistive-text',
8+
},
9+
input: {
10+
dataTestId: 'pie-text-input',
11+
},
12+
},
13+
};
14+
export {
15+
textInput,
16+
};

0 commit comments

Comments
 (0)