Skip to content

Commit 39828b5

Browse files
onurtemizkanonurtemizkan
committed
Export wrapCreateMemoryRouter 
1 parent 1cf960b commit 39828b5

33 files changed

+942
-84
lines changed

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/.gitignore

+29
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# See https://help.github.com/articles/ignoring-files/ for more about ignoring files.
2+
3+
# dependencies
4+
/node_modules
5+
/.pnp
6+
.pnp.js
7+
8+
# testing
9+
/coverage
10+
11+
# production
12+
/build
13+
14+
# misc
15+
.DS_Store
16+
.env.local
17+
.env.development.local
18+
.env.test.local
19+
.env.production.local
20+
21+
npm-debug.log*
22+
yarn-debug.log*
23+
yarn-error.log*
24+
25+
/test-results/
26+
/playwright-report/
27+
/playwright/.cache/
28+
29+
!*.d.ts

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/.npmrc

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
@sentry:registry=http://127.0.0.1:4873
2+
@sentry-internal:registry=http://127.0.0.1:4873

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/package.json

+40
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
{
2+
"name": "react-create-browser-router-test",
3+
"version": "0.1.0",
4+
"private": true,
5+
"dependencies": {
6+
"@sentry/react": "latest || *",
7+
"@types/node": "^18.19.1",
8+
"@types/react": "18.0.0",
9+
"@types/react-dom": "18.0.0",
10+
"react": "18.2.0",
11+
"react-dom": "18.2.0",
12+
"react-router-dom": "^6.4.1",
13+
"react-scripts": "5.0.1",
14+
"typescript": "~5.0.0"
15+
},
16+
"scripts": {
17+
"build": "react-scripts build",
18+
"start": "serve -s build",
19+
"test": "playwright test",
20+
"clean": "npx rimraf node_modules pnpm-lock.yaml",
21+
"test:build": "pnpm install && pnpm build",
22+
"test:build-canary": "pnpm install && pnpm add react@canary react-dom@canary && pnpm build",
23+
"test:assert": "pnpm test"
24+
},
25+
"eslintConfig": {
26+
"extends": ["react-app", "react-app/jest"]
27+
},
28+
"browserslist": {
29+
"production": [">0.2%", "not dead", "not op_mini all"],
30+
"development": ["last 1 chrome version", "last 1 firefox version", "last 1 safari version"]
31+
},
32+
"devDependencies": {
33+
"@playwright/test": "^1.44.1",
34+
"@sentry-internal/test-utils": "link:../../../test-utils",
35+
"serve": "14.0.1"
36+
},
37+
"volta": {
38+
"extends": "../../package.json"
39+
}
40+
}

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/playwright.config.mjs

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
import { getPlaywrightConfig } from '@sentry-internal/test-utils';
2+
3+
const config = getPlaywrightConfig({
4+
startCommand: `pnpm start`,
5+
});
6+
7+
export default config;

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/public/index.html

+24
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
<!DOCTYPE html>
2+
<html lang="en">
3+
<head>
4+
<meta charset="utf-8" />
5+
<meta name="viewport" content="width=device-width, initial-scale=1" />
6+
<meta name="theme-color" content="#000000" />
7+
<meta name="description" content="Web site created using create-react-app" />
8+
<title>React App</title>
9+
</head>
10+
<body>
11+
<noscript>You need to enable JavaScript to run this app.</noscript>
12+
<div id="root"></div>
13+
<!--
14+
This HTML file is a template.
15+
If you open it directly in the browser, you will see an empty page.
16+
17+
You can add webfonts, meta tags, or analytics to this file.
18+
The build step will place the bundled scripts into the <body> tag.
19+
20+
To begin the development, run `npm start` or `yarn start`.
21+
To create a production bundle, use `npm run build` or `yarn build`.
22+
-->
23+
</body>
24+
</html>

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/src/globals.d.ts

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
interface Window {
2+
recordedTransactions?: string[];
3+
capturedExceptionId?: string;
4+
sentryReplayId?: string;
5+
}

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/src/index.tsx

