test(astro): Add E2E test case for catch-all routes

[s1gr1d]

Extracting a test case in a new PR while working on route parametrization.

Continuation of #17054

Part of #16686

[cursor]

Bug: Astro Catch-All Route Naming Issue

The test for Astro catch-all routes ([...path].astro) incorrectly expects the server transaction name to be 'GET /catchAll/[path]'. It should be 'GET /catchAll/[...path]' to correctly reflect the catch-all route parametrization, as also suggested by a todo comment in the test.

dev-packages/e2e-tests/test-applications/astro-5/tests/tracing.dynamic.test.ts#L296-L297

sentry-javascript/dev-packages/e2e-tests/test-applications/astro-5/tests/tracing.dynamic.test.ts

Lines 296 to 297 in fda4f24

	expect(serverPageRequestTxn).toMatchObject({
	transaction: 'GET /catchAll/[path]',

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