Skip to content

Commit d75b9af

Browse files
author
Luca Forstner
authored
test(e2e): Unflake sveltekit e2e tests (#11597)
1 parent 2e7c4f4 commit d75b9af

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

dev-packages/e2e-tests/test-applications/sveltekit-2/test/errors.client.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,17 @@ import { waitForInitialPageload } from './utils';
44

55
test.describe('client-side errors', () => {
66
test('captures error thrown on click', async ({ page }) => {
7-
await page.goto('/client-error');
8-
9-
await expect(page.getByText('Client error')).toBeVisible();
7+
await waitForInitialPageload(page, { route: '/client-error', debug: false });
108

119
const errorEventPromise = waitForError('sveltekit-2', errorEvent => {
1210
return errorEvent?.exception?.values?.[0]?.value === 'Click Error';
1311
});
1412

15-
const clickPromise = page.getByText('Throw error').click();
13+
await page.getByText('Throw error').click();
14+
15+
await expect(errorEventPromise).resolves.toBeDefined();
1616

17-
const [errorEvent, _] = await Promise.all([errorEventPromise, clickPromise]);
17+
const errorEvent = await errorEventPromise;
1818

1919
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
2020

dev-packages/e2e-tests/test-applications/sveltekit/test/errors.client.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,17 @@ import { waitForInitialPageload } from '../utils';
44

55
test.describe('client-side errors', () => {
66
test('captures error thrown on click', async ({ page }) => {
7-
await page.goto('/client-error');
8-
9-
await expect(page.getByText('Client error')).toBeVisible();
7+
await waitForInitialPageload(page, { route: '/client-error', debug: false });
108

119
const errorEventPromise = waitForError('sveltekit', errorEvent => {
1210
return errorEvent?.exception?.values?.[0]?.value === 'Click Error';
1311
});
1412

15-
const clickPromise = page.getByText('Throw error').click();
13+
await page.getByText('Throw error').click();
14+
15+
await expect(errorEventPromise).resolves.toBeDefined();
1616

17-
const [errorEvent, _] = await Promise.all([errorEventPromise, clickPromise]);
17+
const errorEvent = await errorEventPromise;
1818

1919
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
2020

0 commit comments

Comments
 (0)