+66
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
import * as Sentry from '@sentry/react';
2+
import React from 'react';
3+
import ReactDOM from 'react-dom/client';
4+
import {
5+
RouterProvider,
6+
createBrowserRouter,
7+
createRoutesFromChildren,
8+
matchRoutes,
9+
useLocation,
10+
useNavigationType,
11+
} from 'react-router-dom';
12+
import Index from './pages/Index';
13+
import User from './pages/User';
14+
15+
const replay = Sentry.replayIntegration();
16+
17+
Sentry.init({
18+
// environment: 'qa', // dynamic sampling bias to keep transactions
19+
dsn: process.env.REACT_APP_E2E_TEST_DSN,
20+
integrations: [
21+
Sentry.reactRouterV6BrowserTracingIntegration({
22+
useEffect: React.useEffect,
23+
useLocation,
24+
useNavigationType,
25+
createRoutesFromChildren,
26+
matchRoutes,
27+
}),
28+
replay,
29+
],
30+
// We recommend adjusting this value in production, or using tracesSampler
31+
// for finer control
32+
tracesSampleRate: 1.0,
33+
release: 'e2e-test',
34+
35+
tunnel: 'http://localhost:3031',
36+
37+
// Always capture replays, so we can test this properly
38+
replaysSessionSampleRate: 1.0,
39+
replaysOnErrorSampleRate: 0.0,
40+
41+
debug: !!process.env.DEBUG,
42+
});
43+
44+
const sentryCreateBrowserRouter = Sentry.wrapCreateBrowserRouterV6(createBrowserRouter);
45+
46+
const router = sentryCreateBrowserRouter(
47+
[
48+
{
49+
path: '/',
50+
element: <Index />,
51+
},
52+
{
53+
path: '/user/:id',
54+
element: <User />,
55+
},
56+
],
57+
{
58+
// We're testing whether this option is avoided in the integration
59+
// We expect this to be ignored
60+
initialEntries: ['/user/1'],
61+
},
62+
);
63+
64+
const root = ReactDOM.createRoot(document.getElementById('root') as HTMLElement);
65+
66+
root.render(<RouterProvider router={router} />);

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/src/pages/Index.tsx

+23
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
2+
import * as React from 'react';
3+
import { Link } from 'react-router-dom';
4+
5+
const Index = () => {
6+
return (
7+
<>
8+
<input
9+
type="button"
10+
value="Capture Exception"
11+
id="exception-button"
12+
onClick={() => {
13+
throw new Error('I am an error!');
14+
}}
15+
/>
16+
<Link to="/user/5" id="navigation">
17+
navigate
18+
</Link>
19+
</>
20+
);
21+
};
22+
23+
export default Index;

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/src/pages/User.tsx

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
// biome-ignore lint/nursery/noUnusedImports: Need React import for JSX
2+
import * as React from 'react';
3+
4+
const User = () => {
5+
return <p>I am a blank page :)</p>;
6+
};
7+
8+
export default User;

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/src/react-app-env.d.ts

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
/// <reference types="react-scripts" />

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/start-event-proxy.mjs

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
import { startEventProxyServer } from '@sentry-internal/test-utils';
2+
3+
startEventProxyServer({
4+
port: 3031,
5+
proxyServerName: 'react-create-browser-router',
6+
});

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/tests/errors.test.ts

+30
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
import { expect, test } from '@playwright/test';
2+
import { waitForError } from '@sentry-internal/test-utils';
3+
4+
test('Captures exception correctly', async ({ page }) => {
5+
const errorEventPromise = waitForError('react-create-browser-router', event => {
6+
return !event.type && event.exception?.values?.[0]?.value === 'I am an error!';
7+
});
8+
9+
await page.goto('/');
10+
11+
const exceptionButton = page.locator('id=exception-button');
12+
await exceptionButton.click();
13+
14+
const errorEvent = await errorEventPromise;
15+
16+
expect(errorEvent.exception?.values).toHaveLength(1);
17+
expect(errorEvent.exception?.values?.[0]?.value).toBe('I am an error!');
18+
19+
expect(errorEvent.request).toEqual({
20+
headers: expect.any(Object),
21+
url: 'http://localhost:3030/',
22+
});
23+
24+
expect(errorEvent.transaction).toEqual('/');
25+
26+
expect(errorEvent.contexts?.trace).toEqual({
27+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
28+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
29+
});
30+
});

‎dev-packages/e2e-tests/test-applications/react-create-browser-router/tests/transactions.test.ts

