Skip to content

Commit a044f83

Browse files
committed
style: test name consistency
1 parent 11dbf56 commit a044f83

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

core/src/components/select/test/basic/select.e2e.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ configs({ directions: ['ltr'] }).forEach(({ title, config, screenshot }) => {
147147
await expect(focusedOptions).toHaveCount(0);
148148
});
149149

150-
test('it should focus the second option when opened', async ({ page }) => {
150+
test('it should focus the second option when opened with a value', async ({ page }) => {
151151
// ion-app is required to apply the focused styles
152152
await page.setContent(
153153
`
@@ -177,7 +177,7 @@ configs({ directions: ['ltr'] }).forEach(({ title, config, screenshot }) => {
177177
await expect(bananasOption).toHaveClass(/ion-focused/);
178178
});
179179

180-
test('it should focus the second option when opened with a header', async ({ page }) => {
180+
test('it should focus the second option when opened with a value and a header', async ({ page }) => {
181181
test.info().annotations.push({
182182
type: 'issue',
183183
description: 'https://github.com/ionic-team/ionic-framework/issues/30480',
@@ -239,7 +239,7 @@ configs({ directions: ['ltr'] }).forEach(({ title, config, screenshot }) => {
239239
await expect(popover).toBeVisible();
240240
});
241241

242-
test('it should focus the second option when value is set', async ({ page, skip }) => {
242+
test('it should focus the second option when opened with a value', async ({ page, skip }) => {
243243
// TODO (ROU-5437)
244244
skip.browser('webkit', 'Safari 16 only allows text fields and pop-up menus to be focused.');
245245

@@ -300,7 +300,7 @@ configs({ directions: ['ltr'] }).forEach(({ title, config, screenshot }) => {
300300
await expect(modal).toBeVisible();
301301
});
302302

303-
test('it should focus the second option when value is set', async ({ page }) => {
303+
test('it should focus the second option when opened with a value', async ({ page }) => {
304304
// ion-app is required to apply the focused styles
305305
await page.setContent(
306306
`

0 commit comments

Comments
 (0)