Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,10 @@
{
"build-command": "pnpm test:build-latest",
"label": "nextjs-16-cf-workers (latest)"
},
{
"build-command": "pnpm test:build-canary",
"label": "nextjs-16-cf-workers (canary)"
}
]
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import { expect, test } from '@playwright/test';
import { waitForError } from '@sentry-internal/test-utils';

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.describe.skip('Cloudflare Runtime', () => {
test.describe('Cloudflare Runtime', () => {
test('Should report cloudflare as the runtime in API route error events', async ({ request }) => {
const errorEventPromise = waitForError('nextjs-16-cf-workers', errorEvent => {
return !!errorEvent?.exception?.values?.some(value =>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import { expect, test } from '@playwright/test';
import { waitForTransaction } from '@sentry-internal/test-utils';

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should remove sentry-trace and baggage meta tags on ISR dynamic route page load', async ({ page }) => {
test('should remove sentry-trace and baggage meta tags on ISR dynamic route page load', async ({ page }) => {
// Navigate to ISR page
await page.goto('/isr-test/laptop');

Expand All @@ -14,8 +13,7 @@ test.skip('should remove sentry-trace and baggage meta tags on ISR dynamic route
await expect(page.locator('meta[name="baggage"]')).toHaveCount(0);
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should remove sentry-trace and baggage meta tags on ISR static route', async ({ page }) => {
test('should remove sentry-trace and baggage meta tags on ISR static route', async ({ page }) => {
// Navigate to ISR static page
await page.goto('/isr-test/static');

Expand All @@ -27,8 +25,7 @@ test.skip('should remove sentry-trace and baggage meta tags on ISR static route'
await expect(page.locator('meta[name="baggage"]')).toHaveCount(0);
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should remove meta tags for different ISR dynamic route values', async ({ page }) => {
test('should remove meta tags for different ISR dynamic route values', async ({ page }) => {
// Test with 'phone' (one of the pre-generated static params)
await page.goto('/isr-test/phone');
await expect(page.locator('#isr-product-id')).toHaveText('phone');
Expand All @@ -44,8 +41,7 @@ test.skip('should remove meta tags for different ISR dynamic route values', asyn
await expect(page.locator('meta[name="baggage"]')).toHaveCount(0);
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should create unique transactions for ISR pages on each visit', async ({ page }) => {
test('should create unique transactions for ISR pages on each visit', async ({ page }) => {
const traceIds: string[] = [];

// Load the same ISR page 5 times to ensure cached HTML meta tags are consistently removed
Expand Down Expand Up @@ -75,8 +71,7 @@ test.skip('should create unique transactions for ISR pages on each visit', async
expect(uniqueTraceIds.size).toBe(5);
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('ISR route should be identified correctly in the route manifest', async ({ page }) => {
test('ISR route should be identified correctly in the route manifest', async ({ page }) => {
const transactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
return transactionEvent.transaction === '/isr-test/:product' && transactionEvent.contexts?.trace?.op === 'pageload';
});
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import { expect, test } from '@playwright/test';
import { waitForTransaction } from '@sentry-internal/test-utils';

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should create a parameterized transaction when the `app` directory is used', async ({ page }) => {
test('should create a parameterized transaction when the `app` directory is used', async ({ page }) => {
const transactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
return (
transactionEvent.transaction === '/parameterized/:one' && transactionEvent.contexts?.trace?.op === 'pageload'
Expand Down Expand Up @@ -41,8 +40,7 @@ test.skip('should create a parameterized transaction when the `app` directory is
});
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should create a static transaction when the `app` directory is used and the route is not parameterized', async ({
test('should create a static transaction when the `app` directory is used and the route is not parameterized', async ({
page,
}) => {
const transactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
Expand Down Expand Up @@ -83,8 +81,7 @@ test.skip('should create a static transaction when the `app` directory is used a
});
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should create a partially parameterized transaction when the `app` directory is used', async ({ page }) => {
test('should create a partially parameterized transaction when the `app` directory is used', async ({ page }) => {
const transactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
return (
transactionEvent.transaction === '/parameterized/:one/beep' && transactionEvent.contexts?.trace?.op === 'pageload'
Expand Down Expand Up @@ -123,8 +120,7 @@ test.skip('should create a partially parameterized transaction when the `app` di
});
});

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('should create a nested parameterized transaction when the `app` directory is used.', async ({ page }) => {
test('should create a nested parameterized transaction when the `app` directory is used.', async ({ page }) => {
const transactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
return (
transactionEvent.transaction === '/parameterized/:one/beep/:two' &&
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,7 @@ import { expect, test } from '@playwright/test';
import { waitForTransaction } from '@sentry-internal/test-utils';
import { isDevMode } from './isDevMode';

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('Prefetch client spans should have a http.request.prefetch attribute', async ({ page }) => {
test('Prefetch client spans should have a http.request.prefetch attribute', async ({ page }) => {
test.skip(isDevMode, "Prefetch requests don't have the prefetch header in dev mode");

const pageloadTransactionPromise = waitForTransaction('nextjs-16-cf-workers', async transactionEvent => {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
import { expect, test } from '@playwright/test';
import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';

// TODO(https://github.com/opennextjs/opennextjs-cloudflare/issues/1141): Unskip once opennext supports prefetch-hints.json
test.skip('Should capture errors for crashing streaming promises in server components when `Sentry.captureRequestError` is added to the `onRequestError` hook', async ({
test('Should capture errors for crashing streaming promises in server components when `Sentry.captureRequestError` is added to the `onRequestError` hook', async ({
page,
}) => {
const errorEventPromise = waitForError('nextjs-16-cf-workers', errorEvent => {
Expand Down
Loading