+142
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,142 @@
1+
import { expect, test } from '@playwright/test';
2+
import { waitForTransaction } from '@sentry-internal/test-utils';
3+
4+
test('Captures a pageload transaction', async ({ page }) => {
5+
const transactionEventPromise = waitForTransaction('react-create-browser-router', event => {
6+
return event.contexts?.trace?.op === 'pageload';
7+
});
8+
9+
await page.goto('/');
10+
11+
const transactionEvent = await transactionEventPromise;
12+
expect(transactionEvent.contexts?.trace).toEqual({
13+
data: {
14+
deviceMemory: expect.any(String),
15+
effectiveConnectionType: expect.any(String),
16+
hardwareConcurrency: expect.any(String),
17+
'lcp.element': 'body > div#root > input#exception-button[type="button"]',
18+
'lcp.id': 'exception-button',
19+
'lcp.size': 1650,
20+
'sentry.idle_span_finish_reason': 'idleTimeout',
21+
'sentry.op': 'pageload',
22+
'sentry.origin': 'auto.pageload.react.reactrouter_v6',
23+
'sentry.sample_rate': 1,
24+
'sentry.source': 'route',
25+
'performance.timeOrigin': expect.any(Number),
26+
'performance.activationStart': expect.any(Number),
27+
'lcp.renderTime': expect.any(Number),
28+
'lcp.loadTime': expect.any(Number),
29+
},
30+
op: 'pageload',
31+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
32+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
33+
origin: 'auto.pageload.react.reactrouter_v6',
34+
});
35+
36+
expect(transactionEvent).toEqual(
37+
expect.objectContaining({
38+
transaction: '/',
39+
type: 'transaction',
40+
transaction_info: {
41+
source: 'route',
42+
},
43+
}),
44+
);
45+
46+
expect(transactionEvent.spans).toContainEqual({
47+
data: {
48+
'sentry.origin': 'auto.ui.browser.metrics',
49+
'sentry.op': 'browser.domContentLoadedEvent',
50+
},
51+
description: page.url(),
52+
op: 'browser.domContentLoadedEvent',
53+
parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
54+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
55+
start_timestamp: expect.any(Number),
56+
timestamp: expect.any(Number),
57+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
58+
origin: 'auto.ui.browser.metrics',
59+
});
60+
expect(transactionEvent.spans).toContainEqual({
61+
data: {
62+
'sentry.origin': 'auto.ui.browser.metrics',
63+
'sentry.op': 'browser.connect',
64+
},
65+
description: page.url(),
66+
op: 'browser.connect',
67+
parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
68+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
69+
start_timestamp: expect.any(Number),
70+
timestamp: expect.any(Number),
71+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
72+
origin: 'auto.ui.browser.metrics',
73+
});
74+
expect(transactionEvent.spans).toContainEqual({
75+
data: {
76+
'sentry.origin': 'auto.ui.browser.metrics',
77+
'sentry.op': 'browser.request',
78+
},
79+
description: page.url(),
80+
op: 'browser.request',
81+
parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
82+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
83+
start_timestamp: expect.any(Number),
84+
timestamp: expect.any(Number),
85+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
86+
origin: 'auto.ui.browser.metrics',
87+
});
88+
expect(transactionEvent.spans).toContainEqual({
89+
data: {
90+
'sentry.origin': 'auto.ui.browser.metrics',
91+
'sentry.op': 'browser.response',
92+
},
93+
description: page.url(),
94+
op: 'browser.response',
95+
parent_span_id: expect.stringMatching(/[a-f0-9]{16}/),
96+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
97+
start_timestamp: expect.any(Number),
98+
timestamp: expect.any(Number),
99+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
100+
origin: 'auto.ui.browser.metrics',
101+
});
102+
});
103+
104+
test('Captures a navigation transaction', async ({ page }) => {
105+
const transactionEventPromise = waitForTransaction('react-create-browser-router', event => {
106+
return event.contexts?.trace?.op === 'navigation';
107+
});
108+
109+
await page.goto('/');
110+
const linkElement = page.locator('id=navigation');
111+
await linkElement.click();
112+
113+
const transactionEvent = await transactionEventPromise;
114+
expect(transactionEvent.contexts?.trace).toEqual({
115+
data: expect.objectContaining({
116+
deviceMemory: expect.any(String),
117+
effectiveConnectionType: expect.any(String),
118+
hardwareConcurrency: expect.any(String),
119+
'sentry.idle_span_finish_reason': 'idleTimeout',
120+
'sentry.op': 'navigation',
121+
'sentry.origin': 'auto.navigation.react.reactrouter_v6',
122+
'sentry.sample_rate': 1,
123+
'sentry.source': 'route',
124+
}),
125+
op: 'navigation',
126+
span_id: expect.stringMatching(/[a-f0-9]{16}/),
127+
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
128+
origin: 'auto.navigation.react.reactrouter_v6',
129+
});
130+
131+
expect(transactionEvent).toEqual(
132+
expect.objectContaining({
133+
transaction: '/user/:id',
134+
type: 'transaction',
135+
transaction_info: {
136+
source: 'route',
137+
},
138+
}),
139+
);
140+
141+
expect(transactionEvent.spans).toEqual([]);
142+
});

0 commit comments

Comments
 (0)