Skip to content

Commit ab23658

Browse files
author
Luca Forstner
authored
test(e2e): Disable faulty Next.js assertion (#11429)
1 parent 6eddfc9 commit ab23658

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

dev-packages/e2e-tests/test-applications/nextjs-14/tests/request-instrumentation.test.ts

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,16 @@ test('Should send a transaction with a fetch span', async ({ page }) => {
1919
}),
2020
);
2121

22-
expect((await transactionPromise).spans).toContainEqual(
23-
expect.objectContaining({
24-
data: expect.objectContaining({
25-
'http.method': 'GET',
26-
'sentry.op': 'http.client',
27-
'sentry.origin': 'auto.http.otel.http',
28-
}),
29-
description: 'GET http://example.com/',
30-
}),
31-
);
22+
// TODO: Uncomment the below when fixed. For whatever reason that we now have accepted, spans created with Node.js' http.get() will not attach themselves to transactions.
23+
// More info: https://github.com/getsentry/sentry-javascript/pull/11016/files#diff-10fa195789425786c6e5e769380be18790768f0b76319ee41bbb488d9fe50405R4
24+
// expect((await transactionPromise).spans).toContainEqual(
25+
// expect.objectContaining({
26+
// data: expect.objectContaining({
27+
// 'http.method': 'GET',
28+
// 'sentry.op': 'http.client',
29+
// 'sentry.origin': 'auto.http.otel.http',
30+
// }),
31+
// description: 'GET http://example.com/',
32+
// }),
33+
// );
3234
});

0 commit comments

Comments
 (0)