Skip to content

Commit 8a41d45

Browse files
author
Luca Forstner
committed
Fix small number of tests
1 parent f4043ab commit 8a41d45

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ test('Should create a transaction with error status for faulty edge routes', asy
2727
const edgerouteTransactionPromise = waitForTransaction('nextjs-app-dir', async transactionEvent => {
2828
return (
2929
transactionEvent?.transaction === 'GET /api/error-edge-endpoint' &&
30-
transactionEvent?.contexts?.trace?.status === 'internal_error'
30+
transactionEvent?.contexts?.trace?.status === 'unknown_error'
3131
);
3232
});
3333

@@ -37,7 +37,7 @@ test('Should create a transaction with error status for faulty edge routes', asy
3737

3838
const edgerouteTransaction = await edgerouteTransactionPromise;
3939

40-
expect(edgerouteTransaction.contexts?.trace?.status).toBe('internal_error');
40+
expect(edgerouteTransaction.contexts?.trace?.status).toBe('unknown_error');
4141
expect(edgerouteTransaction.contexts?.trace?.op).toBe('http.server');
4242
expect(edgerouteTransaction.contexts?.runtime?.name).toBe('vercel-edge');
4343

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,17 @@ test('Should record exceptions for faulty edge server components', async ({ page
1919
expect(errorEvent.transaction).toBe(`Page Server Component (/edge-server-components/error)`);
2020
});
2121

22-
test('Should record transaction for edge server components', async ({ page }) => {
22+
test.only('Should record transaction for edge server components', async ({ page }) => {
2323
const serverComponentTransactionPromise = waitForTransaction('nextjs-app-dir', async transactionEvent => {
24-
return transactionEvent?.transaction === 'Page Server Component (/edge-server-components)';
24+
console.log('t', transactionEvent.transaction);
25+
return transactionEvent?.transaction === 'GET /edge-server-components';
2526
});
2627

2728
await page.goto('/edge-server-components');
2829

2930
const serverComponentTransaction = await serverComponentTransactionPromise;
3031

32+
expect(serverComponentTransaction).toBe(1);
3133
expect(serverComponentTransaction).toBeDefined();
3234
expect(serverComponentTransaction.request?.headers).toBeDefined();
3335

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ test('Should create a transaction for middleware', async ({ request }) => {
2323
test('Should create a transaction with error status for faulty middleware', async ({ request }) => {
2424
const middlewareTransactionPromise = waitForTransaction('nextjs-app-dir', async transactionEvent => {
2525
return (
26-
transactionEvent?.transaction === 'middleware' && transactionEvent?.contexts?.trace?.status === 'internal_error'
26+
transactionEvent?.transaction === 'middleware' && transactionEvent?.contexts?.trace?.status === 'unknown_error'
2727
);
2828
});
2929

@@ -33,7 +33,7 @@ test('Should create a transaction with error status for faulty middleware', asyn
3333

3434
const middlewareTransaction = await middlewareTransactionPromise;
3535

36-
expect(middlewareTransaction.contexts?.trace?.status).toBe('internal_error');
36+
expect(middlewareTransaction.contexts?.trace?.status).toBe('unknown_error');
3737
expect(middlewareTransaction.contexts?.trace?.op).toBe('middleware.nextjs');
3838
expect(middlewareTransaction.contexts?.runtime?.name).toBe('vercel-edge');
3939
});

0 commit comments

Comments
 (0)