Skip to content

Commit 3a7c793

Browse files
committed
better debug...
1 parent 0c88d9c commit 3a7c793

File tree

2 files changed

+18
-3
lines changed

2 files changed

+18
-3
lines changed

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/client-app-routing-instrumentation.test.ts

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,11 @@ test('Creates a navigation transaction for app router routes', async ({ page })
2020
const randomRoute = String(Math.random());
2121

2222
const clientPageloadTransactionPromise = waitForTransaction('nextjs-13-app-dir', transactionEvent => {
23-
console.log('transaction1', JSON.stringify(transactionEvent, null, 2));
23+
console.log(
24+
'transaction1',
25+
transactionEvent.transaction,
26+
JSON.stringify(transactionEvent.contexts?.trace, null, 2),
27+
);
2428
return (
2529
transactionEvent?.transaction === `/server-component/parameter/${randomRoute}` &&
2630
transactionEvent.contexts?.trace?.op === 'pageload'
@@ -32,15 +36,23 @@ test('Creates a navigation transaction for app router routes', async ({ page })
3236
await page.getByText('Page (/server-component/parameter/[parameter])').isVisible();
3337

3438
const clientNavigationTransactionPromise = waitForTransaction('nextjs-13-app-dir', transactionEvent => {
35-
console.log('transaction2', JSON.stringify(transactionEvent, null, 2));
39+
console.log(
40+
'transaction2',
41+
transactionEvent.transaction,
42+
JSON.stringify(transactionEvent.contexts?.trace, null, 2),
43+
);
3644
return (
3745
transactionEvent?.transaction === '/server-component/parameter/foo/bar/baz' &&
3846
transactionEvent.contexts?.trace?.op === 'navigation'
3947
);
4048
});
4149

4250
const serverComponentTransactionPromise = waitForTransaction('nextjs-13-app-dir', async transactionEvent => {
43-
console.log('transaction3', JSON.stringify(transactionEvent, null, 2));
51+
console.log(
52+
'transaction3',
53+
transactionEvent.transaction,
54+
JSON.stringify(transactionEvent.contexts?.trace, null, 2),
55+
);
4456
return (
4557
transactionEvent?.transaction === 'Page Server Component (/server-component/parameter/[...parameters])' &&
4658
(await clientNavigationTransactionPromise).contexts?.trace?.trace_id ===

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/route-handlers.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import { waitForError, waitForTransaction } from '@sentry-internal/event-proxy-s
33

44
test('Should create a transaction for route handlers', async ({ request }) => {
55
const routehandlerTransactionPromise = waitForTransaction('nextjs-13-app-dir', async transactionEvent => {
6+
console.log('should be GET /route-handlers/[param]', transactionEvent?.transaction);
67
// Sometimes, next.js adds /route there, we accept both variants
78
return (
89
transactionEvent?.transaction === 'GET /route-handlers/[param]' ||
@@ -24,6 +25,7 @@ test('Should create a transaction for route handlers and correctly set span stat
2425
request,
2526
}) => {
2627
const routehandlerTransactionPromise = waitForTransaction('nextjs-13-app-dir', async transactionEvent => {
28+
console.log('should be POST /route-handlers/[param]', transactionEvent?.transaction);
2729
// Sometimes, next.js adds /route there, we accept both variants
2830
return (
2931
transactionEvent?.transaction === 'POST /route-handlers/[param]' ||
@@ -47,6 +49,7 @@ test('Should record exceptions and transactions for faulty route handlers', asyn
4749

4850
const routehandlerTransactionPromise = waitForTransaction('nextjs-13-app-dir', async transactionEvent => {
4951
console.log('transactionEvent3', JSON.stringify(transactionEvent, null, 2));
52+
console.log('should be PUT /route-handlers/[param]', transactionEvent?.transaction);
5053
// Sometimes, next.js adds /route there, we accept both variants
5154
return (
5255
transactionEvent?.transaction === 'PUT /route-handlers/[param]' ||

0 commit comments

Comments
 (0)