Skip to content

Commit 0c39dd1

Browse files
committed
biome
1 parent 9b9791d commit 0c39dd1

File tree

14 files changed

+186
-244
lines changed

14 files changed

+186
-244
lines changed

packages/e2e-tests/test-applications/sveltekit/playwright.config.ts

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
1-
import type { PlaywrightTestConfig } from "@playwright/test";
2-
import { devices } from "@playwright/test";
1+
import type { PlaywrightTestConfig } from '@playwright/test';
2+
import { devices } from '@playwright/test';
33

44
// Fix urls not resolving to localhost on Node v17+
55
// See: https://github.com/axios/axios/issues/3821#issuecomment-1413727575
6-
import { setDefaultResultOrder } from "dns";
7-
setDefaultResultOrder("ipv4first");
6+
import { setDefaultResultOrder } from 'dns';
7+
setDefaultResultOrder('ipv4first');
88

99
const testEnv = process.env.TEST_ENV;
1010

1111
if (!testEnv) {
12-
throw new Error("No test env defined");
12+
throw new Error('No test env defined');
1313
}
1414

1515
const svelteKitPort = 3030;
@@ -19,7 +19,7 @@ const eventProxyPort = 3031;
1919
* See https://playwright.dev/docs/test-configuration.
2020
*/
2121
const config: PlaywrightTestConfig = {
22-
testDir: "./test",
22+
testDir: './test',
2323
/* Maximum time one test can run for. */
2424
timeout: 150_000,
2525
expect: {
@@ -34,9 +34,9 @@ const config: PlaywrightTestConfig = {
3434
/* Fail the build on CI if you accidentally left test.only in the source code. */
3535
forbidOnly: !!process.env.CI,
3636
/* `next dev` is incredibly buggy with the app dir */
37-
retries: testEnv === "development" ? 3 : 0,
37+
retries: testEnv === 'development' ? 3 : 0,
3838
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
39-
reporter: "list",
39+
reporter: 'list',
4040
/* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
4141
use: {
4242
/* Maximum time each action such as `click()` can take. Defaults to 0 (no limit). */
@@ -45,29 +45,29 @@ const config: PlaywrightTestConfig = {
4545
baseURL: `http://localhost:${svelteKitPort}`,
4646

4747
/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
48-
trace: "on-first-retry",
48+
trace: 'on-first-retry',
4949
},
5050

5151
/* Configure projects for major browsers */
5252
projects: [
5353
{
54-
name: "chromium",
54+
name: 'chromium',
5555
use: {
56-
...devices["Desktop Chrome"],
56+
...devices['Desktop Chrome'],
5757
},
5858
},
5959
],
6060

6161
/* Run your local dev server before starting the tests */
6262
webServer: [
6363
{
64-
command: "pnpm ts-node-script start-event-proxy.ts",
64+
command: 'pnpm ts-node-script start-event-proxy.ts',
6565
port: eventProxyPort,
6666
reuseExistingServer: false,
6767
},
6868
{
6969
command:
70-
testEnv === "development"
70+
testEnv === 'development'
7171
? `pnpm wait-port ${eventProxyPort} && pnpm dev --port ${svelteKitPort}`
7272
: `pnpm wait-port ${eventProxyPort} && pnpm preview --port ${svelteKitPort}`,
7373
port: svelteKitPort,
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
1-
import { env } from "$env/dynamic/public";
2-
import * as Sentry from "@sentry/sveltekit";
1+
import { env } from '$env/dynamic/public';
2+
import * as Sentry from '@sentry/sveltekit';
33

4-
console.log("Sentry DSN:", env.PUBLIC_E2E_TEST_DSN);
4+
console.log('Sentry DSN:', env.PUBLIC_E2E_TEST_DSN);
55

66
Sentry.init({
7-
environment: "qa", // dynamic sampling bias to keep transactions
7+
environment: 'qa', // dynamic sampling bias to keep transactions
88
dsn: env.PUBLIC_E2E_TEST_DSN,
99
debug: true,
1010
tunnel: `http://localhost:3031/`, // proxy server
1111
tracesSampleRate: 1.0,
1212
});
1313

1414
const myErrorHandler = ({ error, event }: any) => {
15-
console.error("An error occurred on the client side:", error, event);
15+
console.error('An error occurred on the client side:', error, event);
1616
};
1717

1818
export const handleError = Sentry.handleErrorWithSentry(myErrorHandler);

packages/e2e-tests/test-applications/sveltekit/src/hooks.server.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
import { env } from "$env/dynamic/private";
2-
import * as Sentry from "@sentry/sveltekit";
1+
import { env } from '$env/dynamic/private';
2+
import * as Sentry from '@sentry/sveltekit';
33

44
Sentry.init({
5-
environment: "qa", // dynamic sampling bias to keep transactions
5+
environment: 'qa', // dynamic sampling bias to keep transactions
66
dsn: env.E2E_TEST_DSN,
77
debug: true,
88
tunnel: `http://localhost:3031/`, // proxy server
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
export const GET = () => {
2-
return new Response(JSON.stringify({ users: ["alice", "bob", "carol"] }));
2+
return new Response(JSON.stringify({ users: ['alice', 'bob', 'carol'] }));
33
};
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
export const load = async () => {
2-
throw new Error("Server Load Error");
2+
throw new Error('Server Load Error');
33
return {
4-
msg: "Hello World",
4+
msg: 'Hello World',
55
};
66
};

packages/e2e-tests/test-applications/sveltekit/src/routes/server-route-error/+page.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
export const load = async ({ fetch }) => {
2-
const res = await fetch("/server-route-error");
2+
const res = await fetch('/server-route-error');
33
const data = await res.json();
44
return {
55
msg: data,
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
export const GET = async () => {
2-
throw new Error("Server Route Error");
2+
throw new Error('Server Route Error');
33
return {
4-
msg: "Hello World",
4+
msg: 'Hello World',
55
};
66
};
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
import { browser } from "$app/environment";
1+
import { browser } from '$app/environment';
22

33
export const load = async () => {
4-
throw new Error(`Universal Load Error (${browser ? "browser" : "server"})`);
4+
throw new Error(`Universal Load Error (${browser ? 'browser' : 'server'})`);
55
return {
6-
msg: "Hello World",
6+
msg: 'Hello World',
77
};
88
};
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
export const load = async ({ fetch }) => {
2-
const usersRes = await fetch("/api/users");
2+
const usersRes = await fetch('/api/users');
33
const data = await usersRes.json();
44
return { users: data.users };
55
};
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
export const load = async () => {
22
return {
3-
msg: "Hi everyone!",
3+
msg: 'Hi everyone!',
44
};
55
};
Lines changed: 20 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,55 +1,47 @@
1-
import { expect, test } from "@playwright/test";
2-
import { waitForError } from "../event-proxy-server";
3-
import { waitForInitialPageload } from "./utils";
1+
import { expect, test } from '@playwright/test';
2+
import { waitForError } from '../event-proxy-server';
3+
import { waitForInitialPageload } from './utils';
44

5-
test.describe("capturing client side errors", () => {
6-
test("should capture error thrown on click", async ({ page }) => {
7-
await page.goto("/client-error");
5+
test.describe('capturing client side errors', () => {
6+
test('should capture error thrown on click', async ({ page }) => {
7+
await page.goto('/client-error');
88

9-
await expect(page.getByText("Client error")).toBeVisible();
9+
await expect(page.getByText('Client error')).toBeVisible();
1010

11-
const errorEventPromise = waitForError("sveltekit", (errorEvent) => {
12-
return errorEvent?.exception?.values?.[0]?.value === "Click Error";
11+
const errorEventPromise = waitForError('sveltekit', errorEvent => {
12+
return errorEvent?.exception?.values?.[0]?.value === 'Click Error';
1313
});
1414

15-
const clickPromise = page.getByText("Throw error").click();
15+
const clickPromise = page.getByText('Throw error').click();
1616

17-
const [errorEvent, _] = await Promise.all([
18-
errorEventPromise,
19-
clickPromise,
20-
]);
17+
const [errorEvent, _] = await Promise.all([errorEventPromise, clickPromise]);
2118

22-
const errorEventFrames =
23-
errorEvent.exception?.values?.[0]?.stacktrace?.frames;
19+
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
2420

2521
expect(errorEventFrames?.[errorEventFrames?.length - 1]).toEqual(
2622
expect.objectContaining({
27-
function: expect.stringContaining("HTMLButtonElement"),
23+
function: expect.stringContaining('HTMLButtonElement'),
2824
lineno: 1,
2925
in_app: true,
3026
}),
3127
);
3228

33-
expect(errorEvent.tags).toMatchObject({ runtime: "browser" });
29+
expect(errorEvent.tags).toMatchObject({ runtime: 'browser' });
3430
});
3531

36-
test("should capture universal load error", async ({ page }) => {
32+
test('should capture universal load error', async ({ page }) => {
3733
await waitForInitialPageload(page);
3834
await page.reload();
3935

40-
const errorEventPromise = waitForError("sveltekit", (errorEvent) => {
41-
return (
42-
errorEvent?.exception?.values?.[0]?.value ===
43-
"Universal Load Error (browser)"
44-
);
36+
const errorEventPromise = waitForError('sveltekit', errorEvent => {
37+
return errorEvent?.exception?.values?.[0]?.value === 'Universal Load Error (browser)';
4538
});
4639

4740
// navigating triggers the error on the client
48-
await page.getByText("Universal Load error").click();
41+
await page.getByText('Universal Load error').click();
4942

5043
const errorEvent = await errorEventPromise;
51-
const errorEventFrames =
52-
errorEvent.exception?.values?.[0]?.stacktrace?.frames;
44+
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
5345

5446
expect(errorEventFrames?.[errorEventFrames?.length - 1]).toEqual(
5547
expect.objectContaining({
@@ -58,6 +50,6 @@ test.describe("capturing client side errors", () => {
5850
}),
5951
);
6052

61-
expect(errorEvent.tags).toMatchObject({ runtime: "browser" });
53+
expect(errorEvent.tags).toMatchObject({ runtime: 'browser' });
6254
});
6355
});
Lines changed: 26 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,77 +1,71 @@
1-
import { expect, test } from "@playwright/test";
2-
import { waitForError } from "../event-proxy-server";
1+
import { expect, test } from '@playwright/test';
2+
import { waitForError } from '../event-proxy-server';
33

4-
test.describe("capturing server side errors", () => {
5-
test("should capture universal load error", async ({ page }) => {
6-
const errorEventPromise = waitForError("sveltekit", (errorEvent) => {
7-
return (
8-
errorEvent?.exception?.values?.[0]?.value ===
9-
"Universal Load Error (server)"
10-
);
4+
test.describe('capturing server side errors', () => {
5+
test('should capture universal load error', async ({ page }) => {
6+
const errorEventPromise = waitForError('sveltekit', errorEvent => {
7+
return errorEvent?.exception?.values?.[0]?.value === 'Universal Load Error (server)';
118
});
129

13-
await page.goto("/universal-load-error");
10+
await page.goto('/universal-load-error');
1411

1512
const errorEvent = await errorEventPromise;
16-
const errorEventFrames =
17-
errorEvent.exception?.values?.[0]?.stacktrace?.frames;
13+
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
1814

1915
expect(errorEventFrames?.[errorEventFrames?.length - 1]).toEqual(
2016
expect.objectContaining({
21-
function: "load$1",
17+
function: 'load$1',
2218
lineno: 3,
2319
in_app: true,
2420
}),
2521
);
2622

27-
expect(errorEvent.tags).toMatchObject({ runtime: "node" });
23+
expect(errorEvent.tags).toMatchObject({ runtime: 'node' });
2824
});
2925

30-
test("should capture server load error", async ({ page }) => {
31-
const errorEventPromise = waitForError("sveltekit", (errorEvent) => {
32-
return errorEvent?.exception?.values?.[0]?.value === "Server Load Error";
26+
test('should capture server load error', async ({ page }) => {
27+
const errorEventPromise = waitForError('sveltekit', errorEvent => {
28+
return errorEvent?.exception?.values?.[0]?.value === 'Server Load Error';
3329
});
3430

35-
await page.goto("/server-load-error");
31+
await page.goto('/server-load-error');
3632

3733
const errorEvent = await errorEventPromise;
38-
const errorEventFrames =
39-
errorEvent.exception?.values?.[0]?.stacktrace?.frames;
34+
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
4035

4136
expect(errorEventFrames?.[errorEventFrames?.length - 1]).toEqual(
4237
expect.objectContaining({
43-
function: "load$1",
38+
function: 'load$1',
4439
lineno: 3,
4540
in_app: true,
4641
}),
4742
);
4843

49-
expect(errorEvent.tags).toMatchObject({ runtime: "node" });
44+
expect(errorEvent.tags).toMatchObject({ runtime: 'node' });
5045
});
5146

52-
test("should capture server route (GET) error", async ({ page }) => {
53-
const errorEventPromise = waitForError("sveltekit", (errorEvent) => {
54-
return errorEvent?.exception?.values?.[0]?.value === "Server Route Error";
47+
test('should capture server route (GET) error', async ({ page }) => {
48+
const errorEventPromise = waitForError('sveltekit', errorEvent => {
49+
return errorEvent?.exception?.values?.[0]?.value === 'Server Route Error';
5550
});
5651

57-
await page.goto("/server-route-error");
52+
await page.goto('/server-route-error');
5853

5954
const errorEvent = await errorEventPromise;
60-
const errorEventFrames =
61-
errorEvent.exception?.values?.[0]?.stacktrace?.frames;
55+
const errorEventFrames = errorEvent.exception?.values?.[0]?.stacktrace?.frames;
6256

6357
expect(errorEventFrames?.[errorEventFrames?.length - 1]).toEqual(
6458
expect.objectContaining({
65-
filename: "app:///_server.ts.js",
66-
function: "GET",
59+
filename: 'app:///_server.ts.js',
60+
function: 'GET',
6761
lineno: 2,
6862
in_app: true,
6963
}),
7064
);
7165

7266
expect(errorEvent.tags).toMatchObject({
73-
runtime: "node",
74-
transaction: "GET /server-route-error",
67+
runtime: 'node',
68+
transaction: 'GET /server-route-error',
7569
});
7670
});
7771
});

0 commit comments

Comments
 (0